Re: [PATCH] kmod: verify module name before invoking modprobe

Hi Petr, Please see my inline comments, many thanks. BR Song 在 2024/11/12 20:56, Petr Pavlu 写道: On 11/10/24 12:42, Song Chen wrote: Sometimes when kernel calls request_module to load a module into kernel space, it doesn't pass the module name appropriately, and request_module doesn

[PATCH] kmod: verify module name before invoking modprobe

name_valid to verify the parameters validity one by one, either null or empty. if it fails, no modprobe invoked. Signed-off-by: Song Chen --- kernel/module/kmod.c | 29 - 1 file changed, 28 insertions(+), 1 deletion(-) diff --git a/kernel/module/kmod.c b/kernel/mo

[PATCH] kernel:irq:manage: request threaded irq with a specified priority

fore, this patch introduce a new api to give drivers a chance to assign expected priorities to their irq handler thread. Signed-off-by: Song Chen --- include/linux/interrupt.h | 7 + include/linux/sched.h | 1 + include/linux/sched/prio.h | 1 + kernel/irq/manage.c

[PATCH] kernel: trace: preemptirq_delay_test: add cpu affinity

e same cpu with latency tracer. e.g. cyclictest -p 90 -m -c 0 -i 1000 -a 3 modprobe preemptirq_delay_test test_mode=preempt delay=500 \ burst_size=3 cpu_affinity=3 Signed-off-by: Song Chen --- kernel/trace/Kconfig | 4 kernel/trace/preemptirq_delay_test.c | 14 +

[PATCH] staging: unisys: visornic: enhance visornic to use channel_interrupt

visornic uses timer to check the response queue and drain it if needed periodically. On the other hand, visorbus provides periodic work to serve such request, therefore, timer should be replaced by channel_interrupt. Signed-off-by: Song Chen --- drivers/staging/unisys/visornic/visornic_main.c

[PATCH] staging: unisys: visorhba: enhance visorhba to use channel_interrupt

visorhba uses kthread to obtain the responses from the IO Service Partition periodically, on the other hand, visorbus provides periodic work to serve such request, therefore, kthread should be replaced by channel_interrupt. Signed-off-by: Song Chen --- drivers/staging/unisys/visorhba

[PATCH v2] staging: board: Remove macro board_staging

Macro is not supposed to have flow control in it's statement, remove. Signed-off-by: Song Chen --- Changes in v2: 1, kzm9d_init and armadillo800eva_init are not compatible with the definition of device_initcall, fixed. --- drivers/staging/board/armadillo800eva.c | 15 ++- dr

[PATCH] staging: board: description for CONFIG_STAGING_BOARD

A paragraph to describe what CONFIG_STAGING_BOARD is for, to help developers have better understanding. Signed-off-by: Song Chen --- drivers/staging/board/Kconfig | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/drivers/staging/board/Kconfig b/drivers/staging/board

[PATCH] staging: comedi: remove warnings of comedi_lrange

Checkpatch.pl reports "warning: struct comedi_lrange should normally be const" in some places, which are supposed to be removed. Signed-off-by: Song Chen --- drivers/staging/comedi/drivers/das16.c | 4 ++-- drivers/staging/comedi/drivers/jr3_pci.c | 4 ++-- drivers/staging/come

[PATCH] staging: comedi: clean up debugging code in #if 0 or 1

There are a log of "#if 0" or "#if 1" in driver which cause warning when running checkpatch.pl, they are supposed to be cleaned up before release. Signed-off-by: Song Chen --- drivers/staging/comedi/drivers/cb_pcidas64.c | 95 -- drivers/staging/com

Opportunity!!

How is life with you over there? My name is Mr.Song Chen, I am contacting you for a good business proposal of $12.8mUSD that will benefit both of us, if you are interested,please get back to me and lets benefit from this golden opportunity.I await to hear from you on my private email:(mrsongc