Re: [alsa-devel] [RFC PATCH 8/9] soundwire: intel: remove platform devices and provide new interface

2019-09-18 Thread Pierre-Louis Bossart
On 9/18/19 8:54 AM, Greg KH wrote: On Wed, Sep 18, 2019 at 03:53:02PM +0200, Greg KH wrote: On Wed, Sep 18, 2019 at 08:48:33AM -0500, Pierre-Louis Bossart wrote: On 9/18/19 7:06 AM, Greg KH wrote: On Tue, Sep 17, 2019 at 09:29:52AM -0500, Pierre-Louis Bossart wrote: On 9/17/19 12:55 AM, Greg

Re: [PATCH 5.2 00/85] 5.2.16-stable review

2019-09-18 Thread Naresh Kamboju
On Wed, 18 Sep 2019 at 11:56, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 5.2.16 release. > There are 85 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Resp

Re: [PATCH] KVM: coalesced_mmio: add bounds checking

2019-09-18 Thread Will Deacon
On Wed, Sep 18, 2019 at 04:01:42PM +0200, Paolo Bonzini wrote: > From: Matt Delco > > The first/last indexes are typically shared with a user app. > The app can change the 'last' index that the kernel uses > to store the next result. This change sanity checks the index > before using it for writ

Re: [PATCH v7 01/13] tools lib traceevent: Convert remaining %p[fF] users to %p[sS]

2019-09-18 Thread Andy Shevchenko
On Wed, Sep 18, 2019 at 04:34:07PM +0300, Sakari Ailus wrote: > There are no in-kernel %p[fF] users left. Convert the traceevent tool, > too, to align with the kernel. > function. The _tep_ argument is the trace event parser context. The _name_ is > -the name of the function, the string is copied

Re: Usecases for the per-task latency-nice attribute

2019-09-18 Thread Vincent Guittot
On Wed, 18 Sep 2019 at 16:19, Patrick Bellasi wrote: > > > On Wed, Sep 18, 2019 at 13:41:04 +0100, Parth Shah wrote... > > > Hello everyone, > > Hi Parth, > thanks for staring this discussion. > > [ + patrick.bell...@matbug.net ] my new email address, since with > @arm.com I will not be reachable

[PATCH v3 2/2] leds: tlc591xx: use devm_led_classdev_register_ext()

2019-09-18 Thread Jean-Jacques Hiblot
Use devm_led_classdev_register_ext() to pass the fwnode to the LED core. The fwnode can then be used by the firmware core to create meaningful names. Signed-off-by: Jean-Jacques Hiblot --- drivers/leds/leds-tlc591xx.c | 8 +--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/dr

[PATCH v3 0/2] leds: tlc591xx: switch to managed LED registration

2019-09-18 Thread Jean-Jacques Hiblot
This subject of this series used to be "leds: tlc591xx: switch to OF and managed API" This mini-series updates the tlc591xx driver to use the managed API. The driver is also modified to pass the initialization data to the LED core layer. The goal is to be able to the generic led-backlight [0] driv

[PATCH v3 1/2] leds: tlc591xx: simplify driver by using the managed led API

2019-09-18 Thread Jean-Jacques Hiblot
Use the managed API of the LED class (devm_led_classdev_register() instead of led_classdev_register()). This allows us to remove the code used to track-and-destroy the LED devices Signed-off-by: Jean-Jacques Hiblot Reviewed-by: Tomi Valkeinen --- drivers/leds/leds-tlc591xx.c | 79 +-

Re: [PATCH v1] net/phy: fix DP83865 10 Mbps HDX loopback disable function

2019-09-18 Thread Andrew Lunn
On Wed, Sep 18, 2019 at 05:48:25PM +0300, Peter Mamonov wrote: > According to the DP83865 datasheet "The 10 Mbps HDX loopback can be > disabled in the expanded memory register 0x1C0.1." The driver erroneously > used bit 0 instead of bit 1. Hi Peter This is version 2, not 1. Or if you want to star

[PATCH v3 1/1] platform/x86/intel_cht_int33fe: Split code to USB Micro-B and Type-C variants

2019-09-18 Thread Yauhen Kharuzhy
Existing intel_cht_int33fe ACPI pseudo-device driver assumes that hardware has Type-C connector and register related devices described as I2C connections in the _CRS resource. There is at least one hardware (Lenovo Yoga Book YB1-91L/F) with Micro-B USB connector exists. It has INT33FE device in th

[PATCH v3 0/1] intel_cht_int33fe: Split code to USB Micro-B and Type-C variants

2019-09-18 Thread Yauhen Kharuzhy
Patch to support INT33FE ACPI pseudo-device on hardware with USB Micro-B connector. v3: - Rename TypeB variant to Micro-B (we have only one such device for now and it has Micro-B connector) - Rebase on current linus/master - Remove empty lines and replace "TypeC" by "Type-C" v2: Instead of defi

Re: [PATCH V6 1/2] dt-bindings: mailbox: add binding doc for the ARM SMC/HVC mailbox

2019-09-18 Thread Jassi Brar
On Wed, Sep 18, 2019 at 9:46 AM Andre Przywara wrote: > > On Wed, 18 Sep 2019 09:19:46 -0500 > Jassi Brar wrote: > > Hi, > > > On Wed, Sep 18, 2019 at 4:44 AM Andre Przywara > > wrote: > > > > > > > > > which needs 9 arguments to work. The fact that the fist argument is > > > > always going to

Re: Running an application on a new VT

2019-09-18 Thread Christoph Pleger
Hello, On 2019-09-16 13:53, Christoph Pleger wrote: I am using the program openvt to run another program. openvt is used to open a new virtual terminal and then run the given program on the new VT. In my case, I want to start a KDE Plasma session on the new VT. Unfortunately, this did not work

