Re: [PATCH v4 1/2] scsi: ufs: fix livelock of ufshcd_clear_ua_wluns

2021-01-07 Thread Can Guo
On 2021-01-07 15:47, Jaegeuk Kim wrote: When gate_work/ungate_work gets an error during hibern8_enter or exit, ufshcd_err_handler() ufshcd_scsi_block_requests() ufshcd_reset_and_restore() ufshcd_clear_ua_wluns() -> stuck ufshcd_scsi_unblock_requests() In order to avoid it,

Re: [PATCH v4 2/2] scsi: ufs: handle LINERESET with correct tm_cmd

2021-01-07 Thread Can Guo
On 2021-01-07 15:47, Jaegeuk Kim wrote: From: Jaegeuk Kim This fixes a warning caused by wrong reserve tag usage in __ufshcd_issue_tm_cmd. WARNING: CPU: 7 PID: 7 at block/blk-core.c:630 blk_get_request+0x68/0x70 WARNING: CPU: 4 PID: 157 at block/blk-mq-tag.c:82 blk_mq_get_tag+0x438/0x46c

Re: [PATCH] Hyper-V: pci: x64: Generalize irq/msi set-up and handling

2021-01-07 Thread Borislav Petkov
On Thu, Jan 07, 2021 at 05:05:36AM +, Sunil Muthuswamy wrote: > Currently, operations related to irq/msi in Hyper-V vPCI are > x86-specific code. In order to support virtual PCI on Hyper-V for > other architectures, introduce generic interfaces to replace the > x86-specific ones. There are no

Re: [PATCH 00/10] Remove support for TX49xx

2021-01-07 Thread Geert Uytterhoeven
Hi Nemoto-san, On Thu, Jan 7, 2021 at 2:18 AM Atsushi Nemoto wrote: > On Wed, 6 Jan 2021 21:41:24 +0100, Geert Uytterhoeven > wrote: > >> > Is that sufficient to keep it? > >> > >> for me it is. But now we probaly need some reverts then... > > > > Indeed. Fortunately not all of it, as some

Re: [PATCH v2 1/9] KVM: x86: Add AMD SEV specific Hypercall3

2021-01-07 Thread Ashish Kalra
Hello Steve, Sorry, i realized later that i replied to this email with regard to the current bitmap implementation and not the unencrpyted region list strategy. I am now looking at your thoughts/questions with regard to the unencrypted region list strategy and will reply to them accordingly.

Re: [PATCH v4 2/2] scsi: ufs: handle LINERESET with correct tm_cmd

2021-01-07 Thread Jaegeuk Kim
On 01/07, Can Guo wrote: > On 2021-01-07 15:47, Jaegeuk Kim wrote: > > From: Jaegeuk Kim > > > > This fixes a warning caused by wrong reserve tag usage in > > __ufshcd_issue_tm_cmd. > > > > WARNING: CPU: 7 PID: 7 at block/blk-core.c:630 blk_get_request+0x68/0x70 > > WARNING: CPU: 4 PID: 157 at

Re: [PATCH 5.10 28/63] fbcon: Disable accelerated scrolling

2021-01-07 Thread Geert Uytterhoeven
On Mon, Jan 4, 2021 at 5:58 PM Greg Kroah-Hartman wrote: > From: Daniel Vetter > > commit 39aead8373b3c20bb5965c024dfb51a94e526151 upstream. > > So ever since syzbot discovered fbcon, we have solid proof that it's > full of bugs. And often the solution is to just delete code and remove >

Re: [PATCH v4 2/2] scsi: ufs: handle LINERESET with correct tm_cmd

2021-01-07 Thread Can Guo
On 2021-01-07 16:07, Jaegeuk Kim wrote: On 01/07, Can Guo wrote: On 2021-01-07 15:47, Jaegeuk Kim wrote: > From: Jaegeuk Kim > > This fixes a warning caused by wrong reserve tag usage in > __ufshcd_issue_tm_cmd. > > WARNING: CPU: 7 PID: 7 at block/blk-core.c:630 blk_get_request+0x68/0x70 >

Re: [RFC PATCH kernel] block: initialize block_device::bd_bdi for bdev_cache

2021-01-07 Thread Alexey Kardashevskiy
On 07/01/2021 18:48, Christoph Hellwig wrote: On Thu, Jan 07, 2021 at 10:58:39AM +1100, Alexey Kardashevskiy wrote: And AFAICT the root inode on bdev superblock can get only to bdev_evict_inode() and bdev_free_inode(). Looking at bdev_evict_inode() the only thing that's used there from

[PATCH v7 0/4] drm/panfrost: Add support for mt8183 GPU

