RE: [RFC] remap_pfn_range: store vm_pgoff for alllinear_over_vma_region mappings

From: Pallipadi, Venkatesh
Date: Fri Oct 24 2008 - 00:14:13 EST




>-----Original Message-----
>From: Nick Piggin [mailto:npiggin@xxxxxxx]
>Sent: Thursday, October 23, 2008 6:08 PM
>To: Pallipadi, Venkatesh
>Cc: hugh@xxxxxxxxxxx; apkm@xxxxxxxxxxxxxxxxxxxxx; Ingo Molnar;
>linux-kernel
>Subject: Re: [RFC] remap_pfn_range: store vm_pgoff for all
>linear_over_vma_region mappings
>
>On Thu, Oct 23, 2008 at 03:09:14PM -0700, Venki Pallipadi wrote:
>>
>> While working on x86 PAT, we are having some hurdles with tracking
>> remap_pfn_range() regions, as later we do not have any
>information to say
>> whether that PFNMAP mapping is linear for the entire vma range or
>> it is smaller granularity regions within the vma.
>>
>> A simple solution to this is to use vm_pgoff as an indicator for
>> linear mapping over the vma region. Currently, remap_pfn_range
>> only sets vm_pgoff only for COW mappings. Below patch changes the
>> logic and sets the vm_pgoff irrespective of COW.
>>
>> >From our understanding of the code, this should not break anyone.
>> Just sending it as an RFC to get feedback on whether it is OK to do
>> something like this or are there any corner cases that we may
>> break or watch out for.
>>
>> Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@xxxxxxxxx>
>> Signed-off-by: Suresh Siddha <suresh.b.siddha@xxxxxxxxx>
>
>Hmm, I can'tthink of why it would break, but I would like to see an
>entire patch. Obviously you won't try to decode this vma stuff
>directly from the PAT code, but just call mm helpers... ?
>

Yes. We are thinking of it as a call similar to is_cow_mapping().
May be is_linear_vma_mapping() or something like that. We should
have the rest of the patches by next week.

Thanks,
Venki
--
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/