[PATCH] net: fix uninit value error in __sys_sendmmsg

2020-09-12 Thread Anant Thazhemadam
The crash report showed that there was a local variable; iovstack.i@__sys_sendmmsg created at: ___sys_sendmsg net/socket.c:2388 [inline] __sys_sendmmsg+0x6db/0xc90 net/socket.c:2480 that was left uninitialized. The contents of iovstack are of interest, since the respective pointer is

[PATCH V2] natsemi: switch from 'pci_' to 'dma_' API

2020-09-12 Thread Christophe JAILLET
The wrappers in include/linux/pci-dma-compat.h should go away. The patch has been generated with the coccinelle script below and has been hand modified to replace GFP_ with a correct flag. It has been compile tested. When memory is allocated in 'alloc_ring()' (natsemi.c) GFP_KERNEL can be used

[PATCH v2] arm64: dts: layerscape: Add label to pcie nodes

2020-09-12 Thread Wasim Khan
From: Wasim Khan Add label to pcie nodes so that they are easy to refer. Signed-off-by: Wasim Khan --- Changes in v2: - clubbed separate patches in single patch arch/arm64/boot/dts/freescale/fsl-ls1012a-oxalis.dts | 2 +- arch/arm64/boot/dts/freescale/fsl-ls1012a.dtsi | 5 +++--

RE: [PATCH 2/3] phy: cadence: salvo: Constify cdns_nxp_sequence_pair

2020-09-12 Thread Peter Chen
> > cdns_nxp_sequence_pair[] are never modified and can be made const to allow > the compiler to put them in read-only memory. > > Signed-off-by: Rikard Falkeborn Reviewed-by: Peter Chen Peter > --- > drivers/phy/cadence/phy-cadence-salvo.c | 6 +++--- > 1 file changed, 3 insertions(+), 3

Re: [PATCH v2 3/3] ARM: dts: add Plymovent M2M board

2020-09-12 Thread Shawn Guo
On Fri, Sep 11, 2020 at 08:03:12AM +0200, Marco Felsch wrote: > On 20-09-11 07:09, Oleksij Rempel wrote: > > Plymovent M2M is a control interface produced for the Plymovent filter > > systems. > > > > Signed-off-by: David Jander > > Signed-off-by: Oleksij Rempel > > --- > >

[PATCH] tee/tee_shm.c: Fix error handling path

2020-09-12 Thread Souptick Joarder
When shm->num_pages <= 0, we should avoid calling release_registered_pages() in error handling path. Signed-off-by: Souptick Joarder Cc: John Hubbard --- drivers/tee/tee_shm.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/tee/tee_shm.c

[ANNOUNCE] 4.9.236-rt154

2020-09-12 Thread Clark Williams
Hello RT-list! I'm pleased to announce the 4.9.236-rt154 stable release. You can get this release via the git tree at: git://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-stable-rt.git branch: v4.9-rt Head SHA1: d81b19f462cdf108afa79e7f7717190a280a299e Or to build 4.9.236-rt154

Re: [PATCH] driver/pci: reduce the single block time in pci_read_config

2020-09-12 Thread Jiang Biao
Hi, Bjorn On Thu, 10 Sep 2020 at 09:59, Bjorn Helgaas wrote: > > On Thu, Sep 10, 2020 at 09:54:02AM +0800, Jiang Biao wrote: > > Hi, > > > > On Thu, 10 Sep 2020 at 09:25, Bjorn Helgaas wrote: > > > > > > On Mon, Aug 24, 2020 at 01:20:25PM +0800, Jiang Biao wrote: > > > > From: Jiang Biao > > >

Re: [PATCH] checkpatch: Allow not using -f with files that are in git

2020-09-12 Thread Joe Perches
On Mon, 2020-08-24 at 17:09 -0700, Joe Perches wrote: > If a file exists in git and checkpatch is used without the -f > flag for scanning a file, then checkpatch will scan the file > assuming it's a patch and emit: > > ERROR: Does not appear to be a unified-diff format patch > > Change the

Re: [PATCH] clk: qcom: gcc-msm8939: remove defined but not used variables

2020-09-12 Thread Bjorn Andersson
On Thu 10 Sep 20:37 CDT 2020, Jason Yan wrote: > This addresses the following gcc warning with "make W=1": > > drivers/clk/qcom/gcc-msm8939.c:610:32: warning: > ‘gcc_xo_gpll6_gpll0a_map’ defined but not used > [-Wunused-const-variable=] > static const struct parent_map gcc_xo_gpll6_gpll0a_map[]

Re: [RFC PATCH v1 1/1] sched/fair: select idle cpu from idle cpumask in sched domain

2020-09-12 Thread Jiang Biao
Hi, Aubrey On Fri, 11 Sep 2020 at 23:48, Aubrey Li wrote: > > Added idle cpumask to track idle cpus in sched domain. When a CPU > enters idle, its corresponding bit in the idle cpumask will be set, > and when the CPU exits idle, its bit will be cleared. > > When a task wakes up to select an idle

Re: [PATCH 3/3] spi: spi-geni-qcom: Slightly optimize setup of bidirectional xfters

