Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-12 Thread Like Xu
Hi Yao, On 2020/8/13 11:11, Jin, Yao wrote: Hi Like, On 8/12/2020 9:02 PM, Like Xu wrote: On 2020/8/12 20:15, Arnaldo Carvalho de Melo wrote: Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: Currently if we run 'perf record -e cycles:u', exclude_guest is 0. But it doesn't make se

Re: [RFC PATCH 1/3] mm: Drop locked from isolate_migratepages_block

2020-08-12 Thread Alex Shi
在 2020/8/13 下午12:02, Alexander Duyck 写道: > From: Alexander Duyck > > We can drop the need for the locked variable by making use of the > lruvec_holds_page_lru_lock function. By doing this we can avoid some rcu > locking ugliness for the case where the lruvec is still holding the LRU > lock ass

[PATCH] fs/io_uring.c: Fix uninitialized variable is referenced in io_submit_sqe

2020-08-12 Thread Liu Yong
the commit ("opcode>") caused another vulnerability. After io_get_req(), the sqe_submit struct in req is not initialized, but the following code defaults that req->submit.opcode is available. Signed-off-by: Liu Yong --- fs/io_uring.c | 1 + 1 file changed, 1 insertion(+) diff --git a/fs/io_

[PATCH] Documentation: ABI: iio: fix some spelling mistakes

2020-08-12 Thread Ahmad Fatoum
This fixes some typos found while browsing the documentation. Cc: triv...@kernel.org Signed-off-by: Ahmad Fatoum --- Documentation/ABI/testing/sysfs-bus-iio | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Documentation/ABI/testing/sysfs-bus-iio b/Documentation/ABI/test

[PATCH] staging: rtl8723bs: remove 5 GHz code

2020-08-12 Thread Michael Straube
According to the TODO 5 GHz code should be removed. - find and remove remaining code valid only for 5 GHz. Most of the obvious ones have been removed, but things like channel > 14 still exist. Remove code for channels > 14 from rtw_get_center_ch(). Signed-off-by: Michael Straube --- drivers/

fs/btrfs/volumes.h:206 btrfs_device_set_total_bytes() warn: statement has no effect 31

