Re: [PATCH 1/3] iscsi-target: Convert iscsi_thread_set usage to kthread.h

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

 



On Tue, 2015-03-24 at 18:37 +0200, Sagi Grimberg wrote:
> On 3/23/2015 2:21 PM, Sagi Grimberg wrote:
> > On 3/21/2015 8:16 AM, Nicholas A. Bellinger wrote:
> >> From: Nicholas Bellinger <nab@xxxxxxxxxxxxxxx>
> >>
> >> This patch converts iscsi-target code to use modern kthread.h API
> >> callers for creating RX/TX threads for each new iscsi_conn descriptor,
> >> and releasing associated RX/TX threads during connection shutdown.
> >>
> >> This is done using iscsit_start_kthreads() -> kthread_run() to start
> >> new kthreads from within iscsi_post_login_handler(), and invoking
> >> kthread_stop() from existing iscsit_close_connection() code.
> >>
> >> Also, convert iscsit_logout_post_handler_closesession() code to use
> >> cmpxchg when determing when iscsit_cause_connection_reinstatement()
> >> needs to sleep waiting for completion.
> >
> > I'll run some tests with this.
> 
> Hi Nic,
> 
> I can report that this series (minus patch 3/3 and along with some
> patches I'll send out soon) seems to resolve the list corruption issue
> I've seen.
> 
> Overall the patches look good to me so feel free to include
> patches 1+2 with:
> 
> Tested-by: Sagi Grimberg <sagig@xxxxxxxxxxxx>
> 

Great.  The first two are queued in for-next, and for the moment #1 has
a v3.10+ stable tag as well..

Please let me know if you run into any regressions with these.

Thanks alot Sagi!

--nab

--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[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