From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr1-f66.google.com ([209.85.221.66]:35117 "EHLO mail-wr1-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728107AbeJZXW4 (ORCPT ); Fri, 26 Oct 2018 19:22:56 -0400 Received: by mail-wr1-f66.google.com with SMTP id w5-v6so1647166wrt.2 for ; Fri, 26 Oct 2018 07:45:34 -0700 (PDT) Date: Fri, 26 Oct 2018 15:45:30 +0100 From: Phillip Potter To: Amir Goldstein Cc: viro@zeniv.linux.org.uk, linux-fsdevel@vger.kernel.org Subject: Re: [RFC][PATCH v3 01/10] fs: common implementation of file type Message-ID: <20181026144530.GA1423@pathfinder> References: <20181023201953.GA15687@pathfinder> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Wed, Oct 24, 2018 at 09:16:20AM +0300, Amir Goldstein wrote: > On Tue, Oct 23, 2018 at 11:19 PM Phillip Potter wrote: > > > > Many file systems use a copy&paste implementation > > of dirent to on-disk file type conversions. > > > > Create a common implementation to be used by file systems > > with some useful conversion helpers to reduce open coded > > file type conversions in file system code. > > > > Original patch written by Amir Goldstein. > > Looks good. > I guess you used 'git apply' or just 'patch' > What you usually do when applying someone else mostly unchanged > patches is use 'git am -s -3' so you preserve the original author and > original commit message including the Signed-of-by line. > You can edit your patch by hand to change the From: line to change the > author and add > Signed-off-by: Amir Goldstein > (you sign below me as you changed the patch last) > Dear Amir, I am almost ready to send out the new series with all the suggestions so far incorporated. Would you be happy for me to add to the commit messages slightly to mention David Sterba's point about a brief explanation? It would be the same in each case and I would keep it brief. Also, I have split the functions into a C file as per Jan Kara's suggestion. Are you still happy for me to still include your Signed-off-by tag or would you rather I keep it out until you've had a look? Thanks. Regards, Phil