Message ID | 6d40e289-13ef-26fa-24d6-6b8e052d3a76@redhat.com (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
On Thu, Dec 22, 2016 at 09:28:50AM -0600, Eric Sandeen wrote: > on 32-bit builds: > > mremap_f can't turn a long long into a pointer, and > dump_dirent needs proper %llx & a cast for u64 args > as is done elsewhere. > > Signed-off-by: Eric Sandeen <sandeen@redhat.com> > --- Looks Ok to me: Reviewed-by: Brian Foster <bfoster@redhat.com> > > diff --git a/io/mmap.c b/io/mmap.c > index dc188d0..6f1d330 100644 > --- a/io/mmap.c > +++ b/io/mmap.c > @@ -628,8 +628,8 @@ mremap_f( > switch (c) { > case 'f': > flags = MREMAP_FIXED|MREMAP_MAYMOVE; > - new_addr = (void *)cvtnum(blocksize, sectsize, > - optarg); > + new_addr = (void *)(unsigned long)cvtnum(blocksize, > + sectsize, optarg); > break; > case 'm': > flags = MREMAP_MAYMOVE; > diff --git a/io/readdir.c b/io/readdir.c > index 151b72e..b868d1b 100644 > --- a/io/readdir.c > +++ b/io/readdir.c > @@ -71,9 +71,10 @@ dump_dirent( > long long offset, > struct dirent *dirent) > { > - printf("%08llx: d_ino: 0x%08lx", offset, dirent->d_ino); > + printf("%08llx: d_ino: 0x%08llx", offset, > + (unsigned long long)dirent->d_ino); > #ifdef _DIRENT_HAVE_D_OFF > - printf(" d_off: 0x%08lx", dirent->d_off); > + printf(" d_off: 0x%08llx", (unsigned long long)dirent->d_off); > #endif > #ifdef _DIRENT_HAVE_D_RECLEN > printf(" d_reclen: 0x%x", dirent->d_reclen); > > -- > To unsubscribe from this list: send the line "unsubscribe linux-xfs" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/io/mmap.c b/io/mmap.c index dc188d0..6f1d330 100644 --- a/io/mmap.c +++ b/io/mmap.c @@ -628,8 +628,8 @@ mremap_f( switch (c) { case 'f': flags = MREMAP_FIXED|MREMAP_MAYMOVE; - new_addr = (void *)cvtnum(blocksize, sectsize, - optarg); + new_addr = (void *)(unsigned long)cvtnum(blocksize, + sectsize, optarg); break; case 'm': flags = MREMAP_MAYMOVE; diff --git a/io/readdir.c b/io/readdir.c index 151b72e..b868d1b 100644 --- a/io/readdir.c +++ b/io/readdir.c @@ -71,9 +71,10 @@ dump_dirent( long long offset, struct dirent *dirent) { - printf("%08llx: d_ino: 0x%08lx", offset, dirent->d_ino); + printf("%08llx: d_ino: 0x%08llx", offset, + (unsigned long long)dirent->d_ino); #ifdef _DIRENT_HAVE_D_OFF - printf(" d_off: 0x%08lx", dirent->d_off); + printf(" d_off: 0x%08llx", (unsigned long long)dirent->d_off); #endif #ifdef _DIRENT_HAVE_D_RECLEN printf(" d_reclen: 0x%x", dirent->d_reclen);
on 32-bit builds: mremap_f can't turn a long long into a pointer, and dump_dirent needs proper %llx & a cast for u64 args as is done elsewhere. Signed-off-by: Eric Sandeen <sandeen@redhat.com> --- -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html