my goal was to at least get the non-controversial debug message spamming improvements in, even if figuring out the rest of them (to make sure we are logging important session setup failures enough for users to see the error at least once) will take more work to improve. On Tue, Oct 8, 2024 at 3:25 PM Paulo Alcantara <pc@xxxxxxxxxxxxx> wrote: > > Paulo Alcantara <pc@xxxxxxxxxxxxx> writes: > > > Without this message we at least have > > > > VFS: \\srv has not responded in N seconds. Reconnecting... > > > > which tells the user that connection was lost. And, in case server > > dropped the session (e.g. STATUS_USER_SESSION_DELETED), we'll also have > > the above message printed out as we currently mark tcp ses for > > reconnect. > > Err, no. I forgot that we only print this in server_unresponsive(). We > call cifs_reconnect() when the session is dropped and don't print > anything. -- Thanks, Steve
