Re: [PATCH 10/19] USB: gadget/legacy: remove sb_mutex

2023-09-14 Thread Sergey Shtylyov
On 9/13/23 2:10 PM, Christoph Hellwig wrote: > Creating new a new super_block vs freeing the old one for single instance ^ I can't parse that. :-) > file systems is serialized by the wait for SB_DEAD. > > Remove the superfluous sb_mutex. > > Signed-off-by: Christoph

Re: [PATCH 5.4 03/74] module: merge repetitive strings in module_sig_check()

2021-04-05 Thread Sergey Shtylyov
ame text. >>> Let's put the starting text into the pr_notice() call -- it saves 21 >>> bytes of the object code (x86 gcc 10.2.1). >>> >>> Suggested-by: Joe Perches >>> Reviewed-by: Miroslav Benes >>> Signed-off-by: Sergey Shtylyov >&

Re: [PATCH 5.4 03/74] module: merge repetitive strings in module_sig_check()

2021-04-05 Thread Sergey Shtylyov
On 4/5/21 11:53 AM, Greg Kroah-Hartman wrote: > From: Sergey Shtylyov > > [ Upstream commit 705e9195187d85249fbb0eaa844b1604a98fbc9a ] > > The 'reason' variable in module_sig_check() points to 3 strings across > the *switch* statement, all needlessly starting with the same

[PATCH] sata_mv: add IRQ checks

2021-03-27 Thread Sergey Shtylyov
). Fixes: f351b2d638c3 ("sata_mv: Support SoC controllers") Signed-off-by: Sergey Shtylyov --- drivers/ata/sata_mv.c |4 1 file changed, 4 insertions(+) Index: linux-block/drivers/ata/sata_mv.c === --- linux-block.or

[PATCH v2] module: fix comment style

