Hi Neil, I love your patch! Perhaps something to improve:
[auto build test WARNING on net-next/master] url: https://github.com/0day-ci/linux/commits/Neil-Horman/af_packet-convert-pending-frame-counter-to-atomic_t/20190629-032437 config: x86_64-randconfig-x007-201925 (attached as .config) compiler: gcc-7 (Debian 7.4.0-9) 7.4.0 reproduce: # save the attached .config to linux build tree make ARCH=x86_64 If you fix the issue, kindly add following tag Reported-by: kbuild test robot <l...@intel.com> All warnings (new ones prefixed by >>): net/packet/af_packet.c: In function 'packet_read_pending': >> net/packet/af_packet.c:1168:1: warning: no return statement in function >> returning non-void [-Wreturn-type] } ^ vim +1168 net/packet/af_packet.c b0138408 Daniel Borkmann 2014-01-15 1164 b0138408 Daniel Borkmann 2014-01-15 1165 static unsigned int packet_read_pending(const struct packet_ring_buffer *rb) b0138408 Daniel Borkmann 2014-01-15 1166 { 5e9e5c90 Neil Horman 2019-06-28 1167 atomic_read(&rb->pending_refcnt); b0138408 Daniel Borkmann 2014-01-15 @1168 } b0138408 Daniel Borkmann 2014-01-15 1169 :::::: The code at line 1168 was first introduced by commit :::::: b013840810c221f2b0cf641d01531526052dc1fb packet: use percpu mmap tx frame pending refcount :::::: TO: Daniel Borkmann <dbork...@redhat.com> :::::: CC: David S. Miller <da...@davemloft.net> --- 0-DAY kernel test infrastructure Open Source Technology Center https://lists.01.org/pipermail/kbuild-all Intel Corporation
.config.gz
Description: application/gzip