On Mon, Nov 21, 2016 at 09:40:06AM -0800, Laura Abbott wrote: > On 11/18/2016 06:35 AM, Mark Rutland wrote: > > On Thu, Nov 17, 2016 at 05:16:55PM -0800, Laura Abbott wrote: > >> /* Grab the vDSO code pages. */ > >> for (i = 0; i < vdso_pages; i++) > >> - vdso_pagelist[i + 1] = pfn_to_page(PHYS_PFN(__pa(&vdso_start)) + i); > >> + vdso_pagelist[i + 1] = pfn_to_page(PHYS_PFN(__pa_symbol(&vdso_start)) + i); > > > > Nit: phys_to_page() again. > > I think it makes sense to keep this one as is. It's offsetting > by pfn number and trying force phys_to_page would make it more > difficult to read. My bad; I failed to spot the + i. That sounds good to me; sorry for the noise there. Thanks, Mark. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>