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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 21 Jun 2016 21:25:18 -0400
From:	"Martin K. Petersen" <martin.petersen@...cle.com>
To:	Linus Torvalds <torvalds@...ux-foundation.org>
Cc:	Josh Boyer <jwboyer@...oraproject.org>,
	"Martin K. Petersen" <martin.petersen@...cle.com>,
	Johannes Thumshirn <jthumshirn@...e.de>,
	Thorsten Leemhuis <regressions@...mhuis.info>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Quinn Tran <quinn.tran@...gic.com>
Subject: Re: Reported regressions for 4.7 as of Sunday, 2016-06-19

>>>>> "Linus" == Linus Torvalds <torvalds@...ux-foundation.org> 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@...gic.com>
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@...gic.com>
    Signed-off-by: Himanshu Madhani <himanshu.madhani@...gic.com>
    Signed-off-by: Nicholas Bellinger <nab@...ux-iscsi.org>

Quinn?

-- 
Martin K. Petersen	Oracle Linux Engineering