scsi: mpi3mr: Simplify the workqueue allocation code

Let alloc_ordered_workqueue() format the workqueue name instead of
calling scnprintf() explicitly. Compile-tested only.

Cc: Chandrakanth Patil <chandrakanth.patil@broadcom.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Link: https://patch.msgid.link/20260106185723.2526901-1-bvanassche@acm.org
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

diff --git a/drivers/scsi/mpi3mr/mpi3mr.h b/drivers/scsi/mpi3mr/mpi3mr.h
index 31d68c151b2076aa817014ffa5ea64e6e2d4cc34..82f4ae87d6bc04a55a54c26f98e8b07ceed5b74f 100644 (file)
--- a/drivers/scsi/mpi3mr/mpi3mr.h
+++ b/drivers/scsi/mpi3mr/mpi3mr.h
@@ -1076,7 +1076,6 @@ struct scmd_priv {
  * @fwevt_worker_thread: Firmware event worker thread
  * @fwevt_lock: Firmware event lock
  * @fwevt_list: Firmware event list
- * @watchdog_work_q_name: Fault watchdog worker thread name
  * @watchdog_work_q: Fault watchdog worker thread
  * @watchdog_work: Fault watchdog work
  * @watchdog_lock: Fault watchdog lock
@@ -1265,7 +1264,6 @@ struct mpi3mr_ioc {
        spinlock_t fwevt_lock;
        struct list_head fwevt_list;
 
-       char watchdog_work_q_name[50];
        struct workqueue_struct *watchdog_work_q;
        struct delayed_work watchdog_work;
        spinlock_t watchdog_lock;

diff --git a/drivers/scsi/mpi3mr/mpi3mr_fw.c b/drivers/scsi/mpi3mr/mpi3mr_fw.c
index 8fe6e0bf342e25e0aa506e1daa2524477b872313..b564fe5980a662e2b2dfd8f191871eedb5605c98 100644 (file)
--- a/drivers/scsi/mpi3mr/mpi3mr_fw.c
+++ b/drivers/scsi/mpi3mr/mpi3mr_fw.c
@@ -2878,11 +2878,8 @@ void mpi3mr_start_watchdog(struct mpi3mr_ioc *mrioc)
                return;
 
        INIT_DELAYED_WORK(&mrioc->watchdog_work, mpi3mr_watchdog_work);
-       snprintf(mrioc->watchdog_work_q_name,
-           sizeof(mrioc->watchdog_work_q_name), "watchdog_%s%d", mrioc->name,
-           mrioc->id);
        mrioc->watchdog_work_q = alloc_ordered_workqueue(
-               "%s", WQ_MEM_RECLAIM, mrioc->watchdog_work_q_name);
+               "watchdog_%s%d", WQ_MEM_RECLAIM, mrioc->name, mrioc->id);
        if (!mrioc->watchdog_work_q) {
                ioc_err(mrioc, "%s: failed (line=%d)\n", __func__, __LINE__);
                return;