Re: [PATCH] nvmem: core: Set no-read-write provider to avoid userspace read/write

2019-03-16 Thread kbuild test robot
Hi Gaurav, Thank you for the patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v5.0 next-20190306] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

Re: [PATCH] nvmem: core: Set no-read-write provider to avoid userspace read/write

2019-03-16 Thread kbuild test robot
Hi Gaurav, Thank you for the patch! Yet something to improve: [auto build test ERROR on linus/master] [also build test ERROR on v5.0 next-20190306] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url:

[PATCH v2] kobject: Don't trigger kobject_uevent(KOBJ_REMOVE) twice.

2019-03-16 Thread Tetsuo Handa
syzbot is hitting use-after-free bug in uinput module [1]. This is because kobject_uevent(KOBJ_REMOVE) is called again due to commit 0f4dafc0563c6c49 ("Kobject: auto-cleanup on final unref") after memory allocation fault injection made kobject_uevent(KOBJ_REMOVE) from device_del() from

[PATCH] staging: speakup: fix line over 80 characters.

2019-03-16 Thread Jules Irenge
Fix coding style issues, which solves checkpatch.pl warning: "WARNING: line over 80 characters". --- drivers/staging/speakup/kobjects.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/speakup/kobjects.c b/drivers/staging/speakup/kobjects.c index

Re: [PATCH v2 2/3] scripts/package/mkdebian: avoid implicit effects

2019-03-16 Thread Masahiro Yamada
On Sun, Mar 10, 2019 at 12:44 AM Arseny Maslennikov wrote: > > * The man page for dpkg-source(1) notes: > > > -b, --build directory [format-specific-parameters] > > Build a source package (--build since dpkg 1.17.14). > > <...> > > > > dpkg-source will

Re: [PATCH] Revert "modsign: Abort modules_install when signing fails"

2019-03-16 Thread Masahiro Yamada
On Sat, Mar 16, 2019 at 1:25 AM Douglas Anderson wrote: > > This reverts commit caf6fe91ddf62a96401e21e9b7a07227440f4185. > > The commit was fine but is no longer needed as of commit 3a2429e1faf4 > ("kbuild: change if_changed_rule for multi-line recipe"). Let's go > back to using ";" to be

[PATCH 1/2] staging: vc04_services: Add spaces around Operators

2019-03-16 Thread Emanuel Bennici
Fix Check from the checkpatch.pl Script `CHECK: spaces preferred around that` in vchi.h Signed-off-by: Emanuel Bennici --- drivers/staging/vc04_services/interface/vchi/vchi.h | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git

[PATCH 2/2] staging: vc04_services: Add blank line after struct declaration

2019-03-16 Thread Emanuel Bennici
Signed-off-by: Emanuel Bennici --- drivers/staging/vc04_services/interface/vchi/vchi.h | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/staging/vc04_services/interface/vchi/vchi.h b/drivers/staging/vc04_services/interface/vchi/vchi.h index 7b13cb04c119..43eaf7703777 100644 ---

Re: [RFC] simple_lmk: Introduce Simple Low Memory Killer for Android

2019-03-16 Thread Joel Fernandes
On Sat, Mar 16, 2019 at 12:37:18PM -0700, Suren Baghdasaryan wrote: > On Sat, Mar 16, 2019 at 11:57 AM Christian Brauner > wrote: > > > > On Sat, Mar 16, 2019 at 11:00:10AM -0700, Daniel Colascione wrote: > > > On Sat, Mar 16, 2019 at 10:31 AM Suren Baghdasaryan > > > wrote: > > > > > > > > On

Re: [PATCH v3 3/3] Drivers: hv: vmbus: Fix race condition with new ring_buffer_info mutex

2019-03-16 Thread Kimberly Brown
On Thu, Mar 14, 2019 at 03:45:33PM -0700, Stephen Hemminger wrote: > On Thu, 14 Mar 2019 13:05:15 -0700 > "Kimberly Brown" wrote: > > > Fix a race condition that can result in a ring buffer pointer being set > > to null while a "_show" function is reading the ring buffer's data. This > > problem

linux-next: Fixes tag needs some work in the sound-asoc tree

