| To: | "Darrick J. Wong" <darrick.wong@xxxxxxxxxx> |
|---|---|
| Subject: | Re: [PATCH] xfs: in _attrlist_by_handle, copy the cursor back to userspace |
| From: | Dave Chinner <david@xxxxxxxxxxxxx> |
| Date: | Wed, 3 Aug 2016 07:41:01 +1000 |
| Cc: | Christoph Hellwig <hch@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20160802154035.GI8590@xxxxxxxxxxxxxxxx> |
| References: | <20160802035651.GB8593@xxxxxxxxxxxxxxxx> <20160802122517.GD31492@xxxxxxxxxxxxx> <20160802154035.GI8590@xxxxxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Tue, Aug 02, 2016 at 08:40:35AM -0700, Darrick J. Wong wrote: > On Tue, Aug 02, 2016 at 05:25:17AM -0700, Christoph Hellwig wrote: > > On Mon, Aug 01, 2016 at 08:56:51PM -0700, Darrick J. Wong wrote: > > > When we're iterating inode xattrs by handle, we have to copy the > > > cursor back to userspace so that a subsequent invocation actually > > > retrieves subsequent contents. > > > > Testcase? > > Found it while continuing development of xfs_scrub. I'll send along the > xfstest patch when I've finished polishing it. (It was harder than usual > since > xfs_io doesn't /use/ the attr-by-handle interface... nothing does.) It was (and probably still is) used by SGI's HSM. I thought there was some coverage of the interface in the dmapi part of the xfstests suite, but perhaps it's only tested by SGI's internal hsm test suite... Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [PATCH 20/47] xfs: increase XFS_BTREE_MAXLEVELS to fit the rmapbt, Brian Foster |
|---|---|
| Next by Date: | Re: [PATCH 11/47] xfs: move deferred operations into a separate file, Dave Chinner |
| Previous by Thread: | Re: [PATCH] xfs: in _attrlist_by_handle, copy the cursor back to userspace, Darrick J. Wong |
| Next by Thread: | Re: [PATCH] xfs: in _attrlist_by_handle, copy the cursor back to userspace, Mark Tinguely |
| Indexes: | [Date] [Thread] [Top] [All Lists] |