2020-09-12 Thread kernel test robot
Hi Douglas, I love your patch! Yet something to improve: [auto build test ERROR on spi/for-next] [also build test ERROR on v5.9-rc4 next-20200911] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' as documented in

Re: [PATCH v2 1/2] PM / Domains: Add GENPD_FLAG_NO_SUSPEND/RESUME flags

2020-09-12 Thread Bjorn Andersson
On Thu 10 Sep 03:18 CDT 2020, Ulf Hansson wrote: > On Thu, 10 Sep 2020 at 09:23, Sibi Sankar wrote: > > > > On 2020-08-25 23:23, Bjorn Andersson wrote: > > > On Tue 25 Aug 02:20 CDT 2020, Stephen Boyd wrote: > > >> Quoting Bjorn Andersson (2020-08-24 09:42:12) > > >> > On Fri 21 Aug 14:41 PDT

Re: [PATCH v3 6/8] iommu/arm-smmu: Add impl hook for inherit boot mappings

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 12:13 CDT 2020, Robin Murphy wrote: > On 2020-09-04 16:55, Bjorn Andersson wrote: > > Add a new operation to allow platform implementations to inherit any > > stream mappings from the boot loader. > > Is there a reason we need an explicit step for this? The aim of the > cfg_probe

Re: [PATCH] drm/msm/adreno: fix probe without iommu

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 11:08 CDT 2020, Luca Weiss wrote: > The function iommu_domain_alloc returns NULL on platforms without IOMMU > such as msm8974. This resulted in PTR_ERR(-ENODEV) being assigned to > gpu->aspace so the correct code path wasn't taken. > > Fixes: ccac7ce373c1 ("drm/msm: Refactor

Re: [PATCH 1/3] spi: spi-geni-qcom: Use the FIFO even more

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 20:11 CDT 2020, Doug Anderson wrote: > Hi, > > On Sat, Sep 12, 2020 at 3:53 PM Bjorn Andersson > wrote: > > > > On Sat 12 Sep 16:07 CDT 2020, Douglas Anderson wrote: > > > > > In commit 902481a78ee4 ("spi: spi-geni-qcom: Actually use our FIFO") I > > > explained that the maximum

Re: [PATCH] ARM: imx7ulp: enable cpufreq

2020-09-12 Thread Shawn Guo
On Fri, Sep 11, 2020 at 11:31:44AM +0800, peng@nxp.com wrote: > From: Peng Fan > > Enable cpufreq for i.MX7ULP when imx cpufreq dt driver enabled. > > Signed-off-by: Peng Fan Applied, thanks.

Re: [PATCH 1/4] dt-bindings: fsl: add i.MX7ULP PMC binding doc

2020-09-12 Thread Shawn Guo
On Fri, Sep 11, 2020 at 07:19:59AM +, Aisheng Dong wrote: > > From: Peng Fan > > Sent: Friday, September 11, 2020 11:31 AM > > > > Add i.MX7ULP Power Management Controller binding doc > > > > Signed-off-by: Peng Fan > > --- > > .../bindings/arm/freescale/imx7ulp-pmc.yaml | 33

Re: [PATCH net-next v2 7/7] net: ipa: do not enable GSI interrupt for wakeup

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > We now trigger a system resume when we receive an IPA SUSPEND > interrupt. We should *not* wake up on GSI interrupts. > Reviewed-by: Bjorn Andersson > Signed-off-by: Alex Elder > --- > drivers/net/ipa/gsi.c | 17 - >

Re: [PATCH 1/4] dt-bindings: fsl: add i.MX7ULP PMC binding doc

2020-09-12 Thread Shawn Guo
On Fri, Sep 11, 2020 at 07:45:20AM +, Peng Fan wrote: > > b/Documentation/devicetree/bindings/arm/freescale/imx7ulp-pmc.yaml > > > @@ -0,0 +1,33 @@ > > > +# SPDX-License-Identifier: GPL-2.0 > > > +%YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/arm/freescale/imx7ulp-pmc.yaml# >

Re: [PATCH net-next v2 6/7] net: ipa: enable wakeup on IPA interrupt

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > Now that we handle wakeup interrupts properly, arrange for the IPA > interrupt to be treated as a wakeup interrupt. > Reviewed-by: Bjorn Andersson > Signed-off-by: Alex Elder > --- > drivers/net/ipa/ipa_interrupt.c | 14 ++ > 1

Re: [PATCH 1/6] of: Add basic infrastructure to create early probe arrays

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > We currently probe interrupt controller and timers that need > to be available very early using an infratstructure that creates s/infratstructure/infrastructure/ > struct of_device_id instances in a special section. These are > individual

Re: [PATCH 8/8] arm64: dts: imx8mm-var-som-symphony: Drop unused gpioledgrp

2020-09-12 Thread Shawn Guo
On Tue, Sep 08, 2020 at 05:02:41PM +0200, Krzysztof Kozlowski wrote: > The gpioledgrp in iomux is not used, so it can be dropped. > > Signed-off-by: Krzysztof Kozlowski Applied, thanks.

Re: [PATCH 1/8] ARM: dts: imx6qdl: Correct interrupt flags in examples

