Messages in this thread | | | Date | Tue, 30 Apr 2024 13:21:19 +0200 | Subject | Re: [PATCH V3 2/2] scsi: qla2xxx: Optimisation of exception handling in qla24xx_els_dcmd_iocb | From | Markus Elfring <> |
| |
Would you like to use the summary phrase “Use common error handling code in qla24xx_els_dcmd_iocb()”?
> To avoid duplicate error handling code a bit more, use more common goto > chain in qla24xx_els_dcmd_iocb.
How do you think about the following wording?
Add a jump target so that a bit of exception handling can be better reused at the end of this function implementation.
Can the tag “Suggested-by” be helpful for an improved change description? https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.9-rc6#n586
Regards, Markus
| |