2020-11-07 Thread Sergey Shtylyov
?) the starts of the multi-line comments not being /* on their own line... Signed-off-by: Sergey Shtylyov --- This patch is against the 'modules-next' branch of Jessica Yu's 'linux.git' repo plus my 2 patches posted this week. I'm not sure such patches are welcome, please let me know if they're

[PATCH] module: fix comment style

2020-11-07 Thread Sergey Shtylyov
?) the starts of the multi-line comments not being /* on their own line... Signed-off-by: Sergey Shtylyov --- This patch is against the 'modules-next' branch of Jessica Yu's 'linux.git' repo plus my 2 patches posted this week. I'm not sure such patches are welcome, please let me know if they're

[PATCH 2/2] module: add more 'kernel-doc' comments

2020-11-04 Thread Sergey Shtylyov
Some functions have the proper 'kernel-doc' comments but these don't start with proper /** -- fix that, along with adding () to the function name on the following lines to fully comply with the 'kernel-doc' format. Signed-off-by: Sergey Shtylyov --- kernel/module.c | 16 1

[PATCH 1/2] module: fix up 'kernel-doc' comments

2020-11-04 Thread Sergey Shtylyov
Some 'kernel-doc' function comments do not fully comply with the specified format due to: - missing () after the function name; - "RETURNS:"/"Returns:" instead of "Return:" when documenting the function's result. - empty line before describing the function's arg

[PATCH 0/2] module: add/fix 'kernel-doc' comments

2020-11-04 Thread Sergey Shtylyov
Here are 2 patches against the 'modules-next' branch of Jessica Yu's 'linux.git' repo. I'm cleaning up the 'kernel-doc' function comments... [1/2] module: fix up 'kernel-doc' comments [2/2] module: add more 'kernel-doc' comments

[PATCH v2 3/3] module: only handle errors with the *switch* statement in module_sig_check()

2020-10-31 Thread Sergey Shtylyov
, spell out "nomem" as "lack of memory" in these comments, while at it... Suggested-by: Joe Perches Signed-off-by: Sergey Shtylyov --- Changes in version 2: - new patch. kernel/module.c | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-)

[PATCH v2 2/3] module: avoid *goto*s in module_sig_check()

2020-10-31 Thread Sergey Shtylyov
Let's move the common handling of the non-fatal errors after the *switch* statement -- this avoids *goto*s inside that *switch*... Suggested-by: Joe Perches Signed-off-by: Sergey Shtylyov --- Changes in version 2: - new patch. kernel/module.c | 20 ++-- 1 file changed, 10

[PATCH v2 1/3] module: merge repetitive strings in module_sig_check()

2020-10-31 Thread Sergey Shtylyov
-by: Sergey Shtylyov --- Changes in version 2: - put less starting text into the pr_notice() call; - updated the patch description accordingly, added the "Suggested-by:" tag. kernel/module.c |9 + 1 file changed, 5 insertions(+), 4 deletions(-) Index: linux/kerne

[PATCH v2 0/3] module: refactor module_sig_check()

2020-10-31 Thread Sergey Shtylyov
Here are 3 patches against the 'modules-next' branch of Jessica Yu's 'linux.git' repo. I'm doing some refactoring in module_sig_check()... [1/3] module: merge repetitive strings in module_sig_check() [2/3] module: avoid *goto*s in module_sig_check() [3/3] module: only handle errors with the

Re: [PATCH 0/2] module: some refactoring in module_sig_check()

2020-10-22 Thread Sergey Shtylyov
Hello! On 10/22/20 6:09 PM, Jessica Yu wrote: >> Here are 2 patches against the 'modules-next' branch of Jessica Yu's >> 'linux.git' repo. >> I'm doing some little refactoring in module_sig_check()... >> >> [1/2] module: merge repetitive strings in module_sig_check() >> [2/2] module: unindent

Re: [PATCH 0/2] module: some refactoring in module_sig_check()

2020-10-14 Thread Sergey Shtylyov
On 10/14/20 11:35 AM, Joe Perches wrote: [...] Here are 2 patches against the 'modules-next' branch of Jessica Yu's 'linux.git' repo. I'm doing some little refactoring in module_sig_check()... [1/2] module: merge repetitive strings in module_sig_check() [2/2]

Re: [PATCH 0/2] module: some refactoring in module_sig_check()

2020-10-14 Thread Sergey Shtylyov
Hello! On 14.10.2020 1:44, Joe Perches wrote: Here are 2 patches against the 'modules-next' branch of Jessica Yu's 'linux.git' repo. I'm doing some little refactoring in module_sig_check()... [1/2] module: merge repetitive strings in module_sig_check() [2/2] module: unindent comments in

[PATCH 1/2] module: merge repetitive strings in module_sig_check()

2020-10-13 Thread Sergey Shtylyov
code (x86 gcc 10.2.1). Signed-off-by: Sergey Shtylyov --- kernel/module.c |9 + 1 file changed, 5 insertions(+), 4 deletions(-) Index: linux/kernel/module.c === --- linux.orig/kernel/module.c +++ linux/kernel/module.c

[PATCH 2/2] module: unindent comments in module_sig_check()

2020-10-13 Thread Sergey Shtylyov
-by: Sergey Shtylyov --- kernel/module.c | 17 + 1 file changed, 9 insertions(+), 8 deletions(-) Index: linux/kernel/module.c === --- linux.orig/kernel/module.c +++ linux/kernel/module.c @@ -2901,10 +2901,11 @@ static int

[PATCH 0/2] module: some refactoring in module_sig_check()

2020-10-13 Thread Sergey Shtylyov
Here are 2 patches against the 'modules-next' branch of Jessica Yu's 'linux.git' repo. I'm doing some little refactoring in module_sig_check()... [1/2] module: merge repetitive strings in module_sig_check() [2/2] module: unindent comments in module_sig_check()

Re: [PATCH 0/2] module: n module_sig_check()

2020-10-13 Thread Sergey Shtylyov
Oops, hadn't finished the subject... :-/\

[PATCH 0/2] module: n module_sig_check()

2020-10-13 Thread Sergey Shtylyov
Here are 2 patches against the 'modules-next' branch of Jessica Yu's 'linux.git' repo. I'm doing some little refactoring in module_sig_check()... [1/2] module: merge repetitive strings in module_sig_check() [2/2] module: unindent comments in module_sig_check()