2020-09-12 Thread Shawn Guo
Add Tim who is the board owner. On Tue, Sep 08, 2020 at 05:02:34PM +0200, Krzysztof Kozlowski wrote: > GPIO_ACTIVE_x flags are not correct in the context of interrupt flags. > These are simple defines so they could be used in DTS but they will not > have the same meaning: > 1. GPIO_ACTIVE_HIGH =

Re: [PATCH net-next v2 4/7] net: ipa: manage endpoints separate from clock

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > Currently, when (before) the last IPA clock reference is dropped, > all endpoints are suspended. And whenever the first IPA clock > reference is taken, all endpoints are resumed (or started). > > In most cases there's no need to start endpoints

Re: [PATCH net-next v2 5/7] net: ipa: use device_init_wakeup()

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > The call to wakeup_source_register() in ipa_probe() does not do what > it was intended to do. Call device_init_wakeup() in ipa_setup() > instead, to set the IPA device as wakeup-capable and to initially > enable wakeup capability. > > When we

Re: [PATCH net-next v2 3/7] net: ipa: verify reference flag values

2020-09-12 Thread Bjorn Andersson
On Fri 11 Sep 19:45 CDT 2020, Alex Elder wrote: > We take a single IPA clock reference to keep the clock running until > we get a system suspend operation, and maintain a flag indicating > whether that reference has been taken. When a suspend request > arrives, we drop that reference and clear

[ANNOUNCE] 4.14.198-rt96

2020-09-12 Thread Clark Williams
Hello RT-list! I'm pleased to announce the 4.14.198-rt96 stable release. You can get this release via the git tree at: git://git.kernel.org/pub/scm/linux/kernel/git/rt/linux-stable-rt.git branch: v4.14-rt Head SHA1: 8c4828cbd4220fc1c97c0db534fce850b86aa8d4 Or to build 4.14.198-rt96

BUG: Bad page state in process dirtyc0w_child

2020-09-12 Thread Qian Cai
Occasionally, running this LTP test will trigger an error below on s390: https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/security/dirtyc0w/dirtyc0w.c https://github.com/linux-test-project/ltp/blob/master/testcases/kernel/security/dirtyc0w/dirtyc0w_child.c this .config:

Re: [PATCH 0/6] arch: arm64: Add label to pcie nodes

2020-09-12 Thread Shawn Guo
On Wed, Sep 09, 2020 at 07:14:30PM +0530, Wasim Khan wrote: > From: Wasim Khan > > This patch series adds label to pcie nodes > so that they are easy to refer. > > Wasim Khan (6): > arm64: dts: lx2160a: Add label to pcie nodes > arm64: dts: ls1012a: Add label to pcie nodes > arm64: dts:

Re: [PATCH v5 05/10] powerpc/smp: Dont assume l2-cache to be superset of sibling

2020-09-12 Thread Michael Ellerman
Srikar Dronamraju writes: > * Michael Ellerman [2020-09-11 21:55:23]: > >> Srikar Dronamraju writes: >> > Current code assumes that cpumask of cpus sharing a l2-cache mask will >> > always be a superset of cpu_sibling_mask. >> > >> > Lets stop that assumption. cpu_l2_cache_mask is a superset of

Re: [PATCH] firmware: imx: scu-pd: ignore power domain not owned

2020-09-12 Thread Shawn Guo
On Tue, Sep 08, 2020 at 06:07:01PM +0800, peng@nxp.com wrote: > From: Peng Fan > > Should not register power domain that not owned by current > partition. > > Alought power domains will not be registered when power on failure, > we have to let CPU waste more cycles. > > Whether power on or

Re: [PATCH] Revert "net: linkwatch: add check for netdevice being present to linkwatch_do_dev"

2020-09-12 Thread David Miller
From: Geert Uytterhoeven Date: Sat, 12 Sep 2020 14:33:59 +0200 > "dev" is not the bridge device, but the physical Ethernet interface, which > may already be suspended during s2ram. Hmmm, ok. Looking more deeply NETDEV_CHANGE causes br_port_carrier_check() to run which exits early if

Re: [PATCH v3 1/3] ARM: dts: imx6qdl: tqma6: fix indentation

2020-09-12 Thread Shawn Guo
On Mon, Sep 07, 2020 at 10:04:27AM +0200, Matthias Schiffer wrote: > The PMIC configuration is indented one level too deep. > > Fixes: cac849e9bbc8 ("ARM: dts: imx6qdl: add TQMa6{S,Q,QP} SoM") > Signed-off-by: Matthias Schiffer I change the subject prefix to 'ARM: dts: imx6qdl-tqma6:' and

Re: [RESEND PATCH v2 2/2] drm/mediatek: mtk_dpi: Convert to bridge driver

2020-09-12 Thread Chun-Kuang Hu
Hi, Daniel Vetter 於 2020年9月10日 週四 下午1:48寫道: > > On Thu, Sep 10, 2020 at 06:35:21AM +0800, Chun-Kuang Hu wrote: > > Hi, Andrzej & Neil: > > > > Enric Balletbo i Serra 於 2020年8月26日 週三 > > 下午4:53寫道: > > > > > > > > Convert mtk_dpi to a bridge driver with built-in encoder support for > > >

