* [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 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
* 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
* [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).