linux-next.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* linux-next: Fixes tag needs some work in the scsi-fixes tree
@ 2019-02-05 20:20 Stephen Rothwell
  2019-02-06  3:55 ` Vaibhav Jain
  0 siblings, 1 reply; 13+ messages in thread
From: Stephen Rothwell @ 2019-02-05 20:20 UTC (permalink / raw)
  To: Martin K. Petersen
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Vaibhav Jain

[-- Attachment #1: Type: text/plain, Size: 308 bytes --]

Hi Martin,

In commit

  bb61b843ffd4 ("scsi: cxlflash: Prevent deadlock when adapter probe fails")

Fixes tag

  Fixes: c21e0bbfc485("cxlflash: Base support for IBM CXL Flash Adapter")

has these problem(s):

  - missing space between the SHA1 and the subject

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: linux-next: Fixes tag needs some work in the scsi-fixes tree
  2019-02-05 20:20 linux-next: Fixes tag needs some work in the scsi-fixes tree Stephen Rothwell
@ 2019-02-06  3:55 ` Vaibhav Jain
  2019-02-06  4:34   ` Stephen Rothwell
  2019-02-06 13:03   ` Martin K. Petersen
  0 siblings, 2 replies; 13+ messages in thread
From: Vaibhav Jain @ 2019-02-06  3:55 UTC (permalink / raw)
  To: Stephen Rothwell, Martin K. Petersen
  Cc: Linux Next Mailing List, Linux Kernel Mailing List

Stephen Rothwell <sfr@canb.auug.org.au> writes:

> Hi Martin,
>
> In commit
>
>   bb61b843ffd4 ("scsi: cxlflash: Prevent deadlock when adapter probe fails")
>
> Fixes tag
>
>   Fixes: c21e0bbfc485("cxlflash: Base support for IBM CXL Flash Adapter")
>
> has these problem(s):
>
>   - missing space between the SHA1 and the subject
Thanks for catching this Stephen. I am surprised that the checkpatch.pl
script didnt catch this.

Martin, should I resend the patch with this fixed ?

Thanks,
-- 
Vaibhav Jain <vaibhav@linux.ibm.com>
Linux Technology Center, IBM India Pvt. Ltd.

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: linux-next: Fixes tag needs some work in the scsi-fixes tree
  2019-02-06  3:55 ` Vaibhav Jain
@ 2019-02-06  4:34   ` Stephen Rothwell
  2019-02-06 13:03   ` Martin K. Petersen
  1 sibling, 0 replies; 13+ messages in thread
From: Stephen Rothwell @ 2019-02-06  4:34 UTC (permalink / raw)
  To: Vaibhav Jain
  Cc: Martin K. Petersen, Linux Next Mailing List, Linux Kernel Mailing List

[-- Attachment #1: Type: text/plain, Size: 312 bytes --]

Hi Vaibhav,

On Wed, 06 Feb 2019 09:25:12 +0530 Vaibhav Jain <vaibhav@linux.ibm.com> wrote:
>
> Martin, should I resend the patch with this fixed ?

I wouldn't bother, it is a really picky report (I almost didn't send
it) :-)

Consider it just a reminder for next time.
-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: linux-next: Fixes tag needs some work in the scsi-fixes tree
  2019-02-06  3:55 ` Vaibhav Jain
  2019-02-06  4:34   ` Stephen Rothwell
@ 2019-02-06 13:03   ` Martin K. Petersen
  1 sibling, 0 replies; 13+ messages in thread
From: Martin K. Petersen @ 2019-02-06 13:03 UTC (permalink / raw)
  To: Vaibhav Jain
  Cc: Stephen Rothwell, Martin K. Petersen, Linux Next Mailing List,
	Linux Kernel Mailing List


Vaibhav,

> Martin, should I resend the patch with this fixed ?

No need, I fixed it up yesterday.

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: linux-next: Fixes tag needs some work in the scsi-fixes tree
  2020-09-03  8:47 Stephen Rothwell
@ 2020-09-04  1:51 ` Martin K. Petersen
  0 siblings, 0 replies; 13+ messages in thread
From: Martin K. Petersen @ 2020-09-04  1:51 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Martin K. Petersen, Linux Next Mailing List,
	Linux Kernel Mailing List, Tomas Henzl


Stephen,

>   Fixes: 320e77acb3 scsi: mpt3sas: Irq poll to avoid CPU hard lockups
>   Fixes: a6ffd5bf681 scsi: megaraid_sas: Call disable_irq from process IRQ poll

Ah, my regex missed the short SHAs due to the missing description
quotation marks. Fixed my script and fixed up the commits.

Thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 13+ messages in thread

* linux-next: Fixes tag needs some work in the scsi-fixes tree
@ 2020-09-03  8:47 Stephen Rothwell
  2020-09-04  1:51 ` Martin K. Petersen
  0 siblings, 1 reply; 13+ messages in thread