Re: [PATCH] tlan: switch from 'pci_' to 'dma_' API

2020-09-12 Thread David Miller
From: Christophe JAILLET Date: Sat, 12 Sep 2020 09:43:58 +0200 > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > >

Re: [PATCH] net: tehuti: switch from 'pci_' to 'dma_' API

2020-09-12 Thread David Miller
From: Christophe JAILLET Date: Sat, 12 Sep 2020 16:12:32 +0200 > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > >

Re: [PATCH] rocker: switch from 'pci_' to 'dma_' API

2020-09-12 Thread David Miller
From: Christophe JAILLET Date: Sat, 12 Sep 2020 13:44:18 +0200 > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > >

Re: [PATCH] sc92031: switch from 'pci_' to 'dma_' API

2020-09-12 Thread David Miller
From: Christophe JAILLET Date: Sat, 12 Sep 2020 13:28:58 +0200 > The wrappers in include/linux/pci-dma-compat.h should go away. > > The patch has been generated with the coccinelle script below and has been > hand modified to replace GFP_ with a correct flag. > It has been compile tested. > >

Re: [PATCH net-next] net: ethernet: mlx4: Avoid assigning a value to ring_cons but not used it anymore in mlx4_en_xmit()

2020-09-12 Thread David Miller
From: Luo Jiaxing Date: Sat, 12 Sep 2020 16:08:15 +0800 > We found a set but not used variable 'ring_cons' in mlx4_en_xmit(), it will > cause a warning when build the kernel. And after checking the commit record > of this function, we found that it was introduced by a previous patch. > > So, We

Re: [Linux-kernel-mentees] [PATCH net] tipc: Fix memory leak in tipc_group_create_member()

