* [PATCH] tpm: remove tpm_dev_wq_lock
@ 2019-02-11 10:58 Sebastian Andrzej Siewior
2019-02-11 14:11 ` Jarkko Sakkinen
` (2 more replies)
0 siblings, 3 replies; 17+ messages in thread
From: Sebastian Andrzej Siewior @ 2019-02-11 10:58 UTC (permalink / raw)
To: linux-integrity
Cc: tglx, Jason Gunthorpe, Sebastian Andrzej Siewior, Philip Tricca,
Tadeusz Struk, Jarkko Sakkinen
Added in commit
9e1b74a63f776 ("tpm: add support for nonblocking operation")
but never actually used it.
Cc: Philip Tricca <philip.b.tricca@intel.com>
Cc: Tadeusz Struk <tadeusz.struk@intel.com>
Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
---
drivers/char/tpm/tpm-dev-common.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/char/tpm/tpm-dev-common.c b/drivers/char/tpm/tpm-dev-common.c
index 5eecad233ea1d..eca96e8c669c9 100644
--- a/drivers/char/tpm/tpm-dev-common.c
+++ b/drivers/char/tpm/tpm-dev-common.c
@@ -25,7 +25,6 @@
#include "tpm-dev.h"
static struct workqueue_struct *tpm_dev_wq;
-static DEFINE_MUTEX(tpm_dev_wq_lock);
static void tpm_async_work(struct work_struct *work)
{
--
2.20.1
^ permalink raw reply related [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-02-11 10:58 [PATCH] tpm: remove tpm_dev_wq_lock Sebastian Andrzej Siewior
@ 2019-02-11 14:11 ` Jarkko Sakkinen
2019-10-10 16:03 ` Sebastian Andrzej Siewior
2019-02-12 1:45 ` Tadeusz Struk
2020-12-14 22:07 ` [PATCH V2] tpm: Remove tpm_dev_wq_lock Thomas Gleixner
2 siblings, 1 reply; 17+ messages in thread
From: Jarkko Sakkinen @ 2019-02-11 14:11 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: linux-integrity, tglx, Jason Gunthorpe, Philip Tricca, Tadeusz Struk
On Mon, Feb 11, 2019 at 11:58:35AM +0100, Sebastian Andrzej Siewior wrote:
> Added in commit
>
> 9e1b74a63f776 ("tpm: add support for nonblocking operation")
>
> but never actually used it.
>
> Cc: Philip Tricca <philip.b.tricca@intel.com>
> Cc: Tadeusz Struk <tadeusz.struk@intel.com>
> Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
You should use Fixes-tag e.g.
Fixes: <12 first chars from SHA-1> ("<short summary>")
/Jarkko
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-02-11 10:58 [PATCH] tpm: remove tpm_dev_wq_lock Sebastian Andrzej Siewior
2019-02-11 14:11 ` Jarkko Sakkinen
@ 2019-02-12 1:45 ` Tadeusz Struk
2020-12-14 22:07 ` [PATCH V2] tpm: Remove tpm_dev_wq_lock Thomas Gleixner
2 siblings, 0 replies; 17+ messages in thread
From: Tadeusz Struk @ 2019-02-12 1:45 UTC (permalink / raw)
To: Sebastian Andrzej Siewior, linux-integrity
Cc: tglx, Jason Gunthorpe, Philip Tricca, Jarkko Sakkinen
On 2/11/19 2:58 AM, Sebastian Andrzej Siewior wrote:
> Added in commit
>
> 9e1b74a63f776 ("tpm: add support for nonblocking operation")
>
> but never actually used it.
It was used in one of the early versions of this patch
https://patchwork.kernel.org/patch/10559151/
Not needed later.
Acked-by: Tadeusz Struk <tadeusz.struk@intel.com>
Thanks,
--
Tadeusz
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-02-11 14:11 ` Jarkko Sakkinen
@ 2019-10-10 16:03 ` Sebastian Andrzej Siewior
2019-10-14 19:39 ` Jarkko Sakkinen
0 siblings, 1 reply; 17+ messages in thread
From: Sebastian Andrzej Siewior @ 2019-10-10 16:03 UTC (permalink / raw)
To: Jarkko Sakkinen
Cc: linux-integrity, tglx, Jason Gunthorpe, Philip Tricca, Tadeusz Struk
On 2019-02-11 16:11:45 [+0200], Jarkko Sakkinen wrote:
> On Mon, Feb 11, 2019 at 11:58:35AM +0100, Sebastian Andrzej Siewior wrote:
> > Added in commit
> >
> > 9e1b74a63f776 ("tpm: add support for nonblocking operation")
> >
> > but never actually used it.
> >
> > Cc: Philip Tricca <philip.b.tricca@intel.com>
> > Cc: Tadeusz Struk <tadeusz.struk@intel.com>
> > Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
>
> You should use Fixes-tag e.g.
>
> Fixes: <12 first chars from SHA-1> ("<short summary>")
Is this the only reason why it has not been picked up? A fixes line
which triggers stable backports for something that does need to be
backported?
> /Jarkko
Sebastian
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-10-10 16:03 ` Sebastian Andrzej Siewior
@ 2019-10-14 19:39 ` Jarkko Sakkinen
2019-10-28 20:24 ` Jarkko Sakkinen
0 siblings, 1 reply; 17+ messages in thread
From: Jarkko Sakkinen @ 2019-10-14 19:39 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: linux-integrity, tglx, Jason Gunthorpe, Philip Tricca, Tadeusz Struk
On Thu, Oct 10, 2019 at 06:03:13PM +0200, Sebastian Andrzej Siewior wrote:
> On 2019-02-11 16:11:45 [+0200], Jarkko Sakkinen wrote:
> > On Mon, Feb 11, 2019 at 11:58:35AM +0100, Sebastian Andrzej Siewior wrote:
> > > Added in commit
> > >
> > > 9e1b74a63f776 ("tpm: add support for nonblocking operation")
> > >
> > > but never actually used it.
> > >
> > > Cc: Philip Tricca <philip.b.tricca@intel.com>
> > > Cc: Tadeusz Struk <tadeusz.struk@intel.com>
> > > Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> > > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> >
> > You should use Fixes-tag e.g.
> >
> > Fixes: <12 first chars from SHA-1> ("<short summary>")
>
> Is this the only reason why it has not been picked up? A fixes line
> which triggers stable backports for something that does need to be
> backported?
Fully agree with you. Frankly, I don't really remember anymore why I
responded that way. My guess is that I responded that to a worng patch.
Please just ping immediatelly. Sometimes when dealing with dozens of
patches this kind of human error might happen.
In any case, the patch is applied.
/Jarkko
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-10-14 19:39 ` Jarkko Sakkinen
@ 2019-10-28 20:24 ` Jarkko Sakkinen
2019-10-28 20:26 ` Jarkko Sakkinen
0 siblings, 1 reply; 17+ messages in thread
From: Jarkko Sakkinen @ 2019-10-28 20:24 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: linux-integrity, tglx, Jason Gunthorpe, Philip Tricca, Tadeusz Struk
On Mon, Oct 14, 2019 at 10:39:42PM +0300, Jarkko Sakkinen wrote:
> On Thu, Oct 10, 2019 at 06:03:13PM +0200, Sebastian Andrzej Siewior wrote:
> > On 2019-02-11 16:11:45 [+0200], Jarkko Sakkinen wrote:
> > > On Mon, Feb 11, 2019 at 11:58:35AM +0100, Sebastian Andrzej Siewior wrote:
> > > > Added in commit
> > > >
> > > > 9e1b74a63f776 ("tpm: add support for nonblocking operation")
> > > >
> > > > but never actually used it.
> > > >
> > > > Cc: Philip Tricca <philip.b.tricca@intel.com>
> > > > Cc: Tadeusz Struk <tadeusz.struk@intel.com>
> > > > Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> > > > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> > >
> > > You should use Fixes-tag e.g.
> > >
> > > Fixes: <12 first chars from SHA-1> ("<short summary>")
> >
> > Is this the only reason why it has not been picked up? A fixes line
> > which triggers stable backports for something that does need to be
> > backported?
>
> Fully agree with you. Frankly, I don't really remember anymore why I
> responded that way. My guess is that I responded that to a worng patch.
>
> Please just ping immediatelly. Sometimes when dealing with dozens of
> patches this kind of human error might happen.
>
> In any case, the patch is applied.
OK, so. Gave a relook at this:
This gives checkpatch.pl error:
0012-tpm-remove-tpm_dev_wq_lock.patch
-------------------------------------
ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9e1b74a63f77 ("tpm: add support for nonblocking operation")'
#8:
9e1b74a63f776 ("tpm: add support for nonblocking operation")
total: 1 errors, 0 warnings, 7 lines checked
NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.
Please send me a new patch with a legit fixes line. It is a fix to
regression even if it is a cosmetic one.
I'll drop the current patch from my tree and apply a new one once
I get it from you.
Thanks.
/Jarkko
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-10-28 20:24 ` Jarkko Sakkinen
@ 2019-10-28 20:26 ` Jarkko Sakkinen
2019-11-04 14:39 ` Sebastian Andrzej Siewior
0 siblings, 1 reply; 17+ messages in thread
From: Jarkko Sakkinen @ 2019-10-28 20:26 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: linux-integrity, tglx, Jason Gunthorpe, Philip Tricca, Tadeusz Struk
On Mon, Oct 28, 2019 at 10:24:19PM +0200, Jarkko Sakkinen wrote:
> On Mon, Oct 14, 2019 at 10:39:42PM +0300, Jarkko Sakkinen wrote:
> > On Thu, Oct 10, 2019 at 06:03:13PM +0200, Sebastian Andrzej Siewior wrote:
> > > On 2019-02-11 16:11:45 [+0200], Jarkko Sakkinen wrote:
> > > > On Mon, Feb 11, 2019 at 11:58:35AM +0100, Sebastian Andrzej Siewior wrote:
> > > > > Added in commit
> > > > >
> > > > > 9e1b74a63f776 ("tpm: add support for nonblocking operation")
> > > > >
> > > > > but never actually used it.
> > > > >
> > > > > Cc: Philip Tricca <philip.b.tricca@intel.com>
> > > > > Cc: Tadeusz Struk <tadeusz.struk@intel.com>
> > > > > Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> > > > > Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> > > >
> > > > You should use Fixes-tag e.g.
> > > >
> > > > Fixes: <12 first chars from SHA-1> ("<short summary>")
> > >
> > > Is this the only reason why it has not been picked up? A fixes line
> > > which triggers stable backports for something that does need to be
> > > backported?
> >
> > Fully agree with you. Frankly, I don't really remember anymore why I
> > responded that way. My guess is that I responded that to a worng patch.
> >
> > Please just ping immediatelly. Sometimes when dealing with dozens of
> > patches this kind of human error might happen.
> >
> > In any case, the patch is applied.
>
> OK, so. Gave a relook at this:
>
> This gives checkpatch.pl error:
>
> 0012-tpm-remove-tpm_dev_wq_lock.patch
> -------------------------------------
> ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9e1b74a63f77 ("tpm: add support for nonblocking operation")'
> #8:
> 9e1b74a63f776 ("tpm: add support for nonblocking operation")
>
> total: 1 errors, 0 warnings, 7 lines checked
>
> NOTE: For some of the reported defects, checkpatch may be able to
> mechanically convert to the typical style using --fix or --fix-inplace.
>
> Please send me a new patch with a legit fixes line. It is a fix to
> regression even if it is a cosmetic one.
>
> I'll drop the current patch from my tree and apply a new one once
> I get it from you.
>
> Thanks.
>
> /Jarkko
AFAIK cc stable triggers stable backport, not fixes line alone (not
100% sure about this though). Anyway even my original response was
meant to this patch I recall now that I bumped into that checkpatch
error.
/Jarkko
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-10-28 20:26 ` Jarkko Sakkinen
@ 2019-11-04 14:39 ` Sebastian Andrzej Siewior
2019-11-04 17:37 ` Jerry Snitselaar
0 siblings, 1 reply; 17+ messages in thread
From: Sebastian Andrzej Siewior @ 2019-11-04 14:39 UTC (permalink / raw)
To: Jarkko Sakkinen
Cc: linux-integrity, tglx, Jason Gunthorpe, Philip Tricca, Tadeusz Struk
sorry for that late reply, was traveling…
On 2019-10-28 22:26:37 [+0200], Jarkko Sakkinen wrote:
> > OK, so. Gave a relook at this:
> >
> > This gives checkpatch.pl error:
> >
> > 0012-tpm-remove-tpm_dev_wq_lock.patch
> > -------------------------------------
> > ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9e1b74a63f77 ("tpm: add support for nonblocking operation")'
> > #8:
> > 9e1b74a63f776 ("tpm: add support for nonblocking operation")
> >
> > total: 1 errors, 0 warnings, 7 lines checked
> >
> > NOTE: For some of the reported defects, checkpatch may be able to
> > mechanically convert to the typical style using --fix or --fix-inplace.
> >
> > Please send me a new patch with a legit fixes line. It is a fix to
> > regression even if it is a cosmetic one.
> >
> > I'll drop the current patch from my tree and apply a new one once
> > I get it from you.
Can you please explain what is wrong with that one? It is exactly as
suggested by the error line.
> > Thanks.
> >
> > /Jarkko
>
> AFAIK cc stable triggers stable backport, not fixes line alone (not
> 100% sure about this though). Anyway even my original response was
> meant to this patch I recall now that I bumped into that checkpatch
> error.
The cc: for stable and fixes are high indicators for it to be
considered. These days even a few keywords in the commit message might
let Sasha's script decide to pick/suggest a patch for stable.
> /Jarkko
Sebastian
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-11-04 14:39 ` Sebastian Andrzej Siewior
@ 2019-11-04 17:37 ` Jerry Snitselaar
2019-11-04 17:44 ` Sebastian Andrzej Siewior
0 siblings, 1 reply; 17+ messages in thread
From: Jerry Snitselaar @ 2019-11-04 17:37 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: Jarkko Sakkinen, linux-integrity, tglx, Jason Gunthorpe,
Philip Tricca, Tadeusz Struk
On Mon Nov 04 19, Sebastian Andrzej Siewior wrote:
>sorry for that late reply, was traveling…
>
>On 2019-10-28 22:26:37 [+0200], Jarkko Sakkinen wrote:
>> > OK, so. Gave a relook at this:
>> >
>> > This gives checkpatch.pl error:
>> >
>> > 0012-tpm-remove-tpm_dev_wq_lock.patch
>> > -------------------------------------
>> > ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 9e1b74a63f77 ("tpm: add support for nonblocking operation")'
>> > #8:
>> > 9e1b74a63f776 ("tpm: add support for nonblocking operation")
>> >
>> > total: 1 errors, 0 warnings, 7 lines checked
>> >
>> > NOTE: For some of the reported defects, checkpatch may be able to
>> > mechanically convert to the typical style using --fix or --fix-inplace.
>> >
>> > Please send me a new patch with a legit fixes line. It is a fix to
>> > regression even if it is a cosmetic one.
>> >
>> > I'll drop the current patch from my tree and apply a new one once
>> > I get it from you.
>
>Can you please explain what is wrong with that one? It is exactly as
>suggested by the error line.
>
>> > Thanks.
>> >
>> > /Jarkko
>>
>> AFAIK cc stable triggers stable backport, not fixes line alone (not
>> 100% sure about this though). Anyway even my original response was
>> meant to this patch I recall now that I bumped into that checkpatch
>> error.
>
>The cc: for stable and fixes are high indicators for it to be
>considered. These days even a few keywords in the commit message might
>let Sasha's script decide to pick/suggest a patch for stable.
>
>> /Jarkko
>
>Sebastian
It looks like checkpatch is expecting the word commit to precede the hash on the same line.
I get no errors with the following:
Added in
commit 9e1b74a63f776 ("tpm: add support for nonblocking operation")
but never actually used it.
Fixes: 9e1b74a63f776 ("tpm: add support for nonblocking operation")
Cc: Philip Tricca <philip.b.tricca@intel.com>
Cc: Tadeusz Struk <tadeusz.struk@intel.com>
Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-11-04 17:37 ` Jerry Snitselaar
@ 2019-11-04 17:44 ` Sebastian Andrzej Siewior
2019-11-04 18:27 ` Jerry Snitselaar
0 siblings, 1 reply; 17+ messages in thread
From: Sebastian Andrzej Siewior @ 2019-11-04 17:44 UTC (permalink / raw)
To: Jerry Snitselaar
Cc: Jarkko Sakkinen, linux-integrity, tglx, Jason Gunthorpe,
Philip Tricca, Tadeusz Struk
On 2019-11-04 10:37:09 [-0700], Jerry Snitselaar wrote:
> It looks like checkpatch is expecting the word commit to precede the hash on the same line.
> I get no errors with the following:
That would explain it. That is however not what the TIP tree and other
people do not to mention that reading wise it makes sense to keep the
word `commit' as part of the sentence and add the hash in the next line.
> Added in
>
> commit 9e1b74a63f776 ("tpm: add support for nonblocking operation")
>
> but never actually used it.
>
> Fixes: 9e1b74a63f776 ("tpm: add support for nonblocking operation")
> Cc: Philip Tricca <philip.b.tricca@intel.com>
> Cc: Tadeusz Struk <tadeusz.struk@intel.com>
> Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Sebastian
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-11-04 17:44 ` Sebastian Andrzej Siewior
@ 2019-11-04 18:27 ` Jerry Snitselaar
2019-11-07 16:10 ` Sebastian Andrzej Siewior
0 siblings, 1 reply; 17+ messages in thread
From: Jerry Snitselaar @ 2019-11-04 18:27 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: Jarkko Sakkinen, linux-integrity, tglx, Jason Gunthorpe,
Philip Tricca, Tadeusz Struk
On Mon Nov 04 19, Sebastian Andrzej Siewior wrote:
>On 2019-11-04 10:37:09 [-0700], Jerry Snitselaar wrote:
>> It looks like checkpatch is expecting the word commit to precede the hash on the same line.
>> I get no errors with the following:
>
>That would explain it. That is however not what the TIP tree and other
>people do not to mention that reading wise it makes sense to keep the
>word `commit' as part of the sentence and add the hash in the next line.
>
Yes it reads better. What about the following?
Added in commit 9e1b74a63f776 ("tpm: add support for nonblocking
operation"), but never actually used it.
And then add the Fixes: line above the Cc: and Signed-off-by: ?
>> Added in
>>
>> commit 9e1b74a63f776 ("tpm: add support for nonblocking operation")
>>
>> but never actually used it.
>>
>> Fixes: 9e1b74a63f776 ("tpm: add support for nonblocking operation")
>> Cc: Philip Tricca <philip.b.tricca@intel.com>
>> Cc: Tadeusz Struk <tadeusz.struk@intel.com>
>> Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
>> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
>
>Sebastian
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-11-04 18:27 ` Jerry Snitselaar
@ 2019-11-07 16:10 ` Sebastian Andrzej Siewior
2019-11-07 18:35 ` Jarkko Sakkinen
0 siblings, 1 reply; 17+ messages in thread
From: Sebastian Andrzej Siewior @ 2019-11-07 16:10 UTC (permalink / raw)
To: Jerry Snitselaar
Cc: Jarkko Sakkinen, linux-integrity, tglx, Jason Gunthorpe,
Philip Tricca, Tadeusz Struk
On 2019-11-04 11:27:32 [-0700], Jerry Snitselaar wrote:
> On Mon Nov 04 19, Sebastian Andrzej Siewior wrote:
> > On 2019-11-04 10:37:09 [-0700], Jerry Snitselaar wrote:
> > > It looks like checkpatch is expecting the word commit to precede the hash on the same line.
> > > I get no errors with the following:
> >
> > That would explain it. That is however not what the TIP tree and other
> > people do not to mention that reading wise it makes sense to keep the
> > word `commit' as part of the sentence and add the hash in the next line.
> >
>
> Yes it reads better. What about the following?
>
> Added in commit 9e1b74a63f776 ("tpm: add support for nonblocking
> operation"), but never actually used it.
>
> And then add the Fixes: line above the Cc: and Signed-off-by: ?
Can please get over with? It is a simple patch. It has simple
description.
Sebastian
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-11-07 16:10 ` Sebastian Andrzej Siewior
@ 2019-11-07 18:35 ` Jarkko Sakkinen
2019-11-14 11:16 ` Sebastian Andrzej Siewior
0 siblings, 1 reply; 17+ messages in thread
From: Jarkko Sakkinen @ 2019-11-07 18:35 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: Jerry Snitselaar, linux-integrity, tglx, Jason Gunthorpe,
Philip Tricca, Tadeusz Struk
On Thu, Nov 07, 2019 at 05:10:41PM +0100, Sebastian Andrzej Siewior wrote:
> On 2019-11-04 11:27:32 [-0700], Jerry Snitselaar wrote:
> > On Mon Nov 04 19, Sebastian Andrzej Siewior wrote:
> > > On 2019-11-04 10:37:09 [-0700], Jerry Snitselaar wrote:
> > > > It looks like checkpatch is expecting the word commit to precede the hash on the same line.
> > > > I get no errors with the following:
> > >
> > > That would explain it. That is however not what the TIP tree and other
> > > people do not to mention that reading wise it makes sense to keep the
> > > word `commit' as part of the sentence and add the hash in the next line.
> > >
> >
> > Yes it reads better. What about the following?
> >
> > Added in commit 9e1b74a63f776 ("tpm: add support for nonblocking
> > operation"), but never actually used it.
> >
> > And then add the Fixes: line above the Cc: and Signed-off-by: ?
>
> Can please get over with? It is a simple patch. It has simple
> description.
https://lore.kernel.org/linux-integrity/20191028202419.GA7214@linux.intel.com/
I'm also cool with cc stable as long as the commit is message has the
correct format.
/Jarkko
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-11-07 18:35 ` Jarkko Sakkinen
@ 2019-11-14 11:16 ` Sebastian Andrzej Siewior
2019-11-15 17:34 ` Jarkko Sakkinen
0 siblings, 1 reply; 17+ messages in thread
From: Sebastian Andrzej Siewior @ 2019-11-14 11:16 UTC (permalink / raw)
To: Jarkko Sakkinen
Cc: Jerry Snitselaar, linux-integrity, tglx, Jason Gunthorpe,
Philip Tricca, Tadeusz Struk
On 2019-11-07 20:35:03 [+0200], Jarkko Sakkinen wrote:
> On Thu, Nov 07, 2019 at 05:10:41PM +0100, Sebastian Andrzej Siewior wrote:
> > On 2019-11-04 11:27:32 [-0700], Jerry Snitselaar wrote:
> > > On Mon Nov 04 19, Sebastian Andrzej Siewior wrote:
> > > > On 2019-11-04 10:37:09 [-0700], Jerry Snitselaar wrote:
> > > > > It looks like checkpatch is expecting the word commit to precede the hash on the same line.
> > > > > I get no errors with the following:
> > > >
> > > > That would explain it. That is however not what the TIP tree and other
> > > > people do not to mention that reading wise it makes sense to keep the
> > > > word `commit' as part of the sentence and add the hash in the next line.
> > > >
> > >
> > > Yes it reads better. What about the following?
> > >
> > > Added in commit 9e1b74a63f776 ("tpm: add support for nonblocking
> > > operation"), but never actually used it.
> > >
> > > And then add the Fixes: line above the Cc: and Signed-off-by: ?
> >
> > Can please get over with? It is a simple patch. It has simple
> > description.
>
> https://lore.kernel.org/linux-integrity/20191028202419.GA7214@linux.intel.com/
>
> I'm also cool with cc stable as long as the commit is message has the
> correct format.
This is _really_ getting ridiculous. Holding back a simple patch just
because checkpatch says that the word `commit' is not in a new line. It
is more readable that way not to mention line with the commit id is
getting really long. This is a stupid checkpatch rule which is enforced
here.
> /Jarkko
Sebastian
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] tpm: remove tpm_dev_wq_lock
2019-11-14 11:16 ` Sebastian Andrzej Siewior
@ 2019-11-15 17:34 ` Jarkko Sakkinen
0 siblings, 0 replies; 17+ messages in thread
From: Jarkko Sakkinen @ 2019-11-15 17:34 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: Jerry Snitselaar, linux-integrity, tglx, Jason Gunthorpe,
Philip Tricca, Tadeusz Struk
On Thu, Nov 14, 2019 at 12:16:12PM +0100, Sebastian Andrzej Siewior wrote:
> On 2019-11-07 20:35:03 [+0200], Jarkko Sakkinen wrote:
> > On Thu, Nov 07, 2019 at 05:10:41PM +0100, Sebastian Andrzej Siewior wrote:
> > > On 2019-11-04 11:27:32 [-0700], Jerry Snitselaar wrote:
> > > > On Mon Nov 04 19, Sebastian Andrzej Siewior wrote:
> > > > > On 2019-11-04 10:37:09 [-0700], Jerry Snitselaar wrote:
> > > > > > It looks like checkpatch is expecting the word commit to precede the hash on the same line.
> > > > > > I get no errors with the following:
> > > > >
> > > > > That would explain it. That is however not what the TIP tree and other
> > > > > people do not to mention that reading wise it makes sense to keep the
> > > > > word `commit' as part of the sentence and add the hash in the next line.
> > > > >
> > > >
> > > > Yes it reads better. What about the following?
> > > >
> > > > Added in commit 9e1b74a63f776 ("tpm: add support for nonblocking
> > > > operation"), but never actually used it.
> > > >
> > > > And then add the Fixes: line above the Cc: and Signed-off-by: ?
> > >
> > > Can please get over with? It is a simple patch. It has simple
> > > description.
> >
> > https://lore.kernel.org/linux-integrity/20191028202419.GA7214@linux.intel.com/
> >
> > I'm also cool with cc stable as long as the commit is message has the
> > correct format.
>
> This is _really_ getting ridiculous. Holding back a simple patch just
> because checkpatch says that the word `commit' is not in a new line. It
> is more readable that way not to mention line with the commit id is
> getting really long. This is a stupid checkpatch rule which is enforced
> here.
I'm not sure why formatting a commit message properly is ridicilous.
If it is a bug fix, then it should have fixes tag.
/Jarkko
^ permalink raw reply [flat|nested] 17+ messages in thread
* [PATCH V2] tpm: Remove tpm_dev_wq_lock
2019-02-11 10:58 [PATCH] tpm: remove tpm_dev_wq_lock Sebastian Andrzej Siewior
2019-02-11 14:11 ` Jarkko Sakkinen
2019-02-12 1:45 ` Tadeusz Struk
@ 2020-12-14 22:07 ` Thomas Gleixner
2021-01-04 18:21 ` Jarkko Sakkinen
2 siblings, 1 reply; 17+ messages in thread
From: Thomas Gleixner @ 2020-12-14 22:07 UTC (permalink / raw)
To: Sebastian Andrzej Siewior, linux-integrity
Cc: Jason Gunthorpe, Sebastian Andrzej Siewior, Philip Tricca,
Tadeusz Struk, Jarkko Sakkinen
From: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Never used since it was added.
Fixes: 9e1b74a63f776 ("tpm: add support for nonblocking operation")
Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: linux-integrity@vger.kernel.org
Cc: Jason Gunthorpe <jgg@ziepe.ca>
Cc: Philip Tricca <philip.b.tricca@intel.com>
Cc: Tadeusz Struk <tadeusz.struk@intel.com>
Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
---
V2: Reworded the change log so the !&!%#$ dispute over the formatting
is not longer in the way of getting this triviality done.
---
drivers/char/tpm/tpm-dev-common.c | 1 -
1 file changed, 1 deletion(-)
--- a/drivers/char/tpm/tpm-dev-common.c
+++ b/drivers/char/tpm/tpm-dev-common.c
@@ -20,7 +20,6 @@
#include "tpm-dev.h"
static struct workqueue_struct *tpm_dev_wq;
-static DEFINE_MUTEX(tpm_dev_wq_lock);
static ssize_t tpm_dev_transmit(struct tpm_chip *chip, struct tpm_space *space,
u8 *buf, size_t bufsiz)
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH V2] tpm: Remove tpm_dev_wq_lock
2020-12-14 22:07 ` [PATCH V2] tpm: Remove tpm_dev_wq_lock Thomas Gleixner
@ 2021-01-04 18:21 ` Jarkko Sakkinen
0 siblings, 0 replies; 17+ messages in thread
From: Jarkko Sakkinen @ 2021-01-04 18:21 UTC (permalink / raw)
To: Thomas Gleixner
Cc: Sebastian Andrzej Siewior, linux-integrity, Jason Gunthorpe,
Philip Tricca, Tadeusz Struk, Jarkko Sakkinen
On Mon, Dec 14, 2020 at 11:07:21PM +0100, Thomas Gleixner wrote:
>
> From: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
>
> Never used since it was added.
Hi, apologies for this latency. I was in vacation during 12/14-12/20
last month, and after that Christmas and New Year mixed up schedules a
bit.
Applying today, thanks.
/Jarkko
> Fixes: 9e1b74a63f776 ("tpm: add support for nonblocking operation")
> Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
> Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
> Cc: linux-integrity@vger.kernel.org
> Cc: Jason Gunthorpe <jgg@ziepe.ca>
> Cc: Philip Tricca <philip.b.tricca@intel.com>
> Cc: Tadeusz Struk <tadeusz.struk@intel.com>
> Cc: Jarkko Sakkinen <jarkko.sakkinen@linux.intel.com>
> ---
> V2: Reworded the change log so the !&!%#$ dispute over the formatting
> is not longer in the way of getting this triviality done.
> ---
> drivers/char/tpm/tpm-dev-common.c | 1 -
> 1 file changed, 1 deletion(-)
>
> --- a/drivers/char/tpm/tpm-dev-common.c
> +++ b/drivers/char/tpm/tpm-dev-common.c
> @@ -20,7 +20,6 @@
> #include "tpm-dev.h"
>
> static struct workqueue_struct *tpm_dev_wq;
> -static DEFINE_MUTEX(tpm_dev_wq_lock);
>
> static ssize_t tpm_dev_transmit(struct tpm_chip *chip, struct tpm_space *space,
> u8 *buf, size_t bufsiz)
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2021-01-04 18:22 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-11 10:58 [PATCH] tpm: remove tpm_dev_wq_lock Sebastian Andrzej Siewior
2019-02-11 14:11 ` Jarkko Sakkinen
2019-10-10 16:03 ` Sebastian Andrzej Siewior
2019-10-14 19:39 ` Jarkko Sakkinen
2019-10-28 20:24 ` Jarkko Sakkinen
2019-10-28 20:26 ` Jarkko Sakkinen
2019-11-04 14:39 ` Sebastian Andrzej Siewior
2019-11-04 17:37 ` Jerry Snitselaar
2019-11-04 17:44 ` Sebastian Andrzej Siewior
2019-11-04 18:27 ` Jerry Snitselaar
2019-11-07 16:10 ` Sebastian Andrzej Siewior
2019-11-07 18:35 ` Jarkko Sakkinen
2019-11-14 11:16 ` Sebastian Andrzej Siewior
2019-11-15 17:34 ` Jarkko Sakkinen
2019-02-12 1:45 ` Tadeusz Struk
2020-12-14 22:07 ` [PATCH V2] tpm: Remove tpm_dev_wq_lock Thomas Gleixner
2021-01-04 18:21 ` Jarkko Sakkinen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).