Re: kernel-5.18.8 breaks cifs mounts

[Paulo Alcantara <pc@cjr.nz>]

Julian Sikorski <belegdol@gmail.com> writes:

> Am 30.06.22 um 19:55 schrieb Paulo Alcantara:
>> Julian Sikorski <belegdol@gmail.com> writes:
>> 
>>> Not much is there:
>>>
>>> Jun 30 18:19:34 snowball3 kernel: CIFS: Attempting to mount
>>> \\odroidxu4.local\julian
>>>
>>> Jun 30 18:19:34 snowball3 kernel: CIFS: VFS: cifs_mount failed w/return
>>> code = -22
>>>
>>>
>>> I tried reverting 16d5d91 as it was the only commit referencing smb, but
>>> it did not help unfortunately.
>> 
>> Could you please run
>> 
>> 	echo 'module cifs +p' > /sys/kernel/debug/dynamic_debug/control
>> 	echo 'file fs/cifs/* +p' > /sys/kernel/debug/dynamic_debug/control
>> 	echo 1 > /proc/fs/cifs/cifsFYI
>> 	echo 1 > /sys/module/dns_resolver/parameters/debug
>> 	dmesg --clear
>> 	tcpdump -s 0 -w trace.pcap port 445 & pid=$!
>> 	mount ...
>> 	kill $pid
>> 	dmesg > trace.log
>> 
>> and then send trace.log and trace.pcap.
>
> Attached.
>
>> 
>> What SMB server and version?
>
> Openmediavault 6.0.30 running on top of armbian bullseye. The samba 
> package version is 4.13.13+dfsg-1~deb11u3.

Thanks!

So, it is failing very early during the negproto request:

	SMB2 302 Negotiate Protocol Request
	SMB2 143 Negotiate Protocol Response, Error: STATUS_INVALID_PARAMETER

which seems related to these commits in v5.18.8

	16d5d9100927 smb3: use netname when available on secondary channels
	ca83f50b43a0 smb3: fix empty netname context on secondary channels

Have you tried reverting both in v5.18.8 to see if it works?

I built v5.18.8 but couldn't reproduce it against w22 server, though.