2020-09-12 Thread David Miller
From: Peilin Ye Date: Sat, 12 Sep 2020 06:22:30 -0400 > @@ -291,10 +291,11 @@ static void tipc_group_add_to_tree(struct tipc_group > *grp, > else if (key > nkey) > n = &(*n)->rb_right; > else > - return; > +

Re: [PATCH v2] clk: imx: gate2: Fix a few typos

2020-09-12 Thread Shawn Guo
On Sat, Sep 05, 2020 at 05:10:16PM +0200, Jonathan Neuschäfer wrote: > A few words were misspelled in this comment. > > Signed-off-by: Jonathan Neuschäfer Applied, thanks.

Re: [PATCH 08/13] arm64: dts: imx8mm-var-som-symphony: Use newer interrupts property

2020-09-12 Thread Shawn Guo
On Fri, Sep 04, 2020 at 04:53:07PM +0200, Krzysztof Kozlowski wrote: > The int-gpios was deprecated in favor of generic interrupts property. > > Signed-off-by: Krzysztof Kozlowski Applied patch #8 ~ #13, thanks.

Re: [PATCH 1/3] spi: spi-geni-qcom: Use the FIFO even more

2020-09-12 Thread Doug Anderson
Hi, On Sat, Sep 12, 2020 at 3:53 PM Bjorn Andersson wrote: > > On Sat 12 Sep 16:07 CDT 2020, Douglas Anderson wrote: > > > In commit 902481a78ee4 ("spi: spi-geni-qcom: Actually use our FIFO") I > > explained that the maximum size we could program the FIFO was > > "mas->tx_fifo_depth - 3" but

Re: [PATCH 3/3] spi: spi-geni-qcom: Slightly optimize setup of bidirectional xfters

2020-09-12 Thread Doug Anderson
Hi, On Sat, Sep 12, 2020 at 3:54 PM Bjorn Andersson wrote: > > On Sat 12 Sep 16:08 CDT 2020, Douglas Anderson wrote: > > > When setting up a bidirectional transfer we need to program both the > > TX and RX lengths. We don't need a memory barrier between those two > > writes. Factor out the

Re: [PATCH] mm: memcontrol: Fix out-of-bounds on the buf returned by memory_stat_format

2020-09-12 Thread Andrew Morton
On Sat, 12 Sep 2020 23:51:00 +0800 Muchun Song wrote: > The memory_stat_format() returns a format string, but the return buf > may not including the trailing '\0'. So the users may read the buf > out of bounds. That sounds serious. Is a cc:stable appropriate?

Re: [PATCH 6/6] irqchip/qcom-pdc: Allow modular build

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > Switch the driver to a "hybrid probe" model which preserves the > built-in behaviour while allowing the driver to be optionnally > built as a module for development purposes. > Reviewed-by: Bjorn Andersson > Signed-off-by: Marc Zyngier >

Re: [PATCH 5/6] irqchip/mtk-sysirq: Allow modular build

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > Switch the driver to a "hybrid probe" model which preserves the > built-in behaviour while allowing the driver to be optionnally > built as a module for development purposes. > Reviewed-by: Bjorn Andersson > Signed-off-by: Marc Zyngier >

Re: [PATCH 4/6] irqchip/mtk-cirq: Allow modular build

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > Switch the driver to a "hybrid probe" model which preserves the > built-in behaviour while allowing the driver to be optionnally > built as a module for development purposes. > Reviewed-by: Bjorn Andersson > Signed-off-by: Marc Zyngier >

Re: [PATCH 3/6] irqchip: Introduce IRQCHIP_HYBRID_DRIVER_{BEGIN,END} macros

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > Although we are trying to move to a world where a large number > of irqchip drivers can safely be built as platform drivers > the reality is that most endpoint drivers are not ready for that, > and will fail to probe as they expect their

Re: [PATCH 2/6] irqchip: Make IRQCHIP_MATCH() type safe

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > IRQCHIP_DECLARE() is backed by macros that perform some elementary > type checking on the probe function. Unfortunately, IRQCHIP_MATCH() > doesn't, risking difficult debugging sessions... > > Rewrite IRQCHIP_MATCH() in terms of

Re: [PATCH 1/6] of: Add basic infrastructure to create early probe arrays

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 07:51 CDT 2020, Marc Zyngier wrote: > We currently probe interrupt controller and timers that need > to be available very early using an infratstructure that creates > struct of_device_id instances in a special section. These are > individual structures that are ultimately collated

Re: [PATCH] spi: spi-geni-qcom: Don't wait to start 1st transfer if transmitting

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 13:17 CDT 2020, Douglas Anderson wrote: > If we're sending bytes over SPI, we know the FIFO is empty at the > start of the transfer. There's no reason to wait for the interrupt > telling us to start--we can just start right away. Then if we > transmit everything in one swell foop

Re: [PATCH 2/3] spi: spi-geni-qcom: Don't program CS_TOGGLE again and again

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 16:08 CDT 2020, Douglas Anderson wrote: > We always toggle the chip select manually in spi-geni-qcom so that we > can properly implement the Linux API. There's no reason to program > this to the hardware on every transfer. Program it once at init and > be done with it. > > This

Re: [PATCH 3/3] spi: spi-geni-qcom: Slightly optimize setup of bidirectional xfters

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 16:08 CDT 2020, Douglas Anderson wrote: > When setting up a bidirectional transfer we need to program both the > TX and RX lengths. We don't need a memory barrier between those two > writes. Factor out the __iowmb() and use writel_relaxed(). This > saves a fraction of a

Re: [PATCH 1/3] spi: spi-geni-qcom: Use the FIFO even more

2020-09-12 Thread Bjorn Andersson
On Sat 12 Sep 16:07 CDT 2020, Douglas Anderson wrote: > In commit 902481a78ee4 ("spi: spi-geni-qcom: Actually use our FIFO") I > explained that the maximum size we could program the FIFO was > "mas->tx_fifo_depth - 3" but that I chose "mas->tx_fifo_depth()" > because I was worried about decreased

Re: [PATCH v12 0/5] Add support for DisplayPort driver on SnapDragon

2020-09-12 Thread abhinavk
Thanks Rob for pushing the changes. I confirm that the versions which you have pushed are the latest ones. I have rebased audio changes on top of the latest DP changes and posted them here https://patchwork.freedesktop.org/series/81612/. The other DP change which is missing is the video pattern

[PATCH 2/3] spi: spi-geni-qcom: Don't program CS_TOGGLE again and again

2020-09-12 Thread Douglas Anderson
We always toggle the chip select manually in spi-geni-qcom so that we can properly implement the Linux API. There's no reason to program this to the hardware on every transfer. Program it once at init and be done with it. This saves some part of a microsecond of overhead on each transfer. While

[PATCH 3/3] spi: spi-geni-qcom: Slightly optimize setup of bidirectional xfters

2020-09-12 Thread Douglas Anderson
When setting up a bidirectional transfer we need to program both the TX and RX lengths. We don't need a memory barrier between those two writes. Factor out the __iowmb() and use writel_relaxed(). This saves a fraction of a microsecond of setup overhead on bidirectional transfers.

[PATCH 1/3] spi: spi-geni-qcom: Use the FIFO even more

2020-09-12 Thread Douglas Anderson
In commit 902481a78ee4 ("spi: spi-geni-qcom: Actually use our FIFO") I explained that the maximum size we could program the FIFO was "mas->tx_fifo_depth - 3" but that I chose "mas->tx_fifo_depth()" because I was worried about decreased bandwidth. Since that time: * All the interconnect patches

RE: [PATCH 01/24] inet_net_pton.3: Use 'PRIx32' rather than "%x" when printing 'uint32_t' values

2020-09-12 Thread David Laight
From: Michael Kerrisk > Sent: 11 September 2020 10:31 > > Hi Alex, > > On 9/10/20 11:13 PM, Alejandro Colomar wrote: > > Signed-off-by: Alejandro Colomar > > --- > > man3/inet_net_pton.3 | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/man3/inet_net_pton.3

Re: [PATCH v3 0/10] libperf and arm64 userspace counter access support

2020-09-12 Thread Jiri Olsa
On Fri, Sep 11, 2020 at 03:51:08PM -0600, Rob Herring wrote: > This is resurrecting Raphael's series[1] to enable userspace counter > access on arm64. My previous versions are here[2][3]. > > Changes in v3: > - Dropped removing x86 rdpmc test until libperf tests can run via 'perf test' > -

Re: [PATCH] perf bench: Fix 2 memory sanitizer warnings

2020-09-12 Thread Jiri Olsa
On Fri, Sep 11, 2020 at 10:37:25PM -0700, Ian Rogers wrote: > Memory sanitizer warns if a write is performed where the memory > being read for the write is uninitialized. Avoid this warning by > initializing the memory. > > Signed-off-by: Ian Rogers Acked-by: Jiri Olsa thanks, jirka > --- >

Re: [PATCH v3 0/4] Fixes for setting event freq/periods

2020-09-12 Thread Jiri Olsa
On Fri, Sep 11, 2020 at 07:56:51PM -0700, Ian Rogers wrote: > Some fixes that address issues for regular and pfm4 events with 2 > additional perf_event_attr tests. Various authors, David Sharp isn't > currently at Google. > > v3. moved a loop into a helper following Adrian Hunter's suggestion. >

[PATCH 2/3] phy: cadence: salvo: Constify cdns_nxp_sequence_pair

2020-09-12 Thread Rikard Falkeborn
cdns_nxp_sequence_pair[] are never modified and can be made const to allow the compiler to put them in read-only memory. Signed-off-by: Rikard Falkeborn --- drivers/phy/cadence/phy-cadence-salvo.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [RESEND][RFC PATCH 0/6] Fork brute force attack mitigation (fbfam)

2020-09-12 Thread Ondrej Mosnacek
On Sat, Sep 12, 2020 at 4:51 PM Mel Gorman wrote: > On Sat, Sep 12, 2020 at 11:36:52AM +0200, John Wood wrote: > > On Sat, Sep 12, 2020 at 12:56:18AM -0700, Kees Cook wrote: > > > On Sat, Sep 12, 2020 at 10:03:23AM +1000, James Morris wrote: > > > > On Thu, 10 Sep 2020, Kees Cook wrote: > > > > >

[PATCH 3/3] phy: cadence: torrent: Constify regmap_config structs

2020-09-12 Thread Rikard Falkeborn
The regmap_config structs are never modified and can be made const to allow the compiler to put them in read-only memory. Signed-off-by: Rikard Falkeborn --- drivers/phy/cadence/phy-cadence-torrent.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH 1/3] phy: cadence: Sierra: Constify static structs

