[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240807151732.GH6051@frogsfrogsfrogs>
Date: Wed, 7 Aug 2024 08:17:32 -0700
From: "Darrick J. Wong" <djwong@...nel.org>
To: John Garry <john.g.garry@...cle.com>
Cc: chandan.babu@...cle.com, dchinner@...hat.com, hch@....de,
viro@...iv.linux.org.uk, brauner@...nel.org, jack@...e.cz,
linux-xfs@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-fsdevel@...r.kernel.org, catherine.hoang@...cle.com,
martin.petersen@...cle.com
Subject: Re: [PATCH v3 14/14] xfs: Enable file data forcealign feature
On Wed, Aug 07, 2024 at 02:50:18PM +0100, John Garry wrote:
> On 06/08/2024 20:43, Darrick J. Wong wrote:
> > On Thu, Aug 01, 2024 at 04:30:57PM +0000, John Garry wrote:
> > > From: "Darrick J. Wong"<djwong@...nel.org>
> > >
> > > Enable this feature.
> > >
> > > Signed-off-by: "Darrick J. Wong"<djwong@...nel.org>
> > > Signed-off-by: John Garry<john.g.garry@...cle.com>
> > You really ought to add your own Reviewed-by tag here...
>
> ok, if you prefer. Normally I think it's implied by the context and
> signed-off tag.
SoB means only that either you've written the patch yourself and have
the right to submit it, or that you have the right to pass on a patch
written by someone else who certified the same thing.
An actual code review should be signalled explicitly, not implied by
context.
--D
> Thanks,
> John
>
Powered by blists - more mailing lists