lat 50.0ms
iperf -c host -t 30 -i 10
[ ID] Interval Transfer Bandwidth
[ 3] 0.0-10.0 sec 764 MBytes 641 Mbits/sec
[ 3] 10.0-20.0 sec 730 MBytes 612 Mbits/sec
Signed-off-by: Yang Yingliang
---
include/uapi/linux/pkt_sched.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion
From: Yang Yingliang
When cpu is disabled, all irqs will be migratged to another cpu.
In some cases, a new affinity is different, it needed to be coppied
to irq's affinity. But if the type of irq is LPI, it's affinity will
not be coppied because of irq_set_affinity's return val
From: Yang Yingliang
When cpu is disabled, all irqs will be migratged to another cpu.
In some cases, a new affinity is different, it needed to be coppied
to irq's affinity. But if the type of irq is LPI, it's affinity will
not be coppied because of irq_set_affinity's return val
在 2015/8/31 20:20, Marc Zyngier 写道:
On Sun, 30 Aug 2015 21:15:56 +0800
Hanjun Guo wrote:
On 08/30/2015 02:12 AM, Marc Zyngier wrote:
On 2015-08-29 16:12, Jiang Liu wrote:
On 2015/8/29 21:00, Yang Yingliang wrote:
From: Yang Yingliang
When cpu is disabled, all irqs will be migratged to
Deacon
Cc: Russell King - ARM Linux
Cc: Hanjun Guo
Yang Yingliang (4):
genirq: Introduce irq_settings_set_move_pcntxt() helper
irqchip: GICv3: set non-percpu irqs status with _IRQ_MOVE_PCNTXT
genirq: rename config GENERIC_PENDING_IRQ to GENERIC_IRQ_MIGRATION
arm/arm64: fix a migrating
It's expected to use the helper when interrupt can be moved
in process.
Cc: Jiang Liu
Cc: Thomas Gleixner
Cc: Marc Zyngier
Cc: Mark Rutland
Cc: Will Deacon
Cc: Russell King - ARM Linux
Cc: Hanjun Guo
Signed-off-by: Yang Yingliang
---
include/linux/irq.h | 1 +
kernel/irq/irqd
sell King - ARM Linux
Cc: Hanjun Guo
Signed-off-by: Yang Yingliang
---
arch/arm/Kconfig | 1 +
arch/arm/include/asm/irq.h | 1 -
arch/arm/kernel/irq.c| 62
arch/arm64/Kconfig | 1 +
arch/arm64/include/asm/irq.h | 1 -
ar
: Yang Yingliang
---
drivers/irqchip/irq-gic-v3.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c
index e406bc5..9108387 100644
--- a/drivers/irqchip/irq-gic-v3.c
+++ b/drivers/irqchip/irq-gic-v3.c
@@ -688,6 +688,7 @@ static int
Make the config name more general for moving other migration
interrupts code into kernel/irq/migration.c
Cc: Jiang Liu
Cc: Thomas Gleixner
Cc: Marc Zyngier
Cc: Mark Rutland
Cc: Will Deacon
Cc: Russell King - ARM Linux
Cc: Hanjun Guo
Signed-off-by: Yang Yingliang
---
arch/arc/Kconfig
On 2015/9/7 6:08, Thomas Gleixner wrote:
On Sun, 6 Sep 2015, Yang Yingliang wrote:
It's expected to use the helper when interrupt can be moved
in process.
What's wrong with using the existing helper?
irq_set_status_flags(irq, IRQ_MOVE_PCNTXT);
It looks OK. I will u
On 2015/9/6 13:56, Jiang Liu wrote:
On 2015/9/6 12:23, Yang Yingliang wrote:
Use irq_settings_set_move_pcntxt() helper irqs status with
_IRQ_MOVE_PCNTXT. So that it can do set affinity when calling
irq_set_affinity_locked().
Hi Yingliang,
We could only set _IRQ_MOVE_PCNTCT flag to
On 2015/9/6 13:55, Jiang Liu wrote:
On 2015/9/6 12:23, Yang Yingliang wrote:
When cpu is disabled, all irqs will be migratged to another cpu.
In some cases, a new affinity is different, it needed to be coppied
to irq's affinity. But if the type of irq is LPI, it's affinity w
On 2015/9/6 16:07, Jiang Liu wrote:
On 2015/9/6 12:23, Yang Yingliang wrote:
Hi All,
There is a bug:
When cpu is disabled, all irqs will be migratged to another cpu.
In some cases, a new affinity is different, it needed to be coppied
to irq's affinity. But if the type of irq is LPI,
Hi Thomas,
I got the following call trace when I called some set time syscalls in
userspace.
---start---
Jan 01 00:02:29 Linux kernel: INFO: rcu_sched detected stalls on CPUs/tasks:
Jan 01 00:02:29 Linux kernel: 0: (2 GPs behind) idle=913/1/0
softirq=59289/59291 fqs=488
Jan 01 00:02:2
On 2015/10/22 17:26, Russell King - ARM Linux wrote:
On Wed, Oct 21, 2015 at 09:29:08PM +0100, Russell King - ARM Linux wrote:
On Wed, Oct 21, 2015 at 01:47:49PM +0200, Geert Uytterhoeven wrote:
On Thu, Sep 24, 2015 at 11:32 AM, Yang Yingliang
wrote:
When cpu is disabled, all irqs will be
On 2015/10/22 18:25, Thomas Gleixner wrote:
On Thu, 22 Oct 2015, Ding Tianhong wrote:
On 2015/10/22 15:43, Thomas Gleixner wrote:
Jan 01 00:03:32 Linux kernel: i:0 basenow.tv64:4809284991830
hrtimer_get_softexpires_tv64(timer):444012000 ccpu0
timer:ffdffdec6138, timer->function:ffc
Hi, Marc
Current system only supports (NR_IRQS + 8196) interrupts. It's not
enough on ARM64
server, on D06 we have several pcie devices and each device supports 256
VMs and each
VM uses 64 interrupts. Can we set NR_IRQS to a bigger value to support
more interrupts ?
Thanks,
Yang
.
11 All Redistributors with the same Aff3.Aff2.Aff1 value must share an LPI
Configuration table.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-gic-v3-its.c | 81 --
include/linux/irqchip/arm-gic-v3.h | 6 ++-
2 files changed, 65 insertions
Use alloc_pages_node() to allocate pages on current node.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-gic-v3-its.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
index a03e18e..4ebe131
same Aff3 value must share an LPI Configurationt
table.
10 All Redistributors
11 All Redistributors with the same Aff3.Aff2.Aff1 value must share an LPI
Configuration
table.
Yang Yingliang (4):
irqchip/gic-v3: add common_aff_lpi field in struct rdists
irqchip/gic-v3-its: replace alloc_pages
The config of LPIs need to be same in each prop_page.
So if one prop_page is modified, other prop_page should
be updated too.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-gic-v3-its.c | 39 ++-
1 file changed, 30 insertions(+), 9 deletions(-)
diff
Read CommonLPIAff from GICR_TYPER and check whether the
values are same in each register. If they are different,
prints warning message and set CommonLPIAff to zero.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-gic-v3.c | 20
include/linux/irqchip/arm-gic-v3
On 2018/3/12 17:55, Marc Zyngier wrote:
On 12/03/18 06:49, Yang Yingliang wrote:
This patch set adjust struct rdists to support CommonLPIAff field.
This field is described in spec:
CommonLPIAff, bits [25:24]
The affinity level at which Redistributors share a LPI Configuration table.
00 All
From: Yang Yingliang
While cpus posting a bunch of ITS commands, the cmd_queue and rd_idx will
be wrapped easily. And current way of handling wrapped case is not quite
right.
Such as, in direct case, rd_idx will wrap if other cpus post commands
that make rd_idx increase. When rd_idx wrapped
erate SPIs.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-mbigen.c | 21 +++--
1 file changed, 19 insertions(+), 2 deletions(-)
diff --git a/drivers/irqchip/irq-mbigen.c b/drivers/irqchip/irq-mbigen.c
index f05998f..37c1932 100644
--- a/drivers/irqchip/irq-mbigen.c
+++
mbi_lock is uninitialized, use marco DEFINE_MUTEX
to initialize it.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-gic-v3-mbi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/irqchip/irq-gic-v3-mbi.c b/drivers/irqchip/irq-gic-v3-mbi.c
index ad70e7c..fbfa7ff
SPIs and a bugfix for MBI driver.
Patch #1 is a bugfix for MBI driver.
Pathc #2 ~ #4 is support for MBIGEN generating message based SPIs.
Yang Yingliang (4):
irqchip/gic-v3-mbi: fix uninitialized mbi_lock
irqchip/mbigen: rename register marcros
irqchip/mbigen: add support for a MBIGEN gener
Now MBIGEN can support to generate SPIs by writing
GICD_SETSPIR. Add dt example to help document.
Signed-off-by: Yang Yingliang
---
.../interrupt-controller/hisilicon,mbigen-v2.txt| 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git
a/Documentation
A MBIGEN can also be used for generating SPIs, so let's
rename register macros to make them more resonable.
The first 64-pins of MBIGEN is used by SPIs, so rename
RESERVED_IRQ_PER_MBIGEN_CHIP to SPI_NUM_PER_MBIGEN_CHIP
and change the comment for this marcro.
Signed-off-by: Yang Ying
Hi, Marc
On 2018/10/18 0:30, Marc Zyngier wrote:
On 16/10/18 10:15, Yang Yingliang wrote:
Now with
5052875 ("irqchip/gic-v3: Add support for Message Based Interrupts as an MSI
controller"),
we can support MBIGEN to generate message based SPIs by writing GICD_SETSPIR.
The first
erate SPIs.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-mbigen.c | 29 -
1 file changed, 24 insertions(+), 5 deletions(-)
diff --git a/drivers/irqchip/irq-mbigen.c b/drivers/irqchip/irq-mbigen.c
index f05998f..72db969 100644
--- a/drivers/irqchip/irq-mbig
mbi_lock is uninitialized, use marco DEFINE_MUTEX
to initialize it.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-gic-v3-mbi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/irqchip/irq-gic-v3-mbi.c b/drivers/irqchip/irq-gic-v3-mbi.c
index ad70e7c..fbfa7ff
Now MBIGEN can support to generate SPIs by writing
GICD_SETSPIR. Add dt example to help document.
Signed-off-by: Yang Yingliang
---
.../interrupt-controller/hisilicon,mbigen-v2.txt| 17 -
1 file changed, 16 insertions(+), 1 deletion(-)
diff --git
a/Documentation
SPIs and a bugfix for MBI driver.
Patch #1 is a bugfix for MBI driver.
Pathc #2 ~ #4 is support for MBIGEN generating message based SPIs.
v2:
Move the doorbell address comments to the top.
Yang Yingliang (4):
irqchip/gic-v3-mbi: fix uninitialized mbi_lock
irqchip/mbigen: rename regist
A MBIGEN can also be used for generating SPIs, so let's
rename register macros to make them more resonable.
The first 64-pins of MBIGEN is used by SPIs, so rename
RESERVED_IRQ_PER_MBIGEN_CHIP to SPI_NUM_PER_MBIGEN_CHIP
and change the comment for this marcro.
Signed-off-by: Yang Ying
Add the missing iounmap() before return from tegra_init_fuse()
in the error handling case.
Fixes: 9f94fadd75d3 ("soc/tegra: fuse: Register cell lookups for compatibility")
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/soc/tegra/fuse/fuse-tegra.c | 4 +++-
1 file
base and cpu0_base are not unmapped on error path, add the missing
iounmap() before return msm_dt_timer_init() in the error handling
cases.
Fixes: 6e3321631ac2 ("ARM: msm: Add DT support to msm_timer")
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/clocksource/ti
There is a error message within devm_ioremap_resource
already, so remove the dev_err call to avoid redundant
error message.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/platform/qcom/camss/camss-ispif.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions
base and cpu0_base are not unmapped on error path, add the missing
iounmap() before return msm_dt_timer_init() in the error handling
cases.
Fixes: 6e3321631ac2 ("ARM: msm: Add DT support to msm_timer")
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
Reviewed-by: Manivannan
This patch provides the ability to boot using a device tree that is appended
to the raw binary Image (e.g. cat Image .dtb > Image_w_dtb).
Both BE and LE conditions are tested.
Got references from e2a6a3aa ("ARM: zImage: Allow the appending of a device
tree binary").
Signed
On 2018/2/12 2:45, Marc Zyngier wrote:
Hi, Marc
Sorry for replying so late.
On Sun, 11 Feb 2018 03:42:01 +,
Yang Yingliang wrote:
Hi Yang,
In direct case, rd_idx will wrap if other cpus post commands
that make rd_idx increase. When rd_idx wrapped, the driver prints
timeout messages but
s no online cpu in cpu_mask.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-gic-v3-its.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
index 5416f2b..edd92a9 100644
--- a/drivers/irqchip/irq-gic-v3-i
Hi, Julien
On 2018/6/5 18:16, Julien Thierry wrote:
Hi Yang,
On 05/06/18 07:30, Yang Yingliang wrote:
When the kernel booted with maxcpus=x, 'x' is smaller
than actual cpu numbers, the TAs of offline cpus won't
be set to its->collection.
If LPI is bind to offline cpu, sync
s no online cpu in cpu_mask.
Signed-off-by: Yang Yingliang
---
drivers/irqchip/irq-gic-v3-its.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
index 5416f2b..d8b9539 100644
--- a/drivers/irqchip/irq-gic-v3-i
Hi, Casey
On 2019/4/18 8:24, Casey Schaufler wrote:
On 4/17/2019 4:39 PM, Paul Moore wrote:
On Wed, Apr 17, 2019 at 12:27 PM Oleg Nesterov wrote:
On 04/17, Paul Moore wrote:
On Wed, Apr 17, 2019 at 10:57 AM Oleg Nesterov
wrote:
On 04/17, Paul Moore wrote:
I'm tempted to simply return an e
On 2019/1/26 4:35, Corey Minyard wrote:
On Fri, Jan 25, 2019 at 10:30:59AM +0800, Yang Yingliang wrote:
When we excute the following commands, we got oops
rmmod ipmi_si
cat /proc/ioports
snip...
If io_setup is called successful in try_smi_init() but try_smi_init()
goes out_err before
ested-by: Corey Minyard
Signed-off-by: Yang Yingliang
---
drivers/char/ipmi/ipmi_si_intf.c| 5 +
drivers/char/ipmi/ipmi_si_port_io.c | 3 +++
2 files changed, 8 insertions(+)
diff --git a/drivers/char/ipmi/ipmi_si_intf.c b/drivers/char/ipmi/ipmi_si_intf.c
index dc8603d..f1b9fda 10064
On 2019/1/26 23:08, Corey Minyard wrote:
On Sat, Jan 26, 2019 at 05:14:54PM +0800, Yang Yingliang wrote:
When we excute the following commands, we got oops
rmmod ipmi_si
cat /proc/ioports
snip..
If io_setup is called successful in try_smi_init() but try_smi_init()
goes out_err before
o
Suggested-by: Corey Minyard
Signed-off-by: Yang Yingliang
---
drivers/char/ipmi/ipmi_si_intf.c| 5 +
drivers/char/ipmi/ipmi_si_mem_io.c | 5 +++--
drivers/char/ipmi/ipmi_si_port_io.c | 5 +++--
3 files changed, 11 insertions(+), 4 deletions(-)
diff --git a/drivers/char/ipmi/ipmi_si_intf.c
On 2020/7/30 19:04, Jiri Slaby wrote:
On 13. 07. 20, 12:57, Yang Yingliang wrote:
I got a slab-out-of-bounds report when I doing fuzz test.
[ 334.989515]
==
[ 334.989577] BUG: KASAN: slab-out-of-bounds in vgacon_scroll+0x57a
ot;crypto_ecdh_decode_key"
[drivers/crypto/hisilicon/hpre/hisi_hpre.ko] undefined!
Fixes: 90274769cf79 ("crypto: hisilicon/hpre - add 'CURVE25519' algorithm")
Fixes: 05e7b906aa7c ("crypto: hisilicon/hpre - add 'ECDH' algorithm")
Signed-off-by: Yang Ying
This patchset reduce instructions in copy_template.S
to improve the performance of copy memory, when size
is ge 64 bytes.
Yang Yingliang (3):
arm64: lib: introduce ldp2/stp2 macro
arm64: lib: improve copy performance when size is ge 128 bytes
arm64: lib: improve copy performance when size
Introduce ldp2/stp2 to load/store without add src/dst.
Signed-off-by: Yang Yingliang
---
arch/arm64/include/asm/asm-uaccess.h | 16
arch/arm64/lib/copy_from_user.S | 8
arch/arm64/lib/copy_in_user.S| 8
arch/arm64/lib/copy_to_user.S| 8
patch:
memcpy: 107.12 copy_from_user: 157.50 copy_to_user: 198.85
It's about 25% improvement in memcpy().
Signed-off-by: Yang Yingliang
---
arch/arm64/lib/copy_template.S | 36 +++---
1 file changed, 20 insertions(+), 16 deletions(-)
diff --git a/arch/arm6
: 13.84
It's about 5.27% improvement in memcpy().
Signed-off-by: Yang Yingliang
---
arch/arm64/lib/copy_template.S | 18 ++
1 file changed, 10 insertions(+), 8 deletions(-)
diff --git a/arch/arm64/lib/copy_template.S b/arch/arm64/lib/copy_template.S
index c3cd6f84c9c0..a9cbd47
The spinlock and list head of fnic_list is initialized statically.
It is unnecessary to initialize by spin_lock_init() and INIT_LIST_HEAD().
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/scsi/fnic/fnic_main.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers
IS_ERR() and PTR_ERR() use wrong pointer, it should be
udc->virt_addr, fix it.
Fixes: 1b9f35adb0ff ("usb: gadget: udc: Add Synopsys UDC Platform driver")
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/usb/gadget/udc/snps_udc_plat.c | 4 ++--
1 file changed,
IS_ERR() and PTR_ERR() use wrong pointer, it should be
sensor->dovdd, fix it.
Fixes: e43ccb0a045f ("media: i2c: Add support for the OV5648 image sensor")
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/i2c/ov5648.c | 4 ++--
1 file changed, 2 insertions(+)
IS_ERR() and PTR_ERR() use wrong pointer, it should be
writeback_fid, fix it.
Fixes: 5bfe97d7382b ("9p: Fix writeback fid incorrectly being attached to
dentry")
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
fs/9p/vfs_file.c | 4 ++--
1 file changed, 2 insertions(+), 2
Add the missing destroy_workqueue() before return from
pci_epf_test_init() in the error handling case and add
destroy_workqueue() in pci_epf_test_exit().
Fixes: 349e7a85b25fa ("PCI: endpoint: functions: Add an EP function to test
PCI")
Reported-by: Hulk Robot
Signed-off-by: Yang
: de69e2b3f105 ("staging: rtl8723bs: remove DBG_COUNTER calls from
os_dep/recv_linux.c")
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/staging/rtl8723bs/os_dep/recv_linux.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/staging/rtl872
Use memdup_user_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/staging/greybus/camera.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/greybus/camera.c b/drivers/staging
spinlock can be initialized automatically with DEFINE_SPINLOCK()
rather than explicitly calling spin_lock_init().
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/soc/ixp4xx/ixp4xx-qmgr.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/soc/ixp4xx
Add the missing unlock before return from accept_from_sock()
in the error handling case.
Fixes: 6cde210a9758 ("fs: dlm: add helper for init connection")
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
fs/dlm/lowcomms.c | 1 +
1 file changed, 1 insertion(+)
diff --gi
Signed-off-by: Yang Yingliang
---
drivers/soc/tegra/fuse/fuse-tegra.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/soc/tegra/fuse/fuse-tegra.c
b/drivers/soc/tegra/fuse/fuse-tegra.c
index 94b60a692b51..a0def1219501 100644
--- a/drivers/soc/tegra/fus
After device_get_match_data(), tlc591xx is not checked, add
check for it and also check np after dev_of_node.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/leds/leds-tlc591xx.c | 8 ++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/drivers/leds/leds
by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/i2c/tc358743.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/i2c/adv7604.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/i2c/adv7511-v4l2.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dri
by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/i2c/tda1997x.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/i2c/adv7842.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers
by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/power/supply/generic-adc-battery.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
di
by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/power/supply/s3c_adc_battery.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/phy/ti/phy-twl4030-usb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/d
by calling cancel_delayed_work_sync(), which ensures that
the work is properly cancelled, no longer running, and unable
to re-schedule itself.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/usb/gadget/udc/tegra-xudc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Hi,
On 2021/3/31 18:27, Greg KH wrote:
On Wed, Mar 31, 2021 at 05:42:47PM +0800, Yang Yingliang wrote:
GCC reports the following warning with W=1:
drivers/staging/rtl8723bs/os_dep/recv_linux.c:101:6: warning:
variable ‘ret’ set but not used [-Wunused-but-set-variable]
101 | int ret
Hi,
On 2021/3/31 18:24, Dan Carpenter wrote:
On Wed, Mar 31, 2021 at 05:52:01PM +0800, Yang Yingliang wrote:
Use memdup_user_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/staging/greybus/camera.c | 10
Use memdup_user_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/staging/greybus/camera.c | 13 +++--
1 file changed, 3 insertions(+), 10 deletions(-)
diff --git a/drivers/staging/greybus/camera.c b/drivers
On 2021/4/1 13:59, Greg KH wrote:
On Thu, Apr 01, 2021 at 11:17:52AM +0800, Yang Yingliang wrote:
Use memdup_user_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/staging/greybus/camera.c | 13 +++--
1
On 2021/4/1 15:43, Dan Carpenter wrote:
On Thu, Apr 01, 2021 at 11:17:52AM +0800, Yang Yingliang wrote:
Use memdup_user_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/staging/greybus/camera.c | 13
On 2021/4/1 15:47, Dan Carpenter wrote:
On Thu, Apr 01, 2021 at 10:43:32AM +0300, Dan Carpenter wrote:
On Thu, Apr 01, 2021 at 11:17:52AM +0800, Yang Yingliang wrote:
Use memdup_user_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang
t;)
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
kernel/locking/test-ww_mutex.c | 20
1 file changed, 12 insertions(+), 8 deletions(-)
diff --git a/kernel/locking/test-ww_mutex.c b/kernel/locking/test-ww_mutex.c
index 3e82f449b4ff..6e4faa853e56 100644
--- a/kern
There is an error message within devm_ioremap_resource
already, so remove the dev_err call to avoid redundant
error message.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/platform/ti-vpe/sc.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a
There is an error message within devm_ioremap_resource
already, so remove the dev_err call to avoid redundant
error message.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/platform/qcom/camss/camss-csiphy.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions
Use memdup_user_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
v3:
remove duplicate ';'
v2:
delete unnecessary blank line
use return PTR_ERR(kbuf) instead
---
drivers/staging/greybus/camera.c | 13 +++
There is an error message within devm_ioremap_resource
already, so remove the dev_err call to avoid redundant
error message.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/platform/ti-vpe/csc.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a
There is an error message within devm_ioremap_resource
already, so remove the dev_err call to avoid redundant
error message.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/platform/imx-pxp.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a
Add the missing iounmap() before return from of_fsl_spi_probe()
in the error handling case.
Fixes: 0f0581b24bd0 ("spi: fsl: Convert to use CS GPIO descriptors")
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/spi/spi-fsl-spi.c | 23 ++-
1 file c
Signed-off-by: Yang Yingliang
---
drivers/gpu/drm/nouveau/nouveau_gem.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/nouveau/nouveau_gem.c
b/drivers/gpu/drm/nouveau/nouveau_gem.c
index c88cbb85f101..492e6794c5e6 100644
--- a/drivers/gpu/drm/nouveau
Signed-off-by: Yang Yingliang
---
drivers/net/ethernet/microchip/lan743x_main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/microchip/lan743x_main.c
b/drivers/net/ethernet/microchip/lan743x_main.c
index 1c3e204d727c..e7ab5f3f73fd 100644
--- a/drivers
Signed-off-by: Yang Yingliang
---
tools/lib/bpf/linker.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/lib/bpf/linker.c b/tools/lib/bpf/linker.c
index 46b16cbdcda3..4e08bc07e635 100644
--- a/tools/lib/bpf/linker.c
+++ b/tools/lib/bpf/linker.c
@@ -1895,7 +1895,7
Signed-off-by: Yang Yingliang
---
tools/testing/selftests/kvm/x86_64/xen_shinfo_test.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/kvm/x86_64/xen_shinfo_test.c
b/tools/testing/selftests/kvm/x86_64/xen_shinfo_test.c
index 804ff5ff022d..1f4a0599683c
Signed-off-by: Yang Yingliang
---
include/trace/events/iocost.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/trace/events/iocost.h b/include/trace/events/iocost.h
index e282ce02fa2d..6d1626e7a4ce 100644
--- a/include/trace/events/iocost.h
+++ b/include/trace/events
Remove redundant semi-colon in infinalize_btf_ext().
Signed-off-by: Yang Yingliang
---
v2:
add commit log
---
tools/lib/bpf/linker.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/lib/bpf/linker.c b/tools/lib/bpf/linker.c
index 46b16cbdcda3..4e08bc07e635 100644
Use kmemdup_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/accessibility/speakup/i18n.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/accessibility/speakup/i18n.c
b/drivers
Use kmemdup_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
security/keys/trusted-keys/trusted_tpm1.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/security/keys/trusted-keys/trusted_tpm1.c
b/security
Use kmemdup_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/target/iscsi/iscsi_target_nego.c | 4 +---
drivers/target/iscsi/iscsi_target_parameters.c | 4 +---
2 files changed, 2 insertions(+), 6 deletions
There is a error message within devm_ioremap_resource
already, so remove the dev_err call to avoid redundant
error message.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/media/platform/qcom/camss/camss-csid.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff
Hi,
On 2021/4/2 21:13, Greg KH wrote:
On Fri, Apr 02, 2021 at 05:21:11PM +0800, Yang Yingliang wrote:
Use kmemdup_nul() helper instead of open-coding to
simplify the code.
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
drivers/accessibility/speakup/i18n.c | 4 +---
1 file
Use kmemdup_nul() helper instead of open-coding to
simplify the code in spk_msg_set().
Reported-by: Hulk Robot
Signed-off-by: Yang Yingliang
---
V2:
change the tile and commit log.
---
drivers/accessibility/speakup/i18n.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git
1 - 100 of 223 matches
Mail list logo