2020-09-12 Thread Rikard Falkeborn
The static cdns_reg_pairs and regmap_config structs are not modified and can be made const. This allows the compiler to put them in read-only memory. Signed-off-by: Rikard Falkeborn --- drivers/phy/cadence/phy-cadence-sierra.c | 24 1 file changed, 12 insertions(+), 12

[PATCH 0/3] drivers/phy/cadence: Constify static structs

2020-09-12 Thread Rikard Falkeborn
Constify a number of static structs that are never changed to allow the compiler to put them in read-only memory. Rikard Falkeborn (3): phy: cadence: Sierra: Constify static structs phy: cadence: salvo: Constify cdns_nxp_sequence_pair phy: cadence: torrent: Constify regmap_config structs

Re: [PATCH v3 3/7] ASoC: sun4i-i2s: Add support for H6 I2S

2020-09-12 Thread Samuel Holland
Maxime, On 9/10/20 9:33 AM, Maxime Ripard wrote: > On Thu, Sep 03, 2020 at 09:54:39PM -0500, Samuel Holland wrote: >> On 9/3/20 3:58 PM, Maxime Ripard wrote: >>> On Thu, Sep 03, 2020 at 10:02:31PM +0200, Clément Péron wrote: Hi Maxime, On Wed, 29 Jul 2020 at 17:16, Mark Brown

Re: [PATCH] Using a pointer and kzalloc in place of a struct directly

2020-09-12 Thread Anant Thazhemadam
On 12/09/20 8:25 pm, Greg KH wrote: > On Sat, Sep 12, 2020 at 05:43:38PM +0530, Anant Thazhemadam wrote: >> On 12/09/20 5:17 pm, Greg KH wrote: >>> Note, your "To:" line seemed corrupted, and why not cc: the bpf mailing >>> list as well? >> Oh, I'm sorry about that. I pulled the emails of all

Re: [GIT PULL] Btrfs fixes for 5.9-rc5

2020-09-12 Thread pr-tracker-bot
The pull request you sent on Sat, 12 Sep 2020 19:01:18 +0200: > git://git.kernel.org/pub/scm/linux/kernel/git/kdave/linux.git for-5.9-rc4-tag has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/edf6b0e1e4ddb12e022ce0c17829bad6d4161ea7 Thank you! -- Deet-doot-dot, I am a

Re: [GIT PULL] SMB3 DFS Fix

2020-09-12 Thread pr-tracker-bot
The pull request you sent on Sat, 12 Sep 2020 11:56:45 -0500: > git://git.samba.org/sfrench/cifs-2.6.git tags/5.9-rc4-smb3-fix has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/5a3c558a9f05f4664f569b06f04d6b217785fd21 Thank you! -- Deet-doot-dot, I am a bot.

Re: [GIT PULL] seccomp fixes for v5.9-rc5

2020-09-12 Thread pr-tracker-bot
The pull request you sent on Fri, 11 Sep 2020 17:20:02 -0700: > https://git.kernel.org/pub/scm/linux/kernel/git/kees/linux.git > tags/seccomp-v5.9-rc5 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/ef2e9a563b0cd7965e2a1263125dcbb1c86aa6cc Thank you! --

