xfs
[Top] [All Lists]

Re: [PATCH 0/9] xfs: error sign fixes for 3.15-rc6

To: Jeff Liu <jeff.liu@xxxxxxxxxx>
Subject: Re: [PATCH 0/9] xfs: error sign fixes for 3.15-rc6
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Thu, 15 May 2014 07:28:22 +1000
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <53731B07.9040606@xxxxxxxxxx>
References: <1399971373-6242-1-git-send-email-david@xxxxxxxxxxxxx> <53731B07.9040606@xxxxxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Wed, May 14, 2014 at 03:28:07PM +0800, Jeff Liu wrote:
> Hi Dave,
> 
> On 05/13/2014 04:56 PM, Dave Chinner wrote:
> > Hi folks,
> > 
> > I've been working through the mess that is the
> > error sign impedance mismatch between the core XFS code and the rest
> > of the kernel. I'm about half way through the codebase, and I've
> > found a bunch of incorrect error signs throughout the VFS interface
> > layers.
> > 
> > These were effectively all found by inspection, and is
> > further evidence that we need to convert all of XFS to negative
> > errors as quickly as possible. 
> 
> Would you like to find a volunteer to deal with that? I have a colleague
> is trying to study XFS these days, looks this is a good task to him.

Thanks for the offer, but I've already done half the conversion of
the codebase.  Hence I may as well finish it.  Indeed, The sooner
I've done it, the sooner we can get it into the dev tree for
testing....

> BTW, before doing negative errors converting, should we have
> libxfs infrastructure support at first?

Makes no difference - the libxfs structural changes are just moving
files about and futzing with makefiles. It'll only take me an hour
to redo those patches from scratch, so it doesn't matter what state
the code base is in to start with. I'd prefer that the kernel libxfs
uses negative errors from the start, though ;)

Cheers,

Dave.
-- 
Dave Chinner
david@xxxxxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>