[PATCH v10 10/11] quota: Add proper versioning support to fs_quota_stat

Dave Chinner david at fromorbit.com
Thu Jul 11 02:18:32 CDT 2013


On Wed, Jul 10, 2013 at 11:16:01PM -0500, Chandra Seetharaman wrote:
> On Thu, 2013-07-11 at 11:45 +1000, Dave Chinner wrote:
> > And to play Devil's advocate: it is way too late in the merge cycle
> > to make these sorts of ABI changes to a patch and test/review them
> > adequately.
> 
> There is no ABI issues even in the earlier version, it was an API
> breakage.

Old binaries would have done the wrong thing - that's an ABI
issue. I'm sorry I didn't realise this earlier.

> And with Jan's suggestion even that API breakage is being
> fixed. There is no change in API or ABI. We are just adding a new
> interface.

Yes, we are adding a new API to avoid ABI problems with
re-interpreting the old API. 

But whether it's API or ABI, it doesn't matter - my point is that
it's almost 2 weeks after the merge window was opened and adding new
userspace APIs at the last moment before the merge window closes
tends to be frowned upon....

> Old code and old binary will work as before.

Yes, but we have no new code or binaries to test the new interface,
do we?

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com



More information about the xfs mailing list