Re: v3.12-stable-queue build errors
On 04/27/2015, 06:25 PM, David Miller wrote: > From: ebied...@xmission.com (Eric W. Biederman) > Date: Mon, 27 Apr 2015 10:35:51 -0500 > >> Jiri Slaby writes: >> >>> On 04/25/2015, 04:16 PM, Guenter Roeck wrote: Several powerpc build fail with: drivers/net/ethernet/freescale/gianfar.c: In function 'gfar_start_xmit': drivers/net/ethernet/freescale/gianfar.c:2146:3: error: implicit declaration of function 'dev_consume_skb_any' [-Werror=implicit-function-declaration] dev_consume_skb_any(skb); dev_consume_skb_any() does not exist in 3.12. Introduced by 'gianfar: Carefully free skbs in functions called by netpoll'. >>> >>> Eric, David, >>> >>> what action should I take here? >>> >>> 1) take also: >>> commit e6247027e5173c00efb2084d688d06ff835bc3b0 >>> Author: Eric Dumazet >>> Date: Thu Dec 5 04:45:08 2013 -0800 >>> >>> net: introduce dev_consume_skb_any() >>> >>> >>> 2) drop 'gianfar: Carefully free skbs in functions called by netpoll' >>> from stable-3.12 >> >> This is probably the most reasonable. KISS. > > Yeah just drop the patch. > > I didn't even want to submit this series in the first place but > people kept poking me endlessly about it. Now dropped. Thanks. -- js suse labs -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: v3.12-stable-queue build errors
From: ebied...@xmission.com (Eric W. Biederman) Date: Mon, 27 Apr 2015 10:35:51 -0500 > Jiri Slaby writes: > >> On 04/25/2015, 04:16 PM, Guenter Roeck wrote: >>> Several powerpc build fail with: >>> >>> drivers/net/ethernet/freescale/gianfar.c: In function 'gfar_start_xmit': >>> drivers/net/ethernet/freescale/gianfar.c:2146:3: error: implicit >>> declaration of >>> function 'dev_consume_skb_any' [-Werror=implicit-function-declaration] >>>dev_consume_skb_any(skb); >>> >>> dev_consume_skb_any() does not exist in 3.12. >>> >>> Introduced by 'gianfar: Carefully free skbs in functions called by netpoll'. >> >> Eric, David, >> >> what action should I take here? >> >> 1) take also: >> commit e6247027e5173c00efb2084d688d06ff835bc3b0 >> Author: Eric Dumazet >> Date: Thu Dec 5 04:45:08 2013 -0800 >> >> net: introduce dev_consume_skb_any() >> >> >> 2) drop 'gianfar: Carefully free skbs in functions called by netpoll' >> from stable-3.12 > > This is probably the most reasonable. KISS. Yeah just drop the patch. I didn't even want to submit this series in the first place but people kept poking me endlessly about it. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: v3.12-stable-queue build errors
On Mon, Apr 27, 2015 at 10:35:51AM -0500, Eric W. Biederman wrote: > Jiri Slaby writes: > > > On 04/25/2015, 04:16 PM, Guenter Roeck wrote: > >> Several powerpc build fail with: > >> > >> drivers/net/ethernet/freescale/gianfar.c: In function 'gfar_start_xmit': > >> drivers/net/ethernet/freescale/gianfar.c:2146:3: error: implicit > >> declaration of > >> function 'dev_consume_skb_any' [-Werror=implicit-function-declaration] > >>dev_consume_skb_any(skb); > >> > >> dev_consume_skb_any() does not exist in 3.12. > >> > >> Introduced by 'gianfar: Carefully free skbs in functions called by > >> netpoll'. > > > > Eric, David, > > > > what action should I take here? > > > > 1) take also: > > commit e6247027e5173c00efb2084d688d06ff835bc3b0 > > Author: Eric Dumazet > > Date: Thu Dec 5 04:45:08 2013 -0800 > > > > net: introduce dev_consume_skb_any() > > > > > > 2) drop 'gianfar: Carefully free skbs in functions called by netpoll' > > from stable-3.12 > > This is probably the most reasonable. KISS. > Greg dropped the patch from 3.10. Guenter -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: v3.12-stable-queue build errors
Jiri Slaby writes: > On 04/25/2015, 04:16 PM, Guenter Roeck wrote: >> Several powerpc build fail with: >> >> drivers/net/ethernet/freescale/gianfar.c: In function 'gfar_start_xmit': >> drivers/net/ethernet/freescale/gianfar.c:2146:3: error: implicit declaration >> of >> function 'dev_consume_skb_any' [-Werror=implicit-function-declaration] >>dev_consume_skb_any(skb); >> >> dev_consume_skb_any() does not exist in 3.12. >> >> Introduced by 'gianfar: Carefully free skbs in functions called by netpoll'. > > Eric, David, > > what action should I take here? > > 1) take also: > commit e6247027e5173c00efb2084d688d06ff835bc3b0 > Author: Eric Dumazet > Date: Thu Dec 5 04:45:08 2013 -0800 > > net: introduce dev_consume_skb_any() > > > 2) drop 'gianfar: Carefully free skbs in functions called by netpoll' > from stable-3.12 This is probably the most reasonable. KISS. > 3) any other idea? #define dev_consume_sky_any dev_kfree_skb_any The correctness fix is updating the code to handle any context. The consume vs kfree accouting difference is not particularly interesting. Eric -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: v3.12-stable-queue build errors
On 04/25/2015, 04:16 PM, Guenter Roeck wrote: > Several powerpc build fail with: > > drivers/net/ethernet/freescale/gianfar.c: In function 'gfar_start_xmit': > drivers/net/ethernet/freescale/gianfar.c:2146:3: error: implicit declaration > of > function 'dev_consume_skb_any' [-Werror=implicit-function-declaration] >dev_consume_skb_any(skb); > > dev_consume_skb_any() does not exist in 3.12. > > Introduced by 'gianfar: Carefully free skbs in functions called by netpoll'. Eric, David, what action should I take here? 1) take also: commit e6247027e5173c00efb2084d688d06ff835bc3b0 Author: Eric Dumazet Date: Thu Dec 5 04:45:08 2013 -0800 net: introduce dev_consume_skb_any() 2) drop 'gianfar: Carefully free skbs in functions called by netpoll' from stable-3.12 3) any other idea? thanks, -- js suse labs -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html