Re: [patch 2/8] x86 PAT: set VM_PFNMAP flag in vm_insert_pfn

[Nick Piggin]

On Thu, Nov 20, 2008 at 03:42:32PM -0800, Pallipadi, Venkatesh wrote:
>  
> 
> >-----Original Message-----
> >From: linux-kernel-owner@xxxxxxxxxxxxxxx 
> >[mailto:linux-kernel-owner@xxxxxxxxxxxxxxx] On Behalf Of Ingo Molnar
> >Sent: Tuesday, November 18, 2008 1:38 PM
> >To: Nick Piggin
> >Cc: Pallipadi, Venkatesh; Thomas Gleixner; H Peter Anvin; Hugh 
> >Dickins; Roland Dreier; Jesse Barnes; Jeremy Fitzhardinge; 
> >Arjan van de Ven; linux-kernel@xxxxxxxxxxxxxxx; Siddha, Suresh B
> >Subject: Re: [patch 2/8] x86 PAT: set VM_PFNMAP flag in vm_insert_pfn
> >
> >
> >patchset need acks from MM folks - Andrew, Nick or Hugh - we cannot 
> >just queue them up in the x86/pat tree without agreement from MM 
> >maintainers.
> >
> 
> OK. Feedback based on the discussions on this patchset, this particular patch
> set VM_PFNMAP flag in vm_insert_pfn
> is not really needed and we should rather BUG_ON on that not being set. 
> And there were few more comments on one other generic vm patch from Nick.
> 
> I will resend the entire patchset with changes soon and then we can work
> out the logistics.

So long as it has been reviewed by mm people (cc'ing linux-mm might be
nice), I don't care so much how it makes its way upstream. If it was
a lot of code, it might make sense to slowly work dependencies though
various subsystems... but this isn't too much on the mm side, and it is
primarily support code for x86 subsystem feature so I don't mind myself
if it went via the x86 tree, with appropriate acks.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/