Re: [Bug Report] Wrong value of __NR_userfaultfd in asm-generic/unistd.h

2024-10-21 Thread John Hubbard
On 10/21/24 2:33 AM, David Hildenbrand wrote: Am 21.10.24 um 08:48 schrieb Muhammad Usama Anjum: ... But now comes the tricky part: an architecture defines whether it wants to (a) Use the asm-generic unistd.h (b) Use a custom one E.g., $ cat include/uapi/linux/unistd.h /* SPDX-License-Identi

Re: [Bug Report] Wrong value of __NR_userfaultfd in asm-generic/unistd.h

2024-10-21 Thread David Hildenbrand
Am 21.10.24 um 08:48 schrieb Muhammad Usama Anjum: Hi, The asm-generic/unistd.h file has wrong __NR_userfaultfd syscall number which doesn't even depend on the architecture. This has caused failure of a selftest which was fixed recently [1]. grep -rnIF "#define __NR_userfaultfd" tools/includ

[Bug Report] Wrong value of __NR_userfaultfd in asm-generic/unistd.h

2024-10-20 Thread Muhammad Usama Anjum
Hi, The asm-generic/unistd.h file has wrong __NR_userfaultfd syscall number which doesn't even depend on the architecture. This has caused failure of a selftest which was fixed recently [1]. grep -rnIF "#define __NR_userfaultfd" tools/include/uapi/asm-generic/unistd.h:681:#define __NR_userfaultf

Re: [BUG REPORT] kernel BUG at lib/dynamic_queue_limits.c:99!

2024-07-16 Thread xiujianfeng
Hi, On 2024/7/13 8:44, Jakub Kicinski wrote: > On Fri, 12 Jul 2024 17:43:21 -0700 Jakub Kicinski wrote: >> CC: virtio_net maintainers and Jiri who added BQL > > Oh, sounds like the fix may be already posted: > https://lore.kernel.org/all/20240712080329.197605-2-jean-phili...@linaro.org/ Thanks,

Re: [BUG REPORT] kernel BUG at lib/dynamic_queue_limits.c:99!

2024-07-12 Thread Jakub Kicinski
On Fri, 12 Jul 2024 17:43:21 -0700 Jakub Kicinski wrote: > CC: virtio_net maintainers and Jiri who added BQL Oh, sounds like the fix may be already posted: https://lore.kernel.org/all/20240712080329.197605-2-jean-phili...@linaro.org/

Re: [BUG REPORT] kernel BUG at lib/dynamic_queue_limits.c:99!

2024-07-12 Thread Jakub Kicinski
CC: virtio_net maintainers and Jiri who added BQL On Fri, 12 Jul 2024 10:12:42 +0800 xiujianfeng wrote: > On 2024/7/12 10:08, xiujianfeng wrote: > > I found a problem with my QEMU environment, and the log is as follows. > > > > After I did the bisect to locate the issue, I found > > 8490dd0592e85

Re: [bug report] Memory leak from acpi_ev_install_space_handler()

2021-04-06 Thread John Garry
On 06/04/2021 17:40, Rafael J. Wysocki wrote: On Tue, Apr 6, 2021 at 5:51 PM John Garry wrote: Hi guys, On next-20210406, I enabled CONFIG_DEBUG_KMEMLEAK and CONFIG_DEBUG_TEST_DRIVER_REMOVE for my arm64 system, and see this: Hi Rafael, Why exactly do you think that acpi_ev_install_space

Re: [bug report] Memory leak from acpi_ev_install_space_handler()

2021-04-06 Thread Rafael J. Wysocki
On Tue, Apr 6, 2021 at 5:51 PM John Garry wrote: > > Hi guys, > > On next-20210406, I enabled CONFIG_DEBUG_KMEMLEAK and > CONFIG_DEBUG_TEST_DRIVER_REMOVE for my arm64 system, and see this: Why exactly do you think that acpi_ev_install_space_handler() leaks memory? > root@debian:/home/john# more

[bug report] Memory leak from acpi_ev_install_space_handler()

2021-04-06 Thread John Garry
Hi guys, On next-20210406, I enabled CONFIG_DEBUG_KMEMLEAK and CONFIG_DEBUG_TEST_DRIVER_REMOVE for my arm64 system, and see this: root@debian:/home/john# more /sys/kernel/debug/kmemleak unreferenced object 0x202803c11f00 (size 128): comm "swapper/0", pid 1, jiffies 4294894325 (age 337.524s)

Re: [bug report] node: Add memory-side caching attributes

2021-04-01 Thread Keith Busch
On Thu, Apr 01, 2021 at 12:00:39PM +0300, Dan Carpenter wrote: > Hi Keith, > > I've been trying to figure out ways Smatch can check for device managed > resources. Like adding rules that if we call dev_set_name(&foo->bar) > then it's device managaged and if there is a kfree(foo) without calling >

Re: [bug report] node: Add memory-side caching attributes

2021-04-01 Thread Dan Carpenter
On Thu, Apr 01, 2021 at 08:25:11AM -0300, Jason Gunthorpe wrote: > On Thu, Apr 01, 2021 at 12:00:39PM +0300, Dan Carpenter wrote: > > Hi Keith, > > > > I've been trying to figure out ways Smatch can check for device managed > > resources. Like adding rules that if we call dev_set_name(&foo->bar)

Re: [bug report] node: Add memory-side caching attributes

2021-04-01 Thread Jason Gunthorpe
On Thu, Apr 01, 2021 at 05:06:52PM +0300, Dan Carpenter wrote: > > diff --git a/drivers/base/node.c b/drivers/base/node.c > > index f449dbb2c74666..89c28952863977 100644 > > +++ b/drivers/base/node.c > > @@ -319,25 +319,24 @@ void node_add_cache(unsigned int nid, struct > > node_cache_attrs *cach

Re: [bug report] node: Add memory-side caching attributes

2021-04-01 Thread Jason Gunthorpe
On Thu, Apr 01, 2021 at 12:00:39PM +0300, Dan Carpenter wrote: > Hi Keith, > > I've been trying to figure out ways Smatch can check for device managed > resources. Like adding rules that if we call dev_set_name(&foo->bar) > then it's device managaged and if there is a kfree(foo) without calling >

[bug report] node: Add memory-side caching attributes

2021-04-01 Thread Dan Carpenter
Hi Keith, I've been trying to figure out ways Smatch can check for device managed resources. Like adding rules that if we call dev_set_name(&foo->bar) then it's device managaged and if there is a kfree(foo) without calling device_put(&foo->bar) then that's a resource leak. Of course one of the r

Re: [BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-15 Thread Lucas Stach
Am Montag, dem 15.02.2021 um 10:54 -0500 schrieb Sven Van Asbroeck: > Hi Lucas, > > On Mon, Feb 15, 2021 at 5:15 AM Lucas Stach wrote: > > > > The straight forward way to fix this would be to just disable the PRE > > when the stride is getting too large, which might not work well with > > all us

Re: [BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-15 Thread Sven Van Asbroeck
Hi Lucas, On Mon, Feb 15, 2021 at 5:15 AM Lucas Stach wrote: > > The straight forward way to fix this would be to just disable the PRE > when the stride is getting too large, which might not work well with > all userspace requirements, as it effectively disables the ability to > scan GPU tiled su

Re: [BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-15 Thread Lucas Stach
Hi Sven, Am Freitag, dem 12.02.2021 um 18:52 -0500 schrieb Sven Van Asbroeck: > Philipp, Fabio, > > I was able to verify that the PREs do indeed overrun their allocated ocram > area. > > Section 38.5.1 of the iMX6QuadPlus manual indicates the ocram size > required: width(pixels) x 8 lines x 4 b

Re: [EXT] [bug report] octeontx2-af: cn10k: Uninitialized variables

2021-02-13 Thread Geethasowjanya Akula
bbaraya Sundeep Bhatta; David S. Miller; >Jakub >Kicinski >Cc: net...@vger.kernel.org; linux-kernel@vger.kernel.org; Gustavo A. R. Silva >Subject: [EXT] [bug report] octeontx2-af: cn10k: Uninitialized variables >External Email -

Re: [EXT] [bug report] octeontx2-af: cn10k: Identical code for different branches

2021-02-13 Thread Geethasowjanya Akula
Bhatta; David S. Miller; >Jakub >Kicinski >Cc: net...@vger.kernel.org; linux-kernel@vger.kernel.org; Gustavo A. R. Silva >Subject: [EXT] [bug report] octeontx2-af: cn10k: Identical code for different >branches >External Email ---

Re: [BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-12 Thread Sven Van Asbroeck
Philipp, Fabio, I was able to verify that the PREs do indeed overrun their allocated ocram area. Section 38.5.1 of the iMX6QuadPlus manual indicates the ocram size required: width(pixels) x 8 lines x 4 bytes. For 2048 pixels max, this comes to 64K. This is what the PRE driver allocates. So far, s

[bug report] octeontx2-af: cn10k: Identical code for different branches

2021-02-12 Thread Gustavo A. R. Silva
Hi, In file drivers/net/ethernet/marvell/octeontx2/af/rvu_debugfs.c, function rvu_dbg_init() the same code is executed for both branches: 2431 if (is_rvu_otx2(rvu)) 2432 debugfs_create_file("rvu_pf_cgx_map", 0444, rvu->rvu_dbg.root, 2433

[bug report] octeontx2-af: cn10k: Uninitialized variables

2021-02-12 Thread Gustavo A. R. Silva
Hi, Variables cgx_id and lmac_id are being used uninitialized at lines 731 and 733 in the following function: 723 static int rvu_cgx_config_intlbk(struct rvu *rvu, u16 pcifunc, bool en) 724 { 725 struct mac_ops *mac_ops; 726 u8 cgx_id, lmac_id; 727 728 if (!is_cgx_config_p

Re: [BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-11 Thread Sven Van Asbroeck
Hi Philipp, thank you so much for looking into this, I really appreciate it ! On Thu, Feb 11, 2021 at 9:32 AM Philipp Zabel wrote: > > Another thing that might help to identify who is writing where might be to > clear the whole OCRAM region and dump it after running only decode or only > PRE/PRG

Re: [BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-11 Thread Philipp Zabel
Hi Sven, On Wed, Feb 10, 2021 at 01:29:29PM -0500, Sven Van Asbroeck wrote: > Found it! > > The i.MX6QuadPlus has two pairs of PREs, which use the extended > section of the iRAM. The Classic does not have any PREs or extended > iRAM: > > pre1: pre@21c8000 { >compatible = "fsl,imx6qp-pre"; >

Re: [BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-10 Thread Sven Van Asbroeck
Found it! The i.MX6QuadPlus has two pairs of PREs, which use the extended section of the iRAM. The Classic does not have any PREs or extended iRAM: pre1: pre@21c8000 { compatible = "fsl,imx6qp-pre"; fsl,iram = <&ocram2>; }; pre3: pre@21ca000 { compatible = "fsl,imx6qp-pre";

Re: [BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-10 Thread Sven Van Asbroeck
Bonjour Nicolas, On Wed, Feb 10, 2021 at 11:11 AM Nicolas Dufresne wrote: > > Are you sure you aren't just running out of CMA ? This is the only things that > comes to mind at the moment, sorry if it's not that useful. Thanks for the suggestion! No worries, this is such a strange/weird problem,

Re: [BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-10 Thread Nicolas Dufresne
Hi Sven, Le mercredi 03 février 2021 à 11:33 -0500, Sven Van Asbroeck a écrit : > From: Sven Van Asbroeck > > We have observed that under certain repeatable circumstances, the CODA > mem2mem device consistently generates corrupted frames. This happens only > on an i.MX6qp (Plus) - the classic im

[BUG REPORT] media: coda: mpeg4 decode corruption on i.MX6qp only

2021-02-03 Thread Sven Van Asbroeck
From: Sven Van Asbroeck We have observed that under certain repeatable circumstances, the CODA mem2mem device consistently generates corrupted frames. This happens only on an i.MX6qp (Plus) - the classic imx6q is not affected. This happens when the virtual X screen is wider than 0x900 pixels (1)

Re: [bug report] sched/fair: Prefer prev cpu in asymmetric wakeup path

2020-11-13 Thread Dan Carpenter
On Fri, Nov 13, 2020 at 09:56:37AM +0100, Vincent Guittot wrote: > Hi Dan, > > Le vendredi 13 nov. 2020 à 11:46:57 (+0300), Dan Carpenter a écrit : > > Hello Vincent Guittot, > > > > The patch b4c9c9f15649: "sched/fair: Prefer prev cpu in asymmetric > > wakeup path" from Oct 29, 2020, leads to th

Re: [bug report] sched/fair: Prefer prev cpu in asymmetric wakeup path

2020-11-13 Thread Vincent Guittot
Hi Dan, Le vendredi 13 nov. 2020 à 11:46:57 (+0300), Dan Carpenter a écrit : > Hello Vincent Guittot, > > The patch b4c9c9f15649: "sched/fair: Prefer prev cpu in asymmetric > wakeup path" from Oct 29, 2020, leads to the following static checker > warning: > > kernel/sched/fair.c:6249 selec

[bug report] sched/fair: Prefer prev cpu in asymmetric wakeup path

2020-11-13 Thread Dan Carpenter
Hello Vincent Guittot, The patch b4c9c9f15649: "sched/fair: Prefer prev cpu in asymmetric wakeup path" from Oct 29, 2020, leads to the following static checker warning: kernel/sched/fair.c:6249 select_idle_sibling() error: uninitialized symbol 'task_util'. kernel/sched/fair.c 6

Re: [bug report] tracing, synthetic events: Replace buggy strcat() with seq_buf operations

2020-11-02 Thread Steven Rostedt
On Mon, 2 Nov 2020 10:45:24 +0300 Dan Carpenter wrote: > Hello Steven Rostedt (VMware), > > The patch 761a8c58db6b: "tracing, synthetic events: Replace buggy > strcat() with seq_buf operations" from Oct 23, 2020, leads to the > following static checker warning: > > kernel/trace/trace_even

arm64: BUG report: Hugepages leads cpu failed to come online.

2020-09-16 Thread 李成栋
Hello folks, We observed that Arm VM instance (Centos 8) runs into timeout at boot time when it is launched by Qemu/KVM on an Arm server host that enabled hugepages (2MB/1GB). We also noticed that Will has a workaround that applies to the guest kernel (https://git.kernel.org/pub/scm/linux/kerne

Re: USB: PHY: JZ4770: Add support for new Ingenic SoCs - bug report

2020-07-27 Thread Colin Ian King
Hi, Static analysis with Coverity has detected an issue with the following commit: commit 2a6c0b82e65128c73b5102e00e031c5e58bb3504 Author: 周琰杰 (Zhou Yanjie) Date: Thu Jul 23 14:13:00 2020 +0800 USB: PHY: JZ4770: Add support for new Ingenic SoCs. The analysis from Coverity is as follows:

Re: Bug Report High CPU Usage events_power_efficient

2020-07-14 Thread Martin Zaharinov
Hi Oki i find the problem is come from nf_conntrack_core I isolate problem in this part : queue_delayed_work(system_power_efficient_wq, &conntrack_gc_work.dwork, HZ); When package go to queue delayed in one moment if connection track is to big process to delayed go to lock and start high cp

Re: Bug Report High CPU Usage events_power_efficient

2020-07-08 Thread Greg KH
A: Because it messes up the order in which people normally read text. Q: Why is top-posting such a bad thing? A: Top-posting. Q: What is the most annoying thing in e-mail? A: No. Q: Should I include quotations after my reply? http://daringfireball.net/2007/07/on_top On Wed, Jul 08, 2020 at 11:3

Re: Bug Report High CPU Usage events_power_efficient

2020-07-08 Thread Martin Zaharinov
Yes i search but not find any information. And write hear to help if any have same problem or any of you have idea from where is comme this problem. If need more debug i will only write how to get more information. Martin На ср, 8.07.2020 г. в 10:09 Greg KH написа: > > On Wed, Jul 08, 2020 at

Re: Bug Report High CPU Usage events_power_efficient

2020-07-08 Thread Greg KH
On Wed, Jul 08, 2020 at 09:50:49AM +0300, Martin Zaharinov wrote: > Add Greg , Florian, Eric to this bug > > > On 7 Jul 2020, at 22:54, Martin Zaharinov wrote: > > > > And this is log from /sys/kernel/debug/tracing/trace > > > > > > # entries-in-buffer/entries-written: 32410/32410 #P:64 > >

Re: Bug Report High CPU Usage events_power_efficient

2020-07-07 Thread Martin Zaharinov
Add Greg , Florian, Eric to this bug > On 7 Jul 2020, at 22:54, Martin Zaharinov wrote: > > And this is log from /sys/kernel/debug/tracing/trace > > > # entries-in-buffer/entries-written: 32410/32410 #P:64 > # > # _-=> irqs-off > #

Re: Bug Report High CPU Usage events_power_efficient

2020-07-07 Thread Martin Zaharinov
And this is log from /sys/kernel/debug/tracing/trace # entries-in-buffer/entries-written: 32410/32410 #P:64 # # _-=> irqs-off # / _=> need-resched #| / _---=> hardirq/softirq #

Re: Bug Report High CPU Usage events_power_efficient

2020-07-07 Thread Martin Zaharinov
the problem is hear with kernel 5.7.7 last work kernel without this problem is 5.6.7 hear is more info: cat /proc/57259/stack root@megacableamarilis:~# cat /proc/57259/stack [<0>] gc_worker+0x1be/0x380 [nf_conntrack] [<0>] process_one_work+0x1bc/0x3b0 [<0>] worker_thread+0x4d/0x460 [<0>] kthrea

[PATCH AUTOSEL 5.6 125/606] USB: core: Fix misleading driver bug report

2020-06-08 Thread Sasha Levin
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH 4.4 10/65] USB: core: Fix misleading driver bug report

2020-05-26 Thread Greg Kroah-Hartman
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH 4.14 23/59] USB: core: Fix misleading driver bug report

2020-05-26 Thread Greg Kroah-Hartman
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH 5.4 041/111] USB: core: Fix misleading driver bug report

2020-05-26 Thread Greg Kroah-Hartman
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH 5.6 046/126] USB: core: Fix misleading driver bug report

2020-05-26 Thread Greg Kroah-Hartman
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH 4.19 32/81] USB: core: Fix misleading driver bug report

2020-05-26 Thread Greg Kroah-Hartman
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH 4.9 17/64] USB: core: Fix misleading driver bug report

2020-05-26 Thread Greg Kroah-Hartman
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH AUTOSEL 5.6 38/62] USB: core: Fix misleading driver bug report

2020-05-14 Thread Sasha Levin
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH AUTOSEL 5.4 34/49] USB: core: Fix misleading driver bug report

2020-05-14 Thread Sasha Levin
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH AUTOSEL 4.19 26/31] USB: core: Fix misleading driver bug report

2020-05-14 Thread Sasha Levin
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH AUTOSEL 4.14 35/39] USB: core: Fix misleading driver bug report

2020-05-14 Thread Sasha Levin
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH AUTOSEL 4.4 11/14] USB: core: Fix misleading driver bug report

2020-05-14 Thread Sasha Levin
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

[PATCH AUTOSEL 4.9 24/27] USB: core: Fix misleading driver bug report

2020-05-14 Thread Sasha Levin
From: Alan Stern [ Upstream commit ac854131d9844f79e2fdcef67a7707227538d78a ] The syzbot fuzzer found a race between URB submission to endpoint 0 and device reset. Namely, during the reset we call usb_ep0_reinit() because the characteristics of ep0 may have changed (if the reset follows a firmw

Re: bug report: libceph: follow redirect replies from osds

2019-08-30 Thread Ilya Dryomov
On Fri, Aug 30, 2019 at 4:05 PM Colin Ian King wrote: > > Hi, > > Static analysis with Coverity has picked up an issue with commit: > > commit 205ee1187a671c3b067d7f1e974903b44036f270 > Author: Ilya Dryomov > Date: Mon Jan 27 17:40:20 2014 +0200 > > libceph: follow redirect replies from osd

bug report: libceph: follow redirect replies from osds

2019-08-30 Thread Colin Ian King
Hi, Static analysis with Coverity has picked up an issue with commit: commit 205ee1187a671c3b067d7f1e974903b44036f270 Author: Ilya Dryomov Date: Mon Jan 27 17:40:20 2014 +0200 libceph: follow redirect replies from osds Specifically in function ceph_redirect_decode in net/ceph/osd_client.

re: clk: actions: Don't reference clk_init_data after registration [bug report]

2019-08-16 Thread Colin Ian King
Hi, Static analysis with Coverity Scan on linux-next has found an issue with the following commit: commit 20cac6d02815edcc0b1c87bc3e8858b3d1fda3fa Author: Stephen Boyd Date: Wed Jul 31 12:35:09 2019 -0700 clk: actions: Don't reference clk_init_data after registration The analysis is as f

Re: static analysis bug report in drivers/staging/iio/dac/ad5380.c

2019-08-15 Thread Lars-Peter Clausen
On 8/15/19 12:21 PM, Colin Ian King wrote: > Hi, > > Static analysis with Coverity Scan has detected a potential assignment > bug in ad5380.c: > > 217case IIO_CHAN_INFO_CALIBBIAS: > 218ret = regmap_read(st->regmap, > AD5380_REG_OFFSET(chan->address), > 219

static analysis bug report in drivers/staging/iio/dac/ad5380.c

2019-08-15 Thread Colin Ian King
Hi, Static analysis with Coverity Scan has detected a potential assignment bug in ad5380.c: 217case IIO_CHAN_INFO_CALIBBIAS: 218ret = regmap_read(st->regmap, AD5380_REG_OFFSET(chan->address), 219val); 220if (ret) 221

Re: rcu/nocb: Add bypass callback queueing, bug report

2019-08-13 Thread Paul E. McKenney
On Tue, Aug 13, 2019 at 01:34:02PM +0100, Colin Ian King wrote: > Hi, > > Static analysis on linux-next today found an issue in the following commit: > > commit 1afc4b18724f8f7b7a21fdf66cd43cc4a932812d > Author: Paul E. McKenney > Date: Tue Jul 2 16:03:33 2019 -0700 > > rcu/nocb: Add bypa

re: rcu/nocb: Add bypass callback queueing, bug report

2019-08-13 Thread Colin Ian King
Hi, Static analysis on linux-next today found an issue in the following commit: commit 1afc4b18724f8f7b7a21fdf66cd43cc4a932812d Author: Paul E. McKenney Date: Tue Jul 2 16:03:33 2019 -0700 rcu/nocb: Add bypass callback queueing The coverity report is as follows: 1783// If we ha

Re: drm/exynos: scaler: Reset hardware before starting the operation (bug report)

2019-07-22 Thread Marek Szyprowski
Hi, On 2019-07-05 18:09, Colin Ian King wrote: > Static analysis on today's linux-next has found a potential error in the > following commit: > > commit 280e54c9f614c88292685383cf2d65057586e9fb > Author: Andrzej Pietrasiewicz > Date: Thu Jun 7 13:06:08 2018 +0200 > > drm/exynos: scaler: Re

Re: drm/exynos: scaler: Reset hardware before starting the operation (bug report)

2019-07-05 Thread Colin Ian King
Hi, Static analysis on today's linux-next has found a potential error in the following commit: commit 280e54c9f614c88292685383cf2d65057586e9fb Author: Andrzej Pietrasiewicz Date: Thu Jun 7 13:06:08 2018 +0200 drm/exynos: scaler: Reset hardware before starting the operation In the followi

Re: static analysis bug report: ALSA: swapped client and port arguments in calls to snd_seq_oss_fill_addr

2019-06-28 Thread Takashi Iwai
On Thu, 27 Jun 2019 16:01:40 +0200, Colin Ian King wrote: > > Hi there, > > Static analysis with Coverity has picked up two potential issues with > the call to function snd_seq_oss_fill_addr. The prototype of > snd_seq_oss_fill_addr is as follows: > > static inline void > snd_seq_oss_fill_addr(s

static analysis bug report: ALSA: swapped client and port arguments in calls to snd_seq_oss_fill_addr

2019-06-27 Thread Colin Ian King
Hi there, Static analysis with Coverity has picked up two potential issues with the call to function snd_seq_oss_fill_addr. The prototype of snd_seq_oss_fill_addr is as follows: static inline void snd_seq_oss_fill_addr(struct seq_oss_devinfo *dp, struct snd_seq_event *ev,int dest_client, int dest

Re: media: staging/imx: Improve pipeline searching (bug report)

2019-06-26 Thread Steve Longerbeam
Thanks for catching, On 6/26/19 11:27 AM, Colin Ian King wrote: Hi, Static analysis with Coverity on Linux next has found a potential issue with the following commit: commit 3ef46bc97ca2c918b7657a08220c7340a9bb07a2 Author: Steve Longerbeam Date: Fri May 10 17:50:11 2019 -0400 media: s

re: media: staging/imx: Improve pipeline searching (bug report)

2019-06-26 Thread Colin Ian King
Hi, Static analysis with Coverity on Linux next has found a potential issue with the following commit: commit 3ef46bc97ca2c918b7657a08220c7340a9bb07a2 Author: Steve Longerbeam Date: Fri May 10 17:50:11 2019 -0400 media: staging/imx: Improve pipeline searching The issue is in drivers/sta

[bug report] read-ahead can't work properly

2019-06-24 Thread Weijie Yang
When try the file readahead by posix_fadvise(), I find it can't work properly. For example, posix_fadvise(POSIX_FADV_WILLNEED) a 10MB file, the kernel actually readahead only 512KB data to the page cache, even if there are enough free memory in the machine. When trace to kernel, I find the issu

Re: [bug report][stable] kernel tried to execute NX-protected page - exploit attempt? (uid: 0)

2019-06-09 Thread Joseph Qi
Confirmed the following 3 upstream commits can resolve this issue: d2a68c4effd8 x86/ftrace: Do not call function graph from dynamic trampolines 3c0dab44e227 x86/ftrace: Set trampoline pages as executable 7298e24f9042 x86/kprobes: Set instruction page as executable And they are all included in stab

Re: [bug report][stable] kernel tried to execute NX-protected page - exploit attempt? (uid: 0)

2019-06-09 Thread Greg KH
On Sun, Jun 09, 2019 at 09:10:45PM +0800, Joseph Qi wrote: > Hi Nadav, > Thanks for the comments. > I'll test the 3 patches in the mentioned thread. This should all be fixed in the latest release that happened today. If not, please let us know. thanks, greg k-h

Re: [bug report][stable] kernel tried to execute NX-protected page - exploit attempt? (uid: 0)

2019-06-09 Thread Joseph Qi
Hi Nadav, Thanks for the comments. I'll test the 3 patches in the mentioned thread. Thanks, Joseph On 19/6/8 00:38, Nadav Amit wrote: >> On Jun 7, 2019, at 3:24 AM, Joseph Qi wrote: >> >> Hi all, >> Any idea on this regression? > > Sorry for the late response (I assumed, for some reason, that

Re: [bug report][stable] kernel tried to execute NX-protected page - exploit attempt? (uid: 0)

2019-06-07 Thread Nadav Amit
> On Jun 7, 2019, at 3:24 AM, Joseph Qi wrote: > > Hi all, > Any idea on this regression? Sorry for the late response (I assumed, for some reason, that you also follow the second thread about this issue). Anyhow, it should be fixed by backporting some patches which were mistakenly missed. Se

Re: [bug report][stable] kernel tried to execute NX-protected page - exploit attempt? (uid: 0)

2019-06-07 Thread Joseph Qi
Hi all, Any idea on this regression? Thanks, Joseph On 19/6/5 18:23, Joseph Qi wrote: > Hi, > > I have encountered a kernel BUG when running ltp ftrace-stress-test > on 4.19.48. > > [ 209.704855] LTP: starting ftrace-stress-test (ftrace_stress_test.sh 90) > [ 209.739412] Scheduler tracepoint

[bug report][stable] kernel tried to execute NX-protected page - exploit attempt? (uid: 0)

2019-06-05 Thread Joseph Qi
Hi, I have encountered a kernel BUG when running ltp ftrace-stress-test on 4.19.48. [ 209.704855] LTP: starting ftrace-stress-test (ftrace_stress_test.sh 90) [ 209.739412] Scheduler tracepoints stat_sleep, stat_iowait, stat_blocked and stat_runtime require the kernel parameter schedstats=enabl

Re: Bug report: A commit about serial8250 cause the output disorderly at the phase of startup

2019-04-23 Thread Hongzhi, Song
On 4/23/19 2:48 PM, John Ogness wrote: On 2019-04-22, Hongzhi, Song wrote: Anyone notice this issue? Yes, I am aware of the issue. It is actually a feature, not a bug. ;-) Individual LOG_CONT messages, when classified as emergency messages, are printed immediately to the console. Yes, this

Re: Bug report: A commit about serial8250 cause the output disorderly at the phase of startup

2019-04-22 Thread John Ogness
On 2019-04-22, Hongzhi, Song wrote: > Anyone notice this issue? Yes, I am aware of the issue. It is actually a feature, not a bug. ;-) Individual LOG_CONT messages, when classified as emergency messages, are printed immediately to the console. This makes them appear "disorderly". It is not yet c

Re: Bug report: A commit about serial8250 cause the output disorderly at the phase of startup

2019-04-21 Thread Hongzhi, Song
Hi all, Anyone notice this issue? --Hongzhi On 4/19/19 10:24 AM, Hongzhi, Song wrote: 1. Issue description: Boot kernel( >= linux-rt-devel-v5.0.3 ) with qemu. Then qemu will print following disorderly messages. At the beginning, the messages are disorderly. But then it becomes normally fr

Re: [BUG REPORT] linux-input: keyboard: gpio_keys: False Button Press Event on Wake

2019-04-03 Thread dmitry.torok...@gmail.com
Hi Ken, On Wed, Apr 03, 2019 at 05:50:09PM +, Ken Sloat wrote: > Hello Dmitry, > > I may have found a potential bug in the "gpio_keys" driver. FYI, I am > running the 4.14 LTS kernel on my system, but from my understanding of > the issue, it seems that this would still occur in the latest ver

[BUG REPORT] linux-input: keyboard: gpio_keys: False Button Press Event on Wake

2019-04-03 Thread Ken Sloat
Hello Dmitry, I may have found a potential bug in the "gpio_keys" driver. FYI, I am running the 4.14 LTS kernel on my system, but from my understanding of the issue, it seems that this would still occur in the latest version of the kernel. The problem: In the 4.14 LTS kernel, both key press and

Re: [Fix PATCH] cpu/hotplug: Fix bug report when add "nosmt" parameter with CONFIG_HOTPLUG_CPU=N

2019-03-26 Thread Tianyu Lan
Hi Thomas: Thanks for your review. On Tue, Mar 26, 2019 at 6:39 AM Thomas Gleixner wrote: > > On Mon, 25 Mar 2019, Thomas Gleixner wrote: > > That has nothing to do with 'nosmt'. It's a general bug in the rollback > > code when HOTPLUG_CPU=n. 'nosmt' is using the rollback mechanism and

Re: [Fix PATCH] cpu/hotplug: Fix bug report when add "nosmt" parameter with CONFIG_HOTPLUG_CPU=N

2019-03-25 Thread Thomas Gleixner
On Mon, 25 Mar 2019, Thomas Gleixner wrote: > That has nothing to do with 'nosmt'. It's a general bug in the rollback > code when HOTPLUG_CPU=n. 'nosmt' is using the rollback mechanism and is > just a reliable way to trigger the problem. This happens in the same way > when the bringup of a CPU fail

Re: [Fix PATCH] cpu/hotplug: Fix bug report when add "nosmt" parameter with CONFIG_HOTPLUG_CPU=N

2019-03-25 Thread Greg KH
On Mon, Mar 25, 2019 at 09:51:23PM +0800, lantianyu1...@gmail.com wrote: > From: Lan Tianyu > > When add "nosmt" parameter, kernel still boots up all logical cpus once > and set CR4.MCE on each CPU. This is to avoid shutting down machine > when a broadacasted MCE is observed CR4.MCE=0b. (Detail p

Re: [Fix PATCH] cpu/hotplug: Fix bug report when add "nosmt" parameter with CONFIG_HOTPLUG_CPU=N

2019-03-25 Thread Thomas Gleixner
Tianyu, On Mon, 25 Mar 2019, lantianyu1...@gmail.com wrote: thanks for tracking this down. A few nit picks vs. the subject first: > Subject: [Fix PATCH] cpu/hotplug: Fix bug report when add "nosmt" parameter > with CONFIG_HOTPLUG_CPU=N [PATCH] is sufficient. The extra 'F

[Fix PATCH] cpu/hotplug: Fix bug report when add "nosmt" parameter with CONFIG_HOTPLUG_CPU=N

2019-03-25 Thread lantianyu1986
From: Lan Tianyu When add "nosmt" parameter, kernel still boots up all logical cpus once and set CR4.MCE on each CPU. This is to avoid shutting down machine when a broadacasted MCE is observed CR4.MCE=0b. (Detail please see comment in the cpu_smt_allowed()). Smt cpus will bring up and bring down

Re: [bug report][stable] perf probe: failed to add events

2019-03-25 Thread Adrian Hunter
On 21/03/19 12:10 PM, Greg KH wrote: > On Thu, Feb 28, 2019 at 09:19:08AM +0200, Adrian Hunter wrote: >> On 28/02/19 4:07 AM, Joseph Qi wrote: >>> Hi Adrian, >>> >>> On 19/2/27 20:39, Adrian Hunter wrote: Seems to be fixed by this: From: Adrian Hunter Date: Wed, 27 Feb 2019 05:

Re: [bug report][stable] perf probe: failed to add events

2019-03-21 Thread Greg KH
On Thu, Feb 28, 2019 at 09:19:08AM +0200, Adrian Hunter wrote: > On 28/02/19 4:07 AM, Joseph Qi wrote: > > Hi Adrian, > > > > On 19/2/27 20:39, Adrian Hunter wrote: > >> Seems to be fixed by this: > >> > >> From: Adrian Hunter > >> Date: Wed, 27 Feb 2019 05:35:25 +0200 > >> Subject: [PATCH] perf

Re: static analysis bug report: staging r8712u memcpy of uninitialized variable

2019-03-20 Thread Dan Carpenter
On Mon, Mar 18, 2019 at 11:20:51AM +, Colin Ian King wrote: > Hi, > > Static analysis with cppcheck found a couple of interesting issues with > memcpy'ing of an uninitialized variable. Two occurrences of the same > issue are found in drivers/staging/rtl8712/rtl8712_cmd.c in functions > read_bb

static analysis bug report: staging: rtl8192u: use of uninitialized array

2019-03-18 Thread Colin Ian King
Hi, static analysis with cppcheck has detected use of an uninitialized array tmp_ssid in drivers/staging/rtl8192u/ieee80211/ieee80211_softmac.c in function ieee80211_softmac_new_net() Array tmp_ssid is only initialized when ssidbroad is non-null, however it is being copied and the copy of this is

static analysis bug report: staging r8712u memcpy of uninitialized variable

2019-03-18 Thread Colin Ian King
Hi, Static analysis with cppcheck found a couple of interesting issues with memcpy'ing of an uninitialized variable. Two occurrences of the same issue are found in drivers/staging/rtl8712/rtl8712_cmd.c in functions read_bbreg_hdl and read_rfreg_hdl. For example: static u8 read_bbreg_hdl(struct _

Re: [bug report][stable] perf probe: failed to add events

2019-03-02 Thread Joseph Qi
Hi Adrian, On 19/2/28 15:19, Adrian Hunter wrote: > On 28/02/19 4:07 AM, Joseph Qi wrote: >> Hi Adrian, >> >> On 19/2/27 20:39, Adrian Hunter wrote: >>> Seems to be fixed by this: >>> >>> From: Adrian Hunter >>> Date: Wed, 27 Feb 2019 05:35:25 +0200 >>> Subject: [PATCH] perf probe: Fix getting th

Re: [bug report][stable] perf probe: failed to add events

2019-02-27 Thread Adrian Hunter
On 28/02/19 4:07 AM, Joseph Qi wrote: > Hi Adrian, > > On 19/2/27 20:39, Adrian Hunter wrote: >> Seems to be fixed by this: >> >> From: Adrian Hunter >> Date: Wed, 27 Feb 2019 05:35:25 +0200 >> Subject: [PATCH] perf probe: Fix getting the kernel map >> >> Since commit 4d99e4136580 ("perf machine:

Re: [bug report][stable] perf probe: failed to add events

2019-02-27 Thread Joseph Qi
Hi Adrian, On 19/2/27 20:39, Adrian Hunter wrote: > Seems to be fixed by this: > > From: Adrian Hunter > Date: Wed, 27 Feb 2019 05:35:25 +0200 > Subject: [PATCH] perf probe: Fix getting the kernel map > > Since commit 4d99e4136580 ("perf machine: Workaround missing maps for x86 > PTI entry tram

Re: [bug report][stable] perf probe: failed to add events

2019-02-27 Thread Adrian Hunter
On 26/02/19 4:20 PM, Arnaldo Carvalho de Melo wrote: > Em Tue, Feb 26, 2019 at 02:08:02PM +0100, Greg KH escreveu: >> On Tue, Feb 26, 2019 at 08:32:34PM +0800, Joseph Qi wrote: >>> >>> >>> On 19/2/26 17:05, Greg KH wrote: On Tue, Feb 26, 2019 at 03:31:14PM +0800, Joseph Qi wrote: > Hi, >>>

Re: [bug report][stable] perf probe: failed to add events

2019-02-26 Thread Joseph Qi
On 19/2/26 21:08, Greg KH wrote: > On Tue, Feb 26, 2019 at 08:32:34PM +0800, Joseph Qi wrote: >> >> >> On 19/2/26 17:05, Greg KH wrote: >>> On Tue, Feb 26, 2019 at 03:31:14PM +0800, Joseph Qi wrote: Hi, I'm using kernel v4.19.24 and have found that there is an issue when usin

Re: [bug report][stable] perf probe: failed to add events

2019-02-26 Thread Arnaldo Carvalho de Melo
Em Tue, Feb 26, 2019 at 02:08:02PM +0100, Greg KH escreveu: > On Tue, Feb 26, 2019 at 08:32:34PM +0800, Joseph Qi wrote: > > > > > > On 19/2/26 17:05, Greg KH wrote: > > > On Tue, Feb 26, 2019 at 03:31:14PM +0800, Joseph Qi wrote: > > >> Hi, > > >> > > >> I'm using kernel v4.19.24 and have found

Re: [bug report][stable] perf probe: failed to add events

2019-02-26 Thread Greg KH
On Tue, Feb 26, 2019 at 08:32:34PM +0800, Joseph Qi wrote: > > > On 19/2/26 17:05, Greg KH wrote: > > On Tue, Feb 26, 2019 at 03:31:14PM +0800, Joseph Qi wrote: > >> Hi, > >> > >> I'm using kernel v4.19.24 and have found that there is an issue when > >> using perf probe to define a new dynamic tr

Re: [bug report][stable] perf probe: failed to add events

2019-02-26 Thread Joseph Qi
On 19/2/26 17:05, Greg KH wrote: > On Tue, Feb 26, 2019 at 03:31:14PM +0800, Joseph Qi wrote: >> Hi, >> >> I'm using kernel v4.19.24 and have found that there is an issue when >> using perf probe to define a new dynamic tracepoint. >> >> $ perf probe -a handle_mm_fault >> Failed to write event:

Re: [bug report][stable] perf probe: failed to add events

2019-02-26 Thread Greg KH
On Tue, Feb 26, 2019 at 03:31:14PM +0800, Joseph Qi wrote: > Hi, > > I'm using kernel v4.19.24 and have found that there is an issue when > using perf probe to define a new dynamic tracepoint. > > $ perf probe -a handle_mm_fault > Failed to write event: Numerical result out of range > Error: Fa

[bug report][stable] perf probe: failed to add events

2019-02-25 Thread Joseph Qi
Hi, I'm using kernel v4.19.24 and have found that there is an issue when using perf probe to define a new dynamic tracepoint. $ perf probe -a handle_mm_fault Failed to write event: Numerical result out of range Error: Failed to add events. I've also tried kernel v4.20, and it can pass. So I'v

Re: bug report: iwlwifi: mvm: support mac80211 TXQs model

2019-02-20 Thread Johannes Berg
On Wed, 2019-02-20 at 14:40 +, Colin Ian King wrote: > > ..when the used_hw_queues initialization was removed: > > @@ -360,8 +300,6 @@ int iwl_mvm_mac_ctxt_init(struct iwl_mvm *mvm, > struct ieee80211_vif *vif) > mvm->hw, IEEE80211_IFACE_ITER_RESUME_ALL, > iwl_

  1   2   3   4   5   6   7   >