# git rev-parse -q --verify f0a7d1883d9f78ae7bf15fc258bf9a2b20f35b76^{commit}
f0a7d1883d9f78ae7bf15fc258bf9a2b20f35b76
already have revision, skipping fetch
# git checkout -q -f -B kisskb f0a7d1883d9f78ae7bf15fc258bf9a2b20f35b76
# git clean -qxdf
# < git log -1
# commit f0a7d1883d9f78ae7bf15fc258bf9a2b20f35b76
# Author: David Howells <dhowells@redhat.com>
# Date:   Mon Oct 15 12:43:02 2018 +0100
# 
#     afs: Fix clearance of reply
#     
#     The recent patch to fix the afs_server struct leak didn't actually fix the
#     bug, but rather fixed some of the symptoms.  The problem is that an
#     asynchronous call that holds a resource pointed to by call->reply[0] will
#     find the pointer cleared in the call destructor, thereby preventing the
#     resource from being cleaned up.
#     
#     In the case of the server record leak, the afs_fs_get_capabilities()
#     function in devel code sets up a call with reply[0] pointing at the server
#     record that should be altered when the result is obtained, but this was
#     being cleared before the destructor was called, so the put in the
#     destructor does nothing and the record is leaked.
#     
#     Commit f014ffb025c1 removed the additional ref obtained by
#     afs_install_server(), but the removal of this ref is actually used by the
#     garbage collector to mark a server record as being defunct after the record
#     has expired through lack of use.
#     
#     The offending clearance of call->reply[0] upon completion in
#     afs_process_async_call() has been there from the origin of the code, but
#     none of the asynchronous calls actually use that pointer currently, so it
#     should be safe to remove (note that synchronous calls don't involve this
#     function).
#     
#     Fix this by the following means:
#     
#      (1) Revert commit f014ffb025c1.
#     
#      (2) Remove the clearance of reply[0] from afs_process_async_call().
#     
#     Without this, afs_manage_servers() will suffer an assertion failure if it
#     sees a server record that didn't get used because the usage count is not 1.
#     
#     Fixes: f014ffb025c1 ("afs: Fix afs_server struct leak")
#     Fixes: 08e0e7c82eea ("[AF_RXRPC]: Make the in-kernel AFS filesystem use AF_RXRPC.")
#     Signed-off-by: David Howells <dhowells@redhat.com>
#     Cc: stable <stable@vger.kernel.org>
#     Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
# < /opt/cross/kisskb/br-sparc64-full-2016.08-613-ge98b4dd/bin/sparc64-linux-gcc --version
# < git log --format=%s --max-count=1 f0a7d1883d9f78ae7bf15fc258bf9a2b20f35b76
# < make -s -j 48 ARCH=sparc64 O=/kisskb/build/linus_sparc64-allmodconfig_sparc64 CROSS_COMPILE=/opt/cross/kisskb/br-sparc64-full-2016.08-613-ge98b4dd/bin/sparc64-linux-  allmodconfig
# Added to kconfig CONFIG_BUILD_DOCSRC=n
# Added to kconfig CONFIG_HAVE_FTRACE_MCOUNT_RECORD=n
# Added to kconfig CONFIG_SAMPLES=n
# Added to kconfig CONFIG_MODULE_SIG=n
# yes \n | make -s -j 48 ARCH=sparc64 O=/kisskb/build/linus_sparc64-allmodconfig_sparc64 CROSS_COMPILE=/opt/cross/kisskb/br-sparc64-full-2016.08-613-ge98b4dd/bin/sparc64-linux-  oldconfig
yes: standard output: Broken pipe
# make -s -j 48 ARCH=sparc64 O=/kisskb/build/linus_sparc64-allmodconfig_sparc64 CROSS_COMPILE=/opt/cross/kisskb/br-sparc64-full-2016.08-613-ge98b4dd/bin/sparc64-linux-  
<stdin>:1332:2: warning: #warning syscall io_pgetevents not implemented [-Wcpp]
<stdin>:1335:2: warning: #warning syscall rseq not implemented [-Wcpp]
In file included from /kisskb/src/arch/sparc/include/asm/cmpxchg.h:5:0,
                 from /kisskb/src/arch/sparc/include/asm/atomic_64.h:12,
                 from /kisskb/src/arch/sparc/include/asm/atomic.h:5,
                 from /kisskb/src/include/linux/atomic.h:7,
                 from /kisskb/src/include/asm-generic/bitops/lock.h:5,
                 from /kisskb/src/arch/sparc/include/asm/bitops_64.h:52,
                 from /kisskb/src/arch/sparc/include/asm/bitops.h:5,
                 from /kisskb/src/include/linux/bitops.h:19,
                 from /kisskb/src/include/linux/kernel.h:11,
                 from /kisskb/src/include/linux/list.h:9,
                 from /kisskb/src/include/linux/wait.h:7,
                 from /kisskb/src/include/linux/wait_bit.h:8,
                 from /kisskb/src/include/linux/fs.h:6,
                 from /kisskb/src/fs/ocfs2/file.c:27:
/kisskb/src/fs/ocfs2/file.c: In function 'ocfs2_file_write_iter':
/kisskb/src/arch/sparc/include/asm/cmpxchg_64.h:55:22: warning: value computed is not used [-Wunused-value]
 #define xchg(ptr,x) ((__typeof__(*(ptr)))__xchg((unsigned long)(x),(ptr),sizeof(*(ptr))))
                      ^
/kisskb/src/fs/ocfs2/file.c:2386:3: note: in expansion of macro 'xchg'
   xchg(&iocb->ki_complete, saved_ki_complete);
   ^
/kisskb/src/drivers/input/joystick/analog.c:172:2: warning: #warning Precise timer not defined for this architecture. [-Wcpp]
 #warning Precise timer not defined for this architecture.
  ^
In file included from /kisskb/src/arch/sparc/include/asm/cmpxchg.h:5:0,
                 from /kisskb/src/arch/sparc/include/asm/atomic_64.h:12,
                 from /kisskb/src/arch/sparc/include/asm/atomic.h:5,
                 from /kisskb/src/include/linux/atomic.h:7,
                 from /kisskb/src/include/asm-generic/bitops/lock.h:5,
                 from /kisskb/src/arch/sparc/include/asm/bitops_64.h:52,
                 from /kisskb/src/arch/sparc/include/asm/bitops.h:5,
                 from /kisskb/src/include/linux/bitops.h:19,
                 from /kisskb/src/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c:11:
/kisskb/src/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c: In function 'ixgbevf_xdp_setup':
/kisskb/src/arch/sparc/include/asm/cmpxchg_64.h:55:22: warning: value computed is not used [-Wunused-value]
 #define xchg(ptr,x) ((__typeof__(*(ptr)))__xchg((unsigned long)(x),(ptr),sizeof(*(ptr))))
                      ^
/kisskb/src/drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c:4430:4: note: in expansion of macro 'xchg'
    xchg(&adapter->rx_ring[i]->xdp_prog, adapter->xdp_prog);
    ^
/kisskb/src/scripts/unifdef.c: In function 'Mpass':
/kisskb/src/scripts/unifdef.c:453:28: warning: 'strncpy' output truncated before terminating nul copying 4 bytes from a string of the same length [-Wstringop-truncation]
 static void Mpass (void) { strncpy(keyword, "if  ", 4); Pelif(); }
                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~
  kernel: arch/sparc/boot/image is ready
  kernel: arch/sparc/boot/zImage is ready
Completed OK
# rm -rf /kisskb/build/linus_sparc64-allmodconfig_sparc64
# Build took: 0:12:53.924079