netdev
[Top] [All Lists]

Re: [PATCH 2.5.70] acenic -- update to use alloc_etherdev

To: Stephen Hemminger <shemminger@xxxxxxxx>
Subject: Re: [PATCH 2.5.70] acenic -- update to use alloc_etherdev
From: Jes Sorensen <jes@xxxxxxxxxxxxxxxxxx>
Date: 12 Jun 2003 13:05:11 -0400
Cc: "David S. Miller" <davem@xxxxxxxxxx>, jgarzik@xxxxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <3EE81263.4040205@xxxxxxxx>
References: <20030611144249.7cd63c1c.shemminger@xxxxxxxx> <m3y90845nu.fsf@xxxxxxxxxxxxxxxxxx> <20030611.210445.21901735.davem@xxxxxxxxxx> <3EE81263.4040205@xxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Gnus/5.0808 (Gnus v5.8.8) Emacs/20.7
>>>>> "Stephen" == Stephen Hemminger <shemminger@xxxxxxxx> writes:

Stephen> The funny thing is this alloc_etherdev patch did not change
Stephen> the compatiablity one bit.  Just for grins, took the 2.5
Stephen> driver back into the 2.4.18 and it doesn't build.  The
Stephen> problem is it doesn't know what irqreturn_t is.  The enclosed
Stephen> cribbed from atm/he.c fixes that, but it still redefines
Stephen> local_irq_save etc.

Hi Stephen,

I went back and looked at the comment in 2.4 for when alloc_etherdev
was introduced, but clearly I got the mapping of this to the 2.4.x
release dates wrong. That was my bad.

As for local_irq_save() those patches will still be needed if they are
not present in 2.4.17 (I think this is probably a reasonable cut-off
release), if it's in 2.4.17+ I'll agree we can pull it.

But the irqreturn_t compat stuff still needs to go in since it's
clearly not going to be present in 2.4.18 etc.

Cheers,
Jes

<Prev in Thread] Current Thread [Next in Thread>