Paolo Bonzini
2020-Jul-10 07:46 UTC
[PATCH] scsi: virtio_scsi: Remove unnecessary condition checks
On 10/07/20 09:40, Markus Elfring wrote:>>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/scsi/virtio_scsi.c?id=42f82040ee66db13525dc6f14b8559890b2f4c1c#n980 >>> >>> if (!virtscsi_cmd_cache) { >>> pr_err("kmem_cache_create() for virtscsi_cmd_cache failed\n"); >>> - goto error; >>> + return -ENOMEM; >>> } >> >> Could be doable, but I don't see a particular benefit. > > Can a bit more ?compliance? (with the Linux coding style) matter here?No.>> Having a single error loop is an advantage by itself. > > I do not see that a loop is involved in the implementation of the function ?init?.s/loop/label/ sorry. Paolo