[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 1/4] mm: Initialize lowmem virq when boot-time scrubbing is disabled

>>> On 29.08.17 at 13:51, <wei.liu2@xxxxxxxxxx> wrote:
> On Mon, Aug 28, 2017 at 04:40:26PM -0400, Boris Ostrovsky wrote:
>> scrub_heap_pages() does early return if boot-time scrubbing is
>> disabled, neglecting to initialize lowmem VIRQ.
>> Signed-off-by: Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>
> Seems more appropriate to lift the call to setup_low_mem_virq to the
> caller of scrub_heap_pages.

That's not a good idea imo, as that would mean replicating the
code in ARM and x86. I'd rather see the early return to become
an ordinary if (skipping the rest of the function, preferably
without goto). Or, mainly to limit code churn, introduce a new
wrapper function (of perhaps a different name, as
setup_low_mem_virq() has nothing to do with scrubbing
anyway) and move the setup_low_mem_virq() call
there, making scrub_heap_pages() static.


Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.