* Re: 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! [not found] <20070619190253.GA19419@aepfle.de> @ 2007-06-19 20:10 ` Stephen Hemminger 2007-06-19 20:41 ` Olaf Hering 0 siblings, 1 reply; 6+ messages in thread From: Stephen Hemminger @ 2007-06-19 20:10 UTC (permalink / raw) To: Olaf Hering; +Cc: linux-kernel, netdev On Tue, 19 Jun 2007 21:02:53 +0200 Olaf Hering <olaf@aepfle.de> wrote: > > What happend to __ucmpdi2 from David Woodhouse? > google has a few hits about stuff like this on 32bit powerpc with gcc 4.1.2: > > ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! > > using the drivers/net/s2io* files from 2.6.21 with 2.6.22-rc5 fixes the > compile. > > 25805dcf9d83098cf5492117ad2669cd14cc9b24 adds two u64 >>= 48 followed by > a switch statement (line 2889 and 6816). Probably the "switch(err) {" needs a cast to a smaller type (like u8). -- Stephen Hemminger <shemminger@linux-foundation.org> ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! 2007-06-19 20:10 ` 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! Stephen Hemminger @ 2007-06-19 20:41 ` Olaf Hering 2007-06-21 9:55 ` Sivakumar Subramani 2007-06-27 6:33 ` Jeff Garzik 0 siblings, 2 replies; 6+ messages in thread From: Olaf Hering @ 2007-06-19 20:41 UTC (permalink / raw) To: Stephen Hemminger; +Cc: linux-kernel, netdev On Tue, Jun 19, Stephen Hemminger wrote: > On Tue, 19 Jun 2007 21:02:53 +0200 > Olaf Hering <olaf@aepfle.de> wrote: > > > > > What happend to __ucmpdi2 from David Woodhouse? > > google has a few hits about stuff like this on 32bit powerpc with gcc 4.1.2: > > > > ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! > > > > using the drivers/net/s2io* files from 2.6.21 with 2.6.22-rc5 fixes the > > compile. > > > > 25805dcf9d83098cf5492117ad2669cd14cc9b24 adds two u64 >>= 48 followed by > > a switch statement (line 2889 and 6816). > > Probably the "switch(err) {" needs a cast to a smaller type (like u8). This change removes the calls to __ucmpdi2. --- drivers/net/s2io.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) --- a/drivers/net/s2io.c +++ b/drivers/net/s2io.c @@ -2868,6 +2868,7 @@ static void tx_intr_handler(struct fifo_ struct tx_curr_get_info get_info, put_info; struct sk_buff *skb; struct TxD *txdlp; + u8 err_mask; get_info = fifo_data->tx_curr_get_info; memcpy(&put_info, &fifo_data->tx_curr_put_info, sizeof(put_info)); @@ -2886,8 +2887,8 @@ static void tx_intr_handler(struct fifo_ } /* update t_code statistics */ - err >>= 48; - switch(err) { + err_mask = err >> 48; + switch(err_mask) { case 2: nic->mac_control.stats_info->sw_stat. tx_buf_abort_cnt++; @@ -6805,6 +6806,7 @@ static int rx_osm_handler(struct ring_in u16 l3_csum, l4_csum; unsigned long long err = rxdp->Control_1 & RXD_T_CODE; struct lro *lro; + u8 err_mask; skb->dev = dev; @@ -6813,8 +6815,8 @@ static int rx_osm_handler(struct ring_in if (err & 0x1) { sp->mac_control.stats_info->sw_stat.parity_err_cnt++; } - err >>= 48; - switch(err) { + err_mask = err >> 48; + switch(err_mask) { case 1: sp->mac_control.stats_info->sw_stat. rx_parity_err_cnt++; @@ -6867,9 +6869,9 @@ static int rx_osm_handler(struct ring_in * Note that in this case, since checksum will be incorrect, * stack will validate the same. */ - if (err != 0x5) { - DBG_PRINT(ERR_DBG, "%s: Rx error Value: 0x%llx\n", - dev->name, err); + if (err_mask != 0x5) { + DBG_PRINT(ERR_DBG, "%s: Rx error Value: 0x%x\n", + dev->name, err_mask); sp->stats.rx_crc_errors++; sp->mac_control.stats_info->sw_stat.mem_freed += skb->truesize; ^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! 2007-06-19 20:41 ` Olaf Hering @ 2007-06-21 9:55 ` Sivakumar Subramani 2007-06-26 7:01 ` Andrew Morton 2007-06-27 6:33 ` Jeff Garzik 1 sibling, 1 reply; 6+ messages in thread From: Sivakumar Subramani @ 2007-06-21 9:55 UTC (permalink / raw) To: Olaf Hering, Stephen Hemminger Cc: linux-kernel, netdev, Leonid Grossman, Rastapur Santosh, Ramkrishna Vepa, Sriram Rapuru, Sreenivasa Honnur Hi, We will include this fix in next set of patch submission. Thanks for the fix. Thanks, ~Siva -----Original Message----- From: netdev-owner@vger.kernel.org [mailto:netdev-owner@vger.kernel.org] On Behalf Of Olaf Hering Sent: Wednesday, June 20, 2007 2:11 AM To: Stephen Hemminger Cc: linux-kernel@vger.kernel.org; netdev@vger.kernel.org Subject: Re: 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! On Tue, Jun 19, Stephen Hemminger wrote: > On Tue, 19 Jun 2007 21:02:53 +0200 > Olaf Hering <olaf@aepfle.de> wrote: > > > > > What happend to __ucmpdi2 from David Woodhouse? > > google has a few hits about stuff like this on 32bit powerpc with gcc 4.1.2: > > > > ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! > > > > using the drivers/net/s2io* files from 2.6.21 with 2.6.22-rc5 fixes > > the compile. > > > > 25805dcf9d83098cf5492117ad2669cd14cc9b24 adds two u64 >>= 48 > > followed by a switch statement (line 2889 and 6816). > > Probably the "switch(err) {" needs a cast to a smaller type (like u8). This change removes the calls to __ucmpdi2. --- drivers/net/s2io.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) --- a/drivers/net/s2io.c +++ b/drivers/net/s2io.c @@ -2868,6 +2868,7 @@ static void tx_intr_handler(struct fifo_ struct tx_curr_get_info get_info, put_info; struct sk_buff *skb; struct TxD *txdlp; + u8 err_mask; get_info = fifo_data->tx_curr_get_info; memcpy(&put_info, &fifo_data->tx_curr_put_info, sizeof(put_info)); @@ -2886,8 +2887,8 @@ static void tx_intr_handler(struct fifo_ } /* update t_code statistics */ - err >>= 48; - switch(err) { + err_mask = err >> 48; + switch(err_mask) { case 2: nic->mac_control.stats_info->sw_stat. tx_buf_abort_cnt++; @@ -6805,6 +6806,7 @@ static int rx_osm_handler(struct ring_in u16 l3_csum, l4_csum; unsigned long long err = rxdp->Control_1 & RXD_T_CODE; struct lro *lro; + u8 err_mask; skb->dev = dev; @@ -6813,8 +6815,8 @@ static int rx_osm_handler(struct ring_in if (err & 0x1) { sp->mac_control.stats_info->sw_stat.parity_err_cnt++; } - err >>= 48; - switch(err) { + err_mask = err >> 48; + switch(err_mask) { case 1: sp->mac_control.stats_info->sw_stat. rx_parity_err_cnt++; @@ -6867,9 +6869,9 @@ static int rx_osm_handler(struct ring_in * Note that in this case, since checksum will be incorrect, * stack will validate the same. */ - if (err != 0x5) { - DBG_PRINT(ERR_DBG, "%s: Rx error Value: 0x%llx\n", - dev->name, err); + if (err_mask != 0x5) { + DBG_PRINT(ERR_DBG, "%s: Rx error Value: 0x%x\n", + dev->name, err_mask); sp->stats.rx_crc_errors++; sp->mac_control.stats_info->sw_stat.mem_freed += skb->truesize; - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! 2007-06-21 9:55 ` Sivakumar Subramani @ 2007-06-26 7:01 ` Andrew Morton 2007-06-26 10:07 ` Jeff Garzik 0 siblings, 1 reply; 6+ messages in thread From: Andrew Morton @ 2007-06-26 7:01 UTC (permalink / raw) To: Sivakumar Subramani Cc: Olaf Hering, Stephen Hemminger, linux-kernel, netdev, Leonid Grossman, Rastapur Santosh, Ramkrishna Vepa, Sriram Rapuru, Sreenivasa Honnur, Jeff Garzik On Thu, 21 Jun 2007 05:55:13 -0400 "Sivakumar Subramani" <Sivakumar.Subramani@neterion.com> wrote: > -----Original Message----- > > From: netdev-owner@vger.kernel.org [mailto:netdev-owner@vger.kernel.org] > > On Behalf Of Olaf Hering > > Sent: Wednesday, June 20, 2007 2:11 AM > > To: Stephen Hemminger > > Cc: linux-kernel@vger.kernel.org; netdev@vger.kernel.org > > Subject: Re: 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! > > > > On Tue, Jun 19, Stephen Hemminger wrote: > > > > > On Tue, 19 Jun 2007 21:02:53 +0200 > > > Olaf Hering <olaf@aepfle.de> wrote: > > > > > > > > > > > What happend to __ucmpdi2 from David Woodhouse? > > > > google has a few hits about stuff like this on 32bit powerpc with > > gcc 4.1.2: > > > > > > > > ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! > > > > > > > > using the drivers/net/s2io* files from 2.6.21 with 2.6.22-rc5 fixes > > > > the compile. > > > > > > > > 25805dcf9d83098cf5492117ad2669cd14cc9b24 adds two u64 >>= 48 > > > > followed by a switch statement (line 2889 and 6816). > > > > > > Probably the "switch(err) {" needs a cast to a smaller type (like u8). > > > > This change removes the calls to __ucmpdi2. (fixes quoting, fixes top-posting. Please don't top-post). > Hi, > > We will include this fix in next set of patch submission. Thanks for the > fix. > > --- > > drivers/net/s2io.c | 16 +++++++++------- > > 1 file changed, 9 insertions(+), 7 deletions(-) > > > > --- a/drivers/net/s2io.c > > +++ b/drivers/net/s2io.c > > @@ -2868,6 +2868,7 @@ static void tx_intr_handler(struct fifo_ > > struct tx_curr_get_info get_info, put_info; > > struct sk_buff *skb; > > struct TxD *txdlp; > > + u8 err_mask; > > > > get_info = fifo_data->tx_curr_get_info; > > memcpy(&put_info, &fifo_data->tx_curr_put_info, > > sizeof(put_info)); @@ -2886,8 +2887,8 @@ static void > > tx_intr_handler(struct fifo_ > > } > > > > /* update t_code statistics */ > > - err >>= 48; > > - switch(err) { > > + err_mask = err >> 48; > > + switch(err_mask) { > > case 2: > > > > nic->mac_control.stats_info->sw_stat. > > > > tx_buf_abort_cnt++; > > @@ -6805,6 +6806,7 @@ static int rx_osm_handler(struct ring_in > > u16 l3_csum, l4_csum; > > unsigned long long err = rxdp->Control_1 & RXD_T_CODE; > > struct lro *lro; > > + u8 err_mask; > > > > skb->dev = dev; > > > > @@ -6813,8 +6815,8 @@ static int rx_osm_handler(struct ring_in > > if (err & 0x1) { > > > > sp->mac_control.stats_info->sw_stat.parity_err_cnt++; > > } > > - err >>= 48; > > - switch(err) { > > + err_mask = err >> 48; > > + switch(err_mask) { > > case 1: > > sp->mac_control.stats_info->sw_stat. > > rx_parity_err_cnt++; > > @@ -6867,9 +6869,9 @@ static int rx_osm_handler(struct ring_in > > * Note that in this case, since checksum will be > > incorrect, > > * stack will validate the same. > > */ > > - if (err != 0x5) { > > - DBG_PRINT(ERR_DBG, "%s: Rx error Value: > > 0x%llx\n", > > - dev->name, err); > > + if (err_mask != 0x5) { > > + DBG_PRINT(ERR_DBG, "%s: Rx error Value: 0x%x\n", > > + dev->name, err_mask); > > sp->stats.rx_crc_errors++; > > sp->mac_control.stats_info->sw_stat.mem_freed > > += skb->truesize; > This fix is still not present in anyone's tree and is required for 2.6.22. Where are we up to with it? ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! 2007-06-26 7:01 ` Andrew Morton @ 2007-06-26 10:07 ` Jeff Garzik 0 siblings, 0 replies; 6+ messages in thread From: Jeff Garzik @ 2007-06-26 10:07 UTC (permalink / raw) To: Andrew Morton Cc: Sivakumar Subramani, Olaf Hering, Stephen Hemminger, linux-kernel, netdev, Leonid Grossman, Rastapur Santosh, Ramkrishna Vepa, Sriram Rapuru, Sreenivasa Honnur Andrew Morton wrote: > This fix is still not present in anyone's tree and is required for > 2.6.22. Where are we up to with it? It's in my mbox queue for 2.6.22 (hopefully today). Jeff ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! 2007-06-19 20:41 ` Olaf Hering 2007-06-21 9:55 ` Sivakumar Subramani @ 2007-06-27 6:33 ` Jeff Garzik 1 sibling, 0 replies; 6+ messages in thread From: Jeff Garzik @ 2007-06-27 6:33 UTC (permalink / raw) To: Olaf Hering; +Cc: Stephen Hemminger, linux-kernel, netdev Olaf Hering wrote: > On Tue, Jun 19, Stephen Hemminger wrote: > >> On Tue, 19 Jun 2007 21:02:53 +0200 >> Olaf Hering <olaf@aepfle.de> wrote: >> >>> What happend to __ucmpdi2 from David Woodhouse? >>> google has a few hits about stuff like this on 32bit powerpc with gcc 4.1.2: >>> >>> ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! >>> >>> using the drivers/net/s2io* files from 2.6.21 with 2.6.22-rc5 fixes the >>> compile. >>> >>> 25805dcf9d83098cf5492117ad2669cd14cc9b24 adds two u64 >>= 48 followed by >>> a switch statement (line 2889 and 6816). >> Probably the "switch(err) {" needs a cast to a smaller type (like u8). > > This change removes the calls to __ucmpdi2. > > --- > drivers/net/s2io.c | 16 +++++++++------- > 1 file changed, 9 insertions(+), 7 deletions(-) applied to #upstream-fixes ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2007-06-27 6:33 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20070619190253.GA19419@aepfle.de>
2007-06-19 20:10 ` 2.6.22: ERROR: "__ucmpdi2" [drivers/net/s2io.ko] undefined! Stephen Hemminger
2007-06-19 20:41 ` Olaf Hering
2007-06-21 9:55 ` Sivakumar Subramani
2007-06-26 7:01 ` Andrew Morton
2007-06-26 10:07 ` Jeff Garzik
2007-06-27 6:33 ` Jeff Garzik
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).