From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave McCracken Subject: Re: Re: [Linux PATCH] Make hugepages work in current =?iso-8859-15?q?git=09tree?= Date: Fri, 30 Apr 2010 14:34:48 -0500 Message-ID: <201004301434.48409.dcm@mccr.org> References: <20100428140825.17161.27594.sendpatchset@magnum.int.mccr.org> <201004291643.16613.dcm@mccr.org> <4BD9FF77.1040300@goop.org> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4BD9FF77.1040300@goop.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Jeremy Fitzhardinge Cc: xen-devel@lists.xensource.com, Jan Beulich List-Id: xen-devel@lists.xenproject.org On Thursday 29 April 2010, Jeremy Fitzhardinge wrote: > On 04/29/2010 02:43 PM, Dave McCracken wrote: > > On Thursday 29 April 2010, Jeremy Fitzhardinge wrote: > >> Looks reasonable. I rewrote the commit comment: > >> > >> Subject: [PATCH] x86/hugetlb: use set_pmd for huge pte operations > >> > >> On x86, a huge pte is logically a pte, but structurally a pmd. Among > >> other issues, pmds and ptes overload some flags for multiple uses (PAT > >> vs PSE), so it is necessary to know which structural level a pagetable > >> entry is in order interpret it properly. > >> > >> When huge pages are used within a paravirtualized system, it is > >> therefore appropriate to use the pmd set of function to operate on them, > >> so that the hypervisor can correctly validate the update. > >> > >> Signed-off-by: Dave McCracken > >> Signed-off-by: Jeremy Fitzhardinge > >> > >> Does this look correct? > > > > Yes, that's an excellent summary of the issue. Thank you. > > Have you tested this patch on native as well as under Xen? Yes, it works just fine, though see my separate post about building the current git tree without Xen. I'd forgotten I had to patch it to make it build. Dave McCracken