netdev - [PATCH 1/1 2.6.28] cxgb3 - add one check in interrupt routine

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-next>] [day] [month] [year] [list]

Message-Id: <1223671728.2614.13.camel@wendyx.austin.ibm.com>
Date:	Fri, 10 Oct 2008 15:48:48 -0500
From:	wendy xiong <wendyx@...ibm.com>
To:	jeff@...zik.org, netdev@...r.kernel.org, davem@...emloft.net,
	wenxiong@...ibm.com
Subject: [PATCH 1/1 2.6.28] cxgb3 - add one check in interrupt routine

Hi Jeff,

We are submitting a patch to do the test before accessing A_PL_CLI register in interrupt routine for inclusion in 2.6.28, 
as follows:


diff -Nuarp linux-2.6.27-rc9/drivers/net/cxgb3/sge.c linux-2.6.27-rc9.cxgb3/drivers/net/cxgb3/sge.c
--- linux-2.6.27-rc9/drivers/net/cxgb3/sge.c    2008-10-10 15:09:31.000000000 -0500
+++ linux-2.6.27-rc9.cxgb3/drivers/net/cxgb3/sge.c      2008-10-10 15:11:44.000000000 -0500
@@ -2691,6 +2691,9 @@ static irqreturn_t t3b_intr_napi(int irq
        t3_write_reg(adap, A_PL_CLI, 0);
        map = t3_read_reg(adap, A_SG_DATA_INTR);

+       if (unlikely(pci_channel_offline(adap->pdev)))
+               return IRQ_NONE;
+
        if (unlikely(!map))     /* shared interrupt, most likely */
                return IRQ_NONE;


Thanks,
Wendy




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