Re: Reported regressions for 4.7 as of Sunday, 2016-06-19

From: Quinn Tran
Date: Wed Jun 22 2016 - 00:06:04 EST


Investigating.

Regards,
Quinn Tran







-----Original Message-----
From: "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
Organization: Oracle Corporation
Date: Tuesday, June 21, 2016 at 6:25 PM
To: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
Cc: Josh Boyer <jwboyer@xxxxxxxxxxxxxxxxx>, "Martin K. Petersen" <martin.petersen@xxxxxxxxxx>, Johannes Thumshirn <jthumshirn@xxxxxxx>, Thorsten Leemhuis <regressions@xxxxxxxxxxxxx>, linux-kernel <linux-kernel@xxxxxxxxxxxxxxx>, Quinn Tran <quinn.tran@xxxxxxxxxx>
Subject: Re: Reported regressions for 4.7 as of Sunday, 2016-06-19

>>>>>> "Linus" == Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx> writes:
>
>>> https://bugzilla.redhat.com/show_bug.cgi?id=1348342
>
>This first one appears to be a crash in a USB sound doodad and not
>qla2xxx. Also, this appears to be where the 4.5.5 -> 4.5.6 notion comes
>from. So we can probably ignore 4.5.5 as the last good revision.
>
>Linus> as far as I can tell. And neither of them looks very likely, but
>Linus> what do I know. Adding Martin Petersen and Johannes Thumshirn to
>Linus> the participants just in case they go "Ahh.."
>
>Doubt it's Johannes' tweak. The qla2xxx crash from the two other
>bugzilla entries is in:
>
>(gdb) list *qla24xx_process_response_queue+0x49
>0x27e09 is in qla24xx_process_response_queue (drivers/scsi/qla2xxx/qla_isr.c:2560).
>2555 if (rsp->msix->cpuid != smp_processor_id()) {
>2556 /* if kernel does not notify qla of IRQ's CPU change,
>2557 * then set it here.
>2558 */
>2559 rsp->msix->cpuid = smp_processor_id();
>2560 ha->tgt.rspq_vector_cpuid = rsp->msix->cpuid;
>2561 }
>2562
>2563 while (rsp->ring_ptr->signature != RESPONSE_PROCESSED) {
>2564 pkt = (struct sts_entry_24xx *)rsp->ring_ptr;
>
>That particular code went into 4.5 and comes from:
>
>commit cdb898c52d1dfad4b4800b83a58b3fe5d352edde
>Author: Quinn Tran <quinn.tran@xxxxxxxxxx>
>Date: Thu Dec 17 14:57:05 2015 -0500
>
> qla2xxx: Add irq affinity notification
>
> Register to receive notification of when irq setting change
> occured.
>
> Signed-off-by: Quinn Tran <quinn.tran@xxxxxxxxxx>
> Signed-off-by: Himanshu Madhani <himanshu.madhani@xxxxxxxxxx>
> Signed-off-by: Nicholas Bellinger <nab@xxxxxxxxxxxxxxx>
>
>Quinn?
>
>--
>Martin K. Petersen Oracle Linux Engineering