Re: [PATCH v3 3/3] HID: core: fix dmesg flooding if report field larger than 32bit

2019-09-18 Thread Benjamin Tissoires
On Thu, Aug 29, 2019 at 1:26 AM Joshua Clayton wrote: > > ping? > I'd love to see this get in. > with distro kernel I have effectively no dmesg due to this issue Apologies for the delay. I really thought we should find a better way of fixing this, until I got a laptop affected by it. This series

Re: [PATCH v1] net/phy: fix DP83865 10 Mbps HDX loopback disable function

2019-09-18 Thread Peter Mamonov
Hi Andrew, On Wed, Sep 18, 2019 at 05:26:46PM +0200, Andrew Lunn wrote: > On Wed, Sep 18, 2019 at 05:48:25PM +0300, Peter Mamonov wrote: > > According to the DP83865 datasheet "The 10 Mbps HDX loopback can be > > disabled in the expanded memory register 0x1C0.1." The driver erroneously > > used bi

Re: [PATCH v4] powerpc/setup_64: fix -Wempty-body warnings

2019-09-18 Thread Qian Cai
Michael, ping in case that you might forget this one forever as well. On Mon, 2019-07-15 at 14:32 -0400, Qian Cai wrote: > At the beginning of setup_64.c, it has, > > #ifdef DEBUG > #define DBG(fmt...) udbg_printf(fmt) > #else > #define DBG(fmt...) > #endif > > where DBG() could be com

Re: Usecases for the per-task latency-nice attribute

2019-09-18 Thread Valentin Schneider
On 18/09/2019 15:18, Patrick Bellasi wrote: >> 1. Name: What should be the name for such attr for all the possible usecases? >> = >> Latency nice is the proposed name as of now where the lower value indicates >> that the task doesn't care much for the latency > > If by "lower value" yo

Do we need to correct barriering in circular-buffers.rst?

2019-09-18 Thread David Howells
Will Deacon wrote: > If I'm understanding your code correctly (big 'if'), then you have things > like this in pipe_read(): > > > unsigned int head = READ_ONCE(pipe->head); > unsigned int tail = pipe->tail; > unsigned int mask = pipe->buffers - 1; > > if (tail != head) {

Re: Usecases for the per-task latency-nice attribute

2019-09-18 Thread Patrick Bellasi
On Wed, Sep 18, 2019 at 16:22:32 +0100, Vincent Guittot wrote... > On Wed, 18 Sep 2019 at 16:19, Patrick Bellasi wrote: [...] >> $> Wakeup path tunings >> == >> >> Some additional possible use-cases was already discussed in [3]: >> >> - dynamically tune the policy of

[PATCH v3 2/2] powerpc/irq: inline call_do_irq() and call_do_softirq()

2019-09-18 Thread Christophe Leroy
call_do_irq() and call_do_softirq() are quite similar on PPC32 and PPC64 and are simple enough to be worth inlining. Inlining them avoids an mflr/mtlr pair plus a save/reload on stack. This is inspired from S390 arch. Several other arches do more or less the same. The way sparc arch does seems od

[PATCH v3 1/2] powerpc/irq: bring back ksp_limit management in C functions.

2019-09-18 Thread Christophe Leroy
Commit cbc9565ee826 ("powerpc: Remove ksp_limit on ppc64") moved PPC32 ksp_limit handling in assembly functions call_do_softirq() and call_do_irq() as they are different for PPC32 and PPC64. In preparation of replacing these functions by inline assembly, partialy revert that commit to bring back k

Re: printk() + memory offline deadlock (WAS Re: page_alloc.shuffle=1 + CONFIG_PROVE_LOCKING=y = arm64 hang)

2019-09-18 Thread Sergey Senozhatsky
On (09/18/19 10:39), Qian Cai wrote: > > Perhaps for a quick fix (and a comment that says this needs to be fixed > > properly). I think the changes to printk() that was discussed at > > Plumbers may also solve this properly. > > I assume that the new printk() stuff will also fix this deadlock betw

Re: [PATCH 5/8] ARM: dts: Drop bogus ahclkr clocks for dra7 mcasp 3 to 8

2019-09-18 Thread Tony Lindgren
* Tero Kristo [190917 07:22]: > On 24/07/2019 09:47, Tony Lindgren wrote: > > * Suman Anna [190723 21:02]: > > > Hi Tony, > > > > > > On 7/23/19 6:28 AM, Tony Lindgren wrote: > > > > The ahclkr clkctrl clock bit 28 only exists for mcasp 1 and 2 on dra7. > > > > Otherwise we get the following war

Re: [PATCH 3/3] regulator: core: make regulator_register() EPROBE_DEFER aware

2019-09-18 Thread Dmitry Torokhov
On Wed, Sep 18, 2019 at 1:18 AM Marco Felsch wrote: > > On 19-09-17 17:57, Dmitry Torokhov wrote: > > On Tue, Sep 17, 2019 at 4:42 PM Marco Felsch > > wrote: > > > > > > Sometimes it can happen that the regulator_of_get_init_data() can't > > > retrieve the config due to a not probed device the r

Re: Can KEY_DH_OPERATIONS become tristate? (was: Re: Kernel 5.3.0 stuck during boot on Amiga)

2019-09-18 Thread David Howells
Geert Uytterhoeven wrote: > CC crypto keys people > > TL;DR: CONFIG_CRYPTO_DH=y is reported to cause boot delays of several > minutes on old and slow machines. Why is it doing that? It doesn't do anything unless it is called, so something must be calling it. > Can KEY_DH_OPERATIONS be made tr

