>>> On 20.11.18 at 17:59, <[email protected]> wrote: > On 20/11/2018 16:18, Jan Beulich wrote: >> For domain heap pages assigned to a domain dropping the page reference >> tied to PGC_allocated may not drop the last reference, as otherwise the >> test_and_clear_bit() might already act on an unowned page. >> >> Work around this where possible, but the need to acquire extra page >> references is a fair hint that references should have been acquired in >> other places instead. >> >> Signed-off-by: Jan Beulich <[email protected]> >> --- >> Compile tested only, as I have neither a mem-sharing nor a mem-paging >> environment set up ready to be used for such testing. > > Perhaps we should compile them out by default? It's clear there are no > production users, given the quality of the code and how many security > issues we spot accidentally.
Yeah, well - if we're going to have a perhaps much wider set of config options, then these two surely should become "default n" until they've been brought out of their sorry state. > Either way, the code looks plausible. Acked-by: Andrew Cooper > <[email protected]> Thanks. Jan _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
