RE: [PATCH] e1000: Allow the driver to be used on PA RISC C8000 workstation

RE: [PATCH] e1000: Allow the driver to be used on PA RISC C8000 workstation

[Kirsher, Jeffrey T]

SSB3aWxsIGFkZCBpdCB0byBteSB0cmVlL3F1ZXVlLgoKQ2hlZXIsCkplZmYKDQoNCiAtLS0tLU9y
aWdpbmFsIE1lc3NhZ2UtLS0tLQ0KRnJvbTogCUt5bGUgTWNNYXJ0aW4gW21haWx0bzpreWxlQG1j
bWFydGluLmNhXQ0KU2VudDoJTW9uZGF5LCBEZWNlbWJlciAwNiwgMjAxMCAwODoxMiBBTSBVUyBN
b3VudGFpbiBTdGFuZGFyZCBUaW1lDQpUbzoJR3V5IE1hcnRpbg0KQ2M6CWUxMDAwLWRldmVsQGxp
c3RzLnNvdXJjZWZvcmdlLm5ldDsgTWlrdWxhcyBQYXRvY2thOyBsaW51eC1wYXJpc2NAdmdlci5r
ZXJuZWwub3JnOyBLaXJzaGVyLCBKZWZmcmV5IFQNClN1YmplY3Q6CVJlOiBbUEFUQ0hdIGUxMDAw
OiBBbGxvdyB0aGUgZHJpdmVyIHRvIGJlIHVzZWQgb24gUEEgUklTQyBDODAwMCB3b3Jrc3RhdGlv
bg0KDQpPbiBTdW4sIERlYyAwNSwgMjAxMCBhdCAwNjowNDoyNlBNICswMTAwLCBHdXkgTWFydGlu
IHdyb3RlOg0KPiBIaSBhbGwsDQo+IA0KPiBJIGNhbiBjb25maXJtIHRoYXQgSSd2ZSBzZWVuIHRo
ZSB2ZXJ5IHNhbWUgb24gbXVsdGlwbGUgQzgwMDANCj4gd29ya3N0YXRpb24gdG9vLg0KPiANCj4g
Q2FuIHNvbWVvbmUgcGxlYXNlIGFwcGx5IHRoaXMgcGF0Y2ggPw0KPiANCj4gVGVzdGVkLWJ5OiBH
dXkgTWFydGluIDxnbXNvZnRAdHV4aWNvbWFuLmJlPg0KPiANCg0KSXQgc2hvdWxkIGJlIHNlbnQg
dmlhIG5ldGRldkAgc28gdGhhdCBEYXZlTSBjYW4gcGljayBpdCB1cCBpZiB0aGUgZTEwMDANCmZv
bGtzIGRvIG5vdC4uLg0KDQotLUt5bGUNCg==

Re: [PATCH] e1000: Allow the driver to be used on PA RISC C8000 workstation

[Guy Martin]

Hi Jeff,

Any luck getting this into mainline ?

Regards,
  Guy

On Tue, 7 Dec 2010 05:48:55 -0800
"Kirsher, Jeffrey T" <jeffrey.t.kirsher@intel.com> wrote:

> I will add it to my tree/queue.
> 
> Cheer,
> Jeff
> 
> 
>  -----Original Message-----
> From: 	Kyle McMartin [mailto:kyle@mcmartin.ca]
> Sent:	Monday, December 06, 2010 08:12 AM US Mountain Standard
> Time To:	Guy Martin
> Cc:	e1000-devel@lists.sourceforge.net; Mikulas Patocka;
> linux-parisc@vger.kernel.org; Kirsher, Jeffrey T Subject:	Re:
> [PATCH] e1000: Allow the driver to be used on PA RISC C8000
> workstation
> 
> On Sun, Dec 05, 2010 at 06:04:26PM +0100, Guy Martin wrote:
> > Hi all,
> > 
> > I can confirm that I've seen the very same on multiple C8000
> > workstation too.
> > 
> > Can someone please apply this patch ?
> > 
> > Tested-by: Guy Martin <gmsoft@tuxicoman.be>
> > 
> 
> It should be sent via netdev@ so that DaveM can pick it up if the
> e1000 folks do not...
> 
> --Kyle

Re: [E1000-devel] [PATCH] e1000: Allow the driver to be used on PA RISC C8000 workstation

[Jesse Brandeburg]

On Mon, Feb 28, 2011 at 5:40 AM, Guy Martin <gmsoft@tuxicoman.be> wrote=
:
>
> Hi Jeff,
>
> Any luck getting this into mainline ?