2019-03-16 Thread Stephen Rothwell
Hi all, In commit 399706df420e ("ASoC: rsnd: src: fix compiler warnings") Fixes tag Fixes: linux-next commit 7674bec4fc09 ("ASoC: rsnd: update BSDSR/BSDISR handling") has these problem(s): - 'linux-next commit' was unexpected. In commit ba164a49f8f7 ("ASoC: rsnd: src: Avoid a

linux-next: Fixes tag needs some work in the sound-asoc tree

2019-03-16 Thread Stephen Rothwell
Hi all, In commit 2b13bee38849 ("ASoC: samsung: odroid: Fix clock configuration for 44100 sample rate") Fixes tag Fixes: fbeec965b8d1c ("ASoC: samsung: odroid: Fix 32000 sample rate handling") has these problem(s): - Target SHA1 does not exist Did you mean: 1d22c337dc8f ("ASoC:

Re: mount.nfs: Protocol error after upgrade to linux/master

2019-03-16 Thread Casey Schaufler
On 3/16/2019 1:08 AM, Tetsuo Handa wrote: On 2019/03/16 14:38, Kees Cook wrote: config LSM string "Ordered list of enabled LSMs" + default "yama,loadpin,safesetid,integrity,smack,selinux,tomoyo,apparmor" if DEFAULT_SECURITY_SMACK + default

[PATCH] virtio-fs: fix multiple tag support

2019-03-16 Thread Liu Bo
While doing memremap from pci_dev's system bus address to kernel virtual address, we assign a wrong value to the %end of pgmap.res, which ends up with a wrong resource size in the process of memremap, and that further prevent the second virtiofs pci device from being probed successfully.

Re: [PATCH 18/52] virtio-fs: Map cache using the values from the capabilities

2019-03-16 Thread Liu Bo
On Mon, Dec 10, 2018 at 9:57 AM Vivek Goyal wrote: > > Instead of assuming we had the fixed bar for the cache, use the > value from the capabilities. > > Signed-off-by: Dr. David Alan Gilbert > --- > fs/fuse/virtio_fs.c | 32 +--- > 1 file changed, 17 insertions(+),

[PATCH] staging: mt7621-pci-phy: Add Spaces to Macro Definition

2019-03-16 Thread Emanuel Bennici
Improve Code readability by adding Spaces after #define Signed-off-by: Emanuel Bennici --- .../staging/mt7621-pci-phy/pci-mt7621-phy.c | 126 +- 1 file changed, 63 insertions(+), 63 deletions(-) diff --git a/drivers/staging/mt7621-pci-phy/pci-mt7621-phy.c

[PATCH] IB/iser: remove uninitialized variable len

2019-03-16 Thread Colin King
From: Colin Ian King The variable len is not being inintialized and the uninitialized value is being returned. However, this return path is never reached because the default case in the switch statement returns -ENOSYS. Clean up the code by replacing the return -ENOSYS with a break for the

[PATCH] staging: rtl8723bs: remove unused code

2019-03-16 Thread Colin King
From: Colin Ian King There are two final hunks of code that are only built if SDIO_DYNAMIC_ALLOC_MEM is defined however this is never defined and if it was the code would lead to dereferencing an uninitialized pointer oldmem. It appears that a previous commit 1babeb0c3e59 ("Staging: rtl8723bs:

Re: [PATCH 6/9] arm64: dts: meson: g12a: Add UART A, B & C nodes and pins

2019-03-16 Thread Martin Blumenstingl
Hi Neil, On Sat, Mar 16, 2019 at 3:35 PM Neil Armstrong wrote: [...] > >> + uart_ao_a_c_pins: uart_ao_a_c { > >> + mux { > >> + groups = > >>

Re: [PATCH 5/9] arm64: dts: meson: g12a: add reset controller

2019-03-16 Thread Martin Blumenstingl
Hi Neil, On Sat, Mar 16, 2019 at 3:32 PM Neil Armstrong wrote: [...] > > Do you have any hint how to verify the CBUS offset (0x1004) of the > > reset controller? > > You can find base the address found in the G12A DT for the usb2 phy : >

[PATCH v2] iio: hmc5843: fix potential NULL pointer dereferences

2019-03-16 Thread Kangjie Lu
devm_regmap_init_i2c may fail and return NULL. The fix returns the error when it fails. Signed-off-by: Kangjie Lu --- V2: fix the two together --- drivers/iio/magnetometer/hmc5843_i2c.c | 7 ++- drivers/iio/magnetometer/hmc5843_spi.c | 7 ++- 2 files changed, 12 insertions(+), 2

Re: [PATCH 11/11] arm64: dts: meson-g12a-x96-max: Enable USB

2019-03-16 Thread Martin Blumenstingl
Hi Neil, On Sat, Mar 16, 2019 at 3:46 PM Neil Armstrong wrote: > > > > Le 15/03/2019 22:50, Martin Blumenstingl a écrit : > > Hi Neil, > > > > On Mon, Mar 11, 2019 at 10:58 AM Neil Armstrong > > wrote: > > [...] > >> + { > >> + status = "okay"; > >> +}; > > your patch description states

Re: [PATCH] Use fall-through attribute rather than magic comments

2019-03-16 Thread Gustavo A. R. Silva
Shawn, On 3/15/19 10:38 PM, Shawn Landden wrote: > The -Wimplicit-fallthrough warning recognizes magic comments > https://developers.redhat.com/blog/2017/03/10/wimplicit-fallthrough-in-gcc-7/ > It is my opinion that magic comments are a bad idea. Comments should not > change the interpretation

Re: [GIT PULL] pidfd patches for v5.1-rc1

2019-03-16 Thread pr-tracker-bot
The pull request you sent on Tue, 5 Mar 2019 18:13:01 +0100: > git://git.kernel.org/pub/scm/linux/kernel/git/brauner/linux.git > tags/pidfd-v5.1-rc1 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/a9dce6679d736cb3d612af39bab9f31f8db66f9b Thank you! --

Re: [GIT PULL] Please pull NFS client bugfixes for 5.1

2019-03-16 Thread pr-tracker-bot
The pull request you sent on Sat, 16 Mar 2019 18:33:33 +: > git://git.linux-nfs.org/projects/trondmy/linux-nfs.git tags/nfs-for-5.1-2 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/465c209db83e2cdaeb4a52f4e107a9fc636704db Thank you! -- Deet-doot-dot, I am a

[PATCH 4/4] x86, putuser: switch 64-bit version to normal calling convention

2019-03-16 Thread Alexey Dobriyan
Use usual RDI, RSI for put_user()'s arguments. Do everything in 4 registers, clobber RDX being the next in sequence. Do CLAC as soon as possible. Indent more. Delete unnecessary macros. This essentially undoes what Glauber did 10 years ago, but the "this is the i386 way" is not an argument today

[PATCH 3/4] x86, putuser: cleanup 32-bit version a little

2019-03-16 Thread Alexey Dobriyan
Delete CONFIG_X86_32 references and _ASM_* macros. Don't bother with anything else because i386 is dead. Signed-off-by: Alexey Dobriyan --- arch/x86/lib/putuser_32.S | 36 1 file changed, 16 insertions(+), 20 deletions(-) --- a/arch/x86/lib/putuser_32.S

[PATCH 2/4] x86, putuser: copy pasta into 32 and 64-bit versions

2019-03-16 Thread Alexey Dobriyan
I'm going to rewrite 64-bit version to use normal x86_64 calling convention. Signed-off-by: Alexey Dobriyan --- arch/x86/lib/Makefile |3 - arch/x86/lib/putuser_64.S | 103 ++ 2 files changed, 105 insertions(+), 1 deletion(-) ---

[PATCH 1/4] x86, putuser: add ASM_CALL_CONSTRAINT

2019-03-16 Thread Alexey Dobriyan
Clobber stack pointer given that assembly does CALL. Signed-off-by: Alexey Dobriyan --- arch/x86/include/asm/uaccess.h |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) --- a/arch/x86/include/asm/uaccess.h +++ b/arch/x86/include/asm/uaccess.h @@ -176,7 +176,8 @@

[PATCH 1/3] habanalabs: perform accounting for active CS

2019-03-16 Thread Oded Gabbay
This patch adds accounting for active CS. Active means that the CS was submitted to the H/W queues and was not completed yet. This is necessary to support suspend operation. Because the device will be reset upon suspend, we can only suspend after all active CS have been completed. Hence, we need

[PATCH 2/3] habanalabs: prevent host crash during suspend/resume

2019-03-16 Thread Oded Gabbay
This patch fixes the implementation of suspend/resume of the device so that upon resume of the device, the host won't crash due to PCI completion timeout. Upon suspend, the device is being reset due to PERST. Therefore, upon resume, the driver must initialize the PCI controller as if the driver

[PATCH 3/3] habanalabs: cast to expected type

2019-03-16 Thread Oded Gabbay
This patch fix the following sparse warning: drivers/misc/habanalabs/goya/goya.c:3646:14: warning: incorrect type in assignment (different address spaces) drivers/misc/habanalabs/goya/goya.c:3646:14:expected void *base drivers/misc/habanalabs/goya/goya.c:3646:14:got void [noderef] *

[PATCH 2/4] habanalabs: fix bug when mapping very large memory area

2019-03-16 Thread Oded Gabbay
From: Omer Shpigelman This patch fixes a bug of allocating a too big memory size with kmalloc, which causes a failure. In case of mapping a large memory block, an array of the relevant physical page addresses is allocated. If there are many pages the array might be too big to allocate with

[PATCH 4/4] habanalabs: fix mapping with page size bigger than 4KB

2019-03-16 Thread Oded Gabbay
From: Omer Shpigelman This patch fixes the mapping of virtual address to physical addresses on architectures where PAGE_SIZE is bigger than 4KB. The break down to the device page size was done only for the virtual address while it should have been done for the physical address as well. As a

[PATCH 1/4] habanalabs: fix MMU number of pages calculation

2019-03-16 Thread Oded Gabbay
From: Omer Shpigelman The requested allocation size is 64bit, hence the number of requested pages and the total requested size should 64bit as well. This patch fixes all places where these are treated as 32bit. Signed-off-by: Omer Shpigelman Signed-off-by: Oded Gabbay ---

[PATCH 3/4] habanalabs: complete user context cleanup before hard reset

2019-03-16 Thread Oded Gabbay
From: Omer Shpigelman This patch fixes a bug which led to a crash during hard reset flow. Before a hard reset is executed, we wait a few seconds for the user context cleanup to complete. If it wasn't completed, we kill the user process and move on to the reset flow. Upon killing the user

Re: [PATCH 0/3] Small improvements/bugfixes to the rk3399-opi dt

2019-03-16 Thread Heiko Stuebner
Am Freitag, 15. März 2019, 15:58:15 CET schrieb Alexis Ballier: > This patch series is based on the rockchip/v5.2-armsoc/dts64 branch and > fixes a few mistakes in the rk3399-orangepi.dts I sent a couple weeks ago. applied all 3 for 5.2 Thanks Heiko

Re: KASAN: use-after-free Read in get_mem_cgroup_from_mm

2019-03-16 Thread Andrea Arcangeli
On Sat, Mar 16, 2019 at 05:38:54PM +0800, zhong jiang wrote: > On 2019/3/16 5:39, Andrea Arcangeli wrote: > > On Fri, Mar 08, 2019 at 03:10:08PM +0800, zhong jiang wrote: > >> I can reproduce the issue in arm64 qemu machine. The issue will leave > >> after applying the > >> patch. > >> > >>

Re: [RFC] simple_lmk: Introduce Simple Low Memory Killer for Android

2019-03-16 Thread Suren Baghdasaryan
On Sat, Mar 16, 2019 at 11:57 AM Christian Brauner wrote: > > On Sat, Mar 16, 2019 at 11:00:10AM -0700, Daniel Colascione wrote: > > On Sat, Mar 16, 2019 at 10:31 AM Suren Baghdasaryan > > wrote: > > > > > > On Fri, Mar 15, 2019 at 11:49 AM Joel Fernandes > > > wrote: > > > > > > > > On Fri,

Re: [PATCH v4 7/7] staging: iio: ad5933: move out of staging

2019-03-16 Thread Jonathan Cameron
On Sat, 16 Mar 2019 12:08:11 -0300 Marcelo Schmitt wrote: > Move ad5933 impedance-analyzer driver from staging to mainline. > The ad5933 is a high precision impedance converter system solution > that combines an on-board frequency generator with an > analog-to-digital converter (ADC). This

Re: [PATCH 0/2] x86/CPU: Use correct Cyrix-specific macros

2019-03-16 Thread Andy Lutomirski
On Thu, Mar 14, 2019 at 1:46 PM Matthew Whitehead wrote: > > Replace the incorrect Cyrix-specific macro calls with the correct > setCx86() and getCx86() calls. Also remove the unused setCx86_old(), > getCx86_old() and their related comments in the code. > > Matthew Whitehead (2): > x86/CPU: Use

Re: [RFC] simple_lmk: Introduce Simple Low Memory Killer for Android

2019-03-16 Thread Christian Brauner
On Sat, Mar 16, 2019 at 11:00:10AM -0700, Daniel Colascione wrote: > On Sat, Mar 16, 2019 at 10:31 AM Suren Baghdasaryan wrote: > > > > On Fri, Mar 15, 2019 at 11:49 AM Joel Fernandes > > wrote: > > > > > > On Fri, Mar 15, 2019 at 07:24:28PM +0100, Christian Brauner wrote: > > > [..] > > > > >

Re: [PATCH v4 6/7] staging: iio: ad5933: change attributes to match ABI

2019-03-16 Thread Jonathan Cameron
On Sat, 16 Mar 2019 12:07:55 -0300 Marcelo Schmitt wrote: > Change device attributes' names to match ABI documentation. Names were > chosen such that they tend to be similar to existing ABI so it should > be easier to standardize them when necessary. > > Signed-off-by: Marcelo Schmitt I spoke

Re: [PATCH v5 10/11] staging: iio: ad7780: moving ad7780 out of staging

2019-03-16 Thread Renato Lui Geh
On 03/16, Jonathan Cameron wrote: On Fri, 15 Mar 2019 23:15:55 -0300 Renato Lui Geh wrote: Move ad7780 ADC driver out of staging and into the mainline. The ad7780 is a sigma-delta analog to digital converter. This driver provides reading voltage values and status bits from both the ad778x

Re: [PATCH v4 6/7] staging: iio: ad5933: change attributes to match ABI

2019-03-16 Thread Jonathan Cameron
On Sat, 16 Mar 2019 12:07:55 -0300 Marcelo Schmitt wrote: > Change device attributes' names to match ABI documentation. Names were > chosen such that they tend to be similar to existing ABI so it should > be easier to standardize them when necessary. > > Signed-off-by: Marcelo Schmitt Applied

Re: [PATCH v4 5/7] staging: iio: ad5933: add ABI documentation

2019-03-16 Thread Jonathan Cameron
On Sat, 16 Mar 2019 12:07:32 -0300 Marcelo Schmitt wrote: > Add an ABI documentation for the ad5933 driver. > > Signed-off-by: Marcelo Schmitt Some stray whitespace in here. I cleaned it up. Thanks, Jonathan > --- > .../sysfs-bus-iio-impedance-analyzer-ad5933 | 35 +++ >

Re: [PATCH v4 4/7] staging: iio: ad5933: change help rule message

2019-03-16 Thread Jonathan Cameron
On Sat, 16 Mar 2019 12:07:18 -0300 Marcelo Schmitt wrote: > Remove the previous comment about direct access via sysfs which would > lead one think ad5933 driver has limitations it actually doesn't. > > Signed-off-by: Marcelo Schmitt Applied. Thanks, Jonathan > --- >

Re: [PATCH v4 3/7] staging: iio: ad5933: add SPDX identifier

2019-03-16 Thread Jonathan Cameron
On Sat, 16 Mar 2019 12:07:03 -0300 Marcelo Schmitt wrote: > Add SPDX identifier of GPL-2.0 for the ad5933 driver. > > Signed-off-by: Marcelo Schmitt Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > --- >

Re: [PATCH] net: socket: fix a missing check for nla_nest_start

2019-03-16 Thread David Miller
From: Kangjie Lu Date: Fri, 15 Mar 2019 01:03:05 -0500 > nla_nest_start may fail. The fix check its status and returns > -EMSGSIZE in case it fails. > > Signed-off-by: Kangjie Lu The proper subsystem prefix is "tipc: " not "socket: "

Re: [PATCH v4 2/7] staging: iio: ad5933: organize includes

2019-03-16 Thread Jonathan Cameron
On Sat, 16 Mar 2019 12:06:50 -0300 Marcelo Schmitt wrote: > Organize includes to list them in lexicographic order. > > Signed-off-by: Marcelo Schmitt Applied. Thanks > --- > .../staging/iio/impedance-analyzer/ad5933.c| 18 +- > 1 file changed, 9 insertions(+), 9

Re: [PATCH v4 1/7] staging: iio: ad5933: change multi-line comment style

2019-03-16 Thread Jonathan Cameron
On Sat, 16 Mar 2019 12:06:18 -0300 Marcelo Schmitt wrote: > Make multi-line comments compliant with the preferred code style. > > Signed-off-by: Marcelo Schmitt Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to hopefully ignore. Thanks, Jonathan > ---

[GIT PULL] Please pull NFS client bugfixes for 5.1

2019-03-16 Thread Trond Myklebust
Hi Linus, The following changes since commit 4d6c671ace569d4b0d3f8d92ab3aef18a5d166bc: SUNRPC: Take the transport send lock before binding+connecting (2019-03-10 14:08:19 -0400) are available in the Git repository at: git://git.linux-nfs.org/projects/trondmy/linux-nfs.git

Re: [PATCH v5 11/11] staging: iio: ad7780: add device tree binding

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:16:13 -0300 Renato Lui Geh wrote: > Adds a device tree binding for the ad7780 driver. > > Signed-off-by: Renato Lui Geh Seems like a straight forward binding so applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it.

Re: [PATCH v5 10/11] staging: iio: ad7780: moving ad7780 out of staging

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:15:55 -0300 Renato Lui Geh wrote: > Move ad7780 ADC driver out of staging and into the mainline. > > The ad7780 is a sigma-delta analog to digital converter. This driver provides > reading voltage values and status bits from both the ad778x and ad717x series. > Its

Re: [PATCH v5 09/11] staging: iio: ad7780: add new copyright holder

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:15:26 -0300 Renato Lui Geh wrote: > This patch adds a new copyright holder to the ad7780 driver. > > Signed-off-by: Renato Lui Geh Applied thanks Jonathan > --- > drivers/staging/iio/adc/ad7780.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git

Re: [PATCH v5 08/11] staging: iio: ad7780: add SPDX identifier

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:15:13 -0300 Renato Lui Geh wrote: > Add SPDX identifier (GPL-2.0) to the AD7780 driver. > > Signed-off-by: Renato Lui Geh Applied > --- > drivers/staging/iio/adc/ad7780.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git

Re: [Patch v2 2/2] CIFS: Fix an issue with re-sending rdata when transport returning -EAGAIN

2019-03-16 Thread Pavel Shilovsky
пт, 15 мар. 2019 г. в 00:56, Long Li : > > From: Long Li > > When sending a rdata, transport may return -EAGAIN. In this case > we should re-obtain credits because the session may have been > reconnected. > > Change in v2: adjust_credits before re-sending > > Signed-off-by: Long Li > --- >

Re: [PATCH v5 07/11] staging: iio: ad7780: move regulator to after GPIO init

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:14:59 -0300 Renato Lui Geh wrote: > To maintain consistency between ad7780_probe and ad7780_remove orders, > regulator initialization has been moved to after GPIO initializations. > > Signed-off-by: Renato Lui Geh Applied, Thanks, Jonathan > --- >

Re: [Patch v2 1/2] CIFS: Fix an issue with re-sending wdata when transport returning -EAGAIN

2019-03-16 Thread Pavel Shilovsky
пт, 15 мар. 2019 г. в 00:56, Long Li : > > From: Long Li > > When sending a wdata, transport may return -EAGAIN. In this case > we should re-obtain credits because the session may have been > reconnected. > > Change in v2: adjust_credits before re-sending > > Signed-off-by: Long Li > --- >

Re: [PATCH v5 06/11] staging:iio:ad7780: add chip ID values and mask

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:14:27 -0300 Renato Lui Geh wrote: > The ad7780 supports both the ad778x and ad717x families. Each chip has > a corresponding ID. This patch provides a mask for extracting ID values > from the status bits and also macros for the correct values for the > ad7170, ad7171,

Re: [RFC] simple_lmk: Introduce Simple Low Memory Killer for Android

2019-03-16 Thread Daniel Colascione
On Sat, Mar 16, 2019 at 10:31 AM Suren Baghdasaryan wrote: > > On Fri, Mar 15, 2019 at 11:49 AM Joel Fernandes > wrote: > > > > On Fri, Mar 15, 2019 at 07:24:28PM +0100, Christian Brauner wrote: > > [..] > > > > why do we want to add a new syscall (pidfd_wait) though? Why not just > > > > use

[PATCH v9 7/9] cgroup: make TRACE_CGROUP_PATH irq-safe

2019-03-16 Thread Roman Gushchin
To use the TRACE_CGROUP_PATH() macro with css_set_lock locked, let's make the macro irq-safe. It's necessary in order to trace cgroup freezer state transitions (frozen/not frozen), which are happening with css_set_lock locked. Signed-off-by: Roman Gushchin --- kernel/cgroup/cgroup-internal.h |

[PATCH v9 3/9] cgroup: protect cgroup->nr_(dying_)descendants by css_set_lock

2019-03-16 Thread Roman Gushchin
The number of descendant cgroups and the number of dying descendant cgroups are currently synchronized using the cgroup_mutex. The number of descendant cgroups will be required by the cgroup v2 freezer, which will use it to determine if a cgroup is frozen (depending on total number of descendants

[PATCH v9 4/9] cgroup: cgroup v2 freezer

2019-03-16 Thread Roman Gushchin
Cgroup v1 implements the freezer controller, which provides an ability to stop the workload in a cgroup and temporarily free up some resources (cpu, io, network bandwidth and, potentially, memory) for some other tasks. Cgroup v2 lacks this functionality. This patch implements freezer for cgroup

[PATCH v9 6/9] kselftests: cgroup: add freezer controller self-tests

2019-03-16 Thread Roman Gushchin
This patch implements 9 tests for the freezer controller for cgroup v2: 1) a simple test, which aims to freeze and unfreeze a cgroup with 100 processes 2) a more complicated tree test, which creates a hierarchy of cgroups, puts some processes in some cgroups, and tries to freeze and unfreeze

[PATCH v9 2/9] cgroup: implement __cgroup_task_count() helper

2019-03-16 Thread Roman Gushchin
The helper is identical to the existing cgroup_task_count() except it doesn't take the css_set_lock by itself, assuming that the caller does. Also, move cgroup_task_count() implementation into kernel/cgroup/cgroup.c, as there is nothing specific to cgroup v1. Signed-off-by: Roman Gushchin Cc:

[PATCH v9 5/9] kselftests: cgroup: don't fail on cg_kill_all() error in cg_destroy()

2019-03-16 Thread Roman Gushchin
If the cgroup destruction races with an exit() of a belonging process(es), cg_kill_all() may fail. It's not a good reason to make cg_destroy() fail and leave the cgroup in place, potentially causing next test runs to fail. Signed-off-by: Roman Gushchin Cc: Shuah Khan Cc: Tejun Heo Cc:

[PATCH v9 0/9] freezer for cgroup v2

2019-03-16 Thread Roman Gushchin
This patchset implements freezer for cgroup v2. It provides similar functionality as v1 freezer, but the interface conforms to the cgroup v2 interface design principles, and it provides a better user experience: tasks can be killed, ptrace works, there is no separate controller, which has to be

[PATCH v9 8/9] cgroup: add tracing points for cgroup v2 freezer

2019-03-16 Thread Roman Gushchin
Add cgroup:cgroup_freeze and cgroup:cgroup_unfreeze events, which are using the existing cgroup tracing infrastructure. Add the cgroup_event event class, which is similar to the cgroup class, but contains an additional integer field to store a new value (the level field is dropped). Also add two

[PATCH v9 1/9] cgroup: rename freezer.c into legacy_freezer.c

2019-03-16 Thread Roman Gushchin
Freezer.c will contain an implementation of cgroup v2 freezer, so let's rename the v1 freezer to avoid naming conflicts. Signed-off-by: Roman Gushchin Cc: Tejun Heo Cc: kernel-t...@fb.com --- kernel/cgroup/Makefile| 2 +- kernel/cgroup/{freezer.c => legacy_freezer.c} |

Re: [PATCH v5 05/11] staging: iio: ad7780: set pattern values and masks directly

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:14:14 -0300 Renato Lui Geh wrote: > The AD7780 driver contains status pattern bits designed for checking > whether serial transfers have been correctly performed. Pattern macros > were previously generated through bit fields. This patch sets good > pattern values directly

Re: [PATCH v5 04/11] staging: iio: ad7780: add filter reading to ad778x

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:13:42 -0300 Renato Lui Geh wrote: > This patch adds the new feature of reading the filter odr value for > ad778x chips. This value is stored in the chip's state struct whenever a > read or write call is performed on the chip's driver. > > This feature requires sharing

Re: [PATCH v5 03/11] staging: iio: ad7780: add gain reading to ad778x

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:13:13 -0300 Renato Lui Geh wrote: > This patch adds a new functionality of reading gain values from the > ad778x chips. This value is stored in the chip's state struct and is > updated whenever a read or write call is performed on the driver. > > Signed-off-by: Renato Lui

Re: [PATCH v5 02/11] staging: iio: ad7780: add missing switch default case

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:12:53 -0300 Renato Lui Geh wrote: > This patch simply adds a missing switch default case in read_raw. > > Signed-off-by: Renato Lui Geh Applied, Thanks, Jonathan > --- > drivers/staging/iio/adc/ad7780.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git

Re: [PATCH v5 01/11] staging: iio: ad7780: add gain & filter gpio support

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 23:12:27 -0300 Renato Lui Geh wrote: > Previously, the AD7780 driver only supported gpio for the 'powerdown' > pin. This commit adds suppport for the 'gain' and 'filter' pin. > > Signed-off-by: Renato Lui Geh > Signed-off-by: Giuliano Belinassi > Co-developed-by: Giuliano

Re: [RFC] simple_lmk: Introduce Simple Low Memory Killer for Android

2019-03-16 Thread Suren Baghdasaryan
On Fri, Mar 15, 2019 at 11:49 AM Joel Fernandes wrote: > > On Fri, Mar 15, 2019 at 07:24:28PM +0100, Christian Brauner wrote: > [..] > > > why do we want to add a new syscall (pidfd_wait) though? Why not just use > > > standard poll/epoll interface on the proc fd like Daniel was suggesting. > > >

Re: [PATCH 2/2] dt-bindings: iio: light: Document the VCNL4xx0 device tree bindings

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 14:54:25 -0700 "Angus Ainslie (Purism)" wrote: > devicetree hooks where added to the VCNL4xx0 light and proximity > sensor driver. > > Signed-off-by: Angus Ainslie (Purism) Please avoid wild cards in naming. It goes wrong too often. People can grep if they want to see if

Re: [PATCH 1/2] iio: light: vcnl4000 add support for the VCNL4040 proximity and light sensor

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 14:54:24 -0700 "Angus Ainslie (Purism)" wrote: > The VCNL4040 is almost identical to the VCNL4200 as far as register > layout goes but just need to check a different ID register location. > > This does change the initialization sequence of the VCNL4200 to use word > writes

Re: [PATCH 2/2] staging: iio: adc: ad7192: Convert platform data to DT properties

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 13:29:03 +0200 Mircea Caprioru wrote: > This patch will remove platform data members and replace them with device > tree properties. These properties will be subject to further modifications > and probably replaced with other functionalities at some point in time. From the

Re: [PATCH 2/2] staging: iio: adc: ad7192: Report error directly in ad7192_setup

2019-03-16 Thread Jonathan Cameron
On Sat, 16 Mar 2019 16:45:19 + Jonathan Cameron wrote: > On Fri, 15 Mar 2019 13:14:25 +0200 > Mircea Caprioru wrote: > > > This patch removes the goto out statement thus ensuring that an error is > > report as soon as it occurs. This gives more information and improves > > readability. >

Re: [PATCH 1/2] staging: iio: adc: ad7192: Use DT clock binding

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 13:29:02 +0200 Mircea Caprioru wrote: > This patch replaces the platform data clock select member with DT clock > binding. Through the DT the external clock binding is specified. If this is > not provided then the device will use the internal clock source. > > With the

Re: [PATCH 2/2] staging: iio: adc: ad7192: Report error directly in ad7192_setup

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 13:14:25 +0200 Mircea Caprioru wrote: > This patch removes the goto out statement thus ensuring that an error is > report as soon as it occurs. This gives more information and improves > readability. No problem with the patch, but I'm not sure why it gives more information?

Re: [PATCH 1/2] staging: iio: adc: ad7192: Fix identation

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 13:14:24 +0200 Mircea Caprioru wrote: > This patch fixes the odd indentation inside function ad7192_calibrate_all. > > Signed-off-by: Mircea Caprioru Applied to the togreg branch of iio.git and pushed out as testing Thanks. Jonathan > --- >

Re: [PATCH] iio: adc: fix indentation issue, remove extra tab

2019-03-16 Thread Jonathan Cameron
On Fri, 15 Mar 2019 09:25:38 + "Popa, Stefan Serban" wrote: > On Jo, 2019-03-14 at 23:26 +, Colin King wrote: > > [External] > > > > > > From: Colin Ian King > > > > A return statement is indented one level too deeply; clean this > > up by removing a tab. > > > > Signed-off-by:

Re: [PATCH] iio: pms7003: sleect IIO_TRIGGERED_BUFFER

2019-03-16 Thread Jonathan Cameron
On Thu, 14 Mar 2019 18:27:35 +0100 Tomasz Duszynski wrote: > On Thu, Mar 14, 2019 at 10:00:52AM +0100, Arnd Bergmann wrote: > > Without IIO_TRIGGERED_BUFFER, this driver fails to link: > > Good catch, thanks (btw, perhaps typo in subject can be fixed before > applying patch). > > Acked-by:

Re: [PATCH] iio: cros_ec: Switch to SPDX identifier.

2019-03-16 Thread Jonathan Cameron
On Wed, 13 Mar 2019 12:41:20 +0100 Enric Balletbo i Serra wrote: > Adopt the SPDX license identifier headers to ease license compliance > management. Also fix MODULE_LICENSE for cros_ec_accel_legacy to match > the SPDX and boiler plate license. > > Signed-off-by: Enric Balletbo i Serra

Re: [PATCH v2 3/4] iio:potentiostat:lmp91000: invert if statement

2019-03-16 Thread Jonathan Cameron
On Sat, 9 Mar 2019 18:46:15 + Jonathan Cameron wrote: > On Fri, 8 Mar 2019 16:46:54 -0300 > Anderson Reis wrote: > > > Invert if statement arms in line 214, in order to make the code cleaner, > > solve these checkpatch.pl CHECKs: > > > > - lmp9100.c:214: CHECK: braces {} should be used

Re: [PATCH] iio: cros_ec: Add kernel-doc for cros_ec_sensors_read_lpc

2019-03-16 Thread Jonathan Cameron
On Wed, 13 Mar 2019 12:40:50 +0100 Enric Balletbo i Serra wrote: > From: Gwendal Grignou > > Document cros_ec_sensors_read_lpc, adding an additional note to explain > that this is the safe function for reading the EC data. > > Signed-off-by: Gwendal Grignou > Signed-off-by: Enric Balletbo i

Re: [PATCH] iio: cros_ec: Drop unnecessary include files

2019-03-16 Thread Jonathan Cameron
On Wed, 13 Mar 2019 12:40:32 +0100 Enric Balletbo i Serra wrote: > From: Guenter Roeck > > The cros_ec sensors drivers do not call any sysfs functions > or use any sysfs defines, and thus do not need to include > linux/sysfs.h. Also, some cros_ec drivers include linux/delay.h > and is not

Re: [PATCH v3] iio: cros_ec: Fix the maths for gyro scale calculation

2019-03-16 Thread Jonathan Cameron
On Wed, 13 Mar 2019 12:40:02 +0100 Enric Balletbo i Serra wrote: > From: Gwendal Grignou > > Calculation did not use IIO_DEGREE_TO_RAD and implemented a variant to > avoid precision loss as we aim a nano value. The offset added to avoid > rounding error, though, doesn't give us a close result

Verifique su correo electrónico

2019-03-16 Thread Administrador de cuenta
Web Admin Notificación de correo electrónico Este mensaje se envía desde nuestro centro de mensajes de administración a todos los usuarios de nuestra cuenta de correo electrónico. Estamos eliminando el acceso de todos nuestros clientes de correo web, su cuenta de correo electrónico se

Re: [PATCH v2 2/2] dt-bindings: iio: imu: adis16480: Document external clock

2019-03-16 Thread Jonathan Cameron
On Mon, 11 Mar 2019 11:46:37 +0200 Stefan Popa wrote: > Add documentation for optional use of external clock. All devices > supported by this driver can work with an external clock in sync mode. > Another mode, called Pulse Per Second (PPS) is supported only by adis1649x > devices. The mode is

Re: [PATCH v2 1/2] iio: imu: adis16480: Add support for external clock

2019-03-16 Thread Jonathan Cameron
On Mon, 11 Mar 2019 11:45:29 +0200 Stefan Popa wrote: > Inertial sensor data collection and processing can be controlled by > configuring one of the DIOx lines as an external clock input. This > option is available for all devices supported by this driver. However, > only adis1649x devices

[PATCH v2] PCI: aardvark: Use LTSSM state to build link training flag

2019-03-16 Thread Remi Pommarel
The PCI_EXP_LNKSTA_LT flag in the emulated root device's PCI_EXP_LNKSTA config register does not reflect the actual link training state and is always cleared. The Link Training and Status State Machine (LTSSM) flag in LMI config register could be used as a link training indicator. Indeed if the

Re: [PATCH] staging: iio: adc: ad7192: Add spaces around minus operator

2019-03-16 Thread Jonathan Cameron
On Mon, 11 Mar 2019 11:31:59 +0300 Dan Carpenter wrote: > On Mon, Mar 11, 2019 at 10:12:48AM +0200, Alexandru Ardelean wrote: > > On Sun, Mar 10, 2019 at 11:23 PM Karen Palacio > > wrote: > > > > > > Add spaces around minus operator to fix readibility. > > > > > > Signed-off-by: Karen Palacio

Re: [PATCH 3/3] iio: adc: xilinx: prevent touching unclocked h/w on remove

2019-03-16 Thread Jonathan Cameron
On Sun, 10 Mar 2019 14:58:26 -0400 Sven Van Asbroeck wrote: > In remove, the clock is disabled before canceling the > delayed work. This means that the delayed work may be > touching unclocked hardware. > > Fix by disabling the clock after the delayed work is > fully canceled. This is

Re: [PATCH 2/3] iio: adc: xilinx: fix potential use-after-free on probe

2019-03-16 Thread Jonathan Cameron
On Sun, 10 Mar 2019 14:58:25 -0400 Sven Van Asbroeck wrote: > If probe errors out after request_irq(), its error path > does not explicitly cancel the delayed work, which may > have been scheduled by the interrupt handler. > > This means the delayed work may still be running when > the core

Re: [PATCH 1/3] iio: adc: xilinx: fix potential use-after-free on remove

2019-03-16 Thread Jonathan Cameron
On Sun, 10 Mar 2019 14:58:24 -0400 Sven Van Asbroeck wrote: > When cancel_delayed_work() returns, the delayed work may still > be running. This means that the core could potentially free > the private structure (struct xadc) while the delayed work > is still using it. This is a potential

Re: [PATCH v2 0/4] Patches to allow consistent mmc / mmcblk numbering w/ device tree

2019-03-16 Thread Russell King - ARM Linux admin
On Sat, Mar 16, 2019 at 01:33:58PM +0100, Marek Vasut wrote: > If you have a FS or partition table there, it does. > If you don't, I agree ... that's a problem. eMMC boot partitions are called mmcblkXbootY, and unless you have more than one eMMC device on the system, they can be found either by

Verifique su correo electrónico

2019-03-16 Thread Administrador de cuenta
Web Admin Notificación de correo electrónico Este mensaje se envía desde nuestro centro de mensajes de administración a todos los usuarios de nuestra cuenta de correo electrónico. Estamos eliminando el acceso de todos nuestros clientes de correo web, su cuenta de correo electrónico se

  1   2   >