From: Stephen Rothwell @ 2020-09-03  8:47 UTC (permalink / raw)
  To: Martin K. Petersen
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Tomas Henzl

[-- Attachment #1: Type: text/plain, Size: 817 bytes --]

Hi all,

In commit

  b614d55b970d ("scsi: mpt3sas: Don't call disable_irq from IRQ poll handler")

Fixes tag

  Fixes: 320e77acb3 scsi: mpt3sas: Irq poll to avoid CPU hard lockups

has these problem(s):

  - SHA1 should be at least 12 digits long

In commit

  d2af39141eea ("scsi: megaraid_sas: Don't call disable_irq from process IRQ poll")

Fixes tag

  Fixes: a6ffd5bf681 scsi: megaraid_sas: Call disable_irq from process IRQ poll

has these problem(s):

  - SHA1 should be at least 12 digits long

These can be fixed in the future by setting core.abbrev to 12 (or more)
or (for git v2.11 or later) just making sure it is not set (or set to
"auto").

Also, Fixes tags are usually generated using

  git log -1 --format='Fixes: %h ("%s")' <SHA1>

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: linux-next: Fixes tag needs some work in the scsi-fixes tree
  2020-06-24  7:14 ` SeongJae Park
@ 2020-06-24  7:23   ` Stephen Rothwell
  0 siblings, 0 replies; 13+ messages in thread
From: Stephen Rothwell @ 2020-06-24  7:23 UTC (permalink / raw)
  To: SeongJae Park
  Cc: Martin K. Petersen, Linux Next Mailing List,
	Linux Kernel Mailing List, SeongJae Park

[-- Attachment #1: Type: text/plain, Size: 309 bytes --]

Hi SeongJae,

On Wed, 24 Jun 2020 09:14:47 +0200 SeongJae Park <sjpark@amazon.com> wrote:
>
> Oops, sorry for the mistake.  I will send the fixed version in reply to this.

Unless Martin says otherwise, I am not sure it is worth fixing.  Just
remember for next time.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: linux-next: Fixes tag needs some work in the scsi-fixes tree
  2020-06-24  7:07 Stephen Rothwell
@ 2020-06-24  7:14 ` SeongJae Park
  2020-06-24  7:23   ` Stephen Rothwell
  0 siblings, 1 reply; 13+ messages in thread
From: SeongJae Park @ 2020-06-24  7:14 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Martin K. Petersen, Linux Next Mailing List,
	Linux Kernel Mailing List, SeongJae Park

On Wed, 24 Jun 2020 17:07:42 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:

> 
> [-- Attachment #1: Type: text/plain, Size: 385 bytes --]
> 
> Hi all,
> 
> In commit
> 
>   46da547e21d6 ("scsi: lpfc: Avoid another null dereference in lpfc_sli4_hba_unset()")
> 
> Fixes tag
> 
>   Fixes: 1ffdd2c0440d ("scsi: lpfc: resolve static checker warning inlpfc_sli4_hba_unset")
> 
> has these problem(s):
> 
>   - Subject does not match target commit subject
>     Just use
> 	git log -1 --format='Fixes: %h ("%s")'

Oops, sorry for the mistake.  I will send the fixed version in reply to this.


Thanks,
SeongJae Park

> 
> -- 
> Cheers,
> Stephen Rothwell

^ permalink raw reply	[flat|nested] 13+ messages in thread

* linux-next: Fixes tag needs some work in the scsi-fixes tree
@ 2020-06-24  7:07 Stephen Rothwell
  2020-06-24  7:14 ` SeongJae Park
  0 siblings, 1 reply; 13+ messages in thread
From: Stephen Rothwell @ 2020-06-24  7:07 UTC (permalink / raw)
  To: Martin K. Petersen
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, SeongJae Park

[-- Attachment #1: Type: text/plain, Size: 385 bytes --]

Hi all,

In commit

  46da547e21d6 ("scsi: lpfc: Avoid another null dereference in lpfc_sli4_hba_unset()")

Fixes tag

  Fixes: 1ffdd2c0440d ("scsi: lpfc: resolve static checker warning inlpfc_sli4_hba_unset")

has these problem(s):

  - Subject does not match target commit subject
    Just use
	git log -1 --format='Fixes: %h ("%s")'

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* Re: linux-next: Fixes tag needs some work in the scsi-fixes tree
  2020-05-08 11:33 Stephen Rothwell
@ 2020-05-11 23:54 ` Martin K. Petersen
  0 siblings, 0 replies; 13+ messages in thread
From: Martin K. Petersen @ 2020-05-11 23:54 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Martin K. Petersen, Linux Next Mailing List,
	Linux Kernel Mailing List, Ewan D. Milne


Stephen,

>   Fixes: 4910b524ac9 ("scsi: qla2xxx: Add support for setting port speed")
>
> has these problem(s):
>
>   - SHA1 should be at least 12 digits long

I'm pretty good at spotting 8-digit SHAs but the 11 digits tripped me up
in this case.

I refined the SHA validation logic in my commit hook script to ensure 12
digits going forward.

Thanks!

-- 
Martin K. Petersen	Oracle Linux Engineering

^ permalink raw reply	[flat|nested] 13+ messages in thread

* linux-next: Fixes tag needs some work in the scsi-fixes tree
@ 2020-05-08 11:33 Stephen Rothwell
  2020-05-11 23:54 ` Martin K. Petersen
  0 siblings, 1 reply; 13+ messages in thread
From: Stephen Rothwell @ 2020-05-08 11:33 UTC (permalink / raw)
  To: Martin K. Petersen
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Ewan D. Milne

[-- Attachment #1: Type: text/plain, Size: 449 bytes --]

Hi all,

In commit

  0f3b2f3fb5dc ("scsi: qla2xxx: Do not log message when reading port speed via sysfs")

Fixes tag

  Fixes: 4910b524ac9 ("scsi: qla2xxx: Add support for setting port speed")

has these problem(s):

  - SHA1 should be at least 12 digits long
    Can be fixed by setting core.abbrev to 12 (or more) or (for git v2.11
    or later) just making sure it is not set (or set to "auto").

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* linux-next: Fixes tag needs some work in the scsi-fixes tree
@ 2019-01-29 20:53 Stephen Rothwell
  0 siblings, 0 replies; 13+ messages in thread
From: Stephen Rothwell @ 2019-01-29 20:53 UTC (permalink / raw)
  To: Martin K. Petersen
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Steffen Maier

[-- Attachment #1: Type: text/plain, Size: 317 bytes --]

Hi all,

In commit

  b63195698dea ("scsi: zfcp: fix sysfs block queue limit output for max_segment_size")

Fixes tag

  Fixes: 50c2e9107f ("scsi: introduce a max_segment_size host_template parameters")

has these problem(s):

  - SHA1 should be at least 12 digits long

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

* linux-next: Fixes tag needs some work in the scsi-fixes tree
@ 2019-01-23  5:34 Stephen Rothwell
  0 siblings, 0 replies; 13+ messages in thread
From: Stephen Rothwell @ 2019-01-23  5:34 UTC (permalink / raw)
  To: Martin K. Petersen
  Cc: Linux Next Mailing List, Linux Kernel Mailing List, Christoph Hellwig

[-- Attachment #1: Type: text/plain, Size: 311 bytes --]

Hi Martin,

In commit

  a8cf59a6692c ("scsi: communicate max segment size to the DMA mapping code")

Fixes tag

  Fixes: 50c2e9107f ("scsi: introduce a max_segment_size host_template parameters")

has these problem(s):

  - SHA1 should be at least 12 digits long

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2020-09-04  1:51 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-05 20:20 linux-next: Fixes tag needs some work in the scsi-fixes tree Stephen Rothwell
2019-02-06  3:55 ` Vaibhav Jain
2019-02-06  4:34   ` Stephen Rothwell
2019-02-06 13:03   ` Martin K. Petersen
  -- strict thread matches above, loose matches on Subject: below --
2020-09-03  8:47 Stephen Rothwell
2020-09-04  1:51 ` Martin K. Petersen
2020-06-24  7:07 Stephen Rothwell
2020-06-24  7:14 ` SeongJae Park
2020-06-24  7:23   ` Stephen Rothwell
2020-05-08 11:33 Stephen Rothwell
2020-05-11 23:54 ` Martin K. Petersen
2019-01-29 20:53 Stephen Rothwell
2019-01-23  5:34 Stephen Rothwell

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