From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932893AbeCJMrD (ORCPT ); Sat, 10 Mar 2018 07:47:03 -0500 Received: from mga07.intel.com ([134.134.136.100]:14086 "EHLO mga07.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752136AbeCJMrB (ORCPT ); Sat, 10 Mar 2018 07:47:01 -0500 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.47,451,1515484800"; d="scan'208";a="40805599" Message-ID: Subject: Re: [PATCH 1/3 RESEND] tpm: add longer timeouts for creation commands. From: Jarkko Sakkinen To: Mimi Zohar , Jonathan Corbet Cc: James Bottomley , "Winkler, Tomas" , Jason Gunthorpe , "Usyskin, Alexander" , "linux-integrity@vger.kernel.org" , linux-security-module@vger.kernel.org, linux-kernel@vger.kernel.org Date: Sat, 10 Mar 2018 14:46:55 +0200 In-Reply-To: <1520440503.10396.534.camel@linux.vnet.ibm.com> References: <20180304121205.16934-1-tomas.winkler@intel.com> <20180305125642.GA3425@linux.intel.com> <5B8DA87D05A7694D9FA63FD143655C1B9422E21E@hasmsx108.ger.corp.intel.com> <20180306074932.GB3624@linux.intel.com> <5B8DA87D05A7694D9FA63FD143655C1B9422EFFE@hasmsx108.ger.corp.intel.com> <1520353933.5393.21.camel@HansenPartnership.com> <1520361396.10396.396.camel@linux.vnet.ibm.com> <20180307085454.2c820a13@lwn.net> <1520440503.10396.534.camel@linux.vnet.ibm.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.27.90-1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2018-03-07 at 11:35 -0500, Mimi Zohar wrote: > On Wed, 2018-03-07 at 08:54 -0700, Jonathan Corbet wrote: > > On Tue, 06 Mar 2018 13:36:36 -0500 > > Mimi Zohar wrote: > > > > > I've heard that some maintainers are moving away from cover letters, > > > since they are not include in the git repo and are lost. > > > > If I get a patch series with a cover letter that should be preserved, I > > apply the series in a branch then do a no-ff merge; the cover letter can > > then go into the merge commit. There's no reason why cover letters need to > > be lost. > > Thanks, Jon. That sounds like a really, good idea. > > Some maintainers are saying to put the Changelog after the "---" so > that it isn't included in the patch description. > > One of the reasons for including the Changelog in the patch > description, is to credit people with bug fixes, important rework of > the patch, etc. This is a really good point. By adding the cover letter to the GIT history helps to better track people who to thank or blame :-) /Jarkko From mboxrd@z Thu Jan 1 00:00:00 1970 From: jarkko.sakkinen@linux.intel.com (Jarkko Sakkinen) Date: Sat, 10 Mar 2018 14:46:55 +0200 Subject: [PATCH 1/3 RESEND] tpm: add longer timeouts for creation commands. In-Reply-To: <1520440503.10396.534.camel@linux.vnet.ibm.com> References: <20180304121205.16934-1-tomas.winkler@intel.com> <20180305125642.GA3425@linux.intel.com> <5B8DA87D05A7694D9FA63FD143655C1B9422E21E@hasmsx108.ger.corp.intel.com> <20180306074932.GB3624@linux.intel.com> <5B8DA87D05A7694D9FA63FD143655C1B9422EFFE@hasmsx108.ger.corp.intel.com> <1520353933.5393.21.camel@HansenPartnership.com> <1520361396.10396.396.camel@linux.vnet.ibm.com> <20180307085454.2c820a13@lwn.net> <1520440503.10396.534.camel@linux.vnet.ibm.com> Message-ID: To: linux-security-module@vger.kernel.org List-Id: linux-security-module.vger.kernel.org On Wed, 2018-03-07 at 11:35 -0500, Mimi Zohar wrote: > On Wed, 2018-03-07 at 08:54 -0700, Jonathan Corbet wrote: > > On Tue, 06 Mar 2018 13:36:36 -0500 > > Mimi Zohar wrote: > > > > > I've heard that some maintainers are moving away from cover letters, > > > since they are not include in the git repo and are lost. > > > > If I get a patch series with a cover letter that should be preserved, I > > apply the series in a branch then do a no-ff merge; the cover letter can > > then go into the merge commit. There's no reason why cover letters need to > > be lost. > > Thanks, Jon. That sounds like a really, good idea. > > Some maintainers are saying to put the Changelog after the "---" so > that it isn't included in the patch description. > > One of the reasons for including the Changelog in the patch > description, is to credit people with bug fixes, important rework of > the patch, etc. This is a really good point. By adding the cover letter to the GIT history helps to better track people who to thank or blame :-) /Jarkko -- To unsubscribe from this list: send the line "unsubscribe linux-security-module" in the body of a message to majordomo at vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html