Re: printk() + memory offline deadlock (WAS Re: page_alloc.shuffle=1 + CONFIG_PROVE_LOCKING=y = arm64 hang)

2019-09-18 Thread Sergey Senozhatsky
A correction: On (09/19/19 00:51), Sergey Senozhatsky wrote: [..] > > zone->lock --> console_sem->lock > > So then we have > > zone->lock --> console_sem->lock --> pi_lock --> rq->lock > > vs. the reverse chain > > rq->lock --> console_sem->lock ^^^ zone->lo

[GIT PULL] Mailbox changes for v5.4

2019-09-18 Thread Jassi Brar
Hi Linus, The following changes since commit f74c2bb98776e2de508f4d607cd519873065118e: Linux 5.3-rc8 (2019-09-08 13:33:15 -0700) are available in the Git repository at: git://git.linaro.org/landing-teams/working/fujitsu/integration.git tags/mailbox-v5.4 for you to fetch changes up to 556a09

Re: [PATCH] [v2] arm64: fix unreachable code issue with cmpxchg

2019-09-18 Thread Nathan Chancellor
On Wed, Sep 18, 2019 at 10:02:41AM +0100, Will Deacon wrote: > On Tue, Sep 17, 2019 at 01:34:25PM -0700, Nathan Chancellor wrote: > > On Tue, Sep 10, 2019 at 01:56:22PM +0200, Arnd Bergmann wrote: > > > On arm64 build with clang, sometimes the __cmpxchg_mb is not inlined > > > when CONFIG_OPTIMIZE_

Re: Usecases for the per-task latency-nice attribute

2019-09-18 Thread Vincent Guittot
On Wed, 18 Sep 2019 at 17:46, Patrick Bellasi wrote: > > > On Wed, Sep 18, 2019 at 16:22:32 +0100, Vincent Guittot wrote... > > > On Wed, 18 Sep 2019 at 16:19, Patrick Bellasi > > wrote: > > [...] > > >> $> Wakeup path tunings > >> == > >> > >> Some additional possible us

[PATCH v9 0/2] mtd: spi-nor: add support for is25wp256 spi-nor flash

2019-09-18 Thread Sagar Shrikant Kadam
The patch series adds basic support for 32MiB spi-nor is25wp256 present on HiFive Unleashed A00 board. The flash device gets BFPT_DWORD1_ADDRESS_BYTES_3_ONLY from BFPT table for address width, whereas the flash can support 4 byte address width, so the address width is configured by using the post

[PATCH v9 2/2] mtd: spi-nor: fix nor->addr_width for is25wp256

2019-09-18 Thread Sagar Shrikant Kadam
Use the post bfpt fixup hook for the is25wp256 device as done for is25lp256 device to overwrite the address width advertised by BFPT. For instance the standard devices eg: IS25WP256D-JMLE where J stands for "standard" does not support SFDP. Signed-off-by: Sagar Shrikant Kadam Reviewed-by: Vignes

[PATCH v9 1/2] mtd: spi-nor: add support for is25wp256

2019-09-18 Thread Sagar Shrikant Kadam
Update spi_nor_id table for is25wp256 (32MB) device from ISSI, present on HiFive Unleashed dev board (Rev: A00). Set method to enable quad mode for ISSI device in flash parameters table. Based on code originally written by Wesley Terpstra and/or Palmer Dabbelt https://github.com/riscv/riscv-lin

Re: [PATCH] locking: locktorture: Do not include rwlock.h directly

2019-09-18 Thread Paul E. McKenney
On Tue, Sep 17, 2019 at 11:24:04PM -0700, Davidlohr Bueso wrote: > On Tue, 17 Sep 2019, Paul E. McKenney wrote: > > > On Tue, Sep 17, 2019 at 12:16:14AM -0700, Davidlohr Bueso wrote: > > > On Mon, 16 Sep 2019, Sebastian Andrzej Siewior wrote: > > > > > > > From: Wolfgang M. Reimer > > > > > > >

Re: [PATCH 3/3] regulator: core: make regulator_register() EPROBE_DEFER aware

2019-09-18 Thread Marco Felsch
On 19-09-18 08:53, Dmitry Torokhov wrote: > On Wed, Sep 18, 2019 at 1:18 AM Marco Felsch wrote: > > > > On 19-09-17 17:57, Dmitry Torokhov wrote: > > > On Tue, Sep 17, 2019 at 4:42 PM Marco Felsch > > > wrote: > > > > > > > > Sometimes it can happen that the regulator_of_get_init_data() can't >

Re: [PATCH 3/3] regulator: core: make regulator_register() EPROBE_DEFER aware

2019-09-18 Thread Mark Brown
On Wed, Sep 18, 2019 at 08:53:40AM -0700, Dmitry Torokhov wrote: > On Wed, Sep 18, 2019 at 1:18 AM Marco Felsch wrote: > > Those errors are handled but the behaviour of this funciton is to return > > NULL in such errors which is fine for the caller of this function. I > > only want to handle EPRO

Re: printk() + memory offline deadlock (WAS Re: page_alloc.shuffle=1 + CONFIG_PROVE_LOCKING=y = arm64 hang)

2019-09-18 Thread Qian Cai
On Thu, 2019-09-19 at 00:50 +0900, Sergey Senozhatsky wrote: > On (09/18/19 10:39), Qian Cai wrote: > > > Perhaps for a quick fix (and a comment that says this needs to be fixed > > > properly). I think the changes to printk() that was discussed at > > > Plumbers may also solve this properly. > >

Re: Can KEY_DH_OPERATIONS become tristate? (was: Re: Kernel 5.3.0 stuck during boot on Amiga)

2019-09-18 Thread Geert Uytterhoeven
Hi David, On Wed, Sep 18, 2019 at 5:54 PM David Howells wrote: > Geert Uytterhoeven wrote: > > CC crypto keys people > > > > TL;DR: CONFIG_CRYPTO_DH=y is reported to cause boot delays of several > > minutes on old and slow machines. > > Why is it doing that? It doesn't do anything unless it is

Spende von 5 Millionen Euro

2019-09-18 Thread Shane Missler
Dies ist eine persönliche Mail, die ich an Sie adressiere. Ich bin SHANE MISSLER aus Florida, USA. Wie Sie bereits wissen, habe ich einen Lotto-Jackpot in Höhe von 451 Mio. USD (330 Mio. GBP) gewonnen und das Geld hat mein Leben und mein Familienleben verändert, aber es wird mein Herz nicht verä

Re: [PATCH V2 3/4] ASoC: pcm_dmaengine: Extract snd_dmaengine_pcm_set_runtime_hwparams

2019-09-18 Thread kbuild test robot
/Shengjiu-Wang/ASoC-fsl_asrc-Use-in-out-put_format-instead-of-in-out-put_word_width/20190918-145058 config: arm-viper_defconfig (attached as .config) compiler: arm-linux-gnueabi-gcc (GCC) 7.4.0 reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin

Re: [PATCH V2 2/2] mm/pgtable/debug: Add test validating architecture page table helpers

2019-09-18 Thread Christophe Leroy
Le 18/09/2019 à 07:04, Anshuman Khandual a écrit : On 09/13/2019 03:31 PM, Christophe Leroy wrote: Le 13/09/2019 à 11:02, Anshuman Khandual a écrit : +#if !defined(__PAGETABLE_PMD_FOLDED) && !defined(__ARCH_HAS_4LEVEL_HACK) #ifdefs have to be avoided as much as possible, see below

Re: [PATCH 4.14 00/45] 4.14.145-stable review

2019-09-18 Thread Jon Hunter
On 18/09/2019 07:18, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.14.145 release. > There are 45 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses s

Re: [PATCH 4.19 00/50] 4.19.74-stable review

2019-09-18 Thread Jon Hunter
On 18/09/2019 07:18, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.19.74 release. > There are 50 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses sh

[PATCH v2] HID: hidraw: replace printk() with corresponding pr_xx() variant

2019-09-18 Thread Rishi Gupta
This commit replaces direct invocations of printk with their appropriate pr_info/warn() variant. Signed-off-by: Rishi Gupta --- Changes in v2: - Removed manually adding prefix "hidraw:". drivers/hid/hidraw.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/hid/h

Re: [PATCH 5.2 00/85] 5.2.16-stable review

2019-09-18 Thread Jon Hunter
On 18/09/2019 07:18, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 5.2.16 release. > There are 85 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > Responses sho

[PATCH net v3] net/phy: fix DP83865 10 Mbps HDX loopback disable function

2019-09-18 Thread Peter Mamonov
According to the DP83865 datasheet "the 10 Mbps HDX loopback can be disabled in the expanded memory register 0x1C0.1". The driver erroneously used bit 0 instead of bit 1. Fixes: 4621bf129856 ("phy: Add file missed in previous commit.") Signed-off-by: Peter Mamonov --- Changes since v2: - None Ch

Re: [PATCH net v3] net/phy: fix DP83865 10 Mbps HDX loopback disable function

