Re: [PATCH] fs/smb/client/fs_context: Add hostname option for CIFS module to work with domain-based dfs resources with Kerberos authentication

[Maria Alexeeva <alxvmr@altlinux.org>]

Good day!

Could you please let me know if you have managed to find the messages 
with comments/feedback? This is important for us to make the 
corresponding changes to the proposed patches.

On 7/31/25 18:00, Steve French via samba-technical wrote:
> I don't have any strong opinion on: "dfs_server_hostname" vs
> "dfs_domain_hostname" whichever makes more sense.
> 
> I will look to see if I can find any more threads on this in earlier email
> 
> On Wed, Jul 30, 2025 at 11:54 AM Maria Alexeeva <alxvmr@altlinux.org> wrote:
>>
>> Steve,
>> It seems some of the discussion with review comments fell outside this
>> thread (I can only find vt@altlinux.org Vitaly Chikunov's remarks).
>> Could you please point me to where the rest of the feedback can be
>> found (e.g., about fixing the noisy warning the patch adds and
>> other comments)?
>>
>> An updated patch for fs/smb/client/fs_context has been prepared, renaming
>> the option to dfs_domain_hostname. There's suggestion to further rename
>> it to dfs_server_hostname - what are your thoughts on this?
>>
>> The patches will follow in subsequent messages.
>>
>> Thanks!
>>
>> On 7/25/25 02:50, Steve French via samba-technical wrote:
>>> Maria,
>>> Since this looks like it depends on a cifs-utils change, can you
>>> update your kernel patch with review comments (e.g. changing mount
>>> parm to "dfs_domain_hostname", and there were at least two others in
>>> the thread, e.g. fixing the noisy warning that the patch adds) and
>>> then show the cifs-utils change, so we can make the upcoming merge
>>> window.
>>>
>>> On Thu, Jul 24, 2025 at 5:14 PM Steve French <smfrench@gmail.com> wrote:
>>>>
>>>> I will update the mount parm name, similar to what Tom suggested to
>>>> "dfs_domain_hostname" to be less confusing.
>>>>
>>>> Let me know if you had a v2 of the patch with other changes