2020-08-12 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: dc06fe51d26efc100ac74121607c01a454867c91 commit: 55f3560df975f557c48aa6afc636808f31ecb87a seqlock: Extend seqcount API with associated locks date: 2 weeks ago config: parisc-randconfig-m031-20200811 (attac

Re: [PATCH v5 1/3] arm64: dts: Add a device tree for the Librem 5 phone

2020-08-12 Thread Martin Kepplinger
On 31.07.20 10:27, Martin Kepplinger wrote: > From: "Angus Ainslie (Purism)" > > Add a devicetree description for the Librem 5 phone. 4 hardware revisions > have been available. Some revisions include changes that need different > software to be run. So far, r3 ("Dogwood") is one such example, se

Re: [RFC PATCH v1] dma-fence-array: Deal with sub-fences that are signaled late

2020-08-12 Thread Christian König
Am 13.08.20 um 01:55 schrieb Jordan Crouse: This is an RFC because I'm still trying to grok the correct behavior. Consider a dma_fence_array created two two fence and signal_on_any is true. A reference to dma_fence_array is taken for each waiting fence. Ok, that sounds like you seem to mix a c

linux-next: runtime warning in Linus' tree

2020-08-12 Thread Stephen Rothwell
Hi all, Testing Linus' tree today, my qemu runs (PowerPC powerpc_pseries_le_defconfig) produce the following WARNING: [0.021401][T0] Mount-cache hash table entries: 8192 (order: 0, 65536 bytes, linear) [0.021529][T0] Mountpoint-cache hash table entries: 8192 (order: 0, 65536 byt

Re: Re: [PATCH] ALSA: echoaudio: Fix memory leak in snd_echo_resume()

2020-08-12 Thread dinghao . liu
> On Mon, 10 Aug 2020 08:14:58 +0200, > Dinghao Liu wrote: > > > > When restore_dsp_rettings() fails, chip should be freed > > just like when init_hw() and request_irq() fails. > > > > Fixes: 47b5d028fdce8 ("ALSA: Echoaudio - Add suspend support #2") > > Signed-off-by: Dinghao Liu > > Actually

Re: [PATCH v2] riscv: Setup exception vector for nommu platform

2020-08-12 Thread Atish Patra
On Wed, Aug 12, 2020 at 10:44 PM Damien Le Moal wrote: > > On 2020/08/13 12:40, Qiu Wenbo wrote: > > Exception vector is missing on nommu platform and that is an issue. > > This patch is tested in Sipeed Maix Bit Dev Board. > > > > Fixes: 79b1feba5455 ("RISC-V: Setup exception vector early") > > S

Re: Re: [PATCH] ALSA: pci/asihpi: Fix memory leak in snd_card_asihpi_capture_open()

2020-08-12 Thread dinghao . liu
> On Mon, 10 Aug 2020 08:39:55 +0200, > Dinghao Liu wrote: > > > > When snd_pcm_hw_constraint_pow2() fails, dpcm should be freed > > just like when hpi_instream_open() fails. > > It will be cleaned up via runtime->private_free call in the caller > side (snd_pcm_open -> snd_pcm_release_substream

Re: [PATCH] overflow: Add __must_check attribute to check_*() helpers

2020-08-12 Thread Rasmus Villemoes
On 12/08/2020 23.51, Kees Cook wrote: > Since the destination variable of the check_*_overflow() helpers will > contain a wrapped value on failure, it would be best to make sure callers > really did check the return result of the helper. Adjust the macros to use > a bool-wrapping static inline that

Re: [PATCH v2] drm/of: Consider the state in which the ep is disabled

2020-08-12 Thread Huang Jiachai
ping... 在 2020/7/23 3:01, Heiko Stübner 写道: Am Dienstag, 7. Juli 2020, 13:25:26 CEST schrieb Sandy Huang: don't mask possible_crtcs if remote-point is disabled. Signed-off-by: Sandy Huang Reviewed-by: Heiko Stuebner changes in v2: - drop additional of_node_put, as ep will be put with the n

Re: [PATCH v3 2/2] ASoC: Intel: Add period size constraint on strago board

2020-08-12 Thread Yu-Hsuan Hsu
Pierre-Louis Bossart 於 2020年8月13日 週四 上午12:38寫道: > > > > On 8/12/20 11:08 AM, Lu, Brent wrote: > >>> > >>> I also wonder what's really missing, too :) > >>> > >>> BTW, I took a look back at the thread, and CRAS seems using a very > >>> large buffer, namely: > >>> [ 52.434791] sound pcmC1D0p: PE

[PATCH v2 0/5] Fixes and improvements for Xen pvdrm

2020-08-12 Thread Oleksandr Andrushchenko
From: Oleksandr Andrushchenko Hello, This series contains an assorted set of fixes and improvements for the Xen para-virtualized display driver and grant device driver which I have collected over the last couple of months: 1. Minor fixes to grant device driver and drm/xen-front. 2. New format

[PATCH v2 4/5] xen: Sync up with the canonical protocol definition in Xen

2020-08-12 Thread Oleksandr Andrushchenko
From: Oleksandr Andrushchenko This is the sync up with the canonical definition of the display protocol in Xen. 1. Add protocol version as an integer Version string, which is in fact an integer, is hard to handle in the code that supports different protocol versions. To simplify that also add t

[PATCH v2 5/5] drm/xen-front: Pass dumb buffer data offset to the backend

2020-08-12 Thread Oleksandr Andrushchenko
From: Oleksandr Andrushchenko While importing a dmabuf it is possible that the data of the buffer is put with offset which is indicated by the SGT offset. Respect the offset value and forward it to the backend. Signed-off-by: Oleksandr Andrushchenko Acked-by: Noralf Trønnes --- drivers/gpu/dr

[PATCH v2 2/5] drm/xen-front: Fix misused IS_ERR_OR_NULL checks

2020-08-12 Thread Oleksandr Andrushchenko
From: Oleksandr Andrushchenko The patch c575b7eeb89f: "drm/xen-front: Add support for Xen PV display frontend" from Apr 3, 2018, leads to the following static checker warning: drivers/gpu/drm/xen/xen_drm_front_gem.c:140 xen_drm_front_gem_create() warn: passing zero to 'ERR_CAST'

[PATCH v2 1/5] xen/gntdev: Fix dmabuf import with non-zero sgt offset

2020-08-12 Thread Oleksandr Andrushchenko
From: Oleksandr Andrushchenko It is possible that the scatter-gather table during dmabuf import has non-zero offset of the data, but user-space doesn't expect that. Fix this by failing the import, so user-space doesn't access wrong data. Fixes: bf8dc55b1358 ("xen/gntdev: Implement dma-buf import

[PATCH v2 3/5] drm/xen-front: Add YUYV to supported formats

2020-08-12 Thread Oleksandr Andrushchenko
From: Oleksandr Andrushchenko Add YUYV to supported formats, so the frontend can work with the formats used by cameras and other HW. Signed-off-by: Oleksandr Andrushchenko Acked-by: Noralf Trønnes --- drivers/gpu/drm/xen/xen_drm_front_conn.c | 1 + 1 file changed, 1 insertion(+) diff --git a

Re: [Resend PATCH 2/6] mm/memcg: remove useless check on page->mem_cgroup

2020-08-12 Thread Michal Hocko
On Wed 12-08-20 11:25:53, Alex Shi wrote: > >From 999b0fe5fc65865c3b59ff28500d45572a4a9570 Mon Sep 17 00:00:00 2001 > From: Alex Shi > Date: Wed, 5 Aug 2020 21:02:30 +0800 > Subject: [PATCH 2/6] mm/memcg: bail out early from swap accounting when memcg > is disabled > > If we disabled memcg by cg

[PATCH] Documentation/locking/locktypes: fix the typo

2020-08-12 Thread Huang Shijie
We have three categories locks, not two. Signed-off-by: Huang Shijie --- Documentation/locking/locktypes.rst | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/locking/locktypes.rst b/Documentation/locking/locktypes.rst index 1b577a8bf982..4cefed8048ca 100644 ---

Re: [PATCH v8 05/14] media: rkisp1: add Rockchip ISP1 subdev driver

2020-08-12 Thread Dafna Hirschfeld
Am 07.08.20 um 18:08 schrieb Dafna Hirschfeld: Hi Am 06.08.20 um 14:22 schrieb Tomasz Figa: On Thu, Aug 6, 2020 at 11:21 AM Dafna Hirschfeld wrote: Am 05.08.20 um 23:10 schrieb Dafna Hirschfeld: Hi On 22.07.20 17:24, Tomasz Figa wrote: Hi Dafna, On Sat, Jul 11, 2020 at 01:04:31PM +0

[PATCH] sound/soc/intel: Fix spelling mistake "cant" --> "can't"

2020-08-12 Thread Youling Tang
There is some spelling mistakes in a dev_err message. Fix it. Signed-off-by: Youling Tang --- sound/soc/intel/common/sst-firmware.c | 4 ++-- sound/soc/intel/haswell/sst-haswell-ipc.c | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/sound/soc/intel/common/sst-firmwa

答复: [PATCH] vfio dma_map/unmap: optimized for hugetlbfs pages

2020-08-12 Thread Maoming (maoming, Cloud Infrastructure Service Product Dept.)
Hi, Alex Thanks for taking a look. Also, my apologies for the slow reply. Some replies below: -邮件原件- 发件人: Zhoujian (jay) 发送时间: 2020年7月21日 21:28 收件人: Alex Williamson 抄送: linux-kernel@vger.kernel.org; k...@vger.kernel.org; coh...@redhat.com; Maoming (maoming, Cloud Infrastructure Service P

Re: [PATCH 08/11] extcon: ptn5150: Check current USB mode when probing

2020-08-12 Thread Krzysztof Kozlowski
On Wed, Aug 12, 2020 at 06:29:55PM +0200, Krzysztof Kozlowski wrote: > When machine boots up, the USB could be already in OTG mode. In such > case there will be no interrupt coming to ptn5150 device and driver will > report default state of nothing connected. Detection of USB connection > would h

Re: [PATCH RFC v2 00/18] Add VFIO mediated device support and DEV-MSI support for the idxd driver

2020-08-12 Thread Jason Wang
On 2020/8/13 下午1:26, Tian, Kevin wrote: From: Jason Wang Sent: Thursday, August 13, 2020 12:34 PM On 2020/8/12 下午12:05, Tian, Kevin wrote: The problem is that if we tie all controls via VFIO uAPI, the other subsystem like vDPA is likely to duplicate them. I wonder if there is a way to decou

RE: [PATCH 1/1] arm64: kexec: no need to do irq_chip->irq_mask if it already masked

2020-08-12 Thread Jason Liu
> -Original Message- > From: Marc Zyngier > Sent: Thursday, August 6, 2020 8:26 PM > To: Jason Liu > Cc: Sudeep Holla ; catalin.mari...@arm.com; > w...@kernel.org; sas...@kernel.org; linux-kernel@vger.kernel.org; > linux-arm-ker...@lists.infradead.org > Subject: Re: [PATCH 1/1] arm64:

Re: [PATCH] softirq: add irq off checking for __raise_softirq_irqoff

2020-08-12 Thread Peter Zijlstra
On Thu, Aug 06, 2020 at 12:07:29PM +0800, Jiafei Pan wrote: > __raise_softirq_irqoff will update per-CPU mask of pending softirqs, > it need to be called in irq disabled context in order to keep it atomic > operation, otherwise it will be interrupted by hardware interrupt, > and per-CPU softirqs pe

Re: [PATCH v3 2/2] powerpc/uaccess: Add pre-update addressing to __get_user_asm() and __put_user_asm()

2020-08-12 Thread Christophe Leroy
Le 12/08/2020 à 21:37, Segher Boessenkool a écrit : On Wed, Aug 12, 2020 at 12:25:17PM +, Christophe Leroy wrote: Enable pre-update addressing mode in __get_user_asm() and __put_user_asm() Signed-off-by: Christophe Leroy --- v3: new, splited out from patch 1. It still looks fine to me

Re: [PATCH v4 1/7] pinctrl: qcom: Add msmgpio irqchip flags

2020-08-12 Thread Maulik Shah
Hi, On 8/12/2020 1:02 AM, Stephen Boyd wrote: Can the subject be more specific? "pinctrl: qcom: Set IRQCHIP_SET_TYPE_MASKED flag"? Sure i can update subject in v5. Thanks, Maulik Quoting Maulik Shah (2020-08-10 04:20:54) Add irqchip specific flags for msmgpio irqchip to mask non wakeirqs

Re: [PATCH v2] riscv: Setup exception vector for nommu platform

2020-08-12 Thread Damien Le Moal
On 2020/08/13 12:40, Qiu Wenbo wrote: > Exception vector is missing on nommu platform and that is an issue. > This patch is tested in Sipeed Maix Bit Dev Board. > > Fixes: 79b1feba5455 ("RISC-V: Setup exception vector early") > Suggested-by: Anup Patel > Suggested-by: Atish Patra > Signed-off-by

Re: WARN_ON_ONCE(1) in iomap_dio_actor()

2020-08-12 Thread Dave Chinner
On Mon, Aug 10, 2020 at 10:03:03PM -0400, Qian Cai wrote: > On Sun, Jul 26, 2020 at 04:24:12PM +0100, Christoph Hellwig wrote: > > On Fri, Jul 24, 2020 at 02:24:32PM -0400, Qian Cai wrote: > > > On Fri, Jun 19, 2020 at 05:17:47PM -0700, Matthew Wilcox wrote: > > > > On Fri, Jun 19, 2020 at 05:17:50

[PATCH] rtc: cmos: initialize rtc time when reading alarm

2020-08-12 Thread Victor Ding
cmos_read_alarm() may leave certain fields of a struct rtc_time untouched; therefore, these fields contain garbage if not properly initialized, leading to inconsistent values when converting into time64_t. This patch to set all fields of a struct rtc_time to -1 before calling cmos_read_alarm(). Si

Re: [RFC 2/7] KVM: VMX: Expose IA32_PKRS MSR

2020-08-12 Thread Chenyi Qiang
On 8/13/2020 5:21 AM, Jim Mattson wrote: On Fri, Aug 7, 2020 at 1:46 AM Chenyi Qiang wrote: Protection Keys for Supervisor Pages (PKS) uses IA32_PKRS MSR (PKRS) at index 0x6E1 to allow software to manage supervisor protection key rights. For performance consideration, PKRS intercept will be

Re: [PATCH v2] riscv: Setup exception vector for nommu platform

2020-08-12 Thread Atish Patra
On Wed, Aug 12, 2020 at 8:40 PM Qiu Wenbo wrote: > > Exception vector is missing on nommu platform and that is an issue. > This patch is tested in Sipeed Maix Bit Dev Board. > > Fixes: 79b1feba5455 ("RISC-V: Setup exception vector early") > Suggested-by: Anup Patel > Suggested-by: Atish Patra >

Re: [PATCH 3/3] usb: Add Kconfig and Makefile changes to build brcmstb-usb-pinmap

2020-08-12 Thread Greg Kroah-Hartman
On Wed, Aug 12, 2020 at 04:20:18PM -0400, Al Cooper wrote: > From: Al Cooper > > Add Kconfig and Makefile changes to build brcmstb-usb-pinmap and > update MAINTAINERS for the new driver. This can be part of the previous patch, or at least the Kconfig and Makefile changes should be there so that

Re: [PATCH 2/2] iommu/iova: Free global iova rcache on iova alloc failure

2020-08-12 Thread Vijayanand Jitta
On 8/12/2020 8:46 PM, Joerg Roedel wrote: > On Mon, Aug 03, 2020 at 03:30:48PM +0530, Vijayanand Jitta wrote: >> ping? > > Please repost when v5.9-rc1 is released and add > > Robin Murphy > > on your Cc list. > > Thanks, > > Joerg > Sure, will do. Thanks, Vijay -- QUALCOMM

Re: [PATCH net] bonding: show saner speed for broadcast mode

2020-08-12 Thread Jay Vosburgh
Jarod Wilson wrote: >Broadcast mode bonds transmit a copy of all traffic simultaneously out of >all interfaces, so the "speed" of the bond isn't really the aggregate of >all interfaces, but rather, the speed of the lowest active interface. Did you mean "slowest" here? >Also, the type of

RE: [PATCH RFC v2 00/18] Add VFIO mediated device support and DEV-MSI support for the idxd driver

2020-08-12 Thread Tian, Kevin
> From: Jason Wang > Sent: Thursday, August 13, 2020 12:34 PM > > > On 2020/8/12 下午12:05, Tian, Kevin wrote: > >> The problem is that if we tie all controls via VFIO uAPI, the other > >> subsystem like vDPA is likely to duplicate them. I wonder if there is a > >> way to decouple the vSVA out of

Re: [PATCH v3 2/2] power: supply: sbs-battery: don't assume i2c errors as battery disconnect

2020-08-12 Thread Nicolas Boichat
On Thu, Aug 13, 2020 at 1:11 PM Ikjoon Jang wrote: > > Current sbs-battery considers all smbus errors as disconnection events > when battery-detect pin isn't supplied, and restored to present state back > when any successful transaction is made. > > This can lead to unlimited state changes between

Re: WARNING: locking bug in l2cap_chan_del

2020-08-12 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:06a7a37b ipv4: tunnel: fix compilation on ARCH=um git tree: net console output: https://syzkaller.appspot.com/x/log.txt?x=115caa1690 kernel config: https://syzkaller.appspot.com/x/.config?x=7bb894f55faf8242 dashbo

Re: POC: Alternative solution: Re: [PATCH 0/4] printk: reimplement LOG_CONT handling

2020-08-12 Thread Sergey Senozhatsky
On (20/08/13 02:30), John Ogness wrote: > 2. I haven't yet figured out how to preserve calling context when a > newline appears. For example: > > pr_info("text"); > pr_cont(" 1"); > pr_cont(" 2\n"); > pr_cont("3"); > pr_cont(" 4\n"); > > For "3" the calling context (info, timestamp) is lost becaus

Re: [PATCH 1/3] mm: don't call activate_page() on new ksm pages

2020-08-12 Thread Yang Shi
On Tue, Aug 11, 2020 at 9:04 PM Yu Zhao wrote: > > lru_cache_add_active_or_unevictable() already adds new ksm pages to > active lru. Calling activate_page() isn't really necessary in this > case. > > Signed-off-by: Yu Zhao > --- > mm/swapfile.c | 10 +- > 1 file changed, 5 insertions(+),

drivers/char/agp/i460-agp.c:254:19: sparse: sparse: incorrect type in assignment (different address spaces)

2020-08-12 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: dc06fe51d26efc100ac74121607c01a454867c91 commit: 670d0a4b10704667765f7d18f7592993d02783aa sparse: use identifiers to define address spaces date: 8 weeks ago config: ia64-randconfig-s031-20200813 (attached

Re: [PATCH] ftrace: Fixup lockdep assert held of text_mutex

2020-08-12 Thread Palmer Dabbelt
On Thu, 06 Aug 2020 22:01:01 PDT (-0700), guo...@kernel.org wrote: On Fri, Aug 7, 2020 at 12:01 PM Steven Rostedt wrote: On Fri, 7 Aug 2020 10:59:16 +0800 Guo Ren wrote: > > > > This looks like a bug in the lockdep_assert_held() in whatever arch > > (riscv) is running. > Seems you think it's

Re: [PATCH v2 15/20] fuse, dax: Take ->i_mmap_sem lock during dax page fault

2020-08-12 Thread Dave Chinner
On Wed, Aug 12, 2020 at 05:10:12PM -0400, Vivek Goyal wrote: > On Wed, Aug 12, 2020 at 11:23:45AM +1000, Dave Chinner wrote: > > On Tue, Aug 11, 2020 at 01:55:30PM -0400, Vivek Goyal wrote: > > > On Tue, Aug 11, 2020 at 08:22:38AM +1000, Dave Chinner wrote: > > > > On Fri, Aug 07, 2020 at 03:55:21P

Re: KMSAN: uninit-value in can_receive (2)

2020-08-12 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:ce8056d1 wip: changed copy_from_user where instrumented git tree: https://github.com/google/kmsan.git master console output: https://syzkaller.appspot.com/x/log.txt?x=1041a3be90 kernel config: https://syzkaller.ap

[PATCH v3 1/2] power: supply: sbs-battery: combine get_presence_and_health

2020-08-12 Thread Ikjoon Jang
This patch enables calling sbs_get_battery_presence_and_health() without checking its chip type. No functional changes. Signed-off-by: Ikjoon Jang --- drivers/power/supply/sbs-battery.c | 73 +++--- 1 file changed, 36 insertions(+), 37 deletions(-) diff --git a/drivers/p

[PATCH v3 2/2] power: supply: sbs-battery: don't assume i2c errors as battery disconnect

2020-08-12 Thread Ikjoon Jang
Current sbs-battery considers all smbus errors as disconnection events when battery-detect pin isn't supplied, and restored to present state back when any successful transaction is made. This can lead to unlimited state changes between present and !present when one unsupported command was requeste

[PATCH v3 0/2] power: supply: sbs-battery: fix presence check

2020-08-12 Thread Ikjoon Jang
When gpio detection is not supplied, presence state transitions depend on every smbus transfer result from get_property(). This patch tries to check battery presence again with well supported command before state transition. Changes: v3: check return value of get_presence_and_health() v2: combine

Re: [PATCH] perf parse-events: Set exclude_guest for user-space counting

2020-08-12 Thread Jin, Yao
Hi Arnaldo, On 8/12/2020 8:55 PM, Arnaldo Carvalho de Melo wrote: Em Wed, Aug 12, 2020 at 09:15:04AM -0300, Arnaldo Carvalho de Melo escreveu: Em Wed, Aug 12, 2020 at 02:59:53PM +0800, Jin Yao escreveu: Currently if we run 'perf record -e cycles:u', exclude_guest is 0. But it doesn't make sen

Re: Recursive/circular locking in serial8250_console_write/serial8250_do_startup

2020-08-12 Thread Guenter Roeck
On Wed, Aug 12, 2020 at 08:48:13AM -0700, Guenter Roeck wrote: > Hi, > > crbug.com/1114800 reports a hard lockup due to circular locking in the > 8250 console driver. This is seen if CONFIG_PROVE_LOCKING is enabled. > > Problem is as follows: > - serial8250_do_startup() locks the serial (console)

Re: [PATCH v17 1/3] dt-bindings: Add bindings for Mediatek matrix keypad

2020-08-12 Thread Yingjoe Chen
On Wed, 2020-08-12 at 15:13 -0700, Dmitry Torokhov wrote: > Hi, > > On Tue, Aug 11, 2020 at 09:47:23AM +0800, Yingjoe Chen wrote: > > Hi, > > > > > > On Mon, 2020-08-10 at 14:40 +0800, Fengping Yu wrote: > > > From: "fengping.yu" > > > > > > This patch add devicetree bindings for Mediatek matr

Re: [RFC 7/7] KVM: VMX: Enable PKS for nested VM

2020-08-12 Thread Chenyi Qiang
On 8/11/2020 8:05 AM, Jim Mattson wrote: On Fri, Aug 7, 2020 at 1:47 AM Chenyi Qiang wrote: PKS MSR passes through guest directly. Configure the MSR to match the L0/L1 settings so that nested VM runs PKS properly. Signed-off-by: Chenyi Qiang --- arch/x86/kvm/vmx/nested.c | 32 ++

Re: [RFC PATCH 0/5] Introduce /proc/all/ to gather stats from all processes

2020-08-12 Thread David Ahern
On 8/12/20 1:51 AM, Andrei Vagin wrote: > > I rebased the task_diag patches on top of v5.8: > https://github.com/avagin/linux-task-diag/tree/v5.8-task-diag Thanks for updating the patches. > > /proc/pid files have three major limitations: > * Requires at least three syscalls per process per fil

[PATCH] selftests: rtnetlink: load fou module for kci_test_encap_fou()

2020-08-12 Thread Po-Hsu Lin
The kci_test_encap_fou() test from kci_test_encap() in rtnetlink.sh needs the fou module to work. Otherwise it will fail with: $ ip netns exec "$testns" ip fou add port ipproto 47 RTNETLINK answers: No such file or directory Error talking to the kernel Add the CONFIG_NET_FOU into the c

drivers/char/agp/i460-agp.c:254:19: sparse: expected void gatt

2020-08-12 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: dc06fe51d26efc100ac74121607c01a454867c91 commit: df41017eafd267c08acbfff99d34e4f96bbfbc92 ia64: remove support for machvecs date: 12 months ago config: ia64-randconfig-s031-20200813 (attached as .config) c

Re: [PATCH RFC v2 00/18] Add VFIO mediated device support and DEV-MSI support for the idxd driver

2020-08-12 Thread Jason Wang
On 2020/8/12 下午12:05, Tian, Kevin wrote: The problem is that if we tie all controls via VFIO uAPI, the other subsystem like vDPA is likely to duplicate them. I wonder if there is a way to decouple the vSVA out of VFIO uAPI? vSVA is a per-device (either pdev or mdev) feature thus naturally shou

Re: [PATCH] opp: Fix dev_pm_opp_set_rate() to not return early

2020-08-12 Thread Viresh Kumar
On 11-08-20, 14:09, Stephen Boyd wrote: > This is a goto maze! Any chance we can clean this up? I have sent a short series in reply to this series, please have a look. It should look better now. -- viresh

[PATCH V2 1/4] opp: Enable resources again if they were disabled earlier

2020-08-12 Thread Viresh Kumar
From: Rajendra Nayak dev_pm_opp_set_rate() can now be called with freq = 0 in order to either drop performance or bandwidth votes or to disable regulators on platforms which support them. In such cases, a subsequent call to dev_pm_opp_set_rate() with the same frequency ends up returning early be

[PATCH 2/4] opp: Track device's resources configuration status

2020-08-12 Thread Viresh Kumar
The OPP core needs to track if the resources of devices are enabled or configured or not, as it disables the resources when target_freq is set to 0. Handle that with a separate variable to make it easy to maintain. Also note that we will unconditionally call clk_set_rate() in the case where the r

[PATCH 3/4] opp: Reused enabled flag and remove regulator_enabled

2020-08-12 Thread Viresh Kumar
The common "enabled" flag can be used here instead of "regulator_enabled" now. Signed-off-by: Viresh Kumar --- drivers/opp/core.c | 13 +++-- drivers/opp/opp.h | 2 -- 2 files changed, 3 insertions(+), 12 deletions(-) diff --git a/drivers/opp/core.c b/drivers/opp/core.c index e8882e7f

[PATCH 4/4] opp: Split out _opp_set_rate_zero()

2020-08-12 Thread Viresh Kumar
Create separate routine _opp_set_rate_zero() to handle !target_freq case. Signed-off-by: Viresh Kumar --- drivers/opp/core.c | 52 +--- 1 file changed, 29 insertions(+), 23 deletions(-) diff --git a/drivers/opp/core.c b/drivers/opp/core.c index 5f

Re: [RFC PATCH 00/16] Core scheduling v6

2020-08-12 Thread Li, Aubrey
On 2020/8/13 7:08, Joel Fernandes wrote: > On Wed, Aug 12, 2020 at 10:01:24AM +0800, Li, Aubrey wrote: >> Hi Joel, >> >> On 2020/8/10 0:44, Joel Fernandes wrote: >>> Hi Aubrey, >>> >>> Apologies for replying late as I was still looking into the details. >>> >>> On Wed, Aug 05, 2020 at 11:57:20AM +0

[rcu:dev.2020.08.11a 110/112] ERROR: modpost: "rcu_read_unlock_strict" undefined!

2020-08-12 Thread kernel test robot
Hi Paul, First bad commit (maybe != root cause): tree: https://git.kernel.org/pub/scm/linux/kernel/git/paulmck/linux-rcu.git dev.2020.08.11a head: 9030fc13639a71b15466520f09ca07b80fc30ed1 commit: 601cd69fa4bb0127bb774cc852203fd9fdf5e269 [110/112] rcuperf: Change rcuperf to rcuscale config:

Re: KMSAN: uninit-value in joydev_connect

2020-08-12 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:ce8056d1 wip: changed copy_from_user where instrumented git tree: https://github.com/google/kmsan.git master console output: https://syzkaller.appspot.com/x/log.txt?x=127b9fba90 kernel config: https://syzkaller.ap

Re: [PATCH v2 2/2] power: supply: sbs-battery: don't assume every i2c errors as battery disconnect

2020-08-12 Thread Nicolas Boichat
On Tue, Aug 11, 2020 at 2:59 PM Ikjoon Jang wrote: > > Current sbs-battery considers all smbus errors as diconnection events disconnection > when battery-detect pin isn't supplied, and restored to present state back > on any successful transaction were made. when any... is made > > This can le

RE: [PATCH v3] exfat: remove EXFAT_SB_DIRTY flag

2020-08-12 Thread Namjae Jeon
> Thanks for thinking on this complicated issue. > > > > Most of the NAND flash devices and HDDs have wear leveling and bad sector > > replacement algorithms > applied. > > So I think that the life of the boot sector will not be exhausted first. > > I'm not too worried about the life of the boo

[RFC PATCH 0/3] Re: [PATCH v17 14/21] mm/compaction: do page isolation first in compaction

2020-08-12 Thread Alexander Duyck
Here are the patches I had discussed earlier to address the issues in isolate_migratepages_block. They are based on the tree at: https://github.com/alexshi/linux.git lrunext The first patch is mostly cleanup to address the RCU locking in the function. The second addresses the test_and_set_skip

[RFC PATCH 1/3] mm: Drop locked from isolate_migratepages_block

2020-08-12 Thread Alexander Duyck
From: Alexander Duyck We can drop the need for the locked variable by making use of the lruvec_holds_page_lru_lock function. By doing this we can avoid some rcu locking ugliness for the case where the lruvec is still holding the LRU lock associated with the page. Instead we can just use the lruve

[RFC PATCH 2/3] mm: Drop use of test_and_set_skip in favor of just setting skip

2020-08-12 Thread Alexander Duyck
From: Alexander Duyck The only user of test_and_set_skip was isolate_migratepages_block and it was using it after a call that was testing and clearing the LRU flag. As such it really didn't need to be behind the LRU lock anymore as it wasn't really fulfilling its purpose. With that being the cas

[RFC PATCH 3/3] mm: Identify compound pages sooner in isolate_migratepages_block

2020-08-12 Thread Alexander Duyck
From: Alexander Duyck Since we are holding a reference to the page much sooner in isolate_migratepages_block we could move the PageCompound check out of the LRU locked section and instead just place it after get_page_unless_zero. By doing this we can allow any of the items that might trigger a fa

Re: file metadata via fs API (was: [GIT PULL] Filesystem Information)

2020-08-12 Thread Jeffrey E Altman
On 8/12/2020 2:18 PM, Linus Torvalds (torva...@linux-foundation.org) wrote: > What's wrong with fstatfs()? All the extra magic metadata seems to not > really be anything people really care about. > > What people are actually asking for seems to be some unique mount ID, > and we have 16 bytes of sp

Re: INFO: task hung in pipe_release (2)

2020-08-12 Thread syzbot
syzbot has bisected this issue to: commit fddb5d430ad9fa91b49b1d34d0202ffe2fa0e179 Author: Aleksa Sarai Date: Sat Jan 18 12:07:59 2020 + open: introduce openat2(2) syscall bisection log: https://syzkaller.appspot.com/x/bisect.txt?x=164e716a90 start commit: 6ba1b005 Merge tag 'a

[PATCH net] bonding: show saner speed for broadcast mode

2020-08-12 Thread Jarod Wilson
Broadcast mode bonds transmit a copy of all traffic simultaneously out of all interfaces, so the "speed" of the bond isn't really the aggregate of all interfaces, but rather, the speed of the lowest active interface. Also, the type of the speed field is u32, not unsigned long, so adjust that accor

Re: [PATCH v2] nvme: Use spin_lock_irq() when taking the ctrl->lock

2020-08-12 Thread Logan Gunthorpe
On 2020-08-12 6:32 p.m., Keith Busch wrote: > There's an unrelated whitespace change in nvme_init_identify(). > Otherwise, looks fine. Oops, sorry. can this be fixed up when it's merged? Logan

Re: [PATCH v17 14/21] mm/compaction: do page isolation first in compaction

2020-08-12 Thread Alex Shi
在 2020/8/13 上午10:17, Alexander Duyck 写道: >> zone lock is probability better. you can try and test. > So I spent a good chunk of today looking the code over and what I > realized is that we probably don't even really need to have this code > protected by the zone lock since the LRU bit in the pag

[PATCH] mm: include CMA pages in lowmem_reserve at boot

2020-08-12 Thread Doug Berger
The lowmem_reserve arrays provide a means of applying pressure against allocations from lower zones that were targeted at higher zones. Its values are a function of the number of pages managed by higher zones and are assigned by a call to the setup_per_zone_lowmem_reserve() function. The function

Re: WARNING in compat_do_ebt_get_ctl

2020-08-12 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:fb893de3 Merge tag 'tag-chrome-platform-for-v5.9' of git:/.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=1742b31c90 kernel config: https://syzkaller.appspot.com/x/.config?x=f1fedc6

Re: file metadata via fs API

2020-08-12 Thread Ian Kent
On Wed, 2020-08-12 at 12:50 -0700, Linus Torvalds wrote: > On Wed, Aug 12, 2020 at 12:34 PM Steven Whitehouse < > swhit...@redhat.com> wrote: > > The point of this is to give us the ability to monitor mounts from > > userspace. > > We haven't had that before, I don't see why it's suddenly such a b

[PATCH] drivers/input/misc: Use kobj_to_dev() instead

2020-08-12 Thread Wang Qing
Use kobj_to_dev() instead of container_of() Signed-off-by: Wang Qing --- drivers/input/misc/ims-pcu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/input/misc/ims-pcu.c b/drivers/input/misc/ims-pcu.c index d8dbfc0..0879d96 --- a/drivers/input/misc/ims-pcu.c +++ b/dr

[PATCH] drm/hisilicon: Fix build error of no type of module_init

2020-08-12 Thread Tian Tao
Add missing include to fix build error: hibmc_drm_drv.c:385:1: warning: data definition has no type or storage class [enabled by default] hibmc_drm_drv.c:385:1: error: type defaults to ‘int’ in declaration of ‘module_init’ [-Werror=implicit-int] hibmc_drm_drv.c:385:1: warning: parameter names (with

[PATCH v2] riscv: Setup exception vector for nommu platform

2020-08-12 Thread Qiu Wenbo
Exception vector is missing on nommu platform and that is an issue. This patch is tested in Sipeed Maix Bit Dev Board. Fixes: 79b1feba5455 ("RISC-V: Setup exception vector early") Suggested-by: Anup Patel Suggested-by: Atish Patra Signed-off-by: Qiu Wenbo --- arch/riscv/kernel/head.S | 25

[PATCH] drivers/greybus: Use kobj_to_dev() instead

2020-08-12 Thread Wang Qing
Use kobj_to_dev() instead of container_of() Signed-off-by: Wang Qing --- drivers/greybus/interface.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/greybus/interface.c b/drivers/greybus/interface.c index 67dbe6f..de167f1 --- a/drivers/greybus/interface.c +++ b/

Re: [PATCH] binfmt_flat: revert "binfmt_flat: don't offset the data start"

2020-08-12 Thread Greg Ungerer
Hi Max, On 9/8/20 4:37 am, Max Filippov wrote: binfmt_flat loader uses the gap between text and data to store data segment pointers for the libraries. Even in the absence of shared libraries it stores at least one pointer to the executable's own data segment. Text and data can go back to back in

Re: KMSAN: uninit-value in ath9k_htc_rx_msg

2020-08-12 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:ce8056d1 wip: changed copy_from_user where instrumented git tree: https://github.com/google/kmsan.git master console output: https://syzkaller.appspot.com/x/log.txt?x=12985a1690 kernel config: https://syzkaller.ap

[PATCH v6] Add MediaTek MT6779 devapc driver

2020-08-12 Thread Neal Liu
These patch series introduce a MediaTek MT6779 devapc driver. MediaTek bus fabric provides TrustZone security support and data protection to prevent slaves from being accessed by unexpected masters. The security violation is logged and sent to the processor for further analysis or countermeasure

[PATCH v6 2/2] soc: mediatek: add mt6779 devapc driver

2020-08-12 Thread Neal Liu
MediaTek bus fabric provides TrustZone security support and data protection to prevent slaves from being accessed by unexpected masters. The security violation is logged and sent to the processor for further analysis or countermeasures. Any occurrence of security violation would raise an interrupt

[PATCH v6 1/2] dt-bindings: devapc: add bindings for mtk-devapc

2020-08-12 Thread Neal Liu
Add bindings for mtk-devapc. Signed-off-by: Neal Liu --- .../devicetree/bindings/soc/mediatek/devapc.yaml | 58 1 file changed, 58 insertions(+) create mode 100644 Documentation/devicetree/bindings/soc/mediatek/devapc.yaml diff --git a/Documentation/devicetree/bindings

Re: [PATCH v2] lib/cmdline: prevent unintented access to address

2020-08-12 Thread Randy Dunlap
On 8/12/20 8:07 PM, Seungil Kang wrote: > When args = "\"\0", "i" will be 0 and args[i-1] is used. (*lib/cmdline.c +238) > Because of "i" is an unsigned int type, the function will access at > args[0x]. > It can make a crash. > > Signed-off-by: Seungil Kang > --- > > Thanks for your rev

Re: [PATCH tip/core/rcu 06/12] rcu: Do full report for .need_qs for strict GPs

2020-08-12 Thread Paul E. McKenney
On Thu, Aug 13, 2020 at 02:50:27AM +0200, Jann Horn wrote: > On Thu, Aug 13, 2020 at 12:57 AM wrote: > > The rcu_preempt_deferred_qs_irqrestore() function is invoked at > > the end of an RCU read-side critical section (for example, directly > > from rcu_read_unlock()) and, if .need_qs is set, invo

[PATCH] drivers/dax: Use kobj_to_dev() instead

2020-08-12 Thread Wang Qing
Use kobj_to_dev() instead of container_of() Signed-off-by: Wang Qing --- drivers/dax/bus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dax/bus.c b/drivers/dax/bus.c index df238c8..24625d2 --- a/drivers/dax/bus.c +++ b/drivers/dax/bus.c @@ -331,7 +331,7 @@ static D

drivers/md/dm-ebs-target.c:264:4: warning: Variable 'r' is reassigned a value before the old one has been used.

2020-08-12 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: dc06fe51d26efc100ac74121607c01a454867c91 commit: d3c7b35c20d60650bac8b55c17b194adda03a979 dm: add emulated block size target date: 3 months ago compiler: sparc-linux-gcc (GCC) 9.3.0 If you fix the issue,

drivers/net/ethernet/broadcom/bnx2x/bnx2x_link.c:5638 bnx2x_link_settings_status() warn: signedness bug returning '(-22)'

2020-08-12 Thread kernel test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 7c2a69f610e64c8dec6a06a66e721f4ce1dd783a commit: 2c855d73f2f6107f5b8ebc45f8b934bf7f4419e0 bnx2x: Remove read_status_t function casts date: 9 months ago config: ia64-randconfig-m031-20200811 (attached as .c

Re: [PATCH v8 2/4] scsi: ufs: Introduce HPB feature

2020-08-12 Thread Bart Van Assche
On 2020-08-12 20:00, Daejun Park wrote: > On 2020-08-06 02:11, Daejun Park wrote: >>> +static int ufshpb_create_sysfs(struct ufs_hba *hba, struct ufshpb_lu *hpb) >>> +{ >>> +int ret; >>> + >>> +ufshpb_stat_init(hpb); >>> + >>> +kobject_init(&hpb->kobj, &ufshpb_ktype); >>> +mutex_ini

Re: [PATCH v8 3/4] scsi: ufs: L2P map management for HPB read

2020-08-12 Thread Daejun Park
On 2020-08-06 02:15, Daejun Park wrote: > > +req->end_io_data = (void *)map_req; > > Please leave the (void *) cast out since explicit casts from a non-void > to a void pointer are not necessary in C. OK, I will fix it. > > +static inline struct > > +ufshpb_rsp_field *ufshpb_get_hpb_rsp(str

Re: [PATCH v8 4/4] scsi: ufs: Prepare HPB read for cached sub-region

2020-08-12 Thread Daejun Park
On 2020-08-06 02:18, Daejun Park wrote: > > +static inline u32 ufshpb_get_lpn(struct scsi_cmnd *cmnd) > > +{ > > +return blk_rq_pos(cmnd->request) >> > > +(ilog2(cmnd->device->sector_size) - 9); > > +} > > Please use sectors_to_logical() from drivers/scsi/sd.h instead of open-coding >

[PATCH] dma-debug: fix debug_dma_assert_idle(), use rcu_read_lock()

2020-08-12 Thread Hugh Dickins
Since commit 2a9127fcf229 ("mm: rewrite wait_on_page_bit_common() logic") improved unlock_page(), it has become more noticeable how cow_user_page() in a kernel with CONFIG_DMA_API_DEBUG=y can create and suffer from heavy contention on DMA debug's radix_lock in debug_dma_assert_idle(). It is only d

  1   2   3   4   5   6   7   8   9   10   >