Re: [PATCH v3 24/31] elx: efct: LIO backend interface routines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/21/2020 10:09 PM, Bart Van Assche wrote:
On 4/21/20 9:20 PM, James Smart wrote:
On 4/11/2020 9:57 PM, Bart Van Assche wrote:
On 2020-04-11 20:32, James Smart wrote:
+
+    lio_wq = create_singlethread_workqueue("efct_lio_worker");
+    if (!lio_wq) {
+        efc_log_err(efct, "workqueue create failed\n");
+        return -ENOMEM;
+    }

Is any work queued onto lio_wq that needs to be serialized against other
work queued onto the same queue? If not, can one of the system
workqueues be used, e.g. system_wq?


We are using "lio_wq" here to call the LIO backed during creation or deletion of sessions. LIO api's target_remove_session/ target_setup_session are blocking calls so we don't want to process them in the interrupt thread. "lio_wq" is used for these two events only and this brings the serialization to session management as we create single threaded work queue.

Hi James,

I think the above is very useful information. How about adding that information as a comment above the definition of "lio_wq"?

Thanks,

Bart.


yep - we will. Thanks

-- james




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [SCSI Target Devel]     [Linux SCSI Target Infrastructure]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Linux IIO]     [Samba]     [Device Mapper]

  Powered by Linux