From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id B90217CBF for ; Wed, 20 Mar 2013 20:22:40 -0500 (CDT) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay1.corp.sgi.com (Postfix) with ESMTP id 944078F8065 for ; Wed, 20 Mar 2013 18:22:40 -0700 (PDT) Received: from ipmail06.adl2.internode.on.net (ipmail06.adl2.internode.on.net [150.101.137.129]) by cuda.sgi.com with ESMTP id d54CFofyfHXBvBZG for ; Wed, 20 Mar 2013 18:22:38 -0700 (PDT) Date: Thu, 21 Mar 2013 12:22:37 +1100 From: Dave Chinner Subject: Re: [PATCH 10/21] xfs: add CRC checks to remote symlinks Message-ID: <20130321012237.GI17758@dastard> References: <1363091454-8852-1-git-send-email-david@fromorbit.com> <1363091454-8852-11-git-send-email-david@fromorbit.com> <20130320211457.GM22182@sgi.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20130320211457.GM22182@sgi.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Ben Myers Cc: xfs@oss.sgi.com 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 > > > > 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@fromorbit.com _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs