Re: [PATCH RFC] sched/deadline: Use the revised wakeup rule for suspending constrained dl tasks

2017-05-11 Thread Daniel Bristot de Oliveira
On 05/04/2017 04:17 PM, Peter Zijlstra wrote: > On Mon, Apr 24, 2017 at 05:18:35PM +0200, Daniel Bristot de Oliveira wrote: >> We have been facing some problems with self-suspending constrained >> deadline tasks. The main reason is that the original CBS was not >> designed for such sort of tasks. >

[PATCHv2] Make initramfs honor CONFIG_DEVTMPFS_MOUNT

2017-05-11 Thread Rob Landley
From: Rob Landley Make initramfs honor CONFIG_DEVTMPFS_MOUNT, move /dev/console open after devtmpfs mount, and update help text. Signed-off-by: Rob Landley --- drivers/base/Kconfig | 14 -- init/main.c | 15 +-- 2 files changed, 13 insertions(+), 16 deleti

Re: [PATCH 4.4 16/60] x86/ioapic: Restore IO-APIC irq_chip retrigger callback

2017-05-11 Thread Ben Hutchings
On Thu, 2017-05-11 at 16:12 +0200, Greg Kroah-Hartman wrote: > 4.4-stable review patch. If anyone has any objections, please let me know. > > -- > > From: Ruslan Ruslichenko > > commit a9b4f08770b415f30f2fb0f8329a370c8f554aa3 upstream. > > commit d32932d02e18 removed the irq_r

Re: [PATCH] mdio: mux: Correct mdio_mux_init error path issues

2017-05-11 Thread Florian Fainelli
On 05/10/2017 08:20 AM, Jon Mason wrote: > There is a potential unnecessary refcount decriment on error path of > put_device(&pb->mii_bus->dev), as it is possible to avoid the > of_mdio_find_bus() call if mux_bus is specified by the calling function. > > The same put_device() is not called in the

Re: [PATCH RFC] sched/deadline: Use the revised wakeup rule for suspending constrained dl tasks

2017-05-11 Thread Daniel Bristot de Oliveira
On 05/04/2017 04:23 PM, Peter Zijlstra wrote: > On Thu, May 04, 2017 at 04:17:21PM +0200, Peter Zijlstra wrote: >> On Mon, Apr 24, 2017 at 05:18:35PM +0200, Daniel Bristot de Oliveira wrote: >>> +static void >>> +update_dl_revised_wakeup(struct sched_dl_entity *dl_se, struct rq *rq) >>> +{ >>> +

Re: [PATCH RFC] sched/deadline: Use the revised wakeup rule for suspending constrained dl tasks

2017-05-11 Thread Daniel Bristot de Oliveira
On 05/04/2017 04:26 PM, Peter Zijlstra wrote: > On Thu, May 04, 2017 at 04:17:21PM +0200, Peter Zijlstra wrote: >> We use two difference CBS rules: >> >> 1) the original CBS rule for implicit deadline tasks; >> 2) the revised CBS rule for constrained deadline tasks. >>> @@ -500,6 +550,14 @@ stati

Threads stuck in zap_pid_ns_processes()

2017-05-11 Thread Guenter Roeck
Hi all, the test program attached below almost always results in one of the child processes being stuck in zap_pid_ns_processes(). When this happens, I can see from test logs that nr_hashed == 2 and init_pids==1, but there is only a single thread left in the pid namespace (the one that is stuck).

Re: [PATCH v6 3/5] test: add new driver_data load tester

2017-05-11 Thread Luis R. Rodriguez
On Thu, May 11, 2017 at 07:46:27PM +0900, AKASHI Takahiro wrote: > Luis, > > On Fri, Apr 28, 2017 at 03:45:35AM +0200, Luis R. Rodriguez wrote: > > > > +To test an async call one could do:: > > > > + > > > > +echo anything > /lib/firmware/test-driver_data.bin > > > > > > Your current shel

Re: [PATCH v9 7/9] coresight: add support for CPU debug module

2017-05-11 Thread Mathieu Poirier
On Tue, May 09, 2017 at 10:50:00AM +0800, Leo Yan wrote: > Coresight includes debug module and usually the module connects with CPU > debug logic. ARMv8 architecture reference manual (ARM DDI 0487A.k) has > description for related info in "Part H: External Debug". > > Chapter H7 "The Sample-based

Re: [PATCH] input: cros_ec_keyb: remove extraneous 'const'

2017-05-11 Thread Dmitry Torokhov
On Thu, May 11, 2017 at 01:48:04PM +0200, Arnd Bergmann wrote: > gcc-7 warns about 'const SIMPLE_DEV_PM_OPS', as that macro already constains > a 'const' keyword: > > drivers/input/keyboard/cros_ec_keyb.c:663:14: error: duplicate 'const' > declaration specifier [-Werror=duplicate-decl-specifier]

Re: [PATCH v9 8/9] arm64: dts: hi6220: register debug module

2017-05-11 Thread Mathieu Poirier
On Tue, May 09, 2017 at 10:50:01AM +0800, Leo Yan wrote: > Bind debug module driver for Hi6220. > > Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier > --- > arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 64 > +++ > 1 file changed, 64 insertions(+) > > diff --git

Re: [PATCH v9 9/9] arm64: dts: qcom: msm8916: Add debug unit

2017-05-11 Thread Mathieu Poirier
On Tue, May 09, 2017 at 10:50:02AM +0800, Leo Yan wrote: > Add debug unit on Qualcomm msm8916 based platforms, including the > DragonBoard 410c board. > > Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier > --- > arch/arm64/boot/dts/qcom/msm8916.dtsi | 32 >

