Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2018-12-26 Thread Alan Adamson
Hi, Was this issue ever resolved?  We are seeing this on 4.14.35. Thanks, Alan Adamson On 5/9/18 3:27 PM, Ilkka Sovanto wrote: Hi again, Chris and others! We're hitting this again on 4.14.32. Looks like the path iscsi_queuecommand -> prepd_reject/prepd_fault results in iscsi_complete_task g

Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2018-05-09 Thread Ilkka Sovanto
Hi again, Chris and others! We're hitting this again on 4.14.32. Looks like the path iscsi_queuecommand -> prepd_reject/prepd_fault results in iscsi_complete_task getting called with frwd_lock held instead of the requisite back_lock which might explain the task list corruption under heavy loads.

Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2018-02-07 Thread sitsofe
Hi, On Tuesday, 6 February 2018 11:29:52 UTC, ro...@giantswarm.io wrote: > > Hello, looks like we are struggling from the same issue [1]. > > Was this patch merged to upstream kernel? > > 1 - https://groups.google.com/forum/#!topic/open-iscsi/0qFng-WO3mM > Yes it was merged upstream - https://g

Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2018-02-06 Thread roman
Hello, looks like we are struggling from the same issue [1]. Was this patch merged to upstream kernel? 1 - https://groups.google.com/forum/#!topic/open-iscsi/0qFng-WO3mM On Tuesday, February 21, 2017 at 6:35:54 PM UTC+1, Chris Leech wrote: > > There's a rather long standing regression from commi

Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2017-10-09 Thread Ashish Srivastava (Oracle)
Hi Ilkka, I was going through this particular issue reported here, which also seen with Oracle Linux. I was curious to see the core for which you have posted your below debugging. Can I get the access to this particular core dump. Regards, Ashish On Tuesday, February 21, 2017 at 11:05:54 PM UT

Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2017-03-05 Thread Ilkka Sovanto
Hi! Was running pretty nicely for a week or so, until we got this one: [534630.679965] BUG: unable to handle kernel at 0078 [534630.684035] IP: [] iscsi_xmit_task+0x29/0xc0 [534630.685724] PGD a1fcd3067 [534630.687346] Oops: 0002 [#1] [534630.688846] Modules linked in: [534630.690348]

Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2017-02-24 Thread Ilkka Sovanto
On Fri, Feb 24, 2017 at 5:09 PM, Guilherme G. Piccoli < gpicc...@linux.vnet.ibm.com> wrote: > On 02/23/2017 07:25 PM, Chris Leech wrote: > > Yikes, my git-send-email settings suppressed the important CCs. Sorry! > > > > Guilherme and Ilkka, can you comment about your testing results or > review p

Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2017-02-24 Thread Guilherme G. Piccoli
On 02/23/2017 07:25 PM, Chris Leech wrote: > Yikes, my git-send-email settings suppressed the important CCs. Sorry! > > Guilherme and Ilkka, can you comment about your testing results or review > please? Hi Chris, thanks for looping me. Patch seems nice, I do have some points below, most nitpic

Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2017-02-23 Thread Chris Leech
Yikes, my git-send-email settings suppressed the important CCs. Sorry! Guilherme and Ilkka, can you comment about your testing results or review please? - Chris Leech - Original Message - > There's a rather long standing regression from commit > 659743b [SCSI] libiscsi: Reduce locking

Re: [PATCH] libiscsi: add lock around task lists to fix list corruption regression

2017-02-23 Thread Martin K. Petersen
> "Chris" == Chris Leech writes: Chris> There's a rather long standing regression from commit 659743b Chris> [SCSI] libiscsi: Reduce locking contention in fast path Chris> Depending on iSCSI target behavior, it's possible to hit the case Chris> in iscsi_complete_task where the task is still

[PATCH] libiscsi: add lock around task lists to fix list corruption regression

2017-02-21 Thread Chris Leech
There's a rather long standing regression from commit 659743b [SCSI] libiscsi: Reduce locking contention in fast path Depending on iSCSI target behavior, it's possible to hit the case in iscsi_complete_task where the task is still on a pending list (!list_empty(&task->running)). When that happens