Re: [GIT PULL] libnvdimm fix for v5.9-rc5

2020-09-12 Thread pr-tracker-bot
The pull request you sent on Fri, 11 Sep 2020 22:19:57 +: > git://git.kernel.org/pub/scm/linux/kernel/git/nvdimm/nvdimm.git > tags/libnvdimm-fix-v5.9-rc5 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/4f8b0a5b3f7e5f03b188de9025b60c15559790f9 Thank you! --

RE: [PATCH v3 11/11] scsi: storvsc: Support PAGE_SIZE larger than 4K

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > Hyper-V always use 4k page size (HV_HYP_PAGE_SIZE), so when > communicating with Hyper-V, a guest should always use HV_HYP_PAGE_SIZE > as the unit for page related data. For storvsc, the data is > vmbus_packet_mpb_array. And since

Re: [PATCH] media: vidtv: fix 32-bit warnings

2020-09-12 Thread Joe Perches
On Sat, 2020-09-12 at 11:22 +0200, Mauro Carvalho Chehab wrote: > There are several warnings produced when the driver is built > for 32-bit archs. Solve them. [] > diff --git a/drivers/media/test-drivers/vidtv/vidtv_common.c > b/drivers/media/test-drivers/vidtv/vidtv_common.c [] > diff --git

[PATCH] natsemi: switch from 'pci_' to 'dma_' API

2020-09-12 Thread Christophe JAILLET
The wrappers in include/linux/pci-dma-compat.h should go away. The patch has been generated with the coccinelle script below and has been hand modified to replace GFP_ with a correct flag. It has been compile tested. When memory is allocated in 'alloc_ring()' (natsemi.c) GFP_KERNEL can be used

RE: [PATCH v3 10/11] Driver: hv: util: Make ringbuffer at least take two pages

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When PAGE_SIZE > HV_HYP_PAGE_SIZE, we need the ringbuffer size to be at > least 2 * PAGE_SIZE: one page for the header and at least one page of > the data part (because of the alignment requirement for double mapping). > > So make

RE: [PATCH v3 09/11] HID: hyperv: Make ringbuffer at least take two pages

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When PAGE_SIZE > HV_HYP_PAGE_SIZE, we need the ringbuffer size to be at > least 2 * PAGE_SIZE: one page for the header and at least one page of > the data part (because of the alignment requirement for double mapping). > > So make

RE: [PATCH v3 08/11] Input: hyperv-keyboard: Make ringbuffer at least take two pages

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When PAGE_SIZE > HV_HYP_PAGE_SIZE, we need the ringbuffer size to be at > least 2 * PAGE_SIZE: one page for the header and at least one page of > the data part (because of the alignment requirement for double mapping). > > So

RE: [PATCH v3 07/11] hv_netvsc: Use HV_HYP_PAGE_SIZE for Hyper-V communication

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When communicating with Hyper-V, HV_HYP_PAGE_SIZE should be used since > that's the page size used by Hyper-V and Hyper-V expects all > page-related data using the unit of HY_HYP_PAGE_SIZE, for example, the > "pfn" in

RE: [PATCH v3 06/11] hv: hyperv.h: Introduce some hvpfn helper functions

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > When a guest communicate with the hypervisor, it must use HV_HYP_PAGE to > calculate PFN, so introduce a few hvpfn helper functions as the > counterpart of the page helper functions. This is the preparation for > supporting guest

RE: [PATCH v3 05/11] Drivers: hv: vmbus: Move virt_to_hvpfn() to hyperv header

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > There will be more places other than vmbus where we need to calculate > the Hyper-V page PFN from a virtual address, so move virt_to_hvpfn() to > hyperv generic header. > > Signed-off-by: Boqun Feng > --- > drivers/hv/channel.c

RE: [PATCH v3 04/11] Drivers: hv: Use HV_HYP_PAGE in hv_synic_enable_regs()

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > Both the base_*_gpa should use the guest page number in Hyper-V page, so > use HV_HYP_PAGE instead of PAGE. > > Signed-off-by: Boqun Feng > --- > drivers/hv/hv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) >

RE: [PATCH v3 03/11] Drivers: hv: vmbus: Introduce types of GPADL

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > This patch introduces two types of GPADL: HV_GPADL_{BUFFER, RING}. The > types of GPADL are purely the concept in the guest, IOW the hypervisor > treat them as the same. > > The reason of introducing the types for GPADL is to

RE: [PATCH v3 02/11] Drivers: hv: vmbus: Move __vmbus_open()

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > Pure function movement, no functional changes. The move is made, because > in a later change, __vmbus_open() will rely on some static functions > afterwards, so we separate the move and the modification of > __vmbus_open() in two

RE: [PATCH v3 01/11] Drivers: hv: vmbus: Always use HV_HYP_PAGE_SIZE for gpadl

2020-09-12 Thread Michael Kelley
From: Boqun Feng Sent: Thursday, September 10, 2020 7:35 AM > > Since the hypervisor always uses 4K as its page size, the size of PFNs > used for gpadl should be HV_HYP_PAGE_SIZE rather than PAGE_SIZE, so > adjust this accordingly as the preparation for supporting 16K/64K page > size guests. No