[GIT PULL] arm64 2nd set of updates for 4.12

2017-05-11 Thread Catalin Marinas
Hi Linus, Please pull the arm64 updates below. The mm/vmalloc.c change was acked by Michal Hocko and the arch/arm one by Russell King. Thanks. The following changes since commit 92f66f84d9695d07adf9bc987bbcce4bf9b8e87c: arm64: Fix the DMA mmap and get_sgtable API with DMA_ATTR_FORCE_CONTIGUOUS

Re: [PATCH v9 2/9] doc: Add documentation for Coresight CPU debug

2017-05-11 Thread Mathieu Poirier
On Tue, May 09, 2017 at 10:49:55AM +0800, Leo Yan wrote: > Add detailed documentation for Coresight CPU debug driver, which > contains the info for driver implementation, Mike Leach excellent > summary for "clock and power domain". At the end some examples on how > to enable the debugging functiona

Re: [PATCH] ARM: remove duplicate 'const' annotations'

2017-05-11 Thread Florian Fainelli
On 05/11/2017 04:50 AM, Arnd Bergmann wrote: > gcc-7 warns about some declarations that are more 'const' than necessary: > > arch/arm/mach-at91/pm.c:338:34: error: duplicate 'const' declaration > specifier [-Werror=duplicate-decl-specifier] > static const struct of_device_id const ramc_ids[] __i

Re: [PATCH v9 3/9] doc: Add coresight_cpu_debug.enable to kernel-parameters.txt

2017-05-11 Thread Mathieu Poirier
On Tue, May 09, 2017 at 10:49:56AM +0800, Leo Yan wrote: > Add coresight_cpu_debug.enable to kernel-parameters.txt, this flag is > used to enable/disable the CPU sampling based debugging. > > Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier > --- > Documentation/admin-guide/kernel-paramete

Re: [PATCH] mdio: mux: Correct mdio_mux_init error path issues

2017-05-11 Thread David Miller
From: Florian Fainelli Date: Thu, 11 May 2017 10:05:27 -0700 > On 05/10/2017 08:20 AM, Jon Mason wrote: >> There is a potential unnecessary refcount decriment on error path of >> put_device(&pb->mii_bus->dev), as it is possible to avoid the >> of_mdio_find_bus() call if mux_bus is specified by th

Re: [PATCH v9 4/9] MAINTAINERS: update file entries for Coresight subsystem

2017-05-11 Thread Mathieu Poirier
On Tue, May 09, 2017 at 10:49:57AM +0800, Leo Yan wrote: > Update document file entries for Coresight debug module. > > Signed-off-by: Leo Yan Reviewed-by: Mathieu Poirier > --- > MAINTAINERS | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/MAINTAINERS b/MAINTAINERS > index b948dfa

Re: [PATCH] staging: typec: Fix sparse warnings about incorrect types

2017-05-11 Thread Guenter Roeck
On Wed, May 10, 2017 at 10:51:35PM -0700, Guru Das Srinagesh wrote: > Fix the following sparse warnings about incorrect type usage: > > fusb302.c:1028:32: warning: incorrect type in argument 1 (different base > types) > fusb302.c:1028:32:expected unsigned short [unsigned] [usertype] header >

Re: [drm:qxl] BUG: sleeping function called from invalid context - qxl_bo_kmap_atomic_page()...splat

2017-05-11 Thread Mike Galbraith
On Tue, 2017-05-09 at 04:37 +0200, Mike Galbraith wrote: > On Mon, 2017-05-08 at 16:48 -0300, Gabriel Krisman Bertazi wrote: > > > Thanks for reporting this. Can you confirm the following patch prevents > > the issue? > > Nope, it still gripes. The reason for this gripe is that we find that we

Re: [PATCH 1/2] kbuild: cleanup signing keys with mrproper

2017-05-11 Thread Stephen Hemminger
On Wed, 3 May 2017 13:37:08 +0900 Masahiro Yamada wrote: > +CC David Woodhouse > +CC David Howells > > > 2017-04-15 6:54 GMT+09:00 Stephen Hemminger : > > When 'make mrproper' is run it was supposed to remove the signing > > keys in the certs directory, but only the filename is given > > rather

