On Tue, Dec 3, 2024 at 10:35 AM David Howells <dhowells@xxxxxxxxxx> wrote: > > Steve French <smfrench@xxxxxxxxx> wrote: > > > I sometimes see xfstest generic/207 fail (e.g. to Azure > > http://smb311-linux-testing.southcentralus.cloudapp.azure.com/#/builders/5/builds/281/steps/70/logs/stdio) > > but usually passes. > > > > e.g. "write of 4096 bytes @5455872 finished, expected filesize at > > least 5459968, but got 5455872" > > > > Does anyone else see that fail? or have a better repro scenario to > > narrow it down? > > Can you collect some tracing from it? > > It works for me, but my Azure I/O is so slow that it tends to overlook the > races. > > What patches are applied, btw? This was 6.13-rc1 but it did not always fail. These are the cifs/smb3 patches in 6.13-rc1: $ git log --oneline --no-merges v6.12.. fs/smb/client fs/smb/common 8d7690b3c146 cifs: update internal version number cda88d2fef7a cifs: unlock on error in smb3_reconfigure() 0f0e35790295 cifs: during remount, make sure passwords are in sync b9aef1b13a0a cifs: support mounting with alternate password to allow password rotation c353ee4fb119 smb: Initialize cfid->tcon before performing network ops 3fa640d035e5 smb: During unmount, ensure all cached dir instances drop their dentry 796733054e4a smb: client: fix noisy message when mounting shares 36008fe6e3dc smb: client: don't try following DFS links in cifs_tree_connect() e1481075981d smb: client: allow reconnect when sending ioctl b2fe4a8fa0f6 smb: client: get rid of @nlsc param in cifs_tree_connect() 28ec614f2f9b smb: client: allow more DFS referrals to be cached f4ca4f5a36ea cifs: Fix parsing reparse point with native symlink in SMB1 non-UNICODE session 1f48660667ef cifs: Validate content of WSL reparse point buffers d3d797e32653 cifs: Improve guard for excluding $LXDEV xattr 06a7adf318a3 cifs: Add support for parsing WSL-style symlinks dd26bc067e44 cifs: Validate content of native symlink 723f4ef90452 cifs: Fix parsing native symlinks relative to the export 4bdec0d1f658 smb: client: fix NULL ptr deref in crypto_aead_setkey() ab02d8774181 Update misleading comment in cifs_chan_update_iface 07bdf9272a01 smb: client: change return value in open_cached_dir_by_dentry() if !cfids ceaf1451990e smb: client: disable directory caching when dir_cache_timeout is zero f6e88838400d smb: client: remove unnecessary checks in open_cached_dir() a9685b409a03 smb: prevent use-after-free due to open_cached_dir error paths 7afb86733685 smb: Don't leak cfid when reconnect races with open_cached_dir 0812340811e4 smb: client: handle max length for SMB symlinks 9f544d26b15b smb: client: get rid of bounds check in SMB2_ioctl_init() bc925c1216f0 smb: client: improve compound padding in encryption 9ed9d83a51a9 smb3: request handle caching when caching directories 0d6b0d2e3816 cifs: Recognize SFU char/block devices created by Windows NFS server on Windows Server <<2012 db363b0a1d9e CIFS: New mount option for cifs.upcall namespace resolution 7a2158b73c36 smb/client: Prevent error pointer dereference d413eabff18d fs/smb/client: implement chmod() for SMB3 POSIX Extensions 128630e1dbec smb: cached directories can be more than root file handle 343d7fe6df9e smb: client: fix use-after-free of signing key 7460bf441656 smb: client: Use str_yes_no() helper function f69b0187f874 smb: client: memcpy() with surrounding object base address 6c9903c330ab cifs: Remove pre-historic unused CIFSSMBCopy 9bd812744db2 convert cifs_ioctl_copychunk() -- Thanks, Steve
