# git rev-parse -q --verify 8c7ae38d1ce12a0eaeba655df8562552b3596c7f^{commit}
8c7ae38d1ce12a0eaeba655df8562552b3596c7f
already have revision, skipping fetch
# git checkout -q -f -B kisskb 8c7ae38d1ce12a0eaeba655df8562552b3596c7f
# git clean -qxdf
# < git log -1
# commit 8c7ae38d1ce12a0eaeba655df8562552b3596c7f
# Author: David Howells <dhowells@redhat.com>
# Date:   Wed Mar 27 22:48:02 2019 +0000
# 
#     afs: Fix StoreData op marshalling
#     
#     The marshalling of AFS.StoreData, AFS.StoreData64 and YFS.StoreData64 calls
#     generated by ->setattr() ops for the purpose of expanding a file is
#     incorrect due to older documentation incorrectly describing the way the RPC
#     'FileLength' parameter is meant to work.
#     
#     The older documentation says that this is the length the file is meant to
#     end up at the end of the operation; however, it was never implemented this
#     way in any of the servers, but rather the file is truncated down to this
#     before the write operation is effected, and never expanded to it (and,
#     indeed, it was renamed to 'TruncPos' in 2014).
#     
#     Fix this by setting the position parameter to the new file length and doing
#     a zero-lengh write there.
#     
#     The bug causes Xwayland to SIGBUS due to unexpected non-expansion of a file
#     it then mmaps.  This can be tested by giving the following test program a
#     filename in an AFS directory:
#     
#             #include <stdio.h>
#             #include <stdlib.h>
#             #include <unistd.h>
#             #include <fcntl.h>
#             #include <sys/mman.h>
#             int main(int argc, char *argv[])
#             {
#                     char *p;
#                     int fd;
#                     if (argc != 2) {
#                             fprintf(stderr,
#                                     "Format: test-trunc-mmap <file>\n");
#                             exit(2);
#                     }
#                     fd = open(argv[1], O_RDWR | O_CREAT | O_TRUNC);
#                     if (fd < 0) {
#                             perror(argv[1]);
#                             exit(1);
#                     }
#                     if (ftruncate(fd, 0x140008) == -1) {
#                             perror("ftruncate");
#                             exit(1);
#                     }
#                     p = mmap(NULL, 4096, PROT_READ | PROT_WRITE,
#                              MAP_SHARED, fd, 0);
#                     if (p == MAP_FAILED) {
#                             perror("mmap");
#                             exit(1);
#                     }
#                     p[0] = 'a';
#                     if (munmap(p, 4096) < 0) {
#                             perror("munmap");
#                             exit(1);
#                     }
#                     if (close(fd) < 0) {
#                             perror("close");
#                             exit(1);
#                     }
#                     exit(0);
#             }
#     
#     Fixes: 31143d5d515e ("AFS: implement basic file write support")
#     Reported-by: Jonathan Billings <jsbillin@umich.edu>
#     Tested-by: Jonathan Billings <jsbillin@umich.edu>
#     Signed-off-by: David Howells <dhowells@redhat.com>
#     Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
# < /opt/cross/kisskb/br-aarch64-glibc-2016.08-613-ge98b4dd/bin/aarch64-linux-gcc --version
# < /opt/cross/kisskb/br-aarch64-glibc-2016.08-613-ge98b4dd/bin/aarch64-linux-ld --version
# < git log --format=%s --max-count=1 8c7ae38d1ce12a0eaeba655df8562552b3596c7f
# < make -s -j 80 ARCH=arm64 O=/kisskb/build/linus_arm64-allmodconfig_arm64-gcc5.4 CROSS_COMPILE=/opt/cross/kisskb/br-aarch64-glibc-2016.08-613-ge98b4dd/bin/aarch64-linux-  allmodconfig
# make -s -j 80 ARCH=arm64 O=/kisskb/build/linus_arm64-allmodconfig_arm64-gcc5.4 CROSS_COMPILE=/opt/cross/kisskb/br-aarch64-glibc-2016.08-613-ge98b4dd/bin/aarch64-linux-  
arch/arm64/Makefile:27: ld does not support --fix-cortex-a53-843419; kernel may be susceptible to erratum
arch/arm64/Makefile:40: LSE atomics not supported by binutils
/kisskb/src/drivers/i2c/busses/i2c-sh_mobile.c: In function 'sh_mobile_i2c_isr':
/kisskb/src/drivers/i2c/busses/i2c-sh_mobile.c:399:26: warning: 'data' may be used uninitialized in this function [-Wmaybe-uninitialized]
   pd->msg->buf[real_pos] = data;
                          ^
/kisskb/src/drivers/i2c/busses/i2c-sh_mobile.c:372:16: note: 'data' was declared here
  unsigned char data;
                ^
In file included from /kisskb/src/include/linux/rwsem.h:16:0,
                 from /kisskb/src/include/linux/notifier.h:15,
                 from /kisskb/src/include/linux/clk.h:17,
                 from /kisskb/src/drivers/tty/serial/sh-sci.c:24:
/kisskb/src/drivers/tty/serial/sh-sci.c: In function 'sci_dma_rx_submit':
/kisskb/src/include/linux/spinlock.h:279:3: warning: 'flags' may be used uninitialized in this function [-Wmaybe-uninitialized]
   _raw_spin_unlock_irqrestore(lock, flags); \
   ^
/kisskb/src/drivers/tty/serial/sh-sci.c:1363:16: note: 'flags' was declared here
  unsigned long flags;
                ^
Completed OK
# rm -rf /kisskb/build/linus_arm64-allmodconfig_arm64-gcc5.4
# Build took: 0:15:46.356105