Hi Guy, sorry for the delay,
We haven't been able to get our contacts in HP to give us a decent
response so far, we are following up with them to see whats up.  We
have not lost the patch and are still tracking it internally.

Give us a couple more weeks if that is okay and we should be able to
settle this by then.

> Regards,
> =A0Guy
>
> On Tue, 7 Dec 2010 05:48:55 -0800
> "Kirsher, Jeffrey T" <jeffrey.t.kirsher@intel.com> wrote:
>
> > I will add it to my tree/queue.
> >
> > Cheer,
> > Jeff
> >
> >
> > =A0-----Original Message-----
> > From: =A0 =A0 =A0 =A0 Kyle McMartin [mailto:kyle@mcmartin.ca]
> > Sent: Monday, December 06, 2010 08:12 AM US Mountain Standard
> > Time To: =A0 =A0 =A0Guy Martin
> > Cc: =A0 e1000-devel@lists.sourceforge.net; Mikulas Patocka;
> > linux-parisc@vger.kernel.org; Kirsher, Jeffrey T Subject: =A0 =A0 R=
e:
> > [PATCH] e1000: Allow the driver to be used on PA RISC C8000
> > workstation
> >
> > On Sun, Dec 05, 2010 at 06:04:26PM +0100, Guy Martin wrote:
> > > Hi all,
> > >
> > > I can confirm that I've seen the very same on multiple C8000
> > > workstation too.
> > >
> > > Can someone please apply this patch ?
> > >
> > > Tested-by: Guy Martin <gmsoft@tuxicoman.be>
> > >
> >
> > It should be sent via netdev@ so that DaveM can pick it up if the
> > e1000 folks do not...
> >
> > --Kyle
>
>
> ---------------------------------------------------------------------=
---------
> Free Software Download: Index, Search & Analyze Logs and other IT dat=
a in
> Real-Time with Splunk. Collect, index and harness all the fast moving=
 IT data
> generated by your applications, servers and devices whether physical,=
 virtual
> or in the cloud. Deliver compliance at lower cost and gain new busine=
ss
> insights. http://p.sf.net/sfu/splunk-dev2dev
> _______________________________________________
> E1000-devel mailing list
> E1000-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/e1000-devel
> To learn more about Intel&#174; Ethernet, visit http://communities.in=
tel.com/community/wired
--
To unsubscribe from this list: send the line "unsubscribe linux-parisc"=
 in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [E1000-devel] [PATCH] e1000: Allow the driver to be used on PA RISC C8000 workstation

[Rolf Eike Beer]

