[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20071126104329.34870fe8@bull.net>
Date: Mon, 26 Nov 2007 10:43:29 +0100
From: Sébastien Dugué <sebastien.dugue@...l.net>
To: "peerchen" <peerchen@...il.com>
Cc: "linux-kernel" <linux-kernel@...r.kernel.org>,
"akpm" <akpm@...ux-foundation.org>, "pchen" <pchen@...dia.com>,
"acurrid" <acurrid@...dia.com>, mchan@...adcom.com
Subject: Re: [PATCH 1/2] msi: set 'En' bit of MSI Mapping Capability on HT
platform
On Sun, 25 Nov 2007 11:21:48 +0800 "peerchen" <peerchen@...il.com> wrote:
> According to the HyperTransport spec, 'En' indicate if the MSI Mapping is
> active. So it should be set when enable the MSI.
>
Cool, I had a patch that added a quirk to enable MSI Mapping on Broadcom's
HT1000 so that the builtin BCM5706S Gigabit Ethernet would use MSI. This one
looks even cleaner.
As a side note, the bnx2 driver tries to use MSI by default (in the kernel I'm
using - 2.6.21.5-rt10) and falls back to INTx if no interrupt is received within
a reasonable amount of time. If the HT MSI Mapping is not enabled beforehand,
the first MSI message (before reverting to INTx) is not trapped by the bridge
and lands in memory thus corrupting what is there at that time. In my case
it's the mapcount of a struct page in the freelist which is overwritten with
the MSI message.
Sébastien.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists