Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On (22/09/07 12:24), Satyam Sharma didst pronounce: > > > On Thu, 20 Sep 2007, Satyam Sharma wrote: > > > > BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... > > IIRC I got build failures in: > > > drivers/net/spider_net.c > > > [PATCH -mm] spider_net: Misc build fixes after recent netdev stats changes > > Unbreak the following: > > drivers/net/spider_net.c: In function 'spider_net_release_tx_chain': > drivers/net/spider_net.c:818: error: 'dev' undeclared (first use in this > function) > drivers/net/spider_net.c:818: error: (Each undeclared identifier is reported > only once > drivers/net/spider_net.c:818: error: for each function it appears in.) > drivers/net/spider_net.c: In function 'spider_net_xmit': > drivers/net/spider_net.c:922: error: 'dev' undeclared (first use in this > function) > drivers/net/spider_net.c: In function 'spider_net_pass_skb_up': > drivers/net/spider_net.c:1018: error: 'dev' undeclared (first use in this > function) > drivers/net/spider_net.c: In function 'spider_net_decode_one_descr': > drivers/net/spider_net.c:1215: error: 'dev' undeclared (first use in this > function) > make[2]: *** [drivers/net/spider_net.o] Error 1 > > Signed-off-by: Satyam Sharma <[EMAIL PROTECTED]> I've confirmed that this patch fixes the build error in question. Acked-by: Mel Gorman <[EMAIL PROTECTED]> > > --- > > drivers/net/spider_net.c | 24 +++- > 1 file changed, 11 insertions(+), 13 deletions(-) > > diff -ruNp a/drivers/net/spider_net.c b/drivers/net/spider_net.c > --- a/drivers/net/spider_net.c2007-09-22 06:26:39.0 +0530 > +++ b/drivers/net/spider_net.c2007-09-22 12:12:23.0 +0530 > @@ -795,6 +795,7 @@ spider_net_set_low_watermark(struct spid > static int > spider_net_release_tx_chain(struct spider_net_card *card, int brutal) > { > + struct net_device *dev = card->netdev; > struct spider_net_descr_chain *chain = >tx_chain; > struct spider_net_descr *descr; > struct spider_net_hw_descr *hwdescr; > @@ -919,7 +920,7 @@ spider_net_xmit(struct sk_buff *skb, str > spider_net_release_tx_chain(card, 0); > > if (spider_net_prepare_tx_descr(card, skb) != 0) { > - dev->stats.tx_dropped++; > + netdev->stats.tx_dropped++; > netif_stop_queue(netdev); > return NETDEV_TX_BUSY; > } > @@ -979,16 +980,12 @@ static void > spider_net_pass_skb_up(struct spider_net_descr *descr, > struct spider_net_card *card) > { > - struct spider_net_hw_descr *hwdescr= descr->hwdescr; > - struct sk_buff *skb; > - struct net_device *netdev; > - u32 data_status, data_error; > - > - data_status = hwdescr->data_status; > - data_error = hwdescr->data_error; > - netdev = card->netdev; > + struct spider_net_hw_descr *hwdescr = descr->hwdescr; > + struct sk_buff *skb = descr->skb; > + struct net_device *netdev = card->netdev; > + u32 data_status = hwdescr->data_status; > + u32 data_error = hwdescr->data_error; > > - skb = descr->skb; > skb_put(skb, hwdescr->valid_size); > > /* the card seems to add 2 bytes of junk in front > @@ -1015,8 +1012,8 @@ spider_net_pass_skb_up(struct spider_net > } > > /* update netdevice statistics */ > - dev->stats.rx_packets++; > - dev->stats.rx_bytes += skb->len; > + netdev->stats.rx_packets++; > + netdev->stats.rx_bytes += skb->len; > > /* pass skb up to stack */ > netif_receive_skb(skb); > @@ -1184,6 +1181,7 @@ static int spider_net_resync_tail_ptr(st > static int > spider_net_decode_one_descr(struct spider_net_card *card) > { > + struct net_device *dev = card->netdev; > struct spider_net_descr_chain *chain = >rx_chain; > struct spider_net_descr *descr = chain->tail; > struct spider_net_hw_descr *hwdescr = descr->hwdescr; > @@ -1210,7 +1208,7 @@ spider_net_decode_one_descr(struct spide >(status == SPIDER_NET_DESCR_PROTECTION_ERROR) || >(status == SPIDER_NET_DESCR_FORCE_END) ) { > if (netif_msg_rx_err(card)) > - dev_err(>netdev->dev, > + dev_err(>dev, > "dropping RX descriptor with state %d\n", > status); > dev->stats.rx_dropped++; > goto bad_desc; > -- -- Mel Gorman Part-time Phd Student Linux Technology Center University of Limerick IBM Dublin Software Lab - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On (22/09/07 12:24), Satyam Sharma didst pronounce: On Thu, 20 Sep 2007, Satyam Sharma wrote: BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... IIRC I got build failures in: drivers/net/spider_net.c [PATCH -mm] spider_net: Misc build fixes after recent netdev stats changes Unbreak the following: drivers/net/spider_net.c: In function 'spider_net_release_tx_chain': drivers/net/spider_net.c:818: error: 'dev' undeclared (first use in this function) drivers/net/spider_net.c:818: error: (Each undeclared identifier is reported only once drivers/net/spider_net.c:818: error: for each function it appears in.) drivers/net/spider_net.c: In function 'spider_net_xmit': drivers/net/spider_net.c:922: error: 'dev' undeclared (first use in this function) drivers/net/spider_net.c: In function 'spider_net_pass_skb_up': drivers/net/spider_net.c:1018: error: 'dev' undeclared (first use in this function) drivers/net/spider_net.c: In function 'spider_net_decode_one_descr': drivers/net/spider_net.c:1215: error: 'dev' undeclared (first use in this function) make[2]: *** [drivers/net/spider_net.o] Error 1 Signed-off-by: Satyam Sharma [EMAIL PROTECTED] I've confirmed that this patch fixes the build error in question. Acked-by: Mel Gorman [EMAIL PROTECTED] --- drivers/net/spider_net.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff -ruNp a/drivers/net/spider_net.c b/drivers/net/spider_net.c --- a/drivers/net/spider_net.c2007-09-22 06:26:39.0 +0530 +++ b/drivers/net/spider_net.c2007-09-22 12:12:23.0 +0530 @@ -795,6 +795,7 @@ spider_net_set_low_watermark(struct spid static int spider_net_release_tx_chain(struct spider_net_card *card, int brutal) { + struct net_device *dev = card-netdev; struct spider_net_descr_chain *chain = card-tx_chain; struct spider_net_descr *descr; struct spider_net_hw_descr *hwdescr; @@ -919,7 +920,7 @@ spider_net_xmit(struct sk_buff *skb, str spider_net_release_tx_chain(card, 0); if (spider_net_prepare_tx_descr(card, skb) != 0) { - dev-stats.tx_dropped++; + netdev-stats.tx_dropped++; netif_stop_queue(netdev); return NETDEV_TX_BUSY; } @@ -979,16 +980,12 @@ static void spider_net_pass_skb_up(struct spider_net_descr *descr, struct spider_net_card *card) { - struct spider_net_hw_descr *hwdescr= descr-hwdescr; - struct sk_buff *skb; - struct net_device *netdev; - u32 data_status, data_error; - - data_status = hwdescr-data_status; - data_error = hwdescr-data_error; - netdev = card-netdev; + struct spider_net_hw_descr *hwdescr = descr-hwdescr; + struct sk_buff *skb = descr-skb; + struct net_device *netdev = card-netdev; + u32 data_status = hwdescr-data_status; + u32 data_error = hwdescr-data_error; - skb = descr-skb; skb_put(skb, hwdescr-valid_size); /* the card seems to add 2 bytes of junk in front @@ -1015,8 +1012,8 @@ spider_net_pass_skb_up(struct spider_net } /* update netdevice statistics */ - dev-stats.rx_packets++; - dev-stats.rx_bytes += skb-len; + netdev-stats.rx_packets++; + netdev-stats.rx_bytes += skb-len; /* pass skb up to stack */ netif_receive_skb(skb); @@ -1184,6 +1181,7 @@ static int spider_net_resync_tail_ptr(st static int spider_net_decode_one_descr(struct spider_net_card *card) { + struct net_device *dev = card-netdev; struct spider_net_descr_chain *chain = card-rx_chain; struct spider_net_descr *descr = chain-tail; struct spider_net_hw_descr *hwdescr = descr-hwdescr; @@ -1210,7 +1208,7 @@ spider_net_decode_one_descr(struct spide (status == SPIDER_NET_DESCR_PROTECTION_ERROR) || (status == SPIDER_NET_DESCR_FORCE_END) ) { if (netif_msg_rx_err(card)) - dev_err(card-netdev-dev, + dev_err(dev-dev, dropping RX descriptor with state %d\n, status); dev-stats.rx_dropped++; goto bad_desc; -- -- Mel Gorman Part-time Phd Student Linux Technology Center University of Limerick IBM Dublin Software Lab - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Satyam Sharma wrote: > > BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... > IIRC I got build failures in: > drivers/net/spider_net.c Fixing the above showed up another problem in another file of the same driver (drivers/net/spider_net_ethtool.c) [PATCH -mm] spider_net_ethtool: Keep up with recent netdev stats changes Unbreak the following: drivers/net/spider_net_ethtool.c: In function 'spider_net_get_ethtool_stats': drivers/net/spider_net_ethtool.c:160: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:161: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:162: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:163: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:164: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:165: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:166: error: structure has no member named 'netdev_stats' make[2]: *** [drivers/net/spider_net_ethtool.o] Error 1 Also do another ARRAY_SIZE() cleanup while at it. Signed-off-by: Satyam Sharma <[EMAIL PROTECTED]> --- drivers/net/spider_net_ethtool.c | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff -ruNp a/drivers/net/spider_net_ethtool.c b/drivers/net/spider_net_ethtool.c --- a/drivers/net/spider_net_ethtool.c 2007-09-22 06:26:39.0 +0530 +++ b/drivers/net/spider_net_ethtool.c 2007-09-22 12:43:51.0 +0530 @@ -28,8 +28,6 @@ #include "spider_net.h" -#define SPIDER_NET_NUM_STATS 13 - static struct { const char str[ETH_GSTRING_LEN]; } ethtool_stats_keys[] = { @@ -149,7 +147,7 @@ spider_net_ethtool_get_ringparam(struct static int spider_net_get_stats_count(struct net_device *netdev) { - return SPIDER_NET_NUM_STATS; + return ARRAY_SIZE(ethtool_stats_keys); } static void spider_net_get_ethtool_stats(struct net_device *netdev, @@ -157,13 +155,13 @@ static void spider_net_get_ethtool_stats { struct spider_net_card *card = netdev->priv; - data[0] = card->netdev_stats.tx_packets; - data[1] = card->netdev_stats.tx_bytes; - data[2] = card->netdev_stats.rx_packets; - data[3] = card->netdev_stats.rx_bytes; - data[4] = card->netdev_stats.tx_errors; - data[5] = card->netdev_stats.tx_dropped; - data[6] = card->netdev_stats.rx_dropped; + data[0] = netdev->stats.tx_packets; + data[1] = netdev->stats.tx_bytes; + data[2] = netdev->stats.rx_packets; + data[3] = netdev->stats.rx_bytes; + data[4] = netdev->stats.tx_errors; + data[5] = netdev->stats.tx_dropped; + data[6] = netdev->stats.rx_dropped; data[7] = card->spider_stats.rx_desc_error; data[8] = card->spider_stats.tx_timeouts; data[9] = card->spider_stats.alloc_rx_skb_error; - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Satyam Sharma wrote: > > BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... > IIRC I got build failures in: > drivers/net/spider_net.c [PATCH -mm] spider_net: Misc build fixes after recent netdev stats changes Unbreak the following: drivers/net/spider_net.c: In function 'spider_net_release_tx_chain': drivers/net/spider_net.c:818: error: 'dev' undeclared (first use in this function) drivers/net/spider_net.c:818: error: (Each undeclared identifier is reported only once drivers/net/spider_net.c:818: error: for each function it appears in.) drivers/net/spider_net.c: In function 'spider_net_xmit': drivers/net/spider_net.c:922: error: 'dev' undeclared (first use in this function) drivers/net/spider_net.c: In function 'spider_net_pass_skb_up': drivers/net/spider_net.c:1018: error: 'dev' undeclared (first use in this function) drivers/net/spider_net.c: In function 'spider_net_decode_one_descr': drivers/net/spider_net.c:1215: error: 'dev' undeclared (first use in this function) make[2]: *** [drivers/net/spider_net.o] Error 1 Signed-off-by: Satyam Sharma <[EMAIL PROTECTED]> --- drivers/net/spider_net.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff -ruNp a/drivers/net/spider_net.c b/drivers/net/spider_net.c --- a/drivers/net/spider_net.c 2007-09-22 06:26:39.0 +0530 +++ b/drivers/net/spider_net.c 2007-09-22 12:12:23.0 +0530 @@ -795,6 +795,7 @@ spider_net_set_low_watermark(struct spid static int spider_net_release_tx_chain(struct spider_net_card *card, int brutal) { + struct net_device *dev = card->netdev; struct spider_net_descr_chain *chain = >tx_chain; struct spider_net_descr *descr; struct spider_net_hw_descr *hwdescr; @@ -919,7 +920,7 @@ spider_net_xmit(struct sk_buff *skb, str spider_net_release_tx_chain(card, 0); if (spider_net_prepare_tx_descr(card, skb) != 0) { - dev->stats.tx_dropped++; + netdev->stats.tx_dropped++; netif_stop_queue(netdev); return NETDEV_TX_BUSY; } @@ -979,16 +980,12 @@ static void spider_net_pass_skb_up(struct spider_net_descr *descr, struct spider_net_card *card) { - struct spider_net_hw_descr *hwdescr= descr->hwdescr; - struct sk_buff *skb; - struct net_device *netdev; - u32 data_status, data_error; - - data_status = hwdescr->data_status; - data_error = hwdescr->data_error; - netdev = card->netdev; + struct spider_net_hw_descr *hwdescr = descr->hwdescr; + struct sk_buff *skb = descr->skb; + struct net_device *netdev = card->netdev; + u32 data_status = hwdescr->data_status; + u32 data_error = hwdescr->data_error; - skb = descr->skb; skb_put(skb, hwdescr->valid_size); /* the card seems to add 2 bytes of junk in front @@ -1015,8 +1012,8 @@ spider_net_pass_skb_up(struct spider_net } /* update netdevice statistics */ - dev->stats.rx_packets++; - dev->stats.rx_bytes += skb->len; + netdev->stats.rx_packets++; + netdev->stats.rx_bytes += skb->len; /* pass skb up to stack */ netif_receive_skb(skb); @@ -1184,6 +1181,7 @@ static int spider_net_resync_tail_ptr(st static int spider_net_decode_one_descr(struct spider_net_card *card) { + struct net_device *dev = card->netdev; struct spider_net_descr_chain *chain = >rx_chain; struct spider_net_descr *descr = chain->tail; struct spider_net_hw_descr *hwdescr = descr->hwdescr; @@ -1210,7 +1208,7 @@ spider_net_decode_one_descr(struct spide (status == SPIDER_NET_DESCR_PROTECTION_ERROR) || (status == SPIDER_NET_DESCR_FORCE_END) ) { if (netif_msg_rx_err(card)) - dev_err(>netdev->dev, + dev_err(>dev, "dropping RX descriptor with state %d\n", status); dev->stats.rx_dropped++; goto bad_desc; - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Satyam Sharma wrote: > > BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... > IIRC I got build failures in: > drivers/md/raid6int8.c This turned out to be a gcc bug -- I was using an old cross-compiler. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Satyam Sharma wrote: > > BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... > IIRC I got build failures in: > drivers/ata/pata_scc.c http://lkml.org/lkml/2007/9/21/557 - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Satyam Sharma wrote: BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... IIRC I got build failures in: drivers/ata/pata_scc.c http://lkml.org/lkml/2007/9/21/557 - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Satyam Sharma wrote: BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... IIRC I got build failures in: drivers/md/raid6int8.c This turned out to be a gcc bug -- I was using an old cross-compiler. - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Satyam Sharma wrote: BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... IIRC I got build failures in: drivers/net/spider_net.c [PATCH -mm] spider_net: Misc build fixes after recent netdev stats changes Unbreak the following: drivers/net/spider_net.c: In function 'spider_net_release_tx_chain': drivers/net/spider_net.c:818: error: 'dev' undeclared (first use in this function) drivers/net/spider_net.c:818: error: (Each undeclared identifier is reported only once drivers/net/spider_net.c:818: error: for each function it appears in.) drivers/net/spider_net.c: In function 'spider_net_xmit': drivers/net/spider_net.c:922: error: 'dev' undeclared (first use in this function) drivers/net/spider_net.c: In function 'spider_net_pass_skb_up': drivers/net/spider_net.c:1018: error: 'dev' undeclared (first use in this function) drivers/net/spider_net.c: In function 'spider_net_decode_one_descr': drivers/net/spider_net.c:1215: error: 'dev' undeclared (first use in this function) make[2]: *** [drivers/net/spider_net.o] Error 1 Signed-off-by: Satyam Sharma [EMAIL PROTECTED] --- drivers/net/spider_net.c | 24 +++- 1 file changed, 11 insertions(+), 13 deletions(-) diff -ruNp a/drivers/net/spider_net.c b/drivers/net/spider_net.c --- a/drivers/net/spider_net.c 2007-09-22 06:26:39.0 +0530 +++ b/drivers/net/spider_net.c 2007-09-22 12:12:23.0 +0530 @@ -795,6 +795,7 @@ spider_net_set_low_watermark(struct spid static int spider_net_release_tx_chain(struct spider_net_card *card, int brutal) { + struct net_device *dev = card-netdev; struct spider_net_descr_chain *chain = card-tx_chain; struct spider_net_descr *descr; struct spider_net_hw_descr *hwdescr; @@ -919,7 +920,7 @@ spider_net_xmit(struct sk_buff *skb, str spider_net_release_tx_chain(card, 0); if (spider_net_prepare_tx_descr(card, skb) != 0) { - dev-stats.tx_dropped++; + netdev-stats.tx_dropped++; netif_stop_queue(netdev); return NETDEV_TX_BUSY; } @@ -979,16 +980,12 @@ static void spider_net_pass_skb_up(struct spider_net_descr *descr, struct spider_net_card *card) { - struct spider_net_hw_descr *hwdescr= descr-hwdescr; - struct sk_buff *skb; - struct net_device *netdev; - u32 data_status, data_error; - - data_status = hwdescr-data_status; - data_error = hwdescr-data_error; - netdev = card-netdev; + struct spider_net_hw_descr *hwdescr = descr-hwdescr; + struct sk_buff *skb = descr-skb; + struct net_device *netdev = card-netdev; + u32 data_status = hwdescr-data_status; + u32 data_error = hwdescr-data_error; - skb = descr-skb; skb_put(skb, hwdescr-valid_size); /* the card seems to add 2 bytes of junk in front @@ -1015,8 +1012,8 @@ spider_net_pass_skb_up(struct spider_net } /* update netdevice statistics */ - dev-stats.rx_packets++; - dev-stats.rx_bytes += skb-len; + netdev-stats.rx_packets++; + netdev-stats.rx_bytes += skb-len; /* pass skb up to stack */ netif_receive_skb(skb); @@ -1184,6 +1181,7 @@ static int spider_net_resync_tail_ptr(st static int spider_net_decode_one_descr(struct spider_net_card *card) { + struct net_device *dev = card-netdev; struct spider_net_descr_chain *chain = card-rx_chain; struct spider_net_descr *descr = chain-tail; struct spider_net_hw_descr *hwdescr = descr-hwdescr; @@ -1210,7 +1208,7 @@ spider_net_decode_one_descr(struct spide (status == SPIDER_NET_DESCR_PROTECTION_ERROR) || (status == SPIDER_NET_DESCR_FORCE_END) ) { if (netif_msg_rx_err(card)) - dev_err(card-netdev-dev, + dev_err(dev-dev, dropping RX descriptor with state %d\n, status); dev-stats.rx_dropped++; goto bad_desc; - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: 2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Satyam Sharma wrote: BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... IIRC I got build failures in: drivers/net/spider_net.c Fixing the above showed up another problem in another file of the same driver (drivers/net/spider_net_ethtool.c) [PATCH -mm] spider_net_ethtool: Keep up with recent netdev stats changes Unbreak the following: drivers/net/spider_net_ethtool.c: In function 'spider_net_get_ethtool_stats': drivers/net/spider_net_ethtool.c:160: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:161: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:162: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:163: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:164: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:165: error: structure has no member named 'netdev_stats' drivers/net/spider_net_ethtool.c:166: error: structure has no member named 'netdev_stats' make[2]: *** [drivers/net/spider_net_ethtool.o] Error 1 Also do another ARRAY_SIZE() cleanup while at it. Signed-off-by: Satyam Sharma [EMAIL PROTECTED] --- drivers/net/spider_net_ethtool.c | 18 -- 1 file changed, 8 insertions(+), 10 deletions(-) diff -ruNp a/drivers/net/spider_net_ethtool.c b/drivers/net/spider_net_ethtool.c --- a/drivers/net/spider_net_ethtool.c 2007-09-22 06:26:39.0 +0530 +++ b/drivers/net/spider_net_ethtool.c 2007-09-22 12:43:51.0 +0530 @@ -28,8 +28,6 @@ #include spider_net.h -#define SPIDER_NET_NUM_STATS 13 - static struct { const char str[ETH_GSTRING_LEN]; } ethtool_stats_keys[] = { @@ -149,7 +147,7 @@ spider_net_ethtool_get_ringparam(struct static int spider_net_get_stats_count(struct net_device *netdev) { - return SPIDER_NET_NUM_STATS; + return ARRAY_SIZE(ethtool_stats_keys); } static void spider_net_get_ethtool_stats(struct net_device *netdev, @@ -157,13 +155,13 @@ static void spider_net_get_ethtool_stats { struct spider_net_card *card = netdev-priv; - data[0] = card-netdev_stats.tx_packets; - data[1] = card-netdev_stats.tx_bytes; - data[2] = card-netdev_stats.rx_packets; - data[3] = card-netdev_stats.rx_bytes; - data[4] = card-netdev_stats.tx_errors; - data[5] = card-netdev_stats.tx_dropped; - data[6] = card-netdev_stats.rx_dropped; + data[0] = netdev-stats.tx_packets; + data[1] = netdev-stats.tx_bytes; + data[2] = netdev-stats.rx_packets; + data[3] = netdev-stats.rx_bytes; + data[4] = netdev-stats.tx_errors; + data[5] = netdev-stats.tx_dropped; + data[6] = netdev-stats.rx_dropped; data[7] = card-spider_stats.rx_desc_error; data[8] = card-spider_stats.tx_timeouts; data[9] = card-spider_stats.alloc_rx_skb_error; - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Mel Gorman wrote: > > allmodconfig on ppc64 fails to build with the following error BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... IIRC I got build failures in: drivers/ata/pata_scc.c drivers/md/raid6int8.c drivers/net/spider_net.c drivers/net/pasemi_mac.c drivers/pci/hotplug/rpadlpar_sysfs.c I was in a hurry so didn't record the errors, just noted down the files and disabled them from .config and continued building ... I'll get back to fixing these up tonight (if you don't do that by then already). Satyam - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
2.6.23-rc6-mm1: Build failures on ppc64_defconfig
On Thu, 20 Sep 2007, Mel Gorman wrote: allmodconfig on ppc64 fails to build with the following error BTW ppc64_defconfig didn't quite like 2.6.23-rc6-mm1 either ... IIRC I got build failures in: drivers/ata/pata_scc.c drivers/md/raid6int8.c drivers/net/spider_net.c drivers/net/pasemi_mac.c drivers/pci/hotplug/rpadlpar_sysfs.c I was in a hurry so didn't record the errors, just noted down the files and disabled them from .config and continued building ... I'll get back to fixing these up tonight (if you don't do that by then already). Satyam - To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/