On 31 January 2013 22:10, Rolf Eike Beer <eike-kernel@xxxxxxxxx> wrote: > H. Peter Anvin wrote: >> On 01/31/2013 01:42 PM, James Bottomley wrote: >> > On Thu, 2013-01-31 at 13:29 -0800, H. Peter Anvin wrote: >> >> On 01/31/2013 12:26 PM, James Bottomley wrote: >> >>> [adding autofs list and maintainer for their perusal and ack, although >> >>> this is an obvious fix to me >> >>> >> >>> James] >> >> >> >> The #if list probably should be inverted, and only specific platforms >> >> should have unsigned long... >> > >> > The whitelist is 64 bit only platforms, like ia64, isn't it? So yes, >> > they're far less numerous. >> >> Correct. It is basically the 64-bit platforms without any 32-bit >> userspace, like Alpha and IA64. > > Arm64? It supports 32-bit user-space (CONFIG_COMPAT). But we missed this #ifdef for arm64. I'm not familiar with the autofs subsystem but at a first look it seems that autofs_wqt_t types are not shared with user space as part of a structure. It can come as an argument for an ioctl command but as long as we don't expect this type to hold data larger than 32-bit it should be ok. -- Catalin -- To unsubscribe from this list: send the line "unsubscribe linux-parisc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html