[-- Attachment #1: Type: text/plain, Size: 1172 bytes --]

Am Mittwoch, 2. März 2011, 21:19:24 schrieb Jesse Brandeburg:
> On Mon, Feb 28, 2011 at 5:40 AM, Guy Martin <gmsoft@tuxicoman.be> wrote:
> > Hi Jeff,
> > 
> > Any luck getting this into mainline ?
> 
> Hi Guy, sorry for the delay,
> We haven't been able to get our contacts in HP to give us a decent
> response so far, we are following up with them to see whats up.  We
> have not lost the patch and are still tracking it internally.
> 
> Give us a couple more weeks if that is okay and we should be able to
> settle this by then.

Hi all,

I wonder what exactly you are waiting for? This is a sanity check that we 
disable, so no working systems could get broken by this. And every single 
C8000 seems to be affected by this and is working fine with that patch. So maybe 
people at HP might have a clue _why_ this is screwed, but until then I don't 
see any point in waiting.

So please just add my tested-by and push this upstream soon. Since this is 
basically a hardware quirk I would like to get this into stable also so we may 
run vanilla 2.6.38.1 or something like that on C8000.

Tested-by: Rolf Eike Beer <eike-kernel@sf-tec.de>

Eike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [E1000-devel] [PATCH] e1000: Allow the driver to be used on PA RISC C8000 workstation

[Rolf Eike Beer]

[-- Attachment #1: Type: text/plain, Size: 1682 bytes --]

Am Freitag, 18. März 2011, 17:39:57 schrieb Rolf Eike Beer:
> Am Mittwoch, 2. März 2011, 21:19:24 schrieb Jesse Brandeburg:
> > On Mon, Feb 28, 2011 at 5:40 AM, Guy Martin <gmsoft@tuxicoman.be> wrote:
> > > Hi Jeff,
> > > 
> > > Any luck getting this into mainline ?
> > 
> > Hi Guy, sorry for the delay,
> > We haven't been able to get our contacts in HP to give us a decent
> > response so far, we are following up with them to see whats up.  We
> > have not lost the patch and are still tracking it internally.
> > 
> > Give us a couple more weeks if that is okay and we should be able to
> > settle this by then.

> I wonder what exactly you are waiting for? This is a sanity check that we
> disable, so no working systems could get broken by this. And every single
> C8000 seems to be affected by this and is working fine with that patch. So
> maybe people at HP might have a clue _why_ this is screwed, but until then
> I don't see any point in waiting.
> 
> So please just add my tested-by and push this upstream soon. Since this is
> basically a hardware quirk I would like to get this into stable also so we
> may run vanilla 2.6.38.1 or something like that on C8000.
> 
> Tested-by: Rolf Eike Beer <eike-kernel@sf-tec.de>

For the netdev folks: it's this patch we are talking about

http://www.spinics.net/lists/linux-parisc/msg03091.html

I would love to see that someone finally picks this up and pushes this 
upstream, CC stable. This is absolutely annoying as it breaks every time 
anyone touches the kernel on one of this machines.

Jeff, David, James: can you please make a decision of who takes this and then 
just do it?

Eike

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [E1000-devel] [PATCH] e1000: Allow the driver to be used on PA RISC C8000 workstation

[Rolf Eike Beer]

[-- Attachment #1: Type: text/plain, Size: 1789 bytes --]

Rolf Eike Beer wrote:
> Am Freitag, 18. März 2011, 17:39:57 schrieb Rolf Eike Beer:
> > Am Mittwoch, 2. März 2011, 21:19:24 schrieb Jesse Brandeburg:
> > > On Mon, Feb 28, 2011 at 5:40 AM, Guy Martin <gmsoft@tuxicoman.be> wrote:
> > > > Hi Jeff,
> > > > 
> > > > Any luck getting this into mainline ?
> > > 
> > > Hi Guy, sorry for the delay,
> > > We haven't been able to get our contacts in HP to give us a decent
> > > response so far, we are following up with them to see whats up.  We
> > > have not lost the patch and are still tracking it internally.
> > > 
> > > Give us a couple more weeks if that is okay and we should be able to
> > > settle this by then.
> > 
> > I wonder what exactly you are waiting for? This is a sanity check that
> > we
> > disable, so no working systems could get broken by this. And every
> > single
> > C8000 seems to be affected by this and is working fine with that patch.
> > So maybe people at HP might have a clue _why_ this is screwed, but
> > until then I don't see any point in waiting.
> > 
> > So please just add my tested-by and push this upstream soon. Since this
> > is basically a hardware quirk I would like to get this into stable also
> > so we may run vanilla 2.6.38.1 or something like that on C8000.
> > 
> > Tested-by: Rolf Eike Beer <eike-kernel@sf-tec.de>
> 
> For the netdev folks: it's this patch we are talking about
> 
> http://www.spinics.net/lists/linux-parisc/msg03091.html
> 
> I would love to see that someone finally picks this up and pushes this
> upstream, CC stable. This is absolutely annoying as it breaks every time
> anyone touches the kernel on one of this machines.
> 
> Jeff, David, James: can you please make a decision of who takes this and
> then just do it?

Ping?

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

Re: [PATCH] e1000: Allow the driver to be used on PA RISC C8000 workstation

[Matt Turner]

On Mon, Jun 27, 2011 at 2:42 PM, Rolf Eike Beer <eike-kernel@sf-tec.de> wro=
te:
> Rolf Eike Beer wrote:
>> Am Freitag, 18. M=E4rz 2011, 17:39:57 schrieb Rolf Eike Beer:
>> > Am Mittwoch, 2. M=E4rz 2011, 21:19:24 schrieb Jesse Brandeburg:
>> > > On Mon, Feb 28, 2011 at 5:40 AM, Guy Martin <gmsoft@tuxicoman.be> wr=
ote:
>> > > > Hi Jeff,
>> > > >
>> > > > Any luck getting this into mainline ?
>> > >
>> > > Hi Guy, sorry for the delay,
>> > > We haven't been able to get our contacts in HP to give us a decent
>> > > response so far, we are following up with them to see whats up. =A0We
>> > > have not lost the patch and are still tracking it internally.
>> > >
>> > > Give us a couple more weeks if that is okay and we should be able to
>> > > settle this by then.
>> >
>> > I wonder what exactly you are waiting for? This is a sanity check that
>> > we
>> > disable, so no working systems could get broken by this. And every
>> > single
>> > C8000 seems to be affected by this and is working fine with that patch.
>> > So maybe people at HP might have a clue _why_ this is screwed, but
>> > until then I don't see any point in waiting.
>> >
>> > So please just add my tested-by and push this upstream soon. Since this
>> > is basically a hardware quirk I would like to get this into stable also
>> > so we may run vanilla 2.6.38.1 or something like that on C8000.
>> >
>> > Tested-by: Rolf Eike Beer <eike-kernel@sf-tec.de>
>>
>> For the netdev folks: it's this patch we are talking about
>>
>> http://www.spinics.net/lists/linux-parisc/msg03091.html
>>
>> I would love to see that someone finally picks this up and pushes this
>> upstream, CC stable. This is absolutely annoying as it breaks every time
>> anyone touches the kernel on one of this machines.
>>
>> Jeff, David, James: can you please make a decision of who takes this and
>> then just do it?
>
> Ping?

Ping indeed. Waiting as if HP is going to say "yes, we fucked that up.
Go ahead with the patch." is silly.

---------------------------------------------------------------------------=
---
All of the data generated in your IT infrastructure is seriously valuable.
Why? It contains a definitive record of application performance, security =

threats, fraudulent activity, and more. Splunk takes this data and makes =

sense of it. IT sense. And common sense.
http://p.sf.net/sfu/splunk-d2d-c2
_______________________________________________
E1000-devel mailing list
E1000-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/e1000-devel
To learn more about Intel&#174; Ethernet, visit http://communities.intel.co=
m/community/wired

Re: [E1000-devel] [PATCH] e1000: Allow the driver to be used on PA RISC C8000 workstation

[Grant Grundler]

On Mon, Jun 27, 2011 at 03:33:43PM -0400, Matt Turner wrote:
> On Mon, Jun 27, 2011 at 2:42 PM, Rolf Eike Beer <eike-kernel@sf-tec.de> wrote:
> > Rolf Eike Beer wrote:
> >> Am Freitag, 18. M?rz 2011, 17:39:57 schrieb Rolf Eike Beer:
> >> > Am Mittwoch, 2. M?rz 2011, 21:19:24 schrieb Jesse Brandeburg:
> >> > > On Mon, Feb 28, 2011 at 5:40 AM, Guy Martin <gmsoft@tuxicoman.be> wrote:
> >> > > > Hi Jeff,
> >> > > >
> >> > > > Any luck getting this into mainline ?
> >> > >
> >> > > Hi Guy, sorry for the delay,
> >> > > We haven't been able to get our contacts in HP to give us a decent
> >> > > response so far, we are following up with them to see whats up. ?We
> >> > > have not lost the patch and are still tracking it internally.
> >> > >
> >> > > Give us a couple more weeks if that is okay and we should be able to
> >> > > settle this by then.
> >> >
> >> > I wonder what exactly you are waiting for? This is a sanity check that
> >> > we
> >> > disable, so no working systems could get broken by this. And every
> >> > single
> >> > C8000 seems to be affected by this and is working fine with that patch.
> >> > So maybe people at HP might have a clue _why_ this is screwed, but
> >> > until then I don't see any point in waiting.
> >> >
> >> > So please just add my tested-by and push this upstream soon. Since this
> >> > is basically a hardware quirk I would like to get this into stable also
> >> > so we may run vanilla 2.6.38.1 or something like that on C8000.
> >> >
> >> > Tested-by: Rolf Eike Beer <eike-kernel@sf-tec.de>
> >>
> >> For the netdev folks: it's this patch we are talking about
> >>
> >> http://www.spinics.net/lists/linux-parisc/msg03091.html
> >>
> >> I would love to see that someone finally picks this up and pushes this
> >> upstream, CC stable. This is absolutely annoying as it breaks every time
> >> anyone touches the kernel on one of this machines.
> >>
> >> Jeff, David, James: can you please make a decision of who takes this and
> >> then just do it?
> >
> > Ping?
> 
> Ping indeed. Waiting as if HP is going to say "yes, we fucked that up.
> Go ahead with the patch." is silly.

AFAIK, HP hasn't paid anyone to work on parisc-linux in almost 10 years.
Good luck finding someone in HP who knows about C8000, much less it's
NIC firmware.

Jesse or Jeff K. should be able to accept the patch as-is.
I don't see any problem with it.

hth,
grant (ex-HP employee: 1999-2002 parisc-linux and 2002-2006 ia64-linux drivers)