Re: [RFC][PATCH 00/14] VFS: Introduce superblock configuration context [ver #2]

2017-05-11 Thread Anna Schumaker
Hi David, On 05/11/2017 09:59 AM, David Howells wrote: > > Here are a set of patches to create a superblock configuration context > prior to setting up a new mount, populating it with the parsed > options/binary data, creating the superblock and then effecting the mount. > > This allows namespac

[PATCH] ethernet: aquantia: remove redundant checks on error status

2017-05-11 Thread Colin King
From: Colin Ian King The error status err is initialized as zero and then being checked several times to see if it is less than zero even when it has not been updated. Since these checks are redundant we can remove these as well as err and the error exit label err_exit. Detected by CoverityScan,

Re: [RFC][PATCH] sched/deadline: Remove if statement before clearing throttle and yielded

2017-05-11 Thread Daniel Bristot de Oliveira
On 05/11/2017 04:01 PM, Juri Lelli wrote: > Looks good to me. > > Peter, any particular reason why you wanted to first check the values? +1 -- Daniel

Re: Threads stuck in zap_pid_ns_processes()

2017-05-11 Thread Eric W. Biederman
Guenter Roeck writes: > Hi all, > > the test program attached below almost always results in one of the child > processes being stuck in zap_pid_ns_processes(). When this happens, I can > see from test logs that nr_hashed == 2 and init_pids==1, but there is only > a single thread left in the pid

Re: [PATCH v9 7/9] coresight: add support for CPU debug module

2017-05-11 Thread Suzuki K Poulose
On 09/05/17 03:50, Leo Yan wrote: Coresight includes debug module and usually the module connects with CPU debug logic. ARMv8 architecture reference manual (ARM DDI 0487A.k) has description for related info in "Part H: External Debug". Chapter H7 "The Sample-based Profiling Extension" introduces

Re: [RFC 01/10] x86/mm: Reimplement flush_tlb_page() using flush_tlb_mm_range()

2017-05-11 Thread Borislav Petkov
On Sun, May 07, 2017 at 05:38:30AM -0700, Andy Lutomirski wrote: > flush_tlb_page() was very similar to flush_tlb_mm_range() except that > it had a couple of issues: > > - It was missing an smp_mb() in the case where >current->active_mm != mm. (This is a longstanding bug reported by >Nad

Re: [PATCH 4.10 000/129] 4.10.16-stable review

2017-05-11 Thread Greg Kroah-Hartman
On Thu, May 11, 2017 at 09:59:55AM -0600, Shuah Khan wrote: > On 05/11/2017 08:10 AM, Greg Kroah-Hartman wrote: > > This is the start of the stable review cycle for the 4.10.16 release. > > There are 129 patches in this series, all will be posted as a response > > to this one. If anyone has any is

Re: [PATCH 1/4] vmbus: Improve a size determination in vmbus_device_create()

2017-05-11 Thread Greg KH
On Thu, May 11, 2017 at 09:30:56AM -0700, Stephen Hemminger wrote: > On Thu, 11 May 2017 18:15:46 +0200 > SF Markus Elfring wrote: > > > From: Markus Elfring > > Date: Thu, 11 May 2017 17:30:10 +0200 > > > > Replace the specification of a data structure by a pointer dereference > > as the param

Re: pinctrl-sx150x.c broken in 4.11

2017-05-11 Thread Tony Lindgren
* Nikita Yushchenko [170511 10:01]: > >>> Hmm maybe yeah. I don't quite follow the above the "pinctrl-0 property > >>> of sx150x device tree node, is misinterpreted as hog" part though. > >> > >> sx150x is i2c-gpio device. It has 16 GPIO lines that are communicated > >> with via i2c bus, and an i

Re: [PATCHv2] arm64/cpufeature: don't use mutex in bringup path

2017-05-11 Thread Mark Rutland
On Thu, May 11, 2017 at 05:08:19PM +0100, Marc Zyngier wrote: > On 11/05/17 16:54, Suzuki K Poulose wrote: > > On 11/05/17 16:37, Mark Rutland wrote: > >> On Thu, May 11, 2017 at 04:15:38PM +0100, Suzuki K Poulose wrote: > >>> On 11/05/17 16:01, Mark Rutland wrote: > +static inline bool cpus_h

Re: Kernel modules under new copyleft licence : (was Re: [PATCH v2] module.h: add copyleft-next >= 0.3.1 as GPL compatible)

2017-05-11 Thread Luis R. Rodriguez
Sorry this is an old topic now but a clarification was requested by AKASHI, so following up. On Tue, Aug 09, 2016 at 07:58:27PM -0700, Linus Torvalds wrote: > On Tue, Aug 9, 2016 at 1:14 PM, Luis R. Rodriguez wrote: > > > > I'm personally fine with MODULE_LICENSE("GPL") being used with > > copyl

[PATCH] target: Re-add check to reject control WRITEs with overflow data

2017-05-11 Thread Nicholas A. Bellinger
From: Nicholas Bellinger During v4.3 when the overflow/underflow check was relaxed by commit c72c525022: commit c72c5250224d475614a00c1d7e54a67f77cd3410 Author: Roland Dreier Date: Wed Jul 22 15:08:18 2015 -0700 target: allow underflow/overflow for PR OUT etc. commands to allow

[PULL] A few small documentation updates

2017-05-11 Thread Jonathan Corbet
The following changes since commit 4a1e31c68e9f40be32838944931178b0d9ed9162: Merge tag 'arc-4.12-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/vgupta/arc (2017-05-09 10:10:15 -0700) are available in the git repository at: git://git.lwn.net/linux.git tags/docs-4.12-2 for you to fet

Re: [PATCH 4.10 000/129] 4.10.16-stable review

2017-05-11 Thread Greg Kroah-Hartman
On Thu, May 11, 2017 at 07:45:55PM +0200, Greg Kroah-Hartman wrote: > On Thu, May 11, 2017 at 09:59:55AM -0600, Shuah Khan wrote: > > On 05/11/2017 08:10 AM, Greg Kroah-Hartman wrote: > > > This is the start of the stable review cycle for the 4.10.16 release. > > > There are 129 patches in this ser

Re: [PATCH v6 3/5] test: add new driver_data load tester

2017-05-11 Thread Luis R. Rodriguez
On Thu, May 11, 2017 at 07:46:27PM +0900, AKASHI Takahiro wrote: > On Fri, Apr 28, 2017 at 03:45:35AM +0200, Luis R. Rodriguez wrote: > > > > diff --git a/lib/test_driver_data.c b/lib/test_driver_data.c > > > > new file mode 100644 > > > > index ..11175a3b9f0a > > > > --- /dev/null > >

Re: [PATCH] ethernet: aquantia: remove redundant checks on error status

2017-05-11 Thread David Miller
From: Colin King Date: Thu, 11 May 2017 18:29:29 +0100 > From: Colin Ian King > > The error status err is initialized as zero and then being checked > several times to see if it is less than zero even when it has not > been updated. Since these checks are redundant we can remove these > as well

Re: [PATCH v4 2/2] x86/refcount: Implement fast refcount overflow protection

2017-05-11 Thread Kees Cook
On Wed, May 10, 2017 at 6:24 PM, PaX Team wrote: > On 9 May 2017 at 12:01, Kees Cook wrote: >> Various differences from PaX: >> - uses earlier value reset implementation in assembly >> - uses UD0 and refcount exception handler instead of new int vector >> - uses .text.unlikely instead of custom na

Re: [PATCH v7 1/5] firmware: add extensible driver data params

2017-05-11 Thread Li, Yi
On 5/2/2017 3:49 AM, Luis R. Rodriguez wrote: As the firmware API evolves we keep extending functions with more arguments. Stop this nonsense by proving an extensible data structure which can be used to represent both user parameters and private internal parameters. We introduce 3 data structu

Re: [PATCH] ethernet: aquantia: remove redundant checks on error status

2017-05-11 Thread Colin Ian King
On 11/05/17 19:16, David Miller wrote: > From: Colin King > Date: Thu, 11 May 2017 18:29:29 +0100 > >> From: Colin Ian King >> >> The error status err is initialized as zero and then being checked >> several times to see if it is less than zero even when it has not >> been updated. Since these c

Re: [PATCH] bluetooth: hci_ll: add proper SERDEV dependency

2017-05-11 Thread Sebastian Reichel
Hi, On Thu, May 11, 2017 at 01:41:54PM +0200, Arnd Bergmann wrote: > When SERDEV is a loadable module, and the HCI code is built-in, we > get a link error: > > drivers/bluetooth/built-in.o: In function `ll_close': > hci_ll.c:(.text+0x278d): undefined reference to `serdev_device_close' > drivers/b

Re: [PATCH] Bluetooth: hci_nokia: select CONFIG_BT_HCIUART_H4

2017-05-11 Thread Sebastian Reichel
Hi Arnd, On Thu, May 11, 2017 at 02:11:26PM +0200, Arnd Bergmann wrote: > Like the other H4 variants, this only builds when the base > H4 support is enabled, otherwise we get a build error: > > drivers/bluetooth/hci_nokia.c:624:33: error: array type has incomplete > element type 'struct h4_recv_

[PATCH net] net: phy: Call bus->reset() after releasing PHYs from reset

2017-05-11 Thread Florian Fainelli
The API convention makes it that a given MDIO bus reset should be able to access PHY devices in its reset() callback and perform additional MDIO accesses in order to bring the bus and PHYs in a working state. Commit 69226896ad63 ("mdio_bus: Issue GPIO RESET to PHYs.") broke that contract by first

[PATCH v4] gpiolib: Add stubs for gpiod lookup table interface

2017-05-11 Thread Anatolij Gustschin
Add stubs for gpiod_add_lookup_table() and gpiod_remove_lookup_table() for the !GPIOLIB case to prevent build errors. Signed-off-by: Anatolij Gustschin --- Changes in v4: - move stubs to gpio/machine.h Changes in v3: - add stubs for !GPIOLIB case. Drop prototypes, these are already in gpio/

Re: [PATCH v6 3/5] test: add new driver_data load tester

2017-05-11 Thread Luis R. Rodriguez
On Thu, May 11, 2017 at 07:46:27PM +0900, AKASHI Takahiro wrote: > On Fri, Apr 28, 2017 at 03:45:35AM +0200, Luis R. Rodriguez wrote: > > > > diff --git a/tools/testing/selftests/firmware/driver_data.sh > > > > b/tools/testing/selftests/firmware/driver_data.sh > > ... > > > > > > +TEST_NAME="driv

Re: [PATCH v7 1/5] firmware: add extensible driver data params

2017-05-11 Thread Luis R. Rodriguez
On Thu, May 11, 2017 at 01:17:39PM -0500, Li, Yi wrote: > On 5/2/2017 3:49 AM, Luis R. Rodriguez wrote: > > As the firmware API evolves we keep extending functions with more arguments. > > Stop this nonsense by proving an extensible data structure which can be used > > to represent both user parame

[PATCH][V2] ethernet: aquantia: remove redundant checks on error status

2017-05-11 Thread Colin King
From: Colin Ian King The error status err is initialized as zero and then being checked several times to see if it is less than zero even when it has not been updated. It may seem that the err should be assigned to the return code of the call to the various *offload_en_set calls and then we chec

[PATCH v2 3/7] drm/vc4: Switch DPI to using the panel-bridge helper.

2017-05-11 Thread Eric Anholt
Another 100 lines of boilerplate gone, while allowing for bridges to be connected in the display chain. Signed-off-by: Eric Anholt --- drivers/gpu/drm/vc4/vc4_dpi.c | 164 -- 1 file changed, 30 insertions(+), 134 deletions(-) diff --git a/drivers/gpu/drm/

[PATCH v2 7/7] drm/atmel-hlcdc: Replace the panel usage with drm_panel_bridge.

2017-05-11 Thread Eric Anholt
This cuts 135 lines of boilerplate, at the cost of losing the filtering of get_modes() using atmel_hlcdc_dc_mode_valid(). The atomic check will still check that we don't set an invalid mode, though. Signed-off-by: Eric Anholt --- This patch is just a proposal for atmel-hlcdc if you like it -- m

[PATCH v2 1/7] drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.

2017-05-11 Thread Eric Anholt
Many DRM drivers have common code to make a stub connector implementation that wraps a drm_panel. By wrapping the panel in a DRM bridge, all of the connector code (including calls during encoder enable/disable) goes away. v2: Fix build with CONFIG_DRM=m, drop "dev" argument that should just b

[PATCH v2 4/7] drm/mediatek: Use the panel-bridge helper.

2017-05-11 Thread Eric Anholt
Avoids a bunch of connector boilerplate. Note that this causes panel prepare() to be moved before mtk_dsi_poweron() and unprepare() to be after poweroff(). I think this is the expected usage of the panel API (enable should be when you do things that require the link to be brought up), but there m

[PATCH v2 2/7] drm/vc4: Switch DSI to the panel-bridge layer, and support bridges.

2017-05-11 Thread Eric Anholt
The newer version of the RPi panel driver is going to be a combination of a bridge and a panel, but we should also support panels without a bridge, so the panel-bridge layer lets us do that cleanly. v2: Drop "dev" argument. Signed-off-by: Eric Anholt Acked-by: Daniel Vetter --- drivers/gpu/drm

[PATCH v2 6/7] drm/atmel-hlcdc: Drop custom encoder cleanup func.

2017-05-11 Thread Eric Anholt
drm_encoder_cleanup() finishes with memsetting it to 0, already. Signed-off-by: Eric Anholt --- drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c b/drivers/gpu/drm/atmel-hl

[PATCH v2 5/7] drm/ltdc: Use the panel-bridge helper.

2017-05-11 Thread Eric Anholt
Another 100 lines of boilerplate gone. Bridges aren't supported yet, but will be trivial to add later. Signed-off-by: Eric Anholt --- Also untested. drivers/gpu/drm/stm/ltdc.c | 128 + drivers/gpu/drm/stm/ltdc.h | 2 +- 2 files changed, 13 inserti

Re: Threads stuck in zap_pid_ns_processes()

2017-05-11 Thread Guenter Roeck
On Thu, May 11, 2017 at 12:31:21PM -0500, Eric W. Biederman wrote: > Guenter Roeck writes: > > > Hi all, > > > > the test program attached below almost always results in one of the child > > processes being stuck in zap_pid_ns_processes(). When this happens, I can > > see from test logs that nr_h

Re: [PATCH] mtd: nand: make nand_ooblayout_lp_hamming_ops static

2017-05-11 Thread Brian Norris
On Thu, May 04, 2017 at 04:08:13PM +0200, Boris Brezillon wrote: > On Thu, 4 May 2017 13:11:00 +0100 > Colin King wrote: > > > From: Colin Ian King > > > > nand_ooblayout_lp_hamming_ops can be made static as it does not need to be > > in global scope. > > > > Signed-off-by: Colin Ian King >

Re: [PATCH v2 1/3] kvm: x86: Add a hook for arch specific dirty logging emulation

2017-05-11 Thread Bandan Das
"Huang, Kai" writes: ... > Hi Bandan, > > I was just suggesting. You and Paolo still make the decision :) Sure Kai, I don't mind the name change at all. The maintainer has already picked this up and I don't think the function name change is worth submitting a follow up. Thank you very much for t

Re: [PATCH v6 3/5] test: add new driver_data load tester

2017-05-11 Thread Luis R. Rodriguez
On Thu, May 11, 2017 at 11:26 AM, Luis R. Rodriguez wrote: > > It would seems to make sense to me to only need to verify files when read > for the first time, once its cache I don't see why we would re-verify them ? To be clear, the fw cache feature reads the files from the fs prior to suspend, a

[GIT PULL] target updates for v4.12-rc1

2017-05-11 Thread Nicholas A. Bellinger
Hi Linus, Here are the target-pending updates for v4.12-rc1. Please go ahead and pull from: git://git.kernel.org/pub/scm/linux/kernel/git/nab/target-pending.git for-next Things where alot more calm than previously expected. It's primarily fixes in various areas, with most of the new function

Re: [PATCH 1/3] f2fs: split wio_mutex

2017-05-11 Thread Jaegeuk Kim
On 05/09, Chao Yu wrote: > From: Chao Yu > > Split wio_mutex to adjust different temperature bio cache. This can be rephrased like: Signed-off-by: Chao Yu Signed-off-by: Jaegeuk Kim --- fs/f2fs/f2fs.h| 3 ++- fs/f2fs/segment.c | 4 ++-- fs/f2fs/super.c | 7 --- 3 files changed, 8 i

Re: [PATCH 3/3] f2fs: introduce io_list for serialize data/node IOs

2017-05-11 Thread Jaegeuk Kim
Hi Chao, On 05/09, Chao Yu wrote: > From: Chao Yu > > Serialize data/node IOs by using fifo list instead of mutex lock, > it will help to enhance concurrency of f2fs, meanwhile keeping LFS > IO semantics. I'm not against to give it a try, but not sure how much we can get a benefit from this app

Re: [PATCH] x86/efi: Add EFI_PGT_DUMP support for x86_32, kexec and efi=old_map

2017-05-11 Thread Sai Praneeth Prakhya
On Thu, 2017-05-11 at 13:18 +0200, Borislav Petkov wrote: > On Wed, May 10, 2017 at 03:49:05PM -0700, Sai Praneeth Prakhya wrote: > > From: Sai Praneeth > > > > EFI_PGT_DUMP, as the name suggests dumps efi page tables to dmesg during > > kernel boot. This feature is very useful while debugging pa

Re: [PATCH 1/9] mtd: cfi_cmdset_0020: Drop unnecessary static

2017-05-11 Thread Brian Norris
On Thu, May 04, 2017 at 10:10:46PM +0200, Julia Lawall wrote: > Drop static on a local variable, when the variable is initialized before > any use on every possible execution path through the function. The static > has no benefit, and dropping it reduces the code size. > > The semantic patch that

Re: [PATCH] usb-musb: keep VBUS on when device is disconnected

2017-05-11 Thread Bin Liu
On Mon, Mar 27, 2017 at 10:55:37AM -0700, Tony Lindgren wrote: > * Bin Liu [170327 10:17]: > > On Mon, Mar 27, 2017 at 09:59:47AM -0700, Tony Lindgren wrote: > > > * Moreno Bartalucci [170327 09:23]: > > > > If I understood your patch, however, if the device (anyone, not just my > > > > one) tak

Re: [PATCH] smp_call_function: use inline helpers instead of macros

2017-05-11 Thread kbuild test robot
Hi Arnd, [auto build test ERROR on linus/master] [also build test ERROR on v4.11 next-20170511] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Arnd-Bergmann/smp_call_function-use-inline-helpers

Re: [PATCH] usb-musb: keep VBUS on when device is disconnected

2017-05-11 Thread Tony Lindgren
* Bin Liu [170511 11:53]: > On Mon, Mar 27, 2017 at 10:55:37AM -0700, Tony Lindgren wrote: > > * Bin Liu [170327 10:17]: > > > On Mon, Mar 27, 2017 at 09:59:47AM -0700, Tony Lindgren wrote: > > > > * Moreno Bartalucci [170327 09:23]: > > > > > If I understood your patch, however, if the device (

Re: [RFC] adding of TGID to ftrace output

2017-05-11 Thread Joel Fernandes
On Thu, May 11, 2017 at 7:23 AM, Steven Rostedt wrote: [..] >> To fix the suggestion I made in the very beginning of this email, I >> suggest a combination of trace_find_cmdline type of approach to save >> the tgid of all current threads available, and then using >> trace_sched_fork to find out wh

Re: [RFC PATCH v1 00/30] fs: inode->i_version rework and optimization

2017-05-11 Thread J. Bruce Fields
On Wed, Apr 05, 2017 at 02:14:09PM -0400, J. Bruce Fields wrote: > On Wed, Apr 05, 2017 at 10:05:51AM +0200, Jan Kara wrote: > > 1) Keep i_version as is, make clients also check for i_ctime. > > That would be a protocol revision, which we'd definitely rather avoid. > > But can't we accomplish the

Re: [PATCH] usb-musb: keep VBUS on when device is disconnected

2017-05-11 Thread Bin Liu
On Thu, May 11, 2017 at 11:55:28AM -0700, Tony Lindgren wrote: > * Bin Liu [170511 11:53]: > > On Mon, Mar 27, 2017 at 10:55:37AM -0700, Tony Lindgren wrote: > > > * Bin Liu [170327 10:17]: > > > > On Mon, Mar 27, 2017 at 09:59:47AM -0700, Tony Lindgren wrote: > > > > > * Moreno Bartalucci [1703

Re: [PATCH v4] gpiolib: Add stubs for gpiod lookup table interface

2017-05-11 Thread Andy Shevchenko
On Thu, May 11, 2017 at 9:24 PM, Anatolij Gustschin wrote: > Add stubs for gpiod_add_lookup_table() and gpiod_remove_lookup_table() > for the !GPIOLIB case to prevent build errors. > Thanks! FWIW: Reviewed-by: Andy Shevchenko It will fix this kbuild bot complain: https://lkml.org/lkml/2017/4/2

Re: [PATCH] usb-musb: keep VBUS on when device is disconnected

2017-05-11 Thread Bin Liu
On Thu, May 11, 2017 at 02:01:00PM -0500, Bin Liu wrote: > On Thu, May 11, 2017 at 11:55:28AM -0700, Tony Lindgren wrote: > > * Bin Liu [170511 11:53]: > > > On Mon, Mar 27, 2017 at 10:55:37AM -0700, Tony Lindgren wrote: > > > > * Bin Liu [170327 10:17]: > > > > > On Mon, Mar 27, 2017 at 09:59:47

Re: [PATCH v2 1/2] net: Added mtu parameter to dev_forward_skb calls

2017-05-11 Thread Fredrik Markström
On Thu, May 11, 2017 at 6:01 PM, Stephen Hemminger wrote: > On Thu, 11 May 2017 15:46:27 +0200 > Fredrik Markstrom wrote: > >> From: Fredrik Markström >> >> is_skb_forwardable() currently checks if the packet size is <= mtu of >> the receiving interface. This is not consistent with most of the h

Re: pinctrl-sx150x.c broken in 4.11

2017-05-11 Thread Nikita Yushchenko
>>> Maybe create_pinctrl() could check if the pin controller device >>> for a potential hog points to the device itself and bail out >>> if that's not the case? >> >> Well that's exactly what patch from my first mail in this thread does. >> This indeed fixes my case, but I don't know if it is corre

Re: [PATCH v5 15/17] dt-bindings: qca7000: append UART interface to binding

2017-05-11 Thread Michael Heimpold
Hi, Am Mittwoch, 10. Mai 2017, 10:53:26 CEST schrieb Stefan Wahren: > This merges the serdev binding for the QCA7000 UART driver (Ethernet over > UART) into the existing document. > > Signed-off-by: Stefan Wahren > --- > .../devicetree/bindings/net/qca-qca7000.txt| 32 >

Re: [PATCH v2 7/7] drm/atmel-hlcdc: Replace the panel usage with drm_panel_bridge.

2017-05-11 Thread Boris Brezillon
Hi Eric, On Thu, 11 May 2017 11:31:28 -0700 Eric Anholt wrote: > This cuts 135 lines of boilerplate, at the cost of losing the > filtering of get_modes() using atmel_hlcdc_dc_mode_valid(). The > atomic check will still check that we don't set an invalid mode, > though. Nice. > > Signed-off-b

Re: [PATCH v2 6/7] drm/atmel-hlcdc: Drop custom encoder cleanup func.

2017-05-11 Thread Boris Brezillon
On Thu, 11 May 2017 11:31:27 -0700 Eric Anholt wrote: > drm_encoder_cleanup() finishes with memsetting it to 0, already. > > Signed-off-by: Eric Anholt Acked-by: Boris Brezillon > --- > drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_output.c | 8 +--- > 1 file changed, 1 insertion(+), 7 deleti

Re: [PATCH] usb-musb: keep VBUS on when device is disconnected

2017-05-11 Thread Bin Liu
On Thu, May 11, 2017 at 02:10:05PM -0500, Bin Liu wrote: [...] > > > > The otg state machine implementation in the musb drivers are kind of > > > > strange. > > > > OTG_STATE_A_WAIT_BCON suppose to be a steady state when no usb device is > > > > attached, but the musb drivers use it as a transient

[PATCH V2] x86/efi: Add EFI_PGT_DUMP support for x86_32, kexec

2017-05-11 Thread Sai Praneeth Prakhya
From: Sai Praneeth EFI_PGT_DUMP, as the name suggests dumps efi page tables to dmesg during kernel boot. This feature is very useful while debugging page faults/null pointer dereferences to efi related addresses. Presently, this feature is limited only to x86_64, so let's extend it to other efi c

Re: [RFC][PATCH 00/14] VFS: Introduce superblock configuration context [ver #2]

2017-05-11 Thread David Howells
Anna Schumaker wrote: > Is there any way to split the NFS patch into multiple pieces? Are you okay with a patch or two that add code that is unconnected in that patch, but connected in a later one? David

[PATCH] mm, docs: update memory.stat description with workingset* entries

2017-05-11 Thread Roman Gushchin
Commit 4b4cea91691d ("mm: vmscan: fix IO/refault regression in cache workingset transition") introduced three new entries in memory stat file: - workingset_refault, - workingset_activate, - workingset_nodereclaim. This commit adds a corresponding description to the cgroup v2 docs.

[PATCH] mm: per-cgroup memory reclaim stats

2017-05-11 Thread Roman Gushchin
Track the following reclaim counters for every memory cgroup: PGREFILL, PGSCAN, PGSTEAL, PGACTIVATE, PGDEACTIVATE, PGLAZYFREE and PGLAZYFREED. These values are exposed using the memory.stats interface of cgroup v2. The meaning of each value is the same as for global counters, available using /pro

Re: [PATCH] usb-musb: keep VBUS on when device is disconnected

2017-05-11 Thread Tony Lindgren
* Bin Liu [170511 12:23]: > On Thu, May 11, 2017 at 02:10:05PM -0500, Bin Liu wrote: > [...] > > > > > The otg state machine implementation in the musb drivers are kind of > > > > > strange. > > > > > OTG_STATE_A_WAIT_BCON suppose to be a steady state when no usb device > > > > > is > > > > > at

[PATCH v8 0/5] skb_to_sgvec hardening

2017-05-11 Thread Jason A. Donenfeld
The recent bug with macsec and historical one with virtio have indicated that letting skb_to_sgvec trounce all over an sglist without checking the length is probably a bad idea. And it's not necessary either: an sglist already explicitly marks its last item, and the initialization functions are dil

[PATCH v8 2/5] ipsec: check return value of skb_to_sgvec always

2017-05-11 Thread Jason A. Donenfeld
Signed-off-by: Jason A. Donenfeld Cc: Steffen Klassert Cc: Herbert Xu Cc: "David S. Miller" --- net/ipv4/ah4.c | 8 ++-- net/ipv4/esp4.c | 20 +--- net/ipv6/ah6.c | 8 ++-- net/ipv6/esp6.c | 20 +--- 4 files changed, 38 insertions(+), 18 deletions(-)

[PATCH v8 4/5] macsec: check return value of skb_to_sgvec always

2017-05-11 Thread Jason A. Donenfeld
Signed-off-by: Jason A. Donenfeld Cc: Sabrina Dubroca --- drivers/net/macsec.c | 13 +++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/drivers/net/macsec.c b/drivers/net/macsec.c index cdc347be68f2..dfcb1e9d2ab2 100644 --- a/drivers/net/macsec.c +++ b/drivers/net/macse

[PATCH v8 3/5] rxrpc: check return value of skb_to_sgvec always

2017-05-11 Thread Jason A. Donenfeld
Signed-off-by: Jason A. Donenfeld Cc: David Howells --- net/rxrpc/rxkad.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/net/rxrpc/rxkad.c b/net/rxrpc/rxkad.c index 1bb9b2ccc267..ecab9334e3c1 100644 --- a/net/rxrpc/rxkad.c +++ b/net/rxrpc/rxkad.c @@ -227,7 +22

[PATCH v8 1/5] skbuff: return -EMSGSIZE in skb_to_sgvec to prevent overflow

2017-05-11 Thread Jason A. Donenfeld
This is a defense-in-depth measure in response to bugs like 4d6fa57b4dab ("macsec: avoid heap overflow in skb_to_sgvec"). There's not only a potential overflow of sglist items, but also a stack overflow potential, so we fix this by limiting the amount of recursion this function is allowed to do. No

[PATCH v8 5/5] virtio_net: check return value of skb_to_sgvec always

2017-05-11 Thread Jason A. Donenfeld
Signed-off-by: Jason A. Donenfeld Cc: "Michael S. Tsirkin" Cc: Jason Wang --- drivers/net/virtio_net.c | 9 +++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/drivers/net/virtio_net.c b/drivers/net/virtio_net.c index 9320d96a1632..13fbe4b349c2 100644 --- a/drivers/net/virti

Re: [PATCH v2 1/2] net: Added mtu parameter to dev_forward_skb calls

2017-05-11 Thread Stephen Hemminger
On Thu, 11 May 2017 21:10:11 +0200 Fredrik Markström wrote: > On Thu, May 11, 2017 at 6:01 PM, Stephen Hemminger > wrote: > > On Thu, 11 May 2017 15:46:27 +0200 > > Fredrik Markstrom wrote: > > > >> From: Fredrik Markström > >> > >> is_skb_forwardable() currently checks if the packet size is

Re: [PATCH v5 13/17] dt-bindings: qca7000: rename binding

2017-05-11 Thread Michael Heimpold
Hi, Am Mittwoch, 10. Mai 2017, 10:53:24 CEST schrieb Stefan Wahren: > Before we can merge the QCA7000 UART binding the document needs to be > renamed. > > Signed-off-by: Stefan Wahren > --- > .../devicetree/bindings/net/{qca-qca7000-spi.txt => qca-qca7000.txt} | > 0 1 file changed, 0 inser

Re: [RFC][PATCH] sched/deadline: Remove if statement before clearing throttle and yielded

2017-05-11 Thread Peter Zijlstra
On Thu, May 11, 2017 at 03:01:16PM +0100, Juri Lelli wrote: > > - if (dl_se->dl_yielded) > > - dl_se->dl_yielded = 0; > > - if (dl_se->dl_throttled) > > - dl_se->dl_throttled = 0; > > + dl_se->dl_yielded = 0; > > + dl_se->dl_throttled = 0; > > } > > Looks good to me.

Re: [PATCH][V2] ethernet: aquantia: remove redundant checks on error status

2017-05-11 Thread Lino Sanfilippo
Hi, On 11.05.2017 20:29, Colin King wrote: > From: Colin Ian King > > The error status err is initialized as zero and then being checked > several times to see if it is less than zero even when it has not > been updated. It may seem that the err should be assigned to the > return code of the ca

[kernel-locking] question about structure field initialization

2017-05-11 Thread Gustavo A. R. Silva
Hello everybody, While looking into Coverity ID 1402035 I ran into the following piece of code at kernel/locking/test-ww_mutex.c:197: 197static int test_abba(bool resolve) 198{ 199struct test_abba abba; 200struct ww_acquire_ctx ctx; 201int err, ret; 202 203ww_

Re: [RFC 04/10] x86/mm: Pass flush_tlb_info to flush_tlb_others() etc

2017-05-11 Thread Nadav Amit
> On May 7, 2017, at 5:38 AM, Andy Lutomirski wrote: > > @@ -243,15 +237,15 @@ static void flush_tlb_func(void *info) > return; > } > > - if (f->flush_end == TLB_FLUSH_ALL) { > + if (f->end == TLB_FLUSH_ALL) { > local_flush_tlb(); > trace_

[PATCH] pinctrl: when claiming hog, skip maps not served by same device

2017-05-11 Thread Nikita Yushchenko
When pinctrl device registers, it automatically claims hogs, that is, maps that pinctrl device serves for itself. It is possible that in addition to SoC's pinctrl device, other pinctrl devices get registered. E.g. some gpio expander devies are registered as pinctrl devices. For such devices, pinct

Re: [PATCH][V2] ethernet: aquantia: remove redundant checks on error status

2017-05-11 Thread Pavel Belous
On 11.05.2017 21:29, Colin King wrote: From: Colin Ian King The error status err is initialized as zero and then being checked several times to see if it is less than zero even when it has not been updated. It may seem that the err should be assigned to the return code of the call to the var

Re: [PATCH] usb-musb: keep VBUS on when device is disconnected

2017-05-11 Thread Bin Liu
On Thu, May 11, 2017 at 12:38:11PM -0700, Tony Lindgren wrote: > * Bin Liu [170511 12:23]: > > On Thu, May 11, 2017 at 02:10:05PM -0500, Bin Liu wrote: > > [...] > > > > > > The otg state machine implementation in the musb drivers are kind > > > > > > of strange. > > > > > > OTG_STATE_A_WAIT_BCON

Re: [PATCH] FS: Fixing return type of unsigned_offsets

2017-05-11 Thread Luc Van Oostenryck
On Thu, May 11, 2017 at 8:07 AM, Al Viro wrote: > FVO"needed" equal to "needed to make sparse STFU"? If anything, that's > sparse being wrong - evaluate.c:check_assignment_type() should do > if (t == &ctype_bool) { > if (is_fouled_type(s)) >

Re: Lockdep splat involving all_q_mutex

2017-05-11 Thread Jens Axboe
On 05/10/2017 09:13 PM, Paul E. McKenney wrote: > On Wed, May 10, 2017 at 08:55:54PM -0600, Jens Axboe wrote: >> On 05/10/2017 04:34 PM, Paul E. McKenney wrote: >>> Hello! >>> >>> I got the lockdep splat shown below during some rcutorture testing (which >>> does CPU hotplug operations) on mainline

Re: [PATCH] x86/power/64: Use char arrays for asm function names

2017-05-11 Thread Pavel Machek
On Tue 2017-05-09 14:00:51, Kees Cook wrote: > This switches the hibernate_64.S function names into character arrays > to match other areas of the kernel where this is done (e.g., linker > scripts). Specifically this fixes a compile-time error noticed by the > future CONFIG_FORTIFY_SOURCE routines

<    3   4   5   6   7   8   9   10   11   >