Re: [GIT] Networking

2020-09-12 Thread Alejandro Colomar
On Thu, Sep 3, 2015 at 11:31 AM, Linus Torvalds wrote: >> [-Wsizeof-array-argument] > Ahh. Google shows that it's an old clang warning that gcc has recently > picked up. > But even clang doesn't seem to have any way for a project to say > "please warn about arrays in function argument

[PATCH] media: venus: helpers: Fix ALIGN() of non power of two

2020-09-12 Thread Rikard Falkeborn
ALIGN() expects its second argument to be a power of 2, otherwise incorrect results are produced for some inputs. The output can be both larger or smaller than what is expected. For example, ALIGN(304, 192) equals 320 instead of 384, and ALIGN(65, 192) equals 256 instead of 192. However,

[PATCH 1/2] media: vidtv: fix frequency tuning logic

2020-09-12 Thread Mauro Carvalho Chehab
Right now, there are some issues at the tuning logic: 1) the config struct is not copied at the tuner driver. so, it won't use any frequency table at all; 2) the code that checks for frequency shifts is called at set_params. So, lock_status will never be zeroed; 3) the signal strength will

[PATCH 2/2] media: vidtv: add an initial channel frequency

2020-09-12 Thread Mauro Carvalho Chehab
Use 474 MHz frequency for DVB-T/DVB-C, as this is the first channel that it is valid on most places for DVB-T. In the case of DVB-S, let's add Astra 19.2E initial frequency at the scan files as the default, e. g. 12.5515 GHz. Signed-off-by: Mauro Carvalho Chehab ---

Re: WARNING: refcount bug in io_wqe_worker

2020-09-12 Thread syzbot
syzbot has found a reproducer for the following issue on: HEAD commit:729e3d09 Merge tag 'ceph-for-5.9-rc5' of git://github.com/.. git tree: upstream console output: https://syzkaller.appspot.com/x/log.txt?x=12f2ce0d90 kernel config:

Re: [PATCH 1/5] staging: rtl8723bs: refactor cckrates{only}_included

2020-09-12 Thread Joe Perches
On Sat, 2020-09-12 at 11:22 -0700, Joe Perches wrote: > On Sat, 2020-09-12 at 10:45 +0200, Michael Straube wrote: > > Refactor cckrates_included() and cckratesonly_included() to simplify > > the code and improve readability. [] > diff --git a/drivers/staging/rtl8723bs/core/rtw_wlan_util.c >

[PATCH] MAINTAINERS: make linux-aspeed list remarks consistent

2020-09-12 Thread Lukas Bulwahn
Commit f15a3ea80391 ("MAINTAINERS: Add ASPEED BMC GFX DRM driver entry") does not mention that linux-asp...@lists.ozlabs.org is moderated for non-subscribers, but the other three entries for linux-asp...@lists.ozlabs.org do. By 'majority vote' among entries, let us assume it was just missed here

Re: [PATCH v12 0/5] Add support for DisplayPort driver on SnapDragon

2020-09-12 Thread Rob Clark
Fyi, I've pushed this series and the dp-compliance bits to msm-next-dp[1] I didn't include the dp audio series yet, which seems to need some minor rebasing. (And a small request, when resending, cc freedr...@lists.freedesktop.org, so it shows up in the patchwork instance[2] I use) You might

Re: [PATCH 1/5] staging: rtl8723bs: refactor cckrates{only}_included

2020-09-12 Thread Joe Perches
On Sat, 2020-09-12 at 10:45 +0200, Michael Straube wrote: > Refactor cckrates_included() and cckratesonly_included() to simplify > the code and improve readability. > > Signed-off-by: Michael Straube > --- > drivers/staging/rtl8723bs/core/rtw_wlan_util.c | 14 -- > 1 file changed, 8

[PATCH] spi: spi-geni-qcom: Don't wait to start 1st transfer if transmitting

2020-09-12 Thread Douglas Anderson
If we're sending bytes over SPI, we know the FIFO is empty at the start of the transfer. There's no reason to wait for the interrupt telling us to start--we can just start right away. Then if we transmit everything in one swell foop we don't even need to bother listening for TX interrupts. In a

Re: [v10 0/4] media: vidtv: Implement a virtual DVB driver

2020-09-12 Thread Mauro Carvalho Chehab
Em Sat, 12 Sep 2020 11:49:01 -0300 "Daniel W. S. Almeida" escreveu: > Hi Hans and Mauro & all > > > >Why the dvb_ prefix? All virtual drivers just start with 'vi'. > > > >And wouldn't it make more sense to call dvb_vidtv_bridge.ko just vidtv.ko? > >Just like the other virtual media drivers?

Re: WARNING in syscall_exit_to_user_mode

2020-09-12 Thread Kees Cook
On Fri, Sep 11, 2020 at 03:39:19AM -0700, syzbot wrote: > Hello, > > syzbot found the following issue on: > > HEAD commit:f4d51dff Linux 5.9-rc4 > git tree: upstream > console output: https://syzkaller.appspot.com/x/log.txt?x=12a4e7cd90 > kernel config:

  1   2   3   4   >