Re: [PATCH v4 0/3] ksmbd: refactor smb2_notify() to a blocking wait
Namjae Jeon <[email protected]>
| Newsgroups | org.kernel.vger.linux-cifs |
|---|---|
| Message-ID | <CAKYAXd84UnfdQ6Ef2nqybnhdgURFo9uYpB1mHZpphFyCiMhutA@mail.gmail.com> |
On Mon, Aug 10, 2026 at 1:20 AM ChenXiaoSong <[email protected]> wrote: > > From: ChenXiaoSong <[email protected]> > > The next version of my complete notify implementation may still take some time to modify, > so merging Gael's patch sooner would help avoid repeated rebasing. Can you check two smbtorture failures from this patch-set ? $ ./bin/smbtorture //ksmbd.test/cifsd-test3/ [email protected]%1234 --use-kerberos=required smb2.session.expire2s smbtorture 4.18.0 Using seed 1786452357 time: 2026-08-11 12:45:57.628809 test: expire2s time: 2026-08-11 12:45:57.629426 query info => OK lock => OK 1st notify => PENDING sleep 10 seconds query info => EXPIRED set info => EXPIRED flush => EXPIRED read => EXPIRED write => EXPIRED ioctl => EXPIRED oplock ack => EXPIRED lease ack => EXPIRED query directory => EXPIRED 1st notify => CANCEL 2nd notify => EXPIRED time: 2026-08-11 12:46:07.761775 failure: expire2s [ ../../source4/torture/smb2/session.c:1482: status was NT_STATUS_ACCESS_DENIED, expected NT_STATUS_NETWORK_SESSION_EXPIRED: smb2_notify returned unexpected status ] $ ./bin/smbtorture //ksmbd.test/cifsd-test3/ [email protected]%1234 --use-kerberos=required smb2.session.expire2e smbtorture 4.18.0 Using seed 1786452388 time: 2026-08-11 12:46:28.839654 test: expire2e time: 2026-08-11 12:46:28.840279 query info => OK lock => OK 1st notify => PENDING sleep 10 seconds query info => EXPIRED set info => EXPIRED flush => EXPIRED read => EXPIRED write => EXPIRED ioctl => EXPIRED oplock ack => EXPIRED lease ack => EXPIRED query directory => EXPIRED 1st notify => CANCEL 2nd notify => EXPIRED time: 2026-08-11 12:46:38.954222 failure: expire2e [ ../../source4/torture/smb2/session.c:1482: status was NT_STATUS_ACCESS_DENIED, expected NT_STATUS_NETWORK_SESSION_EXPIRED: smb2_notify returned unexpected status ]