| 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
| ||
|
Date: Tue, 21 Aug 2012 16:27:53 +0100 From: Stefano Stabellini <stefano.stabellini@...citrix.com> To: Konrad Rzeszutek Wilk <konrad.wilk@...cle.com> CC: Stefano Stabellini <Stefano.Stabellini@...citrix.com>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, "xen-devel@...ts.xensource.com" <xen-devel@...ts.xensource.com> Subject: Re: [Xen-devel] [PATCH 11/11] xen/mmu: Release just the MFN list, not MFN list and part of pagetables. On Tue, 21 Aug 2012, Konrad Rzeszutek Wilk wrote: > On Tue, Aug 21, 2012 at 03:18:26PM +0100, Stefano Stabellini wrote: > > On Thu, 16 Aug 2012, Konrad Rzeszutek Wilk wrote: > > > We call memblock_reserve for [start of mfn list] -> [PMD aligned end > > > of mfn list] instead of <start of mfn list> -> <page aligned end of mfn list]. > > > > > > This has the disastrous effect that if at bootup the end of mfn_list is > > > not PMD aligned we end up returning to memblock parts of the region > > > past the mfn_list array. And those parts are the PTE tables with > > > the disastrous effect of seeing this at bootup: > > > > This patch looks wrong to me. > > Its easier to see if you stick the patch in the code. The size = part > was actually also done earlier. Yes, you are right, I see that know. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists