Re: 2.6.29 pat issue

["Pallipadi, Venkatesh" <venkatesh.pallipadi@intel.com>]

On Thu, 2009-02-05 at 04:47 -0800, Thomas Hellström wrote:
> Hi!
> 
> The function in include/linux/mm.h:
> is_linear_pfn_mapping()
> 
> doesn't seem valid to me.
> 
> In particular, we have VMAs to graphics devices in which vma->vm_pgoff
> is non-zero (Points to an offset in the drm device node), and the VMA is
> sparsely populated with pfns pointing to uncached discontigous RAM pages.
> 
> This causes the X86 PAT code to hit the optimized path when it
> shouldn't, and issue a warning.

Only place where vm_pgoff is getting set for a PFNMAP vma is in
remap_pfn_range() which maps the entire range. vm_insert_pfn() which may
have sparsely populated ranges does not set vm_pgoff. What interface are
you using to map discontig pages, where you are seeing these errors?

> Also a question about the philosofy behind this strict checking that all
> PTEs have the same caching attributes: I guess this is only to catch
> bugs in kernel drivers that don't get this right. At the same time, now
> that also user-space VMAs are checked this will probably have a
> significant performance impact. Shouldn't this checking really live
> behind a debug define?

The result of not having the caching attribute right can be really bad
as to hang/crash the system. So, having this only in debug is not the
enough, IM0. Kernel has to enforce UC and WC caching types are
consistent at all times. And we also have to keep the indentity map and
other mappings that may be present for that address consistent.

Thanks,
Venki