Skip to content

convert remap_page_range() to remap_pfn_range() vs. 2.6.9-rc2-mm3

William Lee Irwin III lkml 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest6 of 76 patches 0 repliesNew

6 of 76 patches are in this archive. The cover letter announces 76.

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/76introduce remap_pfn_range()include/linux/mm.h mm/memory.c mm/nommu.c+18 -140 replies
2/76convert atyfb.c to use remap_pfn_range()drivers/video/aty/atyfb_base.c+2 -20 replies
3/76convert gbefb.c to use remap_pfn_range()drivers/video/gbefb.c+2 -20 replies
4/76convert sgivwfb.c to use remap_pfn_range()drivers/video/sgivwfb.c+2 -20 replies
5/76convert igafb.c to use remap_pfn_range()drivers/video/igafb.c+2 -20 replies
6/76convert zr36120.c to use remap_pfn_range()drivers/media/video/zr36120.c+2 -20 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [vm 0/76] convert remap_page_range() to remap_pfn_range() vs. 2.6.9-rc2-mm3

This series resolves a physical address overflow issue in the remap_page_range() API by replacing it with remap_pfn_range(), which accepts its physical address argument as a pfn, hence allowing the use of a single-precision physical address argument without the risk of overflow at the API boundary. The above issue has hobbled support for various 32-bit architectures, including some embedded systems (ppc440 IIRC), caused persistent portability issues for sound drivers for legacy systems (sparc32; unfortunately this patch alone does not fully resolve those), and according to John Fusco's reports, made drivers for some PCI-X hardware infeasible to port to recent ia32 PAE enterprise systems. With this patch series applied, physical address overflows on 32-bit systems caused directly by remap_page_range() are gone forever, and ca. 100LOC of cut-and-waste driver code are swept out of existence alongside them.

vs. 2.6.9-rc2-mm3 and the scheduler header cleanups. The parts touching mm.h should apply with just offsets if those aren't applied beforehand. Successfully tested on x86-64.

-- wli

P.S.: The existing solution to the sparc32 issue was to pass a double
precision representation of the physical address as 2 single- precision arguments in an API (io_remap_page_range()) whose argument corresponding to those two was a single single- precision argument on most/all other architectures. The sparc32-specific issue requires more work beyond these patches to rectify. The most apparent consequence of the API skew is that drivers don't compile on sparc32 when they use io_remap_page_range() due to passing insufficient arguments, or vice-versa for drivers originally written for sparc32.