xfs
[Top] [All Lists]

Re: VFS case-insenstive dcache code (was Re: [PATCH 0/4] XFS: ASCII case

To: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject: Re: VFS case-insenstive dcache code (was Re: [PATCH 0/4] XFS: ASCII case-insensitivity support)
From: Anton Altaparmakov <aia21@xxxxxxxxx>
Date: Thu, 15 May 2008 14:37:45 +0100
Cc: Barry Naujok <bnaujok@xxxxxxx>, xfs@xxxxxxxxxxx, linux-fsdevel@xxxxxxxxxxxxxxx
In-reply-to: <20080515043546.GA22054@xxxxxxxxxxxxx>
References: <20080514075244.738514733@xxxxxxxxxxxxxxxxxxxxxxx> <20080514084344.GA4333@xxxxxxxxxxxxx> <op.ua57wpji3jf8g2@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20080515043546.GA22054@xxxxxxxxxxxxx>
Sender: xfs-bounce@xxxxxxxxxxx
Hi,

On 15 May 2008, at 05:35, Christoph Hellwig wrote:
On Thu, May 15, 2008 at 11:03:51AM +1000, Barry Naujok wrote:
On Wed, 14 May 2008 18:43:44 +1000, Christoph Hellwig <hch@xxxxxxxxxxxxx >
wrote:

All patches looks good to me, but please add a comment describing the
conditional d_reash in the create case.

I'm separating the "return name" patch into two patches: one for the dcache
code which Anton wrote and one with the XFS specific code.

So, how does one apply the Signed-off line and copyright for the dcache
code (below - possibly mangled)?

In this from it should be From: and Signed-off-by: you as you factored
it out of the ntfs code.  Just mention it's originally Anton's ntfs
code in the description.  Of course getting Anton to sign off on it
and prepare a patch to use it in ntfs would be even better.

You can certainly add SOB:

Signed-off-by: Anton Altaparmakov <aia21@xxxxxxxxxx> to the patch. It looks good to me.

Once Linus has applied this I will try and get NTFS switched to using the new function. I have got a few other patches that need applying for NTFS anyway so I will roll them all up in one go into the NTFS git tree and get Linus to apply them...

Best regards,

        Anton
--
Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @)
Unix Support, Computing Service, University of Cambridge, CB2 3QH, UK
Linux NTFS maintainer, http://www.linux-ntfs.org/


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