Re: kernel-5.18.8 breaks cifs mounts

[Julian Sikorski <belegdol@gmail.com>]

Am 30.06.22 um 20:28 schrieb Paulo Alcantara:
> 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.
Good catch! I only noticed

16d5d9100927 smb3: use netname when available on secondary channels

initally, and reverting it did not help. Reverting

ca83f50b43a0 smb3: fix empty netname context on secondary channels

makes the mounts work again.

Best regards,
Julian