From: "Michael Chan" <mchan@broadcom.com>
To: "Andy Gospodarek" <andy@greyhouse.net>
Cc: "David Miller" <davem@davemloft.net>, "netdev" <netdev@vger.kernel.org>
Subject: Re: [TG3]: Workaround MSI bug on 5714/5780.
Date: Thu, 06 Sep 2007 12:05:30 -0700 [thread overview]
Message-ID: <1189105530.4852.8.camel@dell> (raw)
In-Reply-To: <20070906154123.GC10317@gospo.rdu.redhat.com>
On Thu, 2007-09-06 at 11:41 -0400, Andy Gospodarek wrote:
> On Thu, Sep 06, 2007 at 07:34:23AM -0700, David Miller wrote:
> > From: Andy Gospodarek <andy@greyhouse.net>
> > Date: Thu, 6 Sep 2007 09:45:16 -0400
> >
> > > Is is really necessary to get rid of the HT1000 patch?
> >
> > Yes, absolutely and without a single doubt.
> >
> > > commit e3008dedff4bdc96a5f67224cd3d8d12237082a0
> > > Author: Andy Gospodarek <andy@greyhouse.net>
> > > Date: Thu May 10 22:58:57 2007 -0700
> > >
> > > PCI: disable MSI by default on systems with Serverworks HT1000 chips
> > >
> > > This patch was designed to address tg3 and bnx2 messages printed on
> > > systems with HT1000 chips IIRC. Are we going back to those messages on
> > > bnx2 or is there a workaround in that driver needed too?
> >
> > Every report of that problem is with tg3 chips afflicted by this
> > INTX bug which is now worked around properly.
> >
>
> Not according to this:
>
> https://bugzilla.redhat.com/show_bug.cgi?id=227657
>
> Which was precisely why I chose to look at it as a chipset issue.
The HT1000 bridge may very well have an MSI issue. I'm checking with
ServerWorks and I will do some testing to confirm. If confirmed, we can
disable MSI behind the HT1000 bridge instead of globally. The 5714
issue is not caused by the HT1000 as it is not behind the HT1000.
>
> > Even my Niagara T1000 box was hit by this absolutely stupid and
> > wrong MSI quirk, which is how I found this problem to begin with.
> >
> > I'm reverting it.
> >
>
> If that's the case, do we need to consider that some of the bnx2
> hardware (like in this case the 5706) needs a similar workaround?
5706 doesn't have this same MSI problem as the 5714.
next prev parent reply other threads:[~2007-09-06 18:08 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-09-05 20:07 [TG3]: Workaround MSI bug on 5714/5780 Michael Chan
2007-09-05 19:38 ` Andy Gospodarek
2007-09-06 11:02 ` David Miller
2007-09-06 13:45 ` Andy Gospodarek
2007-09-06 14:34 ` David Miller
2007-09-06 14:40 ` Andy Gospodarek
2007-09-06 15:41 ` Andy Gospodarek
2007-09-06 19:05 ` Michael Chan [this message]
2007-09-06 18:26 ` Andy Gospodarek
2007-09-06 19:50 ` David Miller
2007-09-06 20:27 ` Andy Gospodarek
2007-09-08 2:26 ` Michael Chan
2007-09-08 21:25 ` David Miller
2007-09-14 22:33 ` David Miller
2007-09-06 15:11 ` Michael Chan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1189105530.4852.8.camel@dell \
--to=mchan@broadcom.com \
--cc=andy@greyhouse.net \
--cc=davem@davemloft.net \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).