Commit 6411307b authored by Bart Van Assche's avatar Bart Van Assche Committed by Martin K. Petersen
Browse files

scsi: snic: Simplify alloc_workqueue() invocations 



Let alloc_workqueue() format the workqueue name instead of calling
snprintf() explicitly. Not setting shost->work_q_name is safe because
there is no code that reads the value set by the removed code.

Signed-off-by: default avatarBart Van Assche <bvanassche@acm.org>
Link: https://lore.kernel.org/r/20240822195944.654691-15-bvanassche@acm.org


Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent 19d7cda1

drivers/scsi/snic/snic_main.c

+2 −4
Original line number Diff line number Diff line
@@ -300,10 +300,8 @@ snic_add_host(struct Scsi_Host *shost, struct pci_dev *pdev) 
	}



	SNIC_BUG_ON(shost->work_q != NULL);

	snprintf(shost->work_q_name, sizeof(shost->work_q_name), "scsi_wq_%d",

	shost->work_q = alloc_ordered_workqueue("scsi_wq_%d", WQ_MEM_RECLAIM,

						shost->host_no);

	shost->work_q = alloc_ordered_workqueue("%s", WQ_MEM_RECLAIM,

						shost->work_q_name);

	if (!shost->work_q) {

		SNIC_HOST_ERR(shost, "Failed to Create ScsiHost wq.\n");