linux-kernel - Re: [PATCH] mm: don't defer struct page initialization for Xen pv guests

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: <b277a7be-05d4-67e7-f8ad-50f7865e87fe@oracle.com>
Date:   Fri, 16 Feb 2018 09:21:57 -0500
From:   Pavel Tatashin <pasha.tatashin@...CLE.COM>
To:     Juergen Gross <jgross@...e.com>, Michal Hocko <mhocko@...nel.org>
Cc:     linux-kernel@...r.kernel.org, linux-mm@...ck.org,
        xen-devel@...ts.xenproject.org, akpm@...ux-foundation.org,
        stable@...r.kernel.org
Subject: Re: [PATCH] mm: don't defer struct page initialization for Xen pv
 guests

On 02/16/2018 09:02 AM, Juergen Gross wrote:
> On 16/02/18 14:59, Michal Hocko wrote:
>> [CC Pavel]
>>
>> On Fri 16-02-18 14:37:26, Juergen Gross wrote:
>>> Commit f7f99100d8d95dbcf09e0216a143211e79418b9f ("mm: stop zeroing
>>> memory during allocation in vmemmap") broke Xen pv domains in some
>>> configurations, as the "Pinned" information in struct page of early
>>> page tables could get lost.
>>
>> Could you be more specific please?
> 
> In which way? Do you want to see the resulting crash in the commit
> message or some more background information?

Hi Juergen,

Could you please provide details how to reproduce the problem, and crash 
message. Also, information about where exactly xen access struct pages 
before deferred pages are initialized.

Thank you,
Pasha

> 
>>
>>> Avoid this problem by not deferring struct page initialization when
>>> running as Xen pv guest.
>>>
>>> Cc: <stable@...r.kernel.org> #4.15
>> Fixes: f7f99100d8d9 ("mm: stop zeroing memory during allocation in vmemmap")
>>
>> please
> 
> Okay.
> 
> 
> Juergen
>