Re: [PATCH 1/3 RESEND] tpm: add longer timeouts for creation commands.
From: Jason Gunthorpe
Date: Tue Mar 06 2018 - 16:59:18 EST
On Tue, Mar 06, 2018 at 01:36:36PM -0500, Mimi Zohar wrote:
> On Tue, 2018-03-06 at 08:32 -0800, James Bottomley wrote:
> > On Tue, 2018-03-06 at 08:06 +0000, Winkler, Tomas wrote:
> > > >
> > > >
> > > > On Mon, Mar 05, 2018 at 01:09:09PM +0000, Winkler, Tomas wrote:
> > > > >
> > > > > Why you need cover letter?ÂÂWhat are u missing in the patch
> > > > > description
> > > >
> > > > If you submit a *patch set* I *require* a cover letter, yes.
> > >
> > > It's good but it is not must, you are inventing your own rules.
> >
> > As long as the Maintainer is the gatekeeper, you're not going to get
> > very far with this argument. ÂThe fact is that a lot of subsystems have
> > varying rules; often undocumented, some of which are even in conflict,
> > like alphabetic vs reverse christmas tree format for includes.
> >
> > A cover letter is actually one of the more uniform rules. ÂIt's
> > referred to in submitting patches, but not actually documented there.
>
> I've heard that some maintainers are moving away from cover letters,
> since they are not include in the git repo and are lost. ÂI've seen
> Andrew Morton cut and paste the cover letter in the first patch
> description of the patch set.
Andrew has a workflow unlike any other I've seen..
In my view the cover letter should explain why the maintainer should
apply the series, and give any guidance to the review process.
Not duplicate information that belongs in the patch comments. It
shouldn't explain how/why the patch(es) works, etc.
Jason