linux-kernel - Re: [Xen-devel] [PATCH v2] xen/balloon: add runtime control for scrubbing ballooned out pages

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <5B92A01D02000078001E67CE@prv1-mh.provo.novell.com>
Date:   Fri, 07 Sep 2018 09:58:21 -0600
From:   "Jan Beulich" <JBeulich@...e.com>
To:     "Marek Marczykowski" <marmarek@...isiblethingslab.com>
Cc:     "Kai-Heng Feng" <kai.heng.feng@...onical.com>,
        "Thymo van Beers" <thymovanbeers@...il.com>,
        "Bjorn Helgaas" <bhelgaas@...gle.com>,
        "David Rientjes" <rientjes@...gle.com>,
        "Frederic Weisbecker" <frederic@...nel.org>,
        "Ingo Molnar" <mingo@...nel.org>, <tglx@...utronix.de>,
        <gregkh@...uxfoundation.org>,
        "xen-devel" <xen-devel@...ts.xenproject.org>,
        "Jonathan Corbet" <corbet@....net>,
        "Boris Ostrovsky" <boris.ostrovsky@...cle.com>,
        "Konrad Rzeszutek Wilk" <konrad.wilk@...cle.com>,
        "Juergen Gross" <jgross@...e.com>, "Jiri Kosina" <jkosina@...e.cz>,
        <linux-doc@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [Xen-devel] [PATCH v2] xen/balloon: add runtime control
 for scrubbing ballooned out pages

>>> On 07.09.18 at 17:31, <marmarek@...isiblethingslab.com> wrote:
> --- a/drivers/xen/mem-reservation.c
> +++ b/drivers/xen/mem-reservation.c
> @@ -14,6 +14,14 @@
>  
>  #include <xen/interface/memory.h>
>  #include <xen/mem-reservation.h>
> +#include <linux/moduleparam.h>
> +
> +#ifdef CONFIG_XEN_SCRUB_PAGES_DEFAULT
> +bool __read_mostly xen_scrub_pages = true;
> +#else
> +bool __read_mostly xen_scrub_pages = false;
> +#endif

Use IS_ENABLED() here instead of the #ifdef?

Jan