"Peer Chen" <[EMAIL PROTECTED]> writes:
> Eric,
> Any decision for this patch, if not, currently we prefer to add all our
> code to quirks.c.
Sorry. I think adding the code to quirks.c is fine.
For bisection and code inspection purposes I would prefer the code
to come as a patchset of two patch
linux-kernel; akpm; Andy Currid
Subject: Re: [PATCH] msi: set 'En' bit of MSI Mapping Capability
"Peer Chen" <[EMAIL PROTECTED]> writes:
> I feel it's dangerous to set the En bit on Intel platform, If the HT
MSI
> En is set, the MSI should be expected to transf
: set 'En' bit of MSI Mapping Capability
"Peer Chen" <[EMAIL PROTECTED]> writes:
> I feel it's dangerous to set the En bit on Intel platform, If the HT
MSI
> En is set, the MSI should be expected to transform to HT INT message
> format. It may cause inte
"Peer Chen" <[EMAIL PROTECTED]> writes:
> I feel it's dangerous to set the En bit on Intel platform, If the HT MSI
> En is set, the MSI should be expected to transform to HT INT message
> format. It may cause interrupt lost or hardware internal state machine
> failed depend on the hardware design.
From: Eric W. Biederman [mailto:[EMAIL PROTECTED]
Sent: Friday, December 21, 2007 6:48 AM
To: Peer Chen
Cc: peerchen; linux-kernel; akpm; Andy Currid
Subject: Re: [PATCH] msi: set 'En' bit of MSI Mapping Capability
"Peer Chen" <[EMAIL PROTECTED]> writes:
> The q
Andrew Morton <[EMAIL PROTECTED]> writes:
> On Wed, 19 Dec 2007 16:41:25 -0800
> Andrew Morton <[EMAIL PROTECTED]> wrote:
>
>> It would be better to just add a stub implementation of
>> ht_enable_msi_mapping() for all the other architectures - avoid fancy cpp
>> tricks.
>
>
> And by this I really
"Peer Chen" <[EMAIL PROTECTED]> writes:
> The quirk is for our Intel platform, we don't want HT MSI mapping
> enabled in any of our devices.
Why is this a problem? I seem to recall a real hypertransport bus
downstream of the Intel cpu.
If there is a real hypertransport bus in the middle then wh
Chen
Subject: Re: [PATCH] msi: set 'En' bit of MSI Mapping Capability
"peerchen" <[EMAIL PROTECTED]> writes:
> According to the HyperTransport spec, 'En' indicate if the MSI Mapping
is
> active.
> Set the 'En' bit when setup pci and add the
On Wed, 19 Dec 2007 16:41:25 -0800
Andrew Morton <[EMAIL PROTECTED]> wrote:
> It would be better to just add a stub implementation of
> ht_enable_msi_mapping() for all the other architectures - avoid fancy cpp
> tricks.
And by this I really do mean going into each include/asm-*/pci.h and adding
On Tue, 18 Dec 2007 23:00:52 +0800
"peerchen" <[EMAIL PROTECTED]> wrote:
> According to the HyperTransport spec, 'En' indicate if the MSI Mapping is
> active.
> Set the 'En' bit when setup pci and add the quirk for some nvidia devices.
>
> The patch base on kernel 2.6.24-rc5
>
> Signed-off-by:
"peerchen" <[EMAIL PROTECTED]> writes:
> According to the HyperTransport spec, 'En' indicate if the MSI Mapping is
> active.
> Set the 'En' bit when setup pci and add the quirk for some nvidia devices.
>
> The patch base on kernel 2.6.24-rc5
Ok. This is starting to look good.
> Signed-off-by:
11 matches
Mail list logo