From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934289AbdJRDTE (ORCPT ); Tue, 17 Oct 2017 23:19:04 -0400 Received: from ozlabs.org ([103.22.144.67]:47685 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933296AbdJRDTC (ORCPT ); Tue, 17 Oct 2017 23:19:02 -0400 From: Michael Ellerman To: Mimi Zohar , Julia Lawall , Dan Carpenter Cc: Jarkko Sakkinen , SF Markus Elfring , linux-integrity@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Andy Shevchenko , Benjamin Herrenschmidt , Corentin Labbe , Jason Gunthorpe , Jerry Snitselaar , Kenneth Goldman , Nayna Jain , Paul Mackerras , Peter =?utf-8?Q?H=C3=BCwe?= , Stefan Berger , LKML , kernel-janitors@vger.kernel.org Subject: Re: [PATCH 0/4] char-TPM: Adjustments for ten function implementations In-Reply-To: <1508241146.4234.16.camel@linux.vnet.ibm.com> References: <1d3516a2-a8e6-9e95-d438-f115fac84c7f@users.sourceforge.net> <20171016183139.otyh3m5c5yurtmow@linux.intel.com> <20171016183512.3bz6x4b6lbhpbkje@linux.intel.com> <20171017085124.pkrjzghcf5wmcydc@mwanda> <20171017094434.csarc5fw5d32u64w@mwanda> <1508241146.4234.16.camel@linux.vnet.ibm.com> Date: Wed, 18 Oct 2017 14:18:57 +1100 Message-ID: <87lgk9i1ni.fsf@concordia.ellerman.id.au> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Mimi Zohar writes: > On Tue, 2017-10-17 at 12:11 +0200, Julia Lawall wrote: >> On Tue, 17 Oct 2017, Dan Carpenter wrote: >> > On Tue, Oct 17, 2017 at 10:56:42AM +0200, Julia Lawall wrote: >> > > On Tue, 17 Oct 2017, Dan Carpenter wrote: >> > > > On Mon, Oct 16, 2017 at 09:35:12PM +0300, Jarkko Sakkinen wrote: >> > > > > >> > > > > A minor complaint: all commits are missing "Fixes:" tag. >> > > > > >> > > > >> > > > Fixes is only for bug fixes. These don't fix any bugs. >> > > >> > > 0-day seems to put Fixes for everything. Should they be removed when the >> > > old code is undesirable but doesn't actually cause a crash, eg out of date >> > > API. >> > >> > Yeah, I feel like Fixes tags don't belong for API updates and cleanups. >> >> OK, I will remove them from the patches that go through me where they >> don't seem appropriate. > > The "Fixes" tag is an indication that the patch should be backported. No it's not that strong. It's an indication that the patch fixes another commit, which may or may not mean it should be backported depending on the preferences of the backporter. If it *does* need backporting then the Fixes tag helps identify where it should go. The doco is actually pretty well worded IMO: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst#n183 If your patch fixes a bug in a specific commit, e.g. you found an issue using ``git bisect``, please use the 'Fixes:' tag with the first 12 characters of the SHA-1 ID, and the one line summary. and: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst#n602 A Fixes: tag indicates that the patch fixes an issue in a previous commit. It is used to make it easy to determine where a bug originated, which can help review a bug fix. This tag also assists the stable kernel team in determining which stable kernel versions should receive your fix. This is the preferred method for indicating a bug fixed by the patch. See :ref:`describe_changes` for more details. cheers From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Ellerman Date: Wed, 18 Oct 2017 03:18:57 +0000 Subject: Re: [PATCH 0/4] char-TPM: Adjustments for ten function implementations Message-Id: <87lgk9i1ni.fsf@concordia.ellerman.id.au> List-Id: References: <1d3516a2-a8e6-9e95-d438-f115fac84c7f@users.sourceforge.net> <20171016183139.otyh3m5c5yurtmow@linux.intel.com> <20171016183512.3bz6x4b6lbhpbkje@linux.intel.com> <20171017085124.pkrjzghcf5wmcydc@mwanda> <20171017094434.csarc5fw5d32u64w@mwanda> <1508241146.4234.16.camel@linux.vnet.ibm.com> In-Reply-To: <1508241146.4234.16.camel@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Mimi Zohar , Julia Lawall , Dan Carpenter Cc: Jarkko Sakkinen , SF Markus Elfring , linux-integrity@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, Andy Shevchenko , Benjamin Herrenschmidt , Corentin Labbe , Jason Gunthorpe , Jerry Snitselaar , Kenneth Goldman , Nayna Jain , Paul Mackerras , Peter =?utf-8?Q?H=C3=BCwe?= , Stefan Berger , LKML , kernel-janitors@vger.kernel.org Mimi Zohar writes: > On Tue, 2017-10-17 at 12:11 +0200, Julia Lawall wrote: >> On Tue, 17 Oct 2017, Dan Carpenter wrote: >> > On Tue, Oct 17, 2017 at 10:56:42AM +0200, Julia Lawall wrote: >> > > On Tue, 17 Oct 2017, Dan Carpenter wrote: >> > > > On Mon, Oct 16, 2017 at 09:35:12PM +0300, Jarkko Sakkinen wrote: >> > > > > >> > > > > A minor complaint: all commits are missing "Fixes:" tag. >> > > > > >> > > > >> > > > Fixes is only for bug fixes. These don't fix any bugs. >> > > >> > > 0-day seems to put Fixes for everything. Should they be removed when the >> > > old code is undesirable but doesn't actually cause a crash, eg out of date >> > > API. >> > >> > Yeah, I feel like Fixes tags don't belong for API updates and cleanups. >> >> OK, I will remove them from the patches that go through me where they >> don't seem appropriate. > > The "Fixes" tag is an indication that the patch should be backported. No it's not that strong. It's an indication that the patch fixes another commit, which may or may not mean it should be backported depending on the preferences of the backporter. If it *does* need backporting then the Fixes tag helps identify where it should go. The doco is actually pretty well worded IMO: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst#n183 If your patch fixes a bug in a specific commit, e.g. you found an issue using ``git bisect``, please use the 'Fixes:' tag with the first 12 characters of the SHA-1 ID, and the one line summary. and: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst#n602 A Fixes: tag indicates that the patch fixes an issue in a previous commit. It is used to make it easy to determine where a bug originated, which can help review a bug fix. This tag also assists the stable kernel team in determining which stable kernel versions should receive your fix. This is the preferred method for indicating a bug fixed by the patch. See :ref:`describe_changes` for more details. cheers