David Howells <dhowells@xxxxxxxxxx> wrote: > +static inline unsigned int afs_page_dirty_resolution(void) > +{ > + long shift = PAGE_SHIFT - (__AFS_PAGE_PRIV_SHIFT - 1); This should be int, not long, in case we get an explicitly unsigned int number included in the mix (say from thp_order() with THP support). David