2021-01-07 Thread Nicolas Boichat
Hi! Follow-up on the v5 [1], things have gotten significantly better in the last 9 months, thanks to the efforts on Bifrost support by the Collabora team (and probably others I'm not aware of). I've been testing this series on a MT8183/kukui device, with a chromeos-5.10 kernel [2], and got basic

[PATCH v7 1/4] dt-bindings: gpu: mali-bifrost: Add Mediatek MT8183

2021-01-07 Thread Nicolas Boichat
Define a compatible string for the Mali Bifrost GPU found in Mediatek's MT8183 SoCs. Signed-off-by: Nicolas Boichat Reviewed-by: Alyssa Rosenzweig --- Changes in v7: - Fix GPU ID in commit message Changes in v6: - Rebased, actually tested with recent mesa driver. - No change Changes in

[PATCH v7 3/4] drm/panfrost: devfreq: Disable devfreq when num_supplies > 1

2021-01-07 Thread Nicolas Boichat
GPUs with more than a single regulator (e.g. G72 on MT8183) will require platform-specific handling, disable devfreq for now. Signed-off-by: Nicolas Boichat --- Changes in v7: - Fix GPU ID in commit message Changes in v6: - New change drivers/gpu/drm/panfrost/panfrost_devfreq.c | 9

Re: [PATCH 1/1] sched/fair:Avoid unnecessary assignment to cfs_rq->on_list

2021-01-07 Thread Vincent Guittot
On Thu, 7 Jan 2021 at 02:57, wrote: > > From: jun qian > > Obviously, cfs_rq->on_list is already equal to 1 when cfs_rq->on_list > is assigned a value of 1, so an else branch is needed to avoid unnecessary > assignment operations. > > Signed-off-by: jun qian > --- > kernel/sched/fair.c | 4

[PATCH v7 2/4] arm64: dts: mt8183: Add node for the Mali GPU

2021-01-07 Thread Nicolas Boichat
Add a basic GPU node for mt8183. Signed-off-by: Nicolas Boichat --- The binding we use with out-of-tree Mali drivers includes more clocks, this is used for devfreq: the out-of-tree driver switches clk_mux to clk_sub_parent (26Mhz), adjusts clk_main_parent, then switches clk_mux back to

[PATCH v7 4/4] drm/panfrost: Add mt8183-mali compatible string

2021-01-07 Thread Nicolas Boichat
Add support for MT8183's G72 Bifrost. Signed-off-by: Nicolas Boichat --- Changes in v7: - Fix GPU ID in commit message Changes in v6: - Context conflicts, reflow the code. - Use ARRAY_SIZE for power domains too. Changes in v5: - Change power domain name from 2d to core2. Changes in v4:

Re: [PATCH v7 3/4] drm/panfrost: devfreq: Disable devfreq when num_supplies > 1

2021-01-07 Thread Tomeu Vizoso
On 1/7/21 9:26 AM, Nicolas Boichat wrote: GPUs with more than a single regulator (e.g. G72 on MT8183) will require platform-specific handling, disable devfreq for now. Signed-off-by: Nicolas Boichat --- Changes in v7: - Fix GPU ID in commit message Changes in v6: - New change

[tip: x86/platform] x86/platform: Retire arch/x86/platform/goldfish

2021-01-07 Thread tip-bot2 for Roman Kiryanov
The following commit has been merged into the x86/platform branch of tip: Commit-ID: 2b29eed3f201b49feb92fdd0178b10825a5528f4 Gitweb: https://git.kernel.org/tip/2b29eed3f201b49feb92fdd0178b10825a5528f4 Author:Roman Kiryanov AuthorDate:Mon, 16 Nov 2020 18:55:22 -08:00

Re: [PATCH] mm/mmap: replace if (cond) BUG() with BUG_ON()

2021-01-07 Thread Michal Hocko
On Wed 06-01-21 12:10:30, Hugh Dickins wrote: > On Wed, 6 Jan 2021, Andrew Morton wrote: > > On Tue, 5 Jan 2021 20:28:27 -0800 (PST) Hugh Dickins > > wrote: > > > > > Alex, please consider why the authors of these lines (whom you > > > did not Cc) chose to write them without BUG_ON(): it has

Re: [PATCH v5 1/2] dt-bindings: hwlock: add sun6i_hwspinlock

2021-01-07 Thread Wilken Gottwalt
On Wed, 6 Jan 2021 11:14:36 +0100 Maxime Ripard wrote: > Hi, > > On Wed, Dec 23, 2020 at 03:49:19PM -0700, Rob Herring wrote: > > On Wed, Dec 23, 2020 at 4:34 AM Wilken Gottwalt > > wrote: > > > > > > Adds documentation on how to use the sun6i_hwspinlock driver for sun6i > > > compatible SoCs.

Re: [PATCH v2 2/6] mm: hugetlbfs: fix cannot migrate the fallocated HugeTLB page

2021-01-07 Thread Michal Hocko
On Wed 06-01-21 13:07:40, Mike Kravetz wrote: > On 1/6/21 12:02 PM, Michal Hocko wrote: > > On Wed 06-01-21 11:30:25, Mike Kravetz wrote: > >> On 1/6/21 8:35 AM, Michal Hocko wrote: > >>> On Wed 06-01-21 16:47:35, Muchun Song wrote: > Because we only can isolate a active page via

Re: [PATCH v2] soc: qcom: rpmh: Remove serialization of TCS commands

2021-01-07 Thread Maulik Shah
Hi Doug, On 12/4/2020 3:14 AM, Doug Anderson wrote: Hi, On Mon, Nov 23, 2020 at 11:32 PM Maulik Shah wrote: @@ -423,8 +422,7 @@ static irqreturn_t tcs_tx_done(int irq, void *p) cmd = >cmds[j]; sts = read_tcs_cmd(drv, RSC_DRV_CMD_STATUS, i,

[PATCH v3] soc: qcom: rpmh: Remove serialization of TCS commands

2021-01-07 Thread Maulik Shah
From: Lina Iyer Requests sent to RPMH can be sent as fire-n-forget or response required, with the latter ensuring the command has been completed by the hardware accelerator. Commands in a request with tcs_cmd::wait set, would ensure that those select commands are sent as response required, even

Re: [External] Re: [PATCH v2 4/6] mm: hugetlb: add return -EAGAIN for dissolve_free_huge_page

2021-01-07 Thread Michal Hocko
On Thu 07-01-21 11:11:41, Muchun Song wrote: > On Thu, Jan 7, 2021 at 1:07 AM Michal Hocko wrote: > > > > On Wed 06-01-21 16:47:37, Muchun Song wrote: > > > When dissolve_free_huge_page() races with __free_huge_page(), we can > > > do a retry. Because the race window is small. > > > > Is this a

Re: [PATCH v2 3/6] mm: hugetlb: fix a race between freeing and dissolving the page

2021-01-07 Thread Michal Hocko
On Wed 06-01-21 12:58:29, Mike Kravetz wrote: > On 1/6/21 8:56 AM, Michal Hocko wrote: > > On Wed 06-01-21 16:47:36, Muchun Song wrote: > >> There is a race condition between __free_huge_page() > >> and dissolve_free_huge_page(). > >> > >> CPU0: CPU1: > >> > >> //

Re: [PATCH v6 2/3] io_uring: add IOURING_REGISTER_RESTRICTIONS opcode

2021-01-07 Thread Stefano Garzarella
On Mon, Jan 04, 2021 at 01:26:41AM +1100, Daurnimator wrote: On Fri, 28 Aug 2020 at 00:59, Stefano Garzarella wrote: + __u8 register_op; /* IORING_RESTRICTION_REGISTER_OP */ Can you confirm that this intentionally limited the future range of IORING_REGISTER opcodes to 0-255?

[PATCH v7 1/2] lib/string.c: add __sysfs_match_string_with_gaps() helper

2021-01-07 Thread Alexandru Ardelean
The original docstring of the __sysfs_match_string() and match_string() helper, implied that -1 could be used to search through NULL terminated arrays, and positive 'n' could be used to go through arrays that may have NULL elements in the middle of the array. This isn't true. Regardless of the

RE: [PATCH v3 10/21] x86/fpu/xstate: Update xstate save function to support dynamic xstate

2021-01-07 Thread Liu, Jing2
-Original Message- From: Bae, Chang Seok Sent: Wednesday, December 23, 2020 11:57 PM To: b...@suse.de; l...@kernel.org; t...@linutronix.de; mi...@kernel.org; x...@kernel.org Cc: Brown, Len ; Hansen, Dave ; Liu, Jing2 ; Shankar, Ravi V ; linux-kernel@vger.kernel.org; Bae, Chang Seok

[PATCH v7 2/2] iio: Handle enumerated properties with gaps

2021-01-07 Thread Alexandru Ardelean
From: Lars-Peter Clausen Some enums might have gaps or reserved values in the middle of their value range. E.g. consider a 2-bit enum where the values 0, 1 and 3 have a meaning, but 2 is a reserved value and can not be used. Add support for such enums to the IIO enum helper functions. A

Re: [External] Re: [PATCH v2 3/6] mm: hugetlb: fix a race between freeing and dissolving the page

2021-01-07 Thread Michal Hocko
On Thu 07-01-21 13:39:38, Muchun Song wrote: > On Thu, Jan 7, 2021 at 12:56 AM Michal Hocko wrote: > > > > On Wed 06-01-21 16:47:36, Muchun Song wrote: > > > There is a race condition between __free_huge_page() > > > and dissolve_free_huge_page(). > > > > > > CPU0: CPU1: >

Re: [PATCH v5 2/2] hwspinlock: add sun6i hardware spinlock support

2021-01-07 Thread Wilken Gottwalt
On Wed, 6 Jan 2021 11:15:42 +0100 Maxime Ripard wrote: > On Wed, Dec 23, 2020 at 12:35:10PM +0100, Wilken Gottwalt wrote: > > Adds the sun6i_hwspinlock driver for the hardware spinlock unit found in > > most of the sun6i compatible SoCs. > > > > This unit provides at least 32 spinlocks in

Re: [PATCH v4 2/2] scsi: ufs: handle LINERESET with correct tm_cmd

2021-01-07 Thread Jaegeuk Kim
On 01/07, Can Guo wrote: > On 2021-01-07 16:07, Jaegeuk Kim wrote: > > On 01/07, Can Guo wrote: > > > On 2021-01-07 15:47, Jaegeuk Kim wrote: > > > > From: Jaegeuk Kim > > > > > > > > This fixes a warning caused by wrong reserve tag usage in > > > > __ufshcd_issue_tm_cmd. > > > > > > > > WARNING:

Re: [PATCH v7 3/4] drm/panfrost: devfreq: Disable devfreq when num_supplies > 1

2021-01-07 Thread Nicolas Boichat
On Thu, Jan 7, 2021 at 4:31 PM Tomeu Vizoso wrote: > > On 1/7/21 9:26 AM, Nicolas Boichat wrote: > > GPUs with more than a single regulator (e.g. G72 on MT8183) will > > require platform-specific handling, disable devfreq for now. > > > > Signed-off-by: Nicolas Boichat > > --- > > > > Changes in

Re: [PATCH v7 3/4] drm/panfrost: devfreq: Disable devfreq when num_supplies > 1

2021-01-07 Thread Tomeu Vizoso
On 1/7/21 9:49 AM, Nicolas Boichat wrote: On Thu, Jan 7, 2021 at 4:31 PM Tomeu Vizoso wrote: On 1/7/21 9:26 AM, Nicolas Boichat wrote: GPUs with more than a single regulator (e.g. G72 on MT8183) will require platform-specific handling, disable devfreq for now. Signed-off-by: Nicolas Boichat

[PATCH v2 2/2] ARM: dts: add Protonic PRTI6G board

2021-01-07 Thread Oleksij Rempel
Protonic PRTI6G is a reference platform for industrial, safety critical applications Signed-off-by: Oleksij Rempel --- arch/arm/boot/dts/Makefile | 1 + arch/arm/boot/dts/imx6ul-prti6g.dts | 356 2 files changed, 357 insertions(+) create mode 100644

[PATCH v2 1/2] dt-bindings: arm: fsl: add Protonic PRTI6G board

2021-01-07 Thread Oleksij Rempel
Add Protonic Holland PRTI6G, iMX6UL based board Signed-off-by: Oleksij Rempel Acked-by: Rob Herring --- Documentation/devicetree/bindings/arm/fsl.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/Documentation/devicetree/bindings/arm/fsl.yaml

[PATCH v2 0/2] mainline Protonic PRTI6G board

2021-01-07 Thread Oleksij Rempel
changes v2: - reorder pinctls Oleksij Rempel (2): dt-bindings: arm: fsl: add Protonic PRTI6G board ARM: dts: add Protonic PRTI6G board .../devicetree/bindings/arm/fsl.yaml | 1 + arch/arm/boot/dts/Makefile| 1 + arch/arm/boot/dts/imx6ul-prti6g.dts

Re: [External] Re: [PATCH v2 3/6] mm: hugetlb: fix a race between freeing and dissolving the page

2021-01-07 Thread Muchun Song
On Thu, Jan 7, 2021 at 4:41 PM Michal Hocko wrote: > > On Thu 07-01-21 13:39:38, Muchun Song wrote: > > On Thu, Jan 7, 2021 at 12:56 AM Michal Hocko wrote: > > > > > > On Wed 06-01-21 16:47:36, Muchun Song wrote: > > > > There is a race condition between __free_huge_page() > > > > and

Re: [PATCH 3/6] hugetlb: add free page reporting support

2021-01-07 Thread David Hildenbrand
On 07.01.21 04:38, Liang Li wrote: > On Thu, Jan 7, 2021 at 12:08 AM Michal Hocko wrote: >> >> On Tue 05-01-21 22:49:21, Liang Li wrote: >>> hugetlb manages its page in hstate's free page list, not in buddy >>> system, this patch try to make it works for hugetlbfs. It canbe >>> used for memory

Re: [PATCH v4 2/2] scsi: ufs: handle LINERESET with correct tm_cmd

2021-01-07 Thread Can Guo
On 2021-01-07 16:46, Jaegeuk Kim wrote: On 01/07, Can Guo wrote: On 2021-01-07 16:07, Jaegeuk Kim wrote: > On 01/07, Can Guo wrote: > > On 2021-01-07 15:47, Jaegeuk Kim wrote: > > > From: Jaegeuk Kim > > > > > > This fixes a warning caused by wrong reserve tag usage in > > >

[PATCH 1/2] cros_ec_commands: Add EC_CODEC_I2S_RX_RESET

2021-01-07 Thread Yu-Hsuan Hsu
Add the new command EC_CODEC_I2S_RX_RESET in ec_codec_i2s_rx_subcmd, which is used for resetting the EC codec. Signed-off-by: Yu-Hsuan Hsu --- include/linux/platform_data/cros_ec_commands.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/platform_data/cros_ec_commands.h

Re: [PATCH v1] driver core: Fix device link device name collision

2021-01-07 Thread Greg Kroah-Hartman
On Wed, Jan 06, 2021 at 03:26:41PM -0800, Saravana Kannan wrote: > The device link device's name was of the form: > -- > > This can cause name collision as reported here [1] as device names are > not globally unique. Since device names have to be unique within the > bus/class, add the bus/class

[PATCH 2/2] ASoC: cros_ec_codec: Reset I2S RX when probing

2021-01-07 Thread Yu-Hsuan Hsu
It is not guaranteed that I2S RX is disabled when the kernel booting. For example, if the kernel crashes while it is enabled, it will keep enabled until the next time EC reboots. Reset I2S RX when probing to fix this issue. Signed-off-by: Yu-Hsuan Hsu --- sound/soc/codecs/cros_ec_codec.c | 7

[PATCH v8 1/4] dt-bindings: gpu: mali-bifrost: Add Mediatek MT8183

2021-01-07 Thread Nicolas Boichat
Define a compatible string for the Mali Bifrost GPU found in Mediatek's MT8183 SoCs. Signed-off-by: Nicolas Boichat Reviewed-by: Alyssa Rosenzweig --- (no changes since v6) Changes in v6: - Rebased, actually tested with recent mesa driver. - No change Changes in v5: - Rename "2d" power

[PATCH v8 3/4] drm/panfrost: devfreq: Disable devfreq when num_supplies > 1

2021-01-07 Thread Nicolas Boichat
GPUs with more than a single regulator (e.g. G72 on MT8183) will require platform-specific handling, disable devfreq for now. Signed-off-by: Nicolas Boichat Reviewed-by: Tomeu Vizoso --- Changes in v8: - Use DRM_DEV_INFO instead of ERROR Changes in v7: - Fix GPU ID in commit message

[PATCH v8 4/4] drm/panfrost: Add mt8183-mali compatible string

2021-01-07 Thread Nicolas Boichat
Add support for MT8183's G72 Bifrost. Signed-off-by: Nicolas Boichat Reviewed-by: Tomeu Vizoso --- (no changes since v7) Changes in v7: - Fix GPU ID in commit message Changes in v6: - Context conflicts, reflow the code. - Use ARRAY_SIZE for power domains too. Changes in v5: - Change

[PATCH v8 0/4] drm/panfrost: Add support for mt8183 GPU

2021-01-07 Thread Nicolas Boichat
Hi! Follow-up on the v5 [1], things have gotten significantly better in the last 9 months, thanks to the efforts on Bifrost support by the Collabora team (and probably others I'm not aware of). I've been testing this series on a MT8183/kukui device, with a chromeos-5.10 kernel [2], and got basic

Re: [External] Re: [PATCH v2 4/6] mm: hugetlb: add return -EAGAIN for dissolve_free_huge_page

2021-01-07 Thread Muchun Song
On Thu, Jan 7, 2021 at 4:39 PM Michal Hocko wrote: > > On Thu 07-01-21 11:11:41, Muchun Song wrote: > > On Thu, Jan 7, 2021 at 1:07 AM Michal Hocko wrote: > > > > > > On Wed 06-01-21 16:47:37, Muchun Song wrote: > > > > When dissolve_free_huge_page() races with __free_huge_page(), we can > > > >

[PATCH v8 2/4] arm64: dts: mt8183: Add node for the Mali GPU

2021-01-07 Thread Nicolas Boichat
Add a basic GPU node for mt8183. Signed-off-by: Nicolas Boichat --- The binding we use with out-of-tree Mali drivers includes more clocks, this is used for devfreq: the out-of-tree driver switches clk_mux to clk_sub_parent (26Mhz), adjusts clk_main_parent, then switches clk_mux back to

Re: ARC no console output (was Re: [PATCH 1/2] init/console: Use ttynull as a fallback when there is no console)

2021-01-07 Thread John Ogness
Hi Vineet, On 2021-01-06, Vineet Gupta wrote: > This breaks ARC booting (no output on console). Could you provide the kernel boot arguments that you use? This series is partly about addressing users that have used boot arguments that are technically incorrect (even if had worked). Seeing the

[PATCH 0/4] PCI: dwc: Refine the EP code no functionality change

2021-01-07 Thread Zhiqiang Hou
From: Hou Zhiqiang Tune the EP mode code slightly to make it more readable. Hou Zhiqiang (4): PCI: dwc: Change to use an array to store the structure of functions PCI: dwc: Add CFG offset info into function's represented structure PCI: dwc: Rename callback function func_conf_select and

[PATCH 4/4] PCI: dwc: Change the parameter of function dw_pcie_ep_reset_bar()

2021-01-07 Thread Zhiqiang Hou
From: Hou Zhiqiang This helper is endpoint mode specific, so change to use a pointer of 'struct dw_pcie_ep' as the parameter. Signed-off-by: Hou Zhiqiang --- drivers/pci/controller/dwc/pci-dra7xx.c | 2 +- drivers/pci/controller/dwc/pci-layerscape-ep.c | 2 +-

[PATCH 2/4] PCI: dwc: Add CFG offset info into function's represented structure

2021-01-07 Thread Zhiqiang Hou
From: Hou Zhiqiang To avoid multiple calculating of the CFG offset for each function, store the CFG offset info to the function's represented structure, and only do one time calculation during the initialization. Signed-off-by: Hou Zhiqiang --- .../pci/controller/dwc/pcie-designware-ep.c |

[PATCH 3/4] PCI: dwc: Rename callback function func_conf_select and its instance

2021-01-07 Thread Zhiqiang Hou
From: Hou Zhiqiang Rename the callback func_conf_select() and its instance and wrapper to *get_func_cfg_addr(), such that the code becomes more readable. Signed-off-by: Hou Zhiqiang --- drivers/pci/controller/dwc/pci-layerscape-ep.c | 6 +++--- drivers/pci/controller/dwc/pcie-designware-ep.c

Re: Review request 0/1: fs/proc: Fix NULL pointer dereference in

2021-01-07 Thread Gao, Yahu
Eric W. Biederman writes: > [Please note this e-mail is from an EXTERNAL e-mail address] > > Yahu Gao writes: > >> There is a kernel NULL pointer dereference was found in Linux system. >> The details of kernel NULL is shown at bellow. >> >> Currently, we do not have a way to provoke this fault

[PATCH 1/4] PCI: dwc: Change to use an array to store the structure of functions

2021-01-07 Thread Zhiqiang Hou
From: Hou Zhiqiang As there isn't dynamically adding and deleting a function's structure, the list_head is not necessary for this case. Array is easier and more efficient to search. Signed-off-by: Hou Zhiqiang --- .../pci/controller/dwc/pcie-designware-ep.c | 33 ---

[PATCH 1/4] ASoC: rt5645: Introduce mapping for ACPI-defined GPIO

2021-01-07 Thread Chris Chiu
On at least one laptop (ECS EF20EA) the 'hp-detect' GPIO is defined in the DSDT table by the ACPI GpioIo resources in _CRS. The GPIO related information should be mapped to the rt5645 driver to enable the jack detection also on non-DT platforms. Signed-off-by: Chris Chiu ---

[PATCH 2/4] ASoC: rt5645: Add ACPI-defined GPIO for ECS EF20 series

2021-01-07 Thread Chris Chiu
Add the hp-detect gpio for ECS EF20 series laptops based on the _CRS defined in DSDT table. Method (_CRS, 0, NotSerialized) { Name (SBUF, ResourceTemplate () { I2cSerialBusV2 (0x001A, ControllerInitiated, 0x00061A80, AddressingMode7Bit, "\\_SB.PCI0.I2C2", 0x00,

[PATCH 3/4] ASoC: rt5645: add inv_hp_det flag

2021-01-07 Thread Chris Chiu
The ECS EF20EA laptop use gpio for jack detection instead of rt5645 rt5645 JD. However, the GPIO polarity is inverse for hp-detect based on the _DSD property of the RTK2 device. Name (_DSD, Package () { ToUUID("daffd814-6eba-4d8c-8a91-bc9bbf4aa301"), Package () { Package ()

[PATCH] Revert "usb: gadget: Quieten gadget config message"

2021-01-07 Thread Albert Wang
This reverts commit 1cbfb8c4f62d667f6b8b3948949737edb92992cc. The log of USB enumeration result is a useful log and only occupies one line especially when USB3 enumeration failed and then downgrade to USB2. Signed-off-by: Albert Wang --- drivers/usb/gadget/composite.c | 6 +++--- 1 file

[PATCH 0/4] ASoC: rt5645: Enable internal mic and headset on ECS EF20

2021-01-07 Thread Chris Chiu
These patches are trying to fix the jack detection and internal microphone problems on ECS EF20 series laptops which are empowered by Intel Atom x5-Z8350 CPU (CherryTrail) with Realtek rt5645 audio codec. Chris Chiu (4): ASoC: rt5645: Introduce mapping for ACPI-defined GPIO ASoC: rt5645: Add

[PATCH 4/4] ASoC: rt5645: Enable internal microphone and JD on ECS EF20

2021-01-07 Thread Chris Chiu
On ECS EF20 series laptops, the internal mic is on DMIC2/IN2P. And they need the inv_hp_det to make jack detection to work as exoected. Signed-off-by: Chris Chiu --- sound/soc/codecs/rt5645.c | 8 1 file changed, 8 insertions(+) diff --git a/sound/soc/codecs/rt5645.c

[PATCH] [RFC] rt: kernel/sched/core: fix kthread_park() pending too long when CPU un-plugged

2021-01-07 Thread Ran Wang
When doing CPU un-plug stress test, function smpboot_park_threads() would get call to park kernel threads (which including ksoftirqd) on that CPU core, and function wait_task_inactive() would yield for those queued task(s) by calling schedule_hrtimerout() with mode of HRTIMER_MODE_REL. stack

Re: [PATCH v2 1/3] net: sfp: add workaround for Realtek RTL8672 and RTL9601C chips

2021-01-07 Thread Pali Rohár
On Thursday 07 January 2021 03:02:36 Andrew Lunn wrote: > > + /* hwmon interface needs to access 16bit registers in atomic way to > > +* guarantee coherency of the diagnostic monitoring data. If it is not > > +* possible to guarantee coherency because EEPROM is broken in such way > > +

Re: [PATCH] dt-bindings: Add missing array size constraints

2021-01-07 Thread Linus Walleij
On Tue, Jan 5, 2021 at 12:03 AM Rob Herring wrote: > DT properties which can have multiple entries need to specify what the > entries are and define how many entries there can be. In the case of > only a single entry, just 'maxItems: 1' is sufficient. > > Add the missing entry constraints. These

Re: ARC no console output (was Re: [PATCH 1/2] init/console: Use ttynull as a fallback when there is no console)

2021-01-07 Thread Richard Weinberger
[CC'in linux-um since there is a similar issue] On Thu, Jan 7, 2021 at 12:38 AM Vineet Gupta wrote: > > +CC Buildroot folks > > Hi Petr, > > On 11/11/20 5:54 AM, Petr Mladek wrote: > > stdin, stdout, and stderr standard I/O stream are created for the init > > process. They are not available

Re: [PATCH v4 2/2] scsi: ufs: handle LINERESET with correct tm_cmd

2021-01-07 Thread Can Guo
On 2021-01-07 16:58, Can Guo wrote: On 2021-01-07 16:46, Jaegeuk Kim wrote: On 01/07, Can Guo wrote: On 2021-01-07 16:07, Jaegeuk Kim wrote: > On 01/07, Can Guo wrote: > > On 2021-01-07 15:47, Jaegeuk Kim wrote: > > > From: Jaegeuk Kim > > > > > > This fixes a warning caused by wrong reserve

RE: [PATCH] [RFC] rt: kernel/sched/core: fix kthread_park() pending too long when CPU un-plugged

2021-01-07 Thread Ran Wang
Hi, On Thursday, January 7, 2021 5:19 PM, Ran Wang wrote: > > When doing CPU un-plug stress test, function smpboot_park_threads() would get > call to park kernel threads (which including ksoftirqd) on > that CPU core, and function wait_task_inactive() would yield for those queued > task(s) by

Re: [PATCH V4 1/5] gpio: gpio-xilinx: Simplify with dev_err_probe()

2021-01-07 Thread Linus Walleij
On Wed, Jan 6, 2021 at 1:27 PM Srinivas Neeli wrote: > Common pattern of handling deferred probe can be simplified with > dev_err_probe(). Less code and also it prints the error value. > > Signed-off-by: Srinivas Neeli Reviewed-by: Linus Walleij Yours, Linus Walleij

Re: [PATCH net-next] net: sysctl: cleanup net_sysctl_init()

2021-01-07 Thread Alexander Lobakin
From: Jakub Kicinski Date: Wed, 6 Jan 2021 16:30:56 -0800 > On Wed, 06 Jan 2021 20:40:28 + Alexander Lobakin wrote: >> 'net_header' is not used outside of this function, so can be moved >> from BSS onto the stack. >> Declarations of one-element arrays are discouraged, and there's no >> need

Re: [PATCH] media: rkvdec: silence ktest bot build warning

2021-01-07 Thread Hans Verkuil
On 08/12/2020 16:55, Adrian Ratiu wrote: > Some configurations built by the ktest bot produce the following > warn, so mark the struct as __maybe_unused to avoid unnecessary > ML spam. > >>> drivers/staging/media/rkvdec/rkvdec.c:967:34: warning: unused variable >>> 'of_rkvdec_match'

Re: [PATCH v2 6/6] iommu: Delete iommu_dev_has_feature()

2021-01-07 Thread John Garry
On 07/01/2021 01:18, Lu Baolu wrote: On 1/6/21 9:35 PM, John Garry wrote: Function iommu_dev_has_feature() has never been referenced in the tree, and there does not appear to be anything coming soon to use it, so delete it. Hi baolu, It will be used by the device driver which want to

Re: [PATCH V4 2/5] gpio: gpio-xilinx: Reduce spinlock array to array

2021-01-07 Thread Linus Walleij
On Wed, Jan 6, 2021 at 1:27 PM Srinivas Neeli wrote: > Changed spinlock array to single. It is preparation for irq support which > is shared between two channels that's why spinlock should be only one. > > Signed-off-by: Srinivas Neeli Reviewed-by: Linus Walleij Yours, Linus Walleij

Re: [PATCH] HID: uclogic: remove h from printk format specifier

2021-01-07 Thread Jiri Kosina
On Tue, 15 Dec 2020, t...@redhat.com wrote: > From: Tom Rix > > See Documentation/core-api/printk-formats.rst. > h should no longer be used in the format specifier for printk. > > Signed-off-by: Tom Rix Applied, thanks. -- Jiri Kosina SUSE Labs

Re: [PATCH] HID: wiimote: remove h from printk format specifier

2021-01-07 Thread Jiri Kosina
On Tue, 15 Dec 2020, t...@redhat.com wrote: > From: Tom Rix > > See Documentation/core-api/printk-formats.rst. > h should no longer be used in the format specifier for printk. > > Signed-off-by: Tom Rix Applied this one too. Thanks Tom. -- Jiri Kosina SUSE Labs

Re: [PATCH] Revert "usb: gadget: Quieten gadget config message"

2021-01-07 Thread Felipe Balbi
Hi, Albert Wang writes: > This reverts commit 1cbfb8c4f62d667f6b8b3948949737edb92992cccd. > > The log of USB enumeration result is a useful log and only occupies > one line especially when USB3 enumeration failed and then downgrade > to USB2. > > Signed-off-by: Albert Wang you can use dynamic

Re: [PATCH] usb: typec: Send uevent for num_altmodes update

2021-01-07 Thread Greg KH
On Wed, Jan 06, 2021 at 07:49:04PM -0800, Prashant Malani wrote: > Generate a change uevent when the "number_of_alternate_modes" sysfs file > for partners and plugs is updated by a port driver. > > Cc: Heikki Krogerus > Cc: Benson Leung > Signed-off-by: Prashant Malani > --- >

Re: [PATCH] mm: Teach pfn_to_online_page() about ZONE_DEVICE section collisions

2021-01-07 Thread David Hildenbrand
[...] >>> Well, I would love to have no surprises either. So far there was not >>> actual argument why the pmem reserved space cannot be fully initialized. >> >> Yes, I'm still hoping Dan can clarify that. > > Complexity and effective utility (once pfn_to_online_page() is fixed) > are the

Re: [PATCH v1] driver core: Fix device link device name collision

2021-01-07 Thread Michael Walle
Am 2021-01-07 10:00, schrieb Greg Kroah-Hartman: On Wed, Jan 06, 2021 at 03:26:41PM -0800, Saravana Kannan wrote: The device link device's name was of the form: -- This can cause name collision as reported here [1] as device names are not globally unique. Since device names have to be unique

[PATCH v12 0/4] Adding the Sparx5 Serdes driver

2021-01-07 Thread Steen Hegelund
Adding the Sparx5 Serdes driver This series of patches provides the serdes driver for the Microchip Sparx5 ethernet switch. The serdes driver supports the 10G and 25G serdes instances available in the Sparx5. The Sparx5 serdes support several interface modes with several speeds and also allows

[PATCH v12 1/4] dt-bindings: phy: Add sparx5-serdes bindings

2021-01-07 Thread Steen Hegelund
Document the Sparx5 ethernet serdes phy driver bindings. Signed-off-by: Lars Povlsen Signed-off-by: Steen Hegelund Reviewed-by: Rob Herring Reviewed-by: Andrew Lunn --- .../bindings/phy/microchip,sparx5-serdes.yaml | 100 ++ 1 file changed, 100 insertions(+) create mode

[PATCH v12 2/4] phy: Add ethernet serdes configuration option

2021-01-07 Thread Steen Hegelund
Provide a new ethernet phy configuration structure, that allow PHYs used for ethernet to be configured with speed, media type and clock information. Signed-off-by: Lars Povlsen Signed-off-by: Steen Hegelund Reviewed-by: Andrew Lunn --- include/linux/phy/phy-ethernet-serdes.h | 30

[PATCH v12 4/4] arm64: dts: sparx5: Add Sparx5 serdes driver node

2021-01-07 Thread Steen Hegelund
Add Sparx5 serdes driver node, and enable it generally for all reference boards. Signed-off-by: Lars Povlsen Signed-off-by: Steen Hegelund Reviewed-by: Andrew Lunn --- arch/arm64/boot/dts/microchip/sparx5.dtsi | 8 1 file changed, 8 insertions(+) diff --git

Re: [PATCH v2] HID: Add Wireless Radio Control feature for Chicony devices

2021-01-07 Thread Jiri Kosina
On Wed, 23 Dec 2020, Jian-Hong Pan wrote: > Some Chicony's keyboards support airplane mode hotkey (Fn+F2) with > "Wireless Radio Control" feature. For example, the wireless keyboard > [04f2:1236] shipped with ASUS all-in-one desktop. > > After consulting Chicony for this hotkey, learned the

Re: [PATCH v2 1/5] usb: gadget: u_audio: Free requests only after callback

2021-01-07 Thread Felipe Balbi
Jerome Brunet writes: > From: Jack Pham > > As per the kernel doc for usb_ep_dequeue(), it states that "this > routine is asynchronous, that is, it may return before the completion > routine runs". And indeed since v5.0 the dwc3 gadget driver updated > its behavior to place dequeued requests on

Re: [PATCH v2 2/5] usb: gadget: f_uac2: reset wMaxPacketSize

2021-01-07 Thread Felipe Balbi
Jerome Brunet writes: > With commit 913e4a90b6f9 ("usb: gadget: f_uac2: finalize wMaxPacketSize > according to bandwidth") > wMaxPacketSize is computed dynamically but the value is never reset. > > Because of this, the actual maximum packet size can only decrease each time > the audio gadget is

Re: [RESEND PATCH 2/2] misc: add support for retimers interfaces on Intel MAX 10 BMC

2021-01-07 Thread Greg KH
On Thu, Jan 07, 2021 at 02:07:08PM +0800, Xu Yilun wrote: > This driver supports the ethernet retimers (C827) for the Intel PAC > (Programmable Acceleration Card) N3000, which is a FPGA based Smart NIC. > > C827 is an Intel(R) Ethernet serdes transceiver chip that supports > up to 100G transfer.

Re: [PATCH v2 3/5] usb: gadget: u_audio: factorize ssize to alsa fmt conversion

2021-01-07 Thread Felipe Balbi
Jerome Brunet writes: > Factorize format related code common to the capture and playback path. > > Signed-off-by: Jerome Brunet It's never a good idea to send fixes and cleanups/refactors in the same series as that can confuse the person applying your changes. In any case: Acked-by: Felipe

Re: [PATCH] uio: uio_pci_generic: don't fail probe if pdev->irq equals to IRQ_NOTCONNECTED

2021-01-07 Thread gre...@linuxfoundation.org
On Thu, Jan 07, 2021 at 03:50:37PM +0800, 李捷 wrote: > From 0fbcd7e386898d829d3000d094358a91e626ee4a Mon Sep 17 00:00:00 2001 > From: Jie Li > Date: Mon, 7 Dec 2020 08:05:07 +0800 > Subject: [PATCH] uio: uio_pci_generic: don't fail probe if pdev->irq equals to > IRQ_NOTCONNECTED > > Some devices

Re: [PATCH V4 3/5] gpio: gpio-xilinx: Add interrupt support

2021-01-07 Thread Linus Walleij
On Wed, Jan 6, 2021 at 1:27 PM Srinivas Neeli wrote: > Adds interrupt support to the Xilinx GPIO driver so that rising and > falling edge line events can be supported. Since interrupt support is > an optional feature in the Xilinx IP, the driver continues to support > devices which have no

Re: [PATCH v2 4/5] usb: gadget: u_audio: remove struct uac_req

2021-01-07 Thread Felipe Balbi
Jerome Brunet writes: > 'struct uac_req' purpose is to link 'struct usb_request' to the > corresponding 'struct uac_rtd_params'. However member req is never > used. Using the context of the usb request, we can keep track of the > corresponding 'struct uac_rtd_params' just as well, without

[PATCH 0/4] Assorted fixes for RV32

2021-01-07 Thread Atish Patra
This series fixes various issues observed in latest kernel on RV32. The first two patches fixes an resource tree introduced in 5.11-rc1 while the last two fixes the case where 2GB physical memory is used on RV32. There are may be better way to fix the issue pointed out in PATCH 3 as it seems a

Re: [PATCH] objtool: Don't add empty symbols to the rbtree

2021-01-07 Thread Peter Zijlstra
On Wed, Jan 06, 2021 at 09:33:20PM -0600, Josh Poimboeuf wrote: > Building with the Clang assembler shows the following warning: > > arch/x86/kernel/ftrace_64.o: warning: objtool: missing symbol for insn at > offset 0x16 > > The Clang assembler strips section symbols. That ends up giving >

Re: [PATCH v2 5/5] usb: gadget: u_audio: clean up locking

2021-01-07 Thread Felipe Balbi
Jerome Brunet writes: > snd_pcm_stream_lock() is held when the ALSA .trigger() callback is called. > The lock of 'struct uac_rtd_params' is not necessary since all its locking > operation are done under the snd_pcm_stream_lock() too. > > Also, usb_request .complete() is called with irqs

[PATCH 2/4] RISC-V: Set current memblock limit

2021-01-07 Thread Atish Patra
Currently, linux kernel can not use last 4k bytes of addressable space because IS_ERR_VALUE macro treats those as an error. This will be an issue for RV32 as any memblock allocator potentially allocate chunk of memory from the end of DRAM (2GB) leading bad address error even though the address was

[PATCH] X.509: Fix crash caused by NULL pointer

2021-01-07 Thread Tianjia Zhang
On the following call path, `sig->pkey_algo` is not assigned in asymmetric_key_verify_signature(), which causes runtime crash in public_key_verify_signature(). keyctl_pkey_verify asymmetric_key_verify_signature verify_signature public_key_verify_signature This patch simply

[PATCH 1/4] RISC-V: Do not allocate memblock while iterating reserved memblocks

2021-01-07 Thread Atish Patra
Currently, resource tree allocates memory blocks while iterating on the list. It leads to following kernel warning because memblock allocation also invokes memory block reservation API. [0.00] [ cut here ] [0.00] WARNING: CPU: 0 PID: 0 at

[PATCH 3/4] RISC-V: Fix L1_CACHE_BYTES for RV32

2021-01-07 Thread Atish Patra
SMP_CACHE_BYTES/L1_CACHE_BYTES should be defined as 32 instead of 64 for RV32. Otherwise, there will be hole of 32 bytes with each memblock allocation if it is requested to be aligned with SMP_CACHE_BYTES. Signed-off-by: Atish Patra --- arch/riscv/include/asm/cache.h | 4 1 file changed, 4

[PATCH 0/5] vfio/iommu_type1: Some fixes about dirty tracking

2021-01-07 Thread Keqian Zhu
Hi, In patch series[1], I put forward some fixes and optimizations for vfio_iommu_type1. This extracts and improves the fix part of it. [1] https://lore.kernel.org/linux-iommu/20201210073425.25960-1-zhukeqi...@huawei.com/T/#t Thanks, Keqian Keqian Zhu (5): vfio/iommu_type1: Fixes

[PATCH 2/5] vfio/iommu_type1: Populate dirty bitmap for new vfio_dma

2021-01-07 Thread Keqian Zhu
We should populate dirty bitmap for newly added vfio_dma as it can be accessed arbitrarily if vfio_iommu is not promoted to pinned_scope. vfio_dma_populate_bitmap can handle this properly. Fixes: d6a4c185660c ("vfio iommu: Implementation of ioctl for dirty pages tracking") Signed-off-by: Keqian

[PATCH 1/5] vfio/iommu_type1: Fixes vfio_dma_populate_bitmap to avoid dirty lose

2021-01-07 Thread Keqian Zhu
Defer checking whether vfio_dma is of fully-dirty in update_user_bitmap is easy to lose dirty log. For example, after promoting pinned_scope of vfio_iommu, vfio_dma is not considered as fully-dirty, then we may lose dirty log that occurs before vfio_iommu is promoted. The key point is that

  1   2   3   4   5   6   7   8   9   10   >