Message ID | 172780126049.3586479.7813790327650448381.stgit@frogsfrogsfrogs (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | [1/2] src/fiexchange.h: add the start-commit/commit-range ioctls | expand |
On Tue, Oct 01, 2024 at 09:49:27AM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@kernel.org> > > Update this test to check the ioctl structure for XFS_IOC_COMMIT_RANGE. Meh. Can we please not add more to xfs/122, as that's alway just a pain? We can just static_assert the size in xfsprogs (or the xfstests code using it) instead of this mess.
On Tue, Oct 01, 2024 at 10:45:50PM -0700, Christoph Hellwig wrote: > On Tue, Oct 01, 2024 at 09:49:27AM -0700, Darrick J. Wong wrote: > > From: Darrick J. Wong <djwong@kernel.org> > > > > Update this test to check the ioctl structure for XFS_IOC_COMMIT_RANGE. > > Meh. Can we please not add more to xfs/122, as that's alway just > a pain? We can just static_assert the size in xfsprogs (or the > xfstests code using it) instead of this mess. Oh right, we had a plan to autotranslate the xfs/122 stuff to xfs_ondisk.h didn't we... I'll put that back on my list. --D
On Wed, Oct 02, 2024 at 03:47:00PM -0700, Darrick J. Wong wrote: > Oh right, we had a plan to autotranslate the xfs/122 stuff to > xfs_ondisk.h didn't we... I'll put that back on my list. xfs_ondisk.h should be fully covered. You pointed out that were are missing ioctls, though. Let me find some time to add those as well.
On Wed, Oct 02, 2024 at 03:47:00PM -0700, Darrick J. Wong wrote: > On Tue, Oct 01, 2024 at 10:45:50PM -0700, Christoph Hellwig wrote: > > On Tue, Oct 01, 2024 at 09:49:27AM -0700, Darrick J. Wong wrote: > > > From: Darrick J. Wong <djwong@kernel.org> > > > > > > Update this test to check the ioctl structure for XFS_IOC_COMMIT_RANGE. > > > > Meh. Can we please not add more to xfs/122, as that's alway just > > a pain? We can just static_assert the size in xfsprogs (or the > > xfstests code using it) instead of this mess. > > Oh right, we had a plan to autotranslate the xfs/122 stuff to > xfs_ondisk.h didn't we... I'll put that back on my list. Hi Darrick, Do you want to have this patch at first, or just wait for your next version which does the "autotranslate"? Thanks, Zorro > > --D >
On Fri, Oct 11, 2024 at 02:28:58PM +0800, Zorro Lang wrote: > On Wed, Oct 02, 2024 at 03:47:00PM -0700, Darrick J. Wong wrote: > > On Tue, Oct 01, 2024 at 10:45:50PM -0700, Christoph Hellwig wrote: > > > On Tue, Oct 01, 2024 at 09:49:27AM -0700, Darrick J. Wong wrote: > > > > From: Darrick J. Wong <djwong@kernel.org> > > > > > > > > Update this test to check the ioctl structure for XFS_IOC_COMMIT_RANGE. > > > > > > Meh. Can we please not add more to xfs/122, as that's alway just > > > a pain? We can just static_assert the size in xfsprogs (or the > > > xfstests code using it) instead of this mess. > > > > Oh right, we had a plan to autotranslate the xfs/122 stuff to > > xfs_ondisk.h didn't we... I'll put that back on my list. > > Hi Darrick, > > Do you want to have this patch at first, or just wait for your > next version which does the "autotranslate"? Let's drop this for now, machine-converting xfs/122 to xfs_ondisk.h wasn't as hard as I thought it might be. Would you be ok with merging the fiexchange.h patch and the fsstress funshare patch this week? --D > Thanks, > Zorro > > > > > --D > > > >
On Fri, Oct 11, 2024 at 11:19:20AM -0700, Darrick J. Wong wrote: > On Fri, Oct 11, 2024 at 02:28:58PM +0800, Zorro Lang wrote: > > On Wed, Oct 02, 2024 at 03:47:00PM -0700, Darrick J. Wong wrote: > > > On Tue, Oct 01, 2024 at 10:45:50PM -0700, Christoph Hellwig wrote: > > > > On Tue, Oct 01, 2024 at 09:49:27AM -0700, Darrick J. Wong wrote: > > > > > From: Darrick J. Wong <djwong@kernel.org> > > > > > > > > > > Update this test to check the ioctl structure for XFS_IOC_COMMIT_RANGE. > > > > > > > > Meh. Can we please not add more to xfs/122, as that's alway just > > > > a pain? We can just static_assert the size in xfsprogs (or the > > > > xfstests code using it) instead of this mess. > > > > > > Oh right, we had a plan to autotranslate the xfs/122 stuff to > > > xfs_ondisk.h didn't we... I'll put that back on my list. > > > > Hi Darrick, > > > > Do you want to have this patch at first, or just wait for your > > next version which does the "autotranslate"? > > Let's drop this for now, machine-converting xfs/122 to xfs_ondisk.h > wasn't as hard as I thought it might be. Would you be ok with merging > the fiexchange.h patch and the fsstress funshare patch this week? Sure, if you hope so :) > > --D > > > Thanks, > > Zorro > > > > > > > > --D > > > > > > > >
diff --git a/tests/xfs/122.out b/tests/xfs/122.out index 60d8294551..4dc7d7d0a3 100644 --- a/tests/xfs/122.out +++ b/tests/xfs/122.out @@ -76,6 +76,7 @@ sizeof(struct xfs_bulk_ireq) = 64 sizeof(struct xfs_bulkstat) = 192 sizeof(struct xfs_bulkstat_req) = 64 sizeof(struct xfs_clone_args) = 32 +sizeof(struct xfs_commit_range) = 88 sizeof(struct xfs_cud_log_format) = 16 sizeof(struct xfs_cui_log_format) = 16 sizeof(struct xfs_da3_blkinfo) = 56