Re: [PATCH net-next 04/14] gtp: udp recv clean up

2017-09-19 Thread Harald Welte
Hi Tom, I think this patch does too many things at once: * introduce separate rx functions * convert from netif_rx to gro_cells_receive * cosmetic changes like "return -1" to "goto drop" In the context of reviewability and the "one patch per topic", I would prefer to see those separated, thanks.

Re: [PATCH net-next 04/14] gtp: udp recv clean up

2017-09-19 Thread kbuild test robot
Hi Tom, [auto build test ERROR on net-next/master] url: https://github.com/0day-ci/linux/commits/Tom-Herbert/gtp-Additional-feature-support/20170919-143920 config: i386-randconfig-x016-201738 (attached as .config) compiler: gcc-6 (Debian 6.2.0-3) 6.2.0 20160901 reproduce: # save the a

[PATCH net-next 04/14] gtp: udp recv clean up

2017-09-18 Thread Tom Herbert
Create separate UDP receive functions for GTP version 0 and version 1. Set encap_rcv appropriately when configuring a socket. Also, convert to using gro_cells. Signed-off-by: Tom Herbert --- drivers/net/gtp.c | 130 +- 1 file changed, 71 insert