-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dashboard/config/linux: enable SMB_SERVER #5528
Conversation
ff19a5c
to
5c5b34e
Compare
Enabled CONFIG_SMB_SERVER.
5c5b34e
to
52a0d44
Compare
Let's merge it on Monday. |
@@ -9993,7 +9993,10 @@ CONFIG_CIFS_SWN_UPCALL=y | |||
CONFIG_CIFS_SMB_DIRECT=y | |||
CONFIG_CIFS_FSCACHE=y | |||
# CONFIG_CIFS_ROOT is not set | |||
# CONFIG_SMB_SERVER is not set | |||
CONFIG_SMB_SERVER=y | |||
# CONFIG_SMB_SERVER_SMBDIRECT is not set |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do these 3 disabled configs do? Do we need to enable them to get more coverage?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IDK. I wanted to get at least some coverage first.
We can discuss the tuning with @yskelg.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dvyukov are you typically trying to enable as much as possible for every opted-in subsystem?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's discuss with @yskelg then.
The problem with such things is that if one does a change like this, it's likely nobody will ever get back to this later and refine/fix. So I usually try to do at least minimal homework researching around.
Think of it this way: with this change you have just defined the Linux kernel SMB fuzzing strategy for the next decade :)
No description provided.