xfs
[Top] [All Lists]

Re: [PATCH 10/21] xfs: add CRC checks to remote symlinks

To: Ben Myers <bpm@xxxxxxx>
Subject: Re: [PATCH 10/21] xfs: add CRC checks to remote symlinks
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Thu, 21 Mar 2013 12:22:37 +1100
Cc: xfs@xxxxxxxxxxx
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <20130320211457.GM22182@xxxxxxx>
References: <1363091454-8852-1-git-send-email-david@xxxxxxxxxxxxx> <1363091454-8852-11-git-send-email-david@xxxxxxxxxxxxx> <20130320211457.GM22182@xxxxxxx>
User-agent: Mutt/1.5.21 (2010-09-15)
On Wed, Mar 20, 2013 at 04:14:57PM -0500, Ben Myers wrote:
> Hi Dave,
> 
> On Tue, Mar 12, 2013 at 11:30:43PM +1100, Dave Chinner wrote:
> > From: Dave Chinner <dchinner@xxxxxxxxxx>
> > 
> > Add a header to the remote symlink block, containing location and
> > owner information, as well as CRCs and LSN fields. This requires
> > verifiers to be added to the remote symlink buffers for CRC enabled
> > filesystems.
> > 
> > This also fixes a bug reading multiple block symlinks, where the second 
> > block
> > overwrites the first block when copying out the link name.
> 
> Huh.  This commit message doesn't say anything about moving symlink related
> code from xfs_vnodeops.c to a new file xfs_symlink.c.

I'm sure I added it at one point. Maybe I lost it when folding and
splitting various patches as I originally wrote this code in the
userspace libxfs tree...

I'll split it back out.

> If you're going to cut
> and paste code en masse from one file to the next, that's perfectly fine with
> me.  But please do that in a separate patch than the code you're adding for a
> feature so that I can focus on the new code in the review.
> 
> Looks like you need to add a GPL notice.

It isn't actually necessary. All code in the kernel is covered by
the GPLv2, and so notices on individual files are not technically
necessary. Just because it isn't explicitly stated in the file, it
doesn't mean the GPL does not apply to the code - the kernel is
licensed according to what is in the COPYING file (i.e. the full
GPLv2 license w/ clarifications noted by Linus), and so that is the
license that applies to the new file. All that actually matters in
individual files is the copyright statements themselves...

But, seeing as you are concerned, I'll add the boiler plates.

Cheers,

Dave.

-- 
Dave Chinner
david@xxxxxxxxxxxxx

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