2019-09-18 Thread Andrew Lunn
On Wed, Sep 18, 2019 at 07:27:55PM +0300, Peter Mamonov wrote: > According to the DP83865 datasheet "the 10 Mbps HDX loopback can be > disabled in the expanded memory register 0x1C0.1". The driver erroneously > used bit 0 instead of bit 1. > > Fixes: 4621bf129856 ("phy: Add file missed in previous

[PATCH] drivers/net/fjes: fix a potential NULL pointer dereference

2019-09-18 Thread Allen Pais
alloc_workqueue is not checked for errors and as a result, a potential NULL dereference could occur. Signed-off-by: Allen Pais --- drivers/net/fjes/fjes_main.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/drivers/net/fjes/fjes_main.c b/drivers/net/fjes/fjes_main.c index bbbc1dc..2d

[PATCH 1/2] Modify cpupower to schedule itself on cores it is reading MSRs from

2019-09-18 Thread Natarajan, Janakarajan
Modify cpupower to schedule itself on each of the cpus in the system and then get the APERF/MPERF register values. This is advantageous because an IPI is not generated when a read_msr() is executed on the local logical CPU thereby reducing the chance of having APERF and MPERF being out of sync. S

[PATCH] libertas: fix a potential NULL pointer dereference

2019-09-18 Thread Allen Pais
alloc_workqueue is not checked for errors and as a result, a potential NULL dereference could occur. Signed-off-by: Allen Pais --- drivers/net/wireless/marvell/libertas/if_sdio.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/net/wireless/marvell/libertas/if_sdio.c b/drivers/n

[PATCH 2/2] Update cpupower to use the RDPRU instruction

2019-09-18 Thread Natarajan, Janakarajan
AMD Zen 2 introduces the RDPRU instruction which can be used to access some processor registers which are typically only accessible in privilege level 0. ECX specifies the register to read and EDX:EAX will contain the value read. ECX: 0 - Register MPERF 1 - Register APERF This has the added

[PATCH] qla2xxx: fix a potential NULL pointer dereference

2019-09-18 Thread Allen Pais
alloc_workqueue is not checked for errors and as a result, a potential NULL dereference could occur. Signed-off-by: Allen Pais --- drivers/scsi/qla2xxx/qla_os.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/scsi/qla2xxx/qla_os.c b/drivers/scsi/qla2xxx/qla_os.c index 98e60a3..31

Re: [PATCH v3 2/2] powerpc/irq: inline call_do_irq() and call_do_softirq()

2019-09-18 Thread Segher Boessenkool
Hi Christophe, On Wed, Sep 18, 2019 at 03:48:20PM +, Christophe Leroy wrote: > call_do_irq() and call_do_softirq() are quite similar on PPC32 and > PPC64 and are simple enough to be worth inlining. > > Inlining them avoids an mflr/mtlr pair plus a save/reload on stack. But you hardcode the c

Re: [PATCH 0/4] drm/bridge: dw-hdmi: Add support for HDR metadata

2019-09-18 Thread Jonas Karlman
Hi Neil, On 2019-09-18 10:05, Neil Armstrong wrote: > Hi Jonas, > > On 26/05/2019 23:18, Jonas Karlman wrote: >> Add support for HDR metadata using the hdr_output_metadata connector >> property, >> configure Dynamic Range and Mastering InfoFrame accordingly. >> >> A drm_infoframe flag is added to

[PATCH] x86/mm: Remove set_pages_x() and set_pages_nx()

2019-09-18 Thread Larry Finger
In commit 185be15143aa ("x86/mm: Remove set_pages_x() and set_pages_nx()"), the wrappers were removed as they did not provide a real benefit over set_memory_x() and set_memory_nx(). This change causes a problem because the wrappers were exported, but the underlying routines were not. As a result, e

Re: printk meeting at LPC

2019-09-18 Thread Petr Mladek
On Wed 2019-09-18 11:05:28, John Ogness wrote: > On 2019-09-18, Sergey Senozhatsky wrote: > >> Each console has its own iterator. This iterators will need to > >> advance, regardless if the message was printed via write() or > >> write_atomic(). > > > > Great. > > > > ->atomic_write() path will ma

[PATCH v2] x86/mm: Fix 185be15143aa ("Remove set_pages_x() and set_pages_nx()")

2019-09-18 Thread Larry Finger
In commit 185be15143aa ("x86/mm: Remove set_pages_x() and set_pages_nx()"), the wrappers were removed as they did not provide a real benefit over set_memory_x() and set_memory_nx(). This change causes a problem because the wrappers were exported, but the underlying routines were not. As a result, e

Re: [PATCH] x86/mm: Remove set_pages_x() and set_pages_nx()

2019-09-18 Thread Christoph Hellwig
On Wed, Sep 18, 2019 at 11:41:21AM -0500, Larry Finger wrote: > In commit 185be15143aa ("x86/mm: Remove set_pages_x() and set_pages_nx()"), > the wrappers were removed as they did not provide a real benefit over > set_memory_x() and set_memory_nx(). This change causes a problem because > the wrappe

Re: [PATCH v4 1/3] arm64: cpufeature: introduce helper cpu_has_hw_af()

2019-09-18 Thread Catalin Marinas
On Wed, Sep 18, 2019 at 03:20:41PM +0100, Suzuki K Poulose wrote: > On 18/09/2019 14:19, Jia He wrote: > > diff --git a/arch/arm64/include/asm/cpufeature.h > > b/arch/arm64/include/asm/cpufeature.h > > index c96ffa4722d3..206b6e3954cf 100644 > > --- a/arch/arm64/include/asm/cpufeature.h > > +++ b/

Re: [PATCH v4 1/3] arm64: cpufeature: introduce helper cpu_has_hw_af()

2019-09-18 Thread Catalin Marinas
On Wed, Sep 18, 2019 at 07:20:17AM -0700, Matthew Wilcox wrote: > On Wed, Sep 18, 2019 at 09:19:12PM +0800, Jia He wrote: > > +/* Decouple AF from AFDBM. */ > > +bool cpu_has_hw_af(void) > > +{ > > + return (read_cpuid(ID_AA64MMFR1_EL1) & 0xf); > > +} > > + > > Do you really want to call read_cp

Re: Do we need to correct barriering in circular-buffers.rst?

2019-09-18 Thread Linus Torvalds
On Wed, Sep 18, 2019 at 8:43 AM David Howells wrote: > > It mandates using smp_store_release() to update buffer->head in the producer > and buffer->tail in the consumer - but these need pairing with memory barriers > used when reading buffer->head and buffer->tail on the other side. No, the rule

Re: printk meeting at LPC

2019-09-18 Thread Steven Rostedt
On Wed, 18 Sep 2019 18:41:55 +0200 Petr Mladek wrote: > Regarding SysRq. I could imagine introducing another SysRq that > would just call panic(). I mean that it would try to flush the > logs and reboot in the most safe way. You mean sysrq-c ? -- Steve

RE: [PATCH] nvme-pci: Save PCI state before putting drive into deepest state

2019-09-18 Thread Mario.Limonciello
> -Original Message- > From: Rafael J. Wysocki > Sent: Tuesday, September 17, 2019 4:36 PM > To: Keith Busch > Cc: Limonciello, Mario; Jens Axboe; Christoph Hellwig; Sagi Grimberg; linux- > n...@lists.infradead.org; LKML; Hong, Ryan; Wang, Crag; s...@google.com; > Dominguez, Jared > Subjec

Re: [GIT PULL] LED updates for 5.4-rc1

2019-09-18 Thread Linus Torvalds
On Wed, Sep 18, 2019 at 12:30 AM Andy Shevchenko wrote: > > I hope you have a chance to read my previous email. > We still need that function. Your email came in after I did the merge with the fixup, so the kernel now has the broken code (but without the warning). I think (hope) the fixup commit

[PATCH] iio: imu: adis16400: release allocated memory on failure

2019-09-18 Thread Navid Emamdoost
In adis_update_scan_mode, if allocation for adis->buffer fails, previously allocated adis->xfer needs to be released. Signed-off-by: Navid Emamdoost --- drivers/iio/imu/adis_buffer.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/iio/imu/adis_buffer.c b/drivers/ii

RE: [EXT] Re: [v4 2/2] clk: ls1028a: Add clock driver for Display output interface

2019-09-18 Thread Stephen Boyd
Quoting Wen He (2019-09-18 02:20:26) > > -Original Message- > > From: Stephen Boyd > > Quoting Wen He (2019-08-29 03:59:19) > > > diff --git a/drivers/clk/clk-plldig.c b/drivers/clk/clk-plldig.c new > > > file mode 100644 index ..d3239bcf59de > > > --- /dev/null > > > +++ b/dri

[PATCH] iio: imu: adis16400: fix memory leak

2019-09-18 Thread Navid Emamdoost
In adis_update_scan_mode_burst, if adis->buffer allocation fails release the adis->xfer. Signed-off-by: Navid Emamdoost --- drivers/iio/imu/adis_buffer.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/iio/imu/adis_buffer.c b/drivers/iio/imu/adis_buffer.c index 9ac

Re: [PATCH v6 04/10] s390: vfio-ap: filter CRYCB bits for unavailable queue devices

2019-09-18 Thread Halil Pasic
On Fri, 13 Sep 2019 17:26:52 -0400 Tony Krowiak wrote: > +static void vfio_ap_mdev_get_crycb_matrix(struct ap_matrix_mdev *matrix_mdev) > +{ > + unsigned long apid, apqi; > + unsigned long masksz = BITS_TO_LONGS(AP_DEVICES) * > +sizeof(unsigned long); > + > +

Re: [PATCH v6 6/9] leds: multicolor: Introduce a multicolor class definition

2019-09-18 Thread Dan Murphy
Greg +static int led_multicolor_init_color(struct led_classdev_mc_data *data, +struct led_classdev_mc *mcled_cdev, +int color_id, int color_index) +{ + struct led_classdev *led_cdev = mcled_cdev->led_cdev; + st

Re: [PATCH V6 1/2] dt-bindings: mailbox: add binding doc for the ARM SMC/HVC mailbox

2019-09-18 Thread Andre Przywara
On Wed, 18 Sep 2019 10:31:57 -0500 Jassi Brar wrote: Hi, > On Wed, Sep 18, 2019 at 9:46 AM Andre Przywara wrote: > > > > On Wed, 18 Sep 2019 09:19:46 -0500 > > Jassi Brar wrote: > > > > Hi, > > > > > On Wed, Sep 18, 2019 at 4:44 AM Andre Przywara > > > wrote: > > > > > > > > > > > > wh

Re: [PATCH 0/2] nvme: Add kernel API for admin command

2019-09-18 Thread Keith Busch
On Wed, Sep 18, 2019 at 03:26:11PM +0200, Christoph Hellwig wrote: > Even if we had a use case for that the bounce buffer is just too ugly > to live. And I'm really sick and tired of Intel wasting our time for > their out of tree monster given that they haven't even tried helping > to improve the

Re: [PATCH 0/2] pseries/hotplug: Change the default behaviour of cede_offline

2019-09-18 Thread Nathan Lynch
Gautham R Shenoy writes: > The accounting problem in tools such as lparstat with > "cede_offline=on" is affecting customers who are using these tools for > capacity-planning. That problem needs a fix in the short-term, for > which Patch 1 changes the default behaviour of cede_offline from "on" > t

Re: [PATCH 0/2] nvme: Add kernel API for admin command

2019-09-18 Thread Christoph Hellwig
On Wed, Sep 18, 2019 at 11:08:07AM -0600, Keith Busch wrote: > And yes, that bouncing is really nasty, but it's really only needed for > PRP, so maybe let's just not ignore that transfer mode and support > extended metadata iff the controller supports SGLs. We just need a > special SGL setup routin

linux-next: build failure after merge of the driver-core tree

2019-09-18 Thread Mark Brown
Hi all, After merging the driver-core tree, today's linux-next build for arm64 allmodconfig failed like this: /home/broonie/next/next/drivers/i2c/i2c-core-acpi.c: In function 'i2c_acpi_find_adapter_by_handle': /home/broonie/next/next/drivers/i2c/i2c-core-acpi.c:352:10: error: 'i2c_acpi_find_mat

Re: [GIT PULL] LED updates for 5.4-rc1

2019-09-18 Thread Linus Torvalds
On Wed, Sep 18, 2019 at 9:53 AM Linus Torvalds wrote: > > I think (hope) the fixup commit is in one of Greg's pull requests that > are now pending in my queue. Yup, confirmed. I was planning to do them later today, but I'm doing them now to get that regression out of the way. Linu

[PATCH] net: dsa: sja1105: prevent leaking memory

2019-09-18 Thread Navid Emamdoost
In sja1105_static_config_upload, in two cases memory is leaked: when static_config_buf_prepare_for_upload fails and when sja1105_inhibit_tx fails. In both cases config_buf should be released. Signed-off-by: Navid Emamdoost --- drivers/net/dsa/sja1105/sja1105_spi.c | 8 +--- 1 file changed, 5

RE: [PATCH v3 0/5] Introduce variable length mdev alias

2019-09-18 Thread Parav Pandit
Hi Cornelia, > -Original Message- > From: Cornelia Huck > Sent: Tuesday, September 17, 2019 5:14 AM > To: Parav Pandit > Cc: alex.william...@redhat.com; Jiri Pirko ; > kwankh...@nvidia.com; da...@davemloft.net; k...@vger.kernel.org; linux- > ker...@vger.kernel.org; net...@vger.kernel.org

Re: Usecases for the per-task latency-nice attribute

2019-09-18 Thread Tim Chen
On 9/18/19 5:41 AM, Parth Shah wrote: > Hello everyone, > > As per the discussion in LPC2019, new per-task property like latency-nice > can be useful in certain scenarios. The scheduler can take proper decision > by knowing latency requirement of a task from the end-user itself. > > There has alr

[PATCH 2/6] rpmsg: glink: Fix use after free in open_ack TIMEOUT case

2019-09-18 Thread Bjorn Andersson
From: Arun Kumar Neelakantam Extra channel reference put when remote sending OPEN_ACK after timeout causes use-after-free while handling next remote CLOSE command. Remove extra reference put in timeout case to avoid use-after-free. Fixes: b4f8e52b89f6 ("rpmsg: Introduce Qualcomm RPM glink drive

[PATCH 1/6] rpmsg: glink: Fix reuse intents memory leak issue

2019-09-18 Thread Bjorn Andersson
From: Arun Kumar Neelakantam Memory allocated for re-usable intents are not freed during channel cleanup which causes memory leak in system. Check and free all re-usable memory to avoid memory leak. Fixes: 933b45da5d1d ("rpmsg: glink: Add support for TX intents") Cc: sta...@vger.kernel.org Sign

[PATCH 4/6] rpmsg: glink: Fix rpmsg_register_device err handling

2019-09-18 Thread Bjorn Andersson
From: Chris Lew The device release function is set before registering with rpmsg. If rpmsg registration fails, the framework will call device_put(), which invokes the release function. The channel create logic does not need to free rpdev if rpmsg_register_device() fails and release is called. Fi

[PATCH 3/6] rpmsg: glink: Put an extra reference during cleanup

2019-09-18 Thread Bjorn Andersson
From: Chris Lew In a remote processor crash scenario, there is no guarantee the remote processor sent close requests before it went into a bad state. Remove the reference that is normally handled by the close command in the so channel resources can be released. Fixes: b4f8e52b89f6 ("rpmsg: Intro

Re: [PATCH 0/2] Reset realtek bluetooth devices during user suspend

2019-09-18 Thread Abhishek Pandit-Subedi
Sorry, last reply went out with HTML. Re-sending in plain text. On Wed, Sep 18, 2019 at 7:23 AM Alan Stern wrote: > > On Tue, 17 Sep 2019, Abhishek Pandit-Subedi wrote: > > > On a Realtek USB bluetooth device, I wanted a simple and consistent way > > to put the device in reset during suspend (2 r

[PATCH 0/6] rpmsg: glink stability fixes

2019-09-18 Thread Bjorn Andersson
Fixes for issues found in GLINK during reboot testing of a remoteproc. Arun Kumar Neelakantam (2): rpmsg: glink: Fix reuse intents memory leak issue rpmsg: glink: Fix use after free in open_ack TIMEOUT case Bjorn Andersson (2): rpmsg: glink: Don't send pending rx_done during remove rpmsg:

[PATCH 6/6] rpmsg: glink: Free pending deferred work on remove

2019-09-18 Thread Bjorn Andersson
By just cancelling the deferred rx worker during GLINK instance teardown any pending deferred commands are leaked, so free them. Fixes: b4f8e52b89f6 ("rpmsg: Introduce Qualcomm RPM glink driver") Cc: sta...@vger.kernel.org Signed-off-by: Bjorn Andersson --- drivers/rpmsg/qcom_glink_native.c | 14

[PATCH 5/6] rpmsg: glink: Don't send pending rx_done during remove

2019-09-18 Thread Bjorn Andersson
Attempting to transmit rx_done messages after the GLINK instance is being torn down will cause use after free and memory leaks. So cancel the intent_work and free up the pending intents. Fixes: 1d2ea36eead9 ("rpmsg: glink: Add rx done command") Cc: sta...@vger.kernel.org Signed-off-by: Bjorn Ander

[PATCH] net: qrtr: Stop rx_worker before freeing node

2019-09-18 Thread Bjorn Andersson
As the endpoint is unregistered there might still be work pending to handle incoming messages, which will result in a use after free scenario. The plan is to remove the rx_worker, but until then (and for stable@) ensure that the work is stopped before the node is freed. Fixes: bdabad3e363d ("net:

Re: [PATCH] net: dsa: sja1105: prevent leaking memory

2019-09-18 Thread Andrew Lunn
On Wed, Sep 18, 2019 at 12:10:19PM -0500, Navid Emamdoost wrote: > In sja1105_static_config_upload, in two cases memory is leaked: when > static_config_buf_prepare_for_upload fails and when sja1105_inhibit_tx > fails. In both cases config_buf should be released. > > Signed-off-by: Navid Emamdoost

Re: [PATCH RFC] tools/memory-model: Fix data race detection for unordered store and load

2019-09-18 Thread Paul E. McKenney
On Tue, Sep 17, 2019 at 01:39:59PM +0200, Andrea Parri wrote: > On Fri, Sep 06, 2019 at 04:57:22PM -0400, Alan Stern wrote: > > Currently the Linux Kernel Memory Model gives an incorrect response > > for the following litmus test: > > > > C plain-WWC > > > > {} > > > > P0(int *x) > > { > > W

[PATCH] mmc: host: atmel-mci: assign false/true to bool variable

2019-09-18 Thread Saiyam Doshi
Use false/true instead of 0/1 in bool variable assignment. This fixes below coccicheck warning. "WARNING: Assignment of 0/1 to bool variable" Generated by: scripts/coccinelle/misc/boolinit.cocci More information about semantic patching is available at http://coccinelle.lip6.fr/ Signed-off-by: S

Re: [PATCH] nvme-pci: Save PCI state before putting drive into deepest state

2019-09-18 Thread Keith Busch
On Wed, Sep 11, 2019 at 06:42:33PM -0500, Mario Limonciello wrote: > --- > drivers/nvme/host/pci.c | 13 +++-- > 1 file changed, 7 insertions(+), 6 deletions(-) > > diff --git a/drivers/nvme/host/pci.c b/drivers/nvme/host/pci.c > index 732d5b6..9b3fed4 100644 > --- a/drivers/nvme/host/pci

Re: [PATCH 1/4] seccomp: add SECCOMP_RET_USER_NOTIF_ALLOW

2019-09-18 Thread Kees Cook
On Wed, Sep 18, 2019 at 10:48:30AM +0200, Christian Brauner wrote: > This allows the seccomp notifier to continue a syscall. A positive > discussion about this feature was triggered by a post to the > ksummit-discuss mailing list (cf. [3]) and took place during KSummit > (cf. [1]) and again at the

Re: [PATCH 0/2] pseries/hotplug: Change the default behaviour of cede_offline

2019-09-18 Thread Naveen N. Rao
Michael Ellerman wrote: "Naveen N. Rao" writes: Michael Ellerman wrote: "Gautham R. Shenoy" writes: From: "Gautham R. Shenoy" Currently on Pseries Linux Guests, the offlined CPU can be put to one of the following two states: - Long term processor cede (also called extended cede) - Re

Re: [PATCH 2/4] seccomp: add two missing ptrace ifdefines

2019-09-18 Thread Kees Cook
On Wed, Sep 18, 2019 at 11:15:12AM +0200, Tyler Hicks wrote: > On 2019-09-18 10:48:31, Christian Brauner wrote: > > Add tw missing ptrace ifdefines to avoid compilation errors on systems > > that do not provide PTRACE_EVENTMSG_SYSCALL_ENTRY or > > PTRACE_EVENTMSG_SYSCALL_EXIT or: > > > > gcc -Wl,-

[PATCH v2 4/8] dt-bindings: memory-controllers: Convert Samsung Exynos SROM bindings to json-schema

2019-09-18 Thread Krzysztof Kozlowski
Convert Samsung Exynos SROM controller bindings to DT schema format using json-schema. Signed-off-by: Krzysztof Kozlowski --- Changes since v1: 1. Indent example with four spaces (more readable), 2. Split examples into two, 3. Fix pattern for subnode name, 4. Remove checks for #address-cells-ra

[PATCH v2 8/8] dt-bindings: pwm: Convert Samsung PWM bindings to json-schema

2019-09-18 Thread Krzysztof Kozlowski
Convert Samsung PWM (S3C, S5P and Exynos SoCs) bindings to DT schema format using json-schema. Signed-off-by: Krzysztof Kozlowski --- Changes since v1: 1. Indent example with four spaces (more readable), 2. Fix samsung,pwm-outputs after review, 3. Remove double-quotes from clock names. --- ...

[PATCH v2 7/8] dt-bindings: pwm: Convert PWM bindings to json-schema

2019-09-18 Thread Krzysztof Kozlowski
Convert generic PWM bindings to DT schema format using json-schema. The consumer bindings are split to separate file. Signed-off-by: Krzysztof Kozlowski --- Changes since v1: 1. Indent example with four spaces (more readable), 2. Change pattern for pwm nodes, 3. Remove $ref from #cells. --- .

[PATCH v2 3/8] dt-bindings: sram: Convert Samsung Exynos SYSRAM bindings to json-schema

2019-09-18 Thread Krzysztof Kozlowski
Convert Samsung Exynos SYSRAM bindings to DT schema format using json-schema. Signed-off-by: Krzysztof Kozlowski --- TODO: The node naming should be probably fixed (sysram->sram) Changes since v1: 1. Indent example with four spaces (more readable). --- .../devicetree/bindings/sram/samsung-sra

Re: [PATCH] platform/chrome: wilco_ec: Add debugfs test_event file

2019-09-18 Thread Daniel Campello
On Fri, Sep 06, 2019 at 06:20:51PM -0400, Nick Crews wrote: > Thanks for the patch Daniel! A few thoughts that I didn't > have on the review on Gerrit, sorry :) After those changes, > No problem, I will take care of them. > Reviewed-by: Nick Crews > > > u8 sub_cmd; /* Always SUB_CMD_H

[PATCH v2 5/8] dt-bindings: watchdog: Convert Samsung SoC watchdog bindings to json-schema

2019-09-18 Thread Krzysztof Kozlowski
Convert Samsung S3C/S5P/Exynos watchdog bindings to DT schema format using json-schema. Signed-off-by: Krzysztof Kozlowski --- Changes since v1: 1. Indent example with four spaces (more readable), 2. Remove unneeded timeout-sec description. --- .../bindings/watchdog/samsung-wdt.txt | 3

[PATCH v3] platform/chrome: wilco_ec: Add debugfs test_event file

2019-09-18 Thread Daniel Campello
This change introduces a new debugfs file 'test_event' that when written to causes the EC to generate a test event. Signed-off-by: Daniel Campello --- Changes for v2: - Cleaned up and added comments. - Renamed and updated function signature from write_to_mailbox to send_ec_cmd. Changes fo

<    1   2   3   4   5   6   7   8   9   >