Re: [PATCH v2] Staging: rtl8188eu: core: rtw_mlme_ext: Use list_for_each_entry_safe

2017-10-05 Thread Dan Carpenter
On Wed, Oct 04, 2017 at 10:46:56PM +0530, Srishti Sharma wrote: > drivers/staging/rtl8188eu/core/rtw_mlme_ext.c | 8 +++- > 1 file changed, 3 insertions(+), 5 deletions(-) > > diff --git a/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c > b/drivers/staging/rtl8188eu/core/rtw_mlme_ext.c > index

Re: Draft manpage explaining kernel lockdown

2017-10-05 Thread Florian Weimer
On 10/05/2017 01:00 PM, David Howells wrote: Lockdown is typically enabled during boot and may be terminated, if configured, by typing a special key combination on a directly attached physical keyboard. Does this include a Bluetooth keyboard (which might not actually exist and might in reality

Re: [PATCH] [STABLE-4.13] [media] imx-media-of: avoid uninitialized variable warning

2017-10-05 Thread Dan Carpenter
On Wed, Oct 04, 2017 at 03:34:55PM +0200, Arnd Bergmann wrote: > Replaces upstream commit 0b2e9e7947e7 ("media: staging/imx: remove > confusing IS_ERR_OR_NULL usage") > > We get a harmless warning about a potential uninitialized variable > use in the driver: > > drivers/staging/media/imx/imx-medi

Re: [v10 4/6] mm, oom: introduce memory.oom_group

2017-10-05 Thread Roman Gushchin
On Thu, Oct 05, 2017 at 02:06:49PM +0200, Michal Hocko wrote: > On Wed 04-10-17 16:46:36, Roman Gushchin wrote: > > The cgroup-aware OOM killer treats leaf memory cgroups as memory > > consumption entities and performs the victim selection by comparing > > them based on their memory footprint. Then

[tip:ras/urgent] x86/mce: Hide mca_cfg

2017-10-05 Thread tip-bot for Borislav Petkov
Commit-ID: 262e681183ddcdb24d64a2f993e41a226adcec29 Gitweb: https://git.kernel.org/tip/262e681183ddcdb24d64a2f993e41a226adcec29 Author: Borislav Petkov AuthorDate: Mon, 2 Oct 2017 11:28:36 +0200 Committer: Thomas Gleixner CommitDate: Thu, 5 Oct 2017 14:23:06 +0200 x86/mce: Hide mca_cfg

[tip:ras/urgent] RAS/CEC: Use the right length for "cec_disable"

2017-10-05 Thread tip-bot for Nicolas Iooss
Commit-ID: 69a330007091ea8a801dd9fcd897ec52f9529586 Gitweb: https://git.kernel.org/tip/69a330007091ea8a801dd9fcd897ec52f9529586 Author: Nicolas Iooss AuthorDate: Mon, 2 Oct 2017 11:28:35 +0200 Committer: Thomas Gleixner CommitDate: Thu, 5 Oct 2017 14:23:06 +0200 RAS/CEC: Use the right

Re: [PATCH v5 1/7] media: add glossary.rst with a glossary of terms used at V4L2 spec

2017-10-05 Thread Mauro Carvalho Chehab
Em Thu, 5 Oct 2017 11:21:07 +0300 Sakari Ailus escreveu: > Hi Mauro, > > My apologies for the late reply. > > On Tue, Aug 29, 2017 at 10:07:50AM -0300, Mauro Carvalho Chehab wrote: > > Em Tue, 29 Aug 2017 10:47:48 +0300 > > Sakari Ailus escreveu: > > > > > Hi Mauro, > > > > > > Thanks for

Re: [PATCH tip/core/rcu 1/3] membarrier: Provide register expedited private command

2017-10-05 Thread Peter Zijlstra
On Thu, Oct 05, 2017 at 02:12:50PM +0200, Peter Zijlstra wrote: > On Wed, Oct 04, 2017 at 02:37:53PM -0700, Paul E. McKenney wrote: > > diff --git a/arch/powerpc/kernel/membarrier.c > > b/arch/powerpc/kernel/membarrier.c > > new file mode 100644 > > index ..b0d79a5f5981 > > --- /dev/nu

Re: [PATCH] ARM: shmobile: rcar-gen2: fix non-SMP build

2017-10-05 Thread Geert Uytterhoeven
Hi Arnd, On Thu, Oct 5, 2017 at 2:09 PM, Arnd Bergmann wrote: > A bugfix for the SMP case broke the build for the UP case: > > arch/arm/mach-shmobile/headsmp-apmu.o: In function `shmobile_boot_apmu': > (.text+0x34): undefined reference to `secondary_startup' > > The assembler file mixes code that

Re: [PATCH] arm64: use vma_pages helper

2017-10-05 Thread Allen
Use vma_pages function on vma object instead of explicit computation. This is a clean up patch. A version of this is already queued for -next here: https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/commit/?h=for-next/core&id=b4f4a2755665bf847231050b569f587ce0945482 Ah.

Re: [PATCH] spi: spreadtrum adi: add hwspinlock dependency

2017-10-05 Thread Mark Brown
On Thu, Oct 05, 2017 at 02:11:17PM +0200, Arnd Bergmann wrote: > This adds a hard Kconfig dependency on HWSPINLOCK, which is the > simplest solution since we don't care about building for the case > without HWSPINLOCK. Given that there are stubs for HWSPINLOCK it'd be good to also have an || COMP

Re: [PATCH tip/core/rcu 1/3] membarrier: Provide register expedited private command

2017-10-05 Thread Avi Kivity
On 10/05/2017 07:23 AM, Nicholas Piggin wrote: On Wed, 4 Oct 2017 14:37:53 -0700 "Paul E. McKenney" wrote: From: Mathieu Desnoyers Provide a new command allowing processes to register their intent to use the private expedited command. This allows PowerPC to skip the full memory barrier i

Re: [PATCH] arm64: use vma_pages helper

2017-10-05 Thread Robin Murphy
On 05/10/17 13:16, Allen Pais wrote: > Use vma_pages function on vma object instead of explicit computation. > This is a clean up patch. A version of this is already queued for -next here: https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git/commit/?h=for-next/core&id=b4f4a2755665b

Re: [PATCH] rsi: fix integer overflow warning

2017-10-05 Thread Joe Perches
On Thu, 2017-10-05 at 14:05 +0200, Arnd Bergmann wrote: > gcc produces a harmless warning about a recently introduced > signed integer overflow: > > drivers/net/wireless/rsi/rsi_91x_hal.c: In function 'rsi_prepare_mgmt_desc': > include/uapi/linux/swab.h:13:15: error: integer overflow in expression

Re: [PATCH] libnvdimm, namespace: make a couple of functions static

2017-10-05 Thread Johannes Thumshirn
Looks good, Reviewed-by: Johannes Thumshirn -- Johannes Thumshirn Storage jthumsh...@suse.de+49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG N

Re: [PATCH] scsi: aic7xxx: make a couple of functions static

2017-10-05 Thread Johannes Thumshirn
Looks good, Reviewed-by: Johannes Thumshirn -- Johannes Thumshirn Storage jthumsh...@suse.de+49 911 74053 689 SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg GF: Felix Imendörffer, Jane Smithard, Graham Norton HRB 21284 (AG

Re: [PATCH] drm/i915/selftests: fix check for intel IOMMU

2017-10-05 Thread Chris Wilson
Quoting Arnd Bergmann (2017-10-05 13:07:22) > An earlier bugfix tried to work around this build failure: > > drivers/gpu/drm/i915/selftests/mock_gem_device.c: In function > 'mock_gem_device': > drivers/gpu/drm/i915/selftests/mock_gem_device.c:151:20: error: 'struct > dev_archdata' has no member

[PATCH] arm64: use vma_pages helper

2017-10-05 Thread Allen Pais
Use vma_pages function on vma object instead of explicit computation. This is a clean up patch. Signed-off-by: Allen Pais --- arch/arm64/mm/dma-mapping.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/arch/arm64/mm/dma-mapping.c b/arch/arm64/mm/dma-mapping.c index 614a

Re: [PATCH tip/core/rcu 1/3] membarrier: Provide register expedited private command

2017-10-05 Thread Peter Zijlstra
On Wed, Oct 04, 2017 at 02:37:53PM -0700, Paul E. McKenney wrote: > diff --git a/arch/powerpc/kernel/membarrier.c > b/arch/powerpc/kernel/membarrier.c > new file mode 100644 > index ..b0d79a5f5981 > --- /dev/null > +++ b/arch/powerpc/kernel/membarrier.c > @@ -0,0 +1,45 @@ > +void memb

[PATCH] clk: export clk_bulk_prepare

2017-10-05 Thread Arnd Bergmann
The ARM allmodconfig build fails in linux-next, since one clk interface was accidentally never exported but is now used by an mmc driver: ERROR: "clk_bulk_prepare" [drivers/mmc/host/sdhci-msm.ko] undefined! This exports it as well. Fixes: d8034e3379a8 ("mmc: sdhci-msm: Utilize bulk clock API") F

[PATCH] spi: spreadtrum adi: add hwspinlock dependency

2017-10-05 Thread Arnd Bergmann
With CONFIG_HWSPINLOCK=m, the new driver fails to link as a built-in driver: drivers/spi/spi-sprd-adi.o: In function `sprd_adi_remove': spi-sprd-adi.c:(.text+0x18): undefined reference to `hwspin_lock_free' drivers/spi/spi-sprd-adi.o: In function `sprd_adi_probe': spi-sprd-adi.c:(.text+0xfc): unde

[PATCH] drm/rockchip: add PINCTRL dependency for LVDS

2017-10-05 Thread Arnd Bergmann
The new driver fails to build when CONFIG_PINCTRL is disabled: drivers/gpu/drm/rockchip/rockchip_lvds.c: In function 'rockchip_lvds_grf_config': drivers/gpu/drm/rockchip/rockchip_lvds.c:229:39: error: dereferencing pointer to incomplete type 'struct dev_pin_info' if (lvds->pins && !IS_ERR(lvd

[PATCH] ARM: shmobile: rcar-gen2: fix non-SMP build

2017-10-05 Thread Arnd Bergmann
A bugfix for the SMP case broke the build for the UP case: arch/arm/mach-shmobile/headsmp-apmu.o: In function `shmobile_boot_apmu': (.text+0x34): undefined reference to `secondary_startup' The assembler file mixes code that is used for SMP with code that we also need on a single-CPU build, so I'm

[PATCH] drm/i915: avoid potential uninitialized variable use

2017-10-05 Thread Arnd Bergmann
One of the recent changes introduced a warning about undefined behavior in the sanity checking: drivers/gpu/drm/i915/intel_ddi.c: In function 'intel_ddi_hdmi_level': drivers/gpu/drm/i915/intel_ddi.c:654:6: error: 'n_hdmi_entries' may be used uninitialized in this function [-Werror=maybe-uninitial

[PATCH] drm/i915/selftests: fix check for intel IOMMU

2017-10-05 Thread Arnd Bergmann
An earlier bugfix tried to work around this build failure: drivers/gpu/drm/i915/selftests/mock_gem_device.c: In function 'mock_gem_device': drivers/gpu/drm/i915/selftests/mock_gem_device.c:151:20: error: 'struct dev_archdata' has no member named 'iommu' Checking for CONFIG_IOMMU_API is not suffi

Re: [PATCH] vfs: hard-ban creating files with control characters in the name

2017-10-05 Thread Alexey Dobriyan
> Anything with bytes 1-31,127 will get -EACCES. You're making a mistake of catering to Unix shells which are pure garbage as programming languages instead of fixing them or switching to saner alternatives and then "fixing" kernel to sort-of workaround Unix shells deficiencies. Formally, you patc

[PATCH] drm/omap: displays: panel-dpi: add backlight dependency

2017-10-05 Thread Arnd Bergmann
The new backlight code causes a link failure when backlight support itself is disabled: drivers/gpu/drm/omapdrm/displays/panel-dpi.o: In function `panel_dpi_probe_of': panel-dpi.c:(.text+0x35c): undefined reference to `of_find_backlight_by_node' This adds a Kconfig dependency like we have for the

Re: [v10 4/6] mm, oom: introduce memory.oom_group

2017-10-05 Thread Michal Hocko
On Wed 04-10-17 16:46:36, Roman Gushchin wrote: > The cgroup-aware OOM killer treats leaf memory cgroups as memory > consumption entities and performs the victim selection by comparing > them based on their memory footprint. Then it kills the biggest task > inside the selected memory cgroup. > > B

[PATCH] rsi: fix integer overflow warning

2017-10-05 Thread Arnd Bergmann
gcc produces a harmless warning about a recently introduced signed integer overflow: drivers/net/wireless/rsi/rsi_91x_hal.c: In function 'rsi_prepare_mgmt_desc': include/uapi/linux/swab.h:13:15: error: integer overflow in expression [-Werror=overflow] (((__u16)(x) & (__u16)0x00ffU) << 8) | \

[PATCH 3/3] kconfig: Clean up modules handling and fix crash

2017-10-05 Thread Ulf Magnusson
Kconfig currently doesn't handle 'm' appearing in a Kconfig file before the modules symbol is defined (the symbol with 'option modules'). The problem is the following code, which runs during parsing: /* change 'm' into 'm' && MODULES */ if (e->left.sym == &symbol_mod)

[PATCH 2/3] kconfig: Clarify expression rewriting

2017-10-05 Thread Ulf Magnusson
menu_finalize() is one of the more opaque parts of Kconfig, and I need to make some changes to it to fix an issue related to modules. Add some comments related to expression rewriting and dependency propagation as a review aid. They will also help other people trying to understand the code. Signed

[PATCH 1/3] kconfig: Rename menu_check_dep() to rewrite_m()

2017-10-05 Thread Ulf Magnusson
More directly describes the only thing it does. Signed-off-by: Ulf Magnusson --- scripts/kconfig/menu.c | 16 ++-- 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/scripts/kconfig/menu.c b/scripts/kconfig/menu.c index e935793..8354dfa 100644 --- a/scripts/kconfig/menu.c

[PATCH 0/3] kconfig: Make 'm' safe before modules symbol is defined

2017-10-05 Thread Ulf Magnusson
Hello, This patchset fixes a segfault that occurs if 'm' appears in certain expressions before the modules symbol is defined. The problem is that m is rewritten to m && already during parsing. Doing it in menu_finalize(), which runs after parsing, fixes the problem. To aid the review and people

[PATCH] scsi: aic7xxx: make a couple of functions static

2017-10-05 Thread Colin King
From: Colin Ian King Functions ahc_devlimited_syncrate and ahc_linux_initialize_scsi_bus are declared static in their prototypes but are missing in their definitions, so add the missing static. Cleans up sparse warnings: symbol 'ahc_devlimited_syncrate' was not declared. Should it be static? sym

[PATCH v1 4/6] platform/x86: intel_ips: Keep pointer to struct device

2017-10-05 Thread Andy Shevchenko
...instead of keeping pointer to struct pci_dev. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/intel_ips.c | 51 1 file changed, 25 insertions(+), 26 deletions(-) diff --git a/drivers/platform/x86/intel_ips.c b/drivers/platform/x86/intel_ips.c

Re: [PATCH V9 13/15] mmc: block: Add CQE and blk-mq support

2017-10-05 Thread Bartlomiej Zolnierkiewicz
Hi, On Wednesday, October 04, 2017 07:23:07 PM Hunter, Adrian wrote: > > -Original Message- > > From: Bartlomiej Zolnierkiewicz [mailto:b.zolnier...@samsung.com] > > Sent: Wednesday, October 4, 2017 12:40 PM > > To: Linus Walleij > > Cc: Hunter, Adrian ; Ulf Hansson > > ; linux-mmc ; lin

[PATCH v1 5/6] platform/x86: intel_ips: Remove unneeded fields and label

2017-10-05 Thread Andy Shevchenko
There are fields in the struct ips_mcp_limits which are not used anywhere and a label which we may get rid of. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/intel_ips.c | 7 ++- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/platform/x86/intel_ips.c b/drivers

Re: [PATCH v3 02/22] dt-bindings: arm: add support for ARM System Control and Management Interface(SCMI) protocol

2017-10-05 Thread Arnd Bergmann
On Wed, Oct 4, 2017 at 4:47 PM, Sudeep Holla wrote: > On 04/10/17 15:17, Arnd Bergmann wrote: >> On Wed, Oct 4, 2017 at 3:53 PM, Sudeep Holla wrote: >>> On 04/10/17 13:35, Arnd Bergmann wrote: On Wed, Oct 4, 2017 at 1:07 PM, Sudeep Holla wrote: >> There are probably several ways of doing t

[PATCH] PM / OPP: Rename dev_pm_opp_register_put_opp_helper()

2017-10-05 Thread Viresh Kumar
The routine is named incorrectly since the first attempt as there is nothing like a put_opp() helper. We wanted to unregister the set_opp() helper here and so it should rather be named as dev_pm_opp_unregister_set_opp_helper(). Signed-off-by: Viresh Kumar --- drivers/opp/core.c | 6 +++--- i

[PATCH v1 6/6] platform/x86: intel_ips: Remove FSF address from GPL notice

2017-10-05 Thread Andy Shevchenko
This patch removes the FSF address from the GPL notice to fix a checkpatch.pl CHECK message. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/intel_ips.c | 4 drivers/platform/x86/intel_ips.h | 4 2 files changed, 8 deletions(-) diff --git a/drivers/platform/x86/intel_ips.c b/d

[PATCH v1 1/6] platform/x86: intel_ips: Simplify error handling via devres API

2017-10-05 Thread Andy Shevchenko
Use devm_ and pcim_ functions to make error handling simpler and code smaller and tidier. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/intel_ips.c | 62 ++-- 1 file changed, 15 insertions(+), 47 deletions(-) diff --git a/drivers/platform/x86/intel_

[PATCH v1 2/6] platform/x86: intel_ips: Switch to new PCI IRQ allocation API

2017-10-05 Thread Andy Shevchenko
This makes code cleaner. Signed-off-by: Andy Shevchenko --- drivers/platform/x86/intel_ips.c | 18 +- 1 file changed, 13 insertions(+), 5 deletions(-) diff --git a/drivers/platform/x86/intel_ips.c b/drivers/platform/x86/intel_ips.c index 063d9a1624b4..6f6900065bc2 100644 --- a/d

[PATCH v1 3/6] platform/x86: intel_ips: Use PCI_VDEVICE() macro

2017-10-05 Thread Andy Shevchenko
Intel vendor ID is defined globally, thus we may use PCI_VDEVICE(). Signed-off-by: Andy Shevchenko --- drivers/platform/x86/intel_ips.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/platform/x86/intel_ips.c b/drivers/platform/x86/intel_ips.c index 6f6900065bc2..48

Re: [PATCH 1/8] pinctrl: meson: remove offset from pinctrl

2017-10-05 Thread Jerome Brunet
On Thu, 2017-10-05 at 13:22 +0200, Linus Walleij wrote: > On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > > > Offset on meson pinctrl and gpios is something that was carried from the > > vendor driver, where there is a weird link between the 2 > > controllers. Since these 2 controllers ar

Re: [PATCH 8/8] pinctrl: meson-gxbb: add missing GPIOX_22 pin

2017-10-05 Thread Linus Walleij
On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > GPIOX_22 is declared as a gpio but the id is no present in the pin > table. This hole trigger an error while reading the pingroup debugfs entry > > GPIOX_22 is no routed externally. For all we know, it could an internal > pin of SoC > > Sign

[PATCH RESEND v3 5/5] sched/fair: Fix use of find_idlest_group when local group is idlest

2017-10-05 Thread Brendan Jackman
find_idlest_group returns NULL when the local group is idlest. The caller then continues the find_idlest_group search at a lower level of the current CPU's sched_domain hierarchy. find_idlest_group_cpu is not consulted and, crucially, @new_cpu is not updated. This means the search is pointless and

Re: [v10 3/6] mm, oom: cgroup-aware OOM killer

2017-10-05 Thread Roman Gushchin
On Thu, Oct 05, 2017 at 01:12:30PM +0200, Michal Hocko wrote: > On Thu 05-10-17 11:27:07, Roman Gushchin wrote: > > On Wed, Oct 04, 2017 at 02:24:26PM -0700, Shakeel Butt wrote: > [...] > > > Sorry about the confusion. There are two things. First, should we do a > > > css_get on the newly selected

[PATCH RESEND v3 4/5] sched/fair: Fix use of find_idlest_group when no groups are allowed

2017-10-05 Thread Brendan Jackman
When p is allowed on none of the CPUs in the sched_domain, we currently return NULL from find_idlest_group, and pointlessly continue the search on lower sched_domain levels (where p is also not allowed) before returning prev_cpu regardless (as we have not updated new_cpu). Add an explicit check fo

[PATCH RESEND v3 0/5]

2017-10-05 Thread Brendan Jackman
This patchset optimises away an unused comparison, and fixes some corner cases in the find_idlest_group path of select_task_rq_fair. Changes v2 -> v3: - 1/5 through 4/5 unchanged. - 5/5: Essentially undid [1]. Vincent pointed out that this isn't really a bugfix but more of a change in poli

[PATCH RESEND v3 1/5] sched/fair: Move select_task_rq_fair slow-path into its own function

2017-10-05 Thread Brendan Jackman
In preparation for changes that would otherwise require adding a new level of indentation to the while(sd) loop, create a new function find_idlest_cpu which contains this loop, and rename the existing find_idlest_cpu to find_idlest_group_cpu. Code inside the while(sd) loop is unchanged. @new_cpu i

[PATCH RESEND v3 2/5] sched/fair: Remove unnecessary comparison with -1

2017-10-05 Thread Brendan Jackman
Since commit 83a0a96a5f26 ("sched/fair: Leverage the idle state info when choosing the "idlest" cpu") find_idlest_grou_cpu (formerly find_idlest_cpu) no longer returns -1. Signed-off-by: Brendan Jackman Cc: Dietmar Eggemann Cc: Vincent Guittot Cc: Josef Bacik Cc: Ingo Molnar Cc: Morten Rasmus

[PATCH RESEND v3 3/5] sched/fair: Fix find_idlest_group when local group is not allowed

2017-10-05 Thread Brendan Jackman
When the local group is not allowed we do not modify this_*_load from their initial value of 0. That means that the load checks at the end of find_idlest_group cause us to incorrectly return NULL. Fixing the initial values to ULONG_MAX means we will instead return the idlest remote group in that ca

Re: [PATCH 7/8] pinctrl: meson-gx: TEST_N belongs to the AO controller

2017-10-05 Thread Linus Walleij
On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > On meson-gx platforms, TEST_N has been incorrectly declared in the EE > controller while it belongs to AO controller. > > Move the pin to the appropriate controller > > Signed-off-by: Jerome Brunet Patch applied. Yours, Linus Walleij

Re: [PATCH 6/8] pinctrl: meson: get rid of pin_base

2017-10-05 Thread Linus Walleij
On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > pin_base was used with the manually set pin offset in meson pinctrl. This > is no longer the case, pin_base is 0 on every meson pinctrl controllers > and should go away. > > Tested-by: Martin Blumenstingl > Signed-off-by: Jerome Brunet Pa

Re: [BUG] get_maintainers: Inconsistent behavior on multiple runs

2017-10-05 Thread Joe Perches
On Thu, 2017-10-05 at 12:21 +0530, Viresh Kumar wrote: > Hi Joe, Hi Viresh. > Recently I was Cc'd for some of the patches from Manu, as > get_maintainers incorrectly mentioned me. > > Apart from the fact that I never contributed to the mentioned > file (drivers/phy/qualcomm/phy-qcom-qmp.c), the

Re: [v10 3/6] mm, oom: cgroup-aware OOM killer

2017-10-05 Thread Michal Hocko
On Wed 04-10-17 16:46:35, Roman Gushchin wrote: > Traditionally, the OOM killer is operating on a process level. > Under oom conditions, it finds a process with the highest oom score > and kills it. > > This behavior doesn't suit well the system with many running > containers: > > 1) There is no

[PATCH] x86/hyperv: don't use percpu areas for pcpu_flush/pcpu_flush_ex structures

2017-10-05 Thread Vitaly Kuznetsov
hv_do_hypercall() does virt_to_phys() translation and with some configs (CONFIG_SLAB) this doesn't work for percpu areas, we pass wrong memory to hypervisor and get #GP. We could use working slow_virt_to_phys() instead but doing so kills the performance. Move pcpu_flush/pcpu_flush_ex structures ou

Re: [PATCH v3 03/33] ring-buffer: Add interface for setting absolute time stamps

2017-10-05 Thread Tom Zanussi
Hi Steve, On Wed, 2017-10-04 at 13:14 -0400, Steven Rostedt wrote: > On Fri, 22 Sep 2017 14:59:43 -0500 > Tom Zanussi wrote: > > > + /* > > +* New timestamps may not be consistent with the previous setting. > > +* Reset the buffer so that it doesn't have incomparable timestamps. > > I

[PATCH] mm/slob: remove an unnecessary check for __GFP_ZERO

2017-10-05 Thread miles.chen
From: Miles Chen Current flow guarantees a valid pointer when handling the __GFP_ZERO case. So remove the unnecessary NULL pointer check. Signed-off-by: Miles Chen --- mm/slob.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/mm/slob.c b/mm/slob.c index a8bd6fa..a72649c 100

Re: [PATCH 5/8] pinctrl: meson: remove offset continued - meson8b

2017-10-05 Thread Linus Walleij
On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > Tested-by: Martin Blumenstingl > Signed-off-by: Jerome Brunet Patch applied. Linus Walleij

[PATCH] bootwrapper: mspsc.c: fix pointer-to-int-cast warnings

2017-10-05 Thread Michal Suchanek
I get these warnings: ../arch/powerpc/boot/mpsc.c: In function 'mpsc_get_virtreg_of_phandle': ../arch/powerpc/boot/mpsc.c:113:35: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast] ../arch/powerpc/boot/mpsc.c: In function 'mpsc_console_init': ../arch/powerpc/boot/mpsc

[PATCH] Staging: rtl8723bs: Externs should be avoided in .C file

2017-10-05 Thread Srinivasan Shanmugam
Fix checkpatch issues: "CHECK: Alignment should match open parenthesis" Signed-off-by: Srinivasan Shanmugam --- drivers/staging/rtl8723bs/core/rtw_recv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rtl8723bs/core/rtw_recv.c b/drivers/staging/rtl8723bs/cor

[tip:x86/cache] x86/intel_rdt: Remove redundant assignment

2017-10-05 Thread tip-bot for Jithu Joseph
Commit-ID: 3916a4135c696fa226a1abe6d6a0ff7f5edd9a7c Gitweb: https://git.kernel.org/tip/3916a4135c696fa226a1abe6d6a0ff7f5edd9a7c Author: Jithu Joseph AuthorDate: Wed, 4 Oct 2017 15:48:57 -0700 Committer: Thomas Gleixner CommitDate: Thu, 5 Oct 2017 13:20:32 +0200 x86/intel_rdt: Remove re

[tip:x86/cache] x86/intel_rdt/cqm: Make integer rmid_limbo_count static

2017-10-05 Thread tip-bot for Colin Ian King
Commit-ID: 5fd88b60e11b7d81b2c944c1b45834c4a6aa0157 Gitweb: https://git.kernel.org/tip/5fd88b60e11b7d81b2c944c1b45834c4a6aa0157 Author: Colin Ian King AuthorDate: Mon, 2 Oct 2017 15:59:31 +0100 Committer: Thomas Gleixner CommitDate: Thu, 5 Oct 2017 13:20:32 +0200 x86/intel_rdt/cqm: Mak

Re: [PATCH v3 21/22] cpufreq: add support for CPU DVFS based on SCMI message protocol

2017-10-05 Thread Sudeep Holla
On 05/10/17 12:20, Arnd Bergmann wrote: > On Wed, Oct 4, 2017 at 5:01 PM, Sudeep Holla wrote: + +static struct platform_driver scmi_cpufreq_platdrv = { + .driver = { + .name = "scmi-cpufreq", + }, + .probe = scmi_cpufreq_pr

Re: [PATCH 2/3] i2c: Cocci spatch "of_table"

2017-10-05 Thread Wolfram Sang
On Thu, Sep 21, 2017 at 08:24:27AM +0200, Thomas Meyer wrote: > Make sure (of/i2c/platform)_device_id tables are NULL terminated. > Found by coccinelle spatch "misc/of_table.cocci" > > Signed-off-by: Thomas Meyer Applied to for-current, thanks! But please use descriptive subjects, I cahanged it

Re: [PATCH 4/8] pinctrl: meson: remove offset continued - meson8

2017-10-05 Thread Linus Walleij
On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > Tested-by: Martin Blumenstingl > Signed-off-by: Jerome Brunet Patch applied. Yours, Linus Walleij

Re: [PATCH 3/8] pinctrl: meson: remove offset continued - gxl

2017-10-05 Thread Linus Walleij
On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > Signed-off-by: Jerome Brunet Patch applied. Yours, Linus Walleij

Re: [PATCH 2/8] pinctrl: meson: remove offset continued - gxbb

2017-10-05 Thread Linus Walleij
On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > Signed-off-by: Jerome Brunet Patch applied. Yours, Linus Walleij

[PATCH] KVM: PPC: Book3S HV: Delete an error message for a failed memory allocation in kvmppc_allocate_hpt()

2017-10-05 Thread SF Markus Elfring
From: Markus Elfring Date: Thu, 5 Oct 2017 13:16:51 +0200 Omit an extra message for a memory allocation failure in this function. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- arch/powerpc/kvm/book3s_64_mmu_hv.c | 1 - 1 file changed, 1 deletion(-

Re: [PATCH] i2c: piix4: Disable completely the IMC during SMBUS_BLOCK_DATA

2017-10-05 Thread Wolfram Sang
On Thu, Sep 21, 2017 at 09:37:43AM +0200, Ricardo Ribalda Delgado wrote: > SMBUS_BLOCK_DATA transactions might fail due to a race condition with > the IMC, even when the IMC semaphore is used. > > This bug has been reported and confirmed by AMD, who suggested as a > solution an IMC firmware upgrad

Re: [PATCH 1/8] pinctrl: meson: remove offset from pinctrl

2017-10-05 Thread Linus Walleij
On Wed, Sep 20, 2017 at 3:39 PM, Jerome Brunet wrote: > Offset on meson pinctrl and gpios is something that was carried from the > vendor driver, where there is a weird link between the 2 > controllers. Since these 2 controllers are independent, this offset adds > an unnecessary complexity. > > T

Re: [PATCH v3 21/22] cpufreq: add support for CPU DVFS based on SCMI message protocol

2017-10-05 Thread Arnd Bergmann
On Wed, Oct 4, 2017 at 5:01 PM, Sudeep Holla wrote: >>> + >>> +static struct platform_driver scmi_cpufreq_platdrv = { >>> + .driver = { >>> + .name = "scmi-cpufreq", >>> + }, >>> + .probe = scmi_cpufreq_probe, >>> + .remove = scmi_cpufreq_re

Re: [PATCH v2 00/16] gpio: Tight IRQ chip integration and banked infrastructure

2017-10-05 Thread Linus Walleij
On Tue, Oct 3, 2017 at 8:26 PM, Grygorii Strashko wrote: > On 10/02/2017 02:55 AM, Linus Walleij wrote: >> On Thu, Sep 28, 2017 at 4:22 PM, Grygorii Strashko >> wrote: >> >>> Sry, but I do not agree with this series. >>> - no prof that it can be re-used by other drivers than tegra >>> (at least

Re: [PATCH v1] i2c:i2c-stm32f7: fix setup structure

2017-10-05 Thread Wolfram Sang
On Thu, Sep 21, 2017 at 03:30:09PM +0200, Pierre-Yves MORDRET wrote: > I2C drive setup structure is not properly allocated. > Make it static instead of pointer to store driver data. > > Signed-off-by: Pierre-Yves MORDRET Ouch! How did it work before? Applied to for-current, thanks! Please provi

Re: [PATCH v2 00/16] gpio: Tight IRQ chip integration and banked infrastructure

2017-10-05 Thread Linus Walleij
On Mon, Oct 2, 2017 at 9:55 AM, Linus Walleij wrote: > This is not necessarily a blocker if it can be shown that others than > TI/OMAP can reuse it. > > I've looked at things like the imagination pistachio: > > pinctrl@18101C00 { > compatible = "img,pistachio-system-pinctrl"; > re

Re: [v10 3/6] mm, oom: cgroup-aware OOM killer

2017-10-05 Thread Michal Hocko
On Wed 04-10-17 16:31:38, Johannes Weiner wrote: > On Wed, Oct 04, 2017 at 01:17:14PM -0700, David Rientjes wrote: > > On Wed, 4 Oct 2017, Roman Gushchin wrote: > > > > > > > @@ -828,6 +828,12 @@ static void __oom_kill_process(struct > > > > > task_struct *victim) > > > > > struct mm_struct

Re: [v10 3/6] mm, oom: cgroup-aware OOM killer

2017-10-05 Thread Michal Hocko
On Thu 05-10-17 11:27:07, Roman Gushchin wrote: > On Wed, Oct 04, 2017 at 02:24:26PM -0700, Shakeel Butt wrote: [...] > > Sorry about the confusion. There are two things. First, should we do a > > css_get on the newly selected memcg within the for loop when we still > > have a reference to it? > >

Re: [PATCH V9 13/15] mmc: block: Add CQE and blk-mq support

2017-10-05 Thread Ulf Hansson
On 5 October 2017 at 12:55, Bartlomiej Zolnierkiewicz wrote: > On Wednesday, October 04, 2017 11:48:27 AM Ulf Hansson wrote: >> [...] >> >> >> MQ code that doesn't perform and therefore we cannot switch seamlessly >> >> to MQ. >> > >> > I think that switching seamlessly to blk-mq in short/medium-t

Re: [PATCH v3 07/14] xhci: Add Intel cherrytrail extended cap / otg phy mux handling

2017-10-05 Thread Mathias Nyman
On 22.09.2017 21:37, Hans de Goede wrote: The Intel cherrytrail xhci controller has an extended cap mmio-range which contains registers to control the muxing to the xhci (host mode) or the dwc3 (device mode) and vbus-detection for the otg usb-phy. Having a mux driver included in the xhci code (o

Re: [PATCH] MAINTAINERS: Add entry for Synopsys DesignWare APB GPIO driver

2017-10-05 Thread Linus Walleij
On Fri, Sep 22, 2017 at 11:33 PM, Hoan Tran wrote: > This patch adds maintainer for Synopsys DesignWare APB GPIO driver. > > Signed-off-by: Hoan Tran Patch applied. I added an F: entry for the DT bindings as well. Thank you for taking on this driver! Yours, Linus Walleij

Re: [PATCH v2 2/2] dt-bindings: i2c: Update documentation for "mv78230-a0-i2c"

2017-10-05 Thread Wolfram Sang
On Mon, Oct 02, 2017 at 04:23:43PM +0200, Gregory CLEMENT wrote: > Hi Wolfram, > > On ven., sept. 29 2017, Kalyan Kinthada > wrote: > > > This commit modifies the documentation for > > "marvell,mv78230-a0-i2c" compatible string. > > > > The "marvell,mv78230-a0-i2c" compatible string enables t

Re: Extreme time jitter with suspend/resume cycles

2017-10-05 Thread Miroslav Lichvar
On Wed, Oct 04, 2017 at 05:16:31PM -0700, John Stultz wrote: > On Wed, Oct 4, 2017 at 9:11 AM, Gabriel Beddingfield > wrote: > > We found that the problem is an interaction between the NTP code and > > what I call the "delta_delta hack." (see [1] and [2]) This code > > allocates a static variable

Re: Draft manpage explaining kernel lockdown

2017-10-05 Thread Johannes Berg
On Thu, 2017-10-05 at 12:00 +0100, David Howells wrote: > > Only validly signed wifi databases may be use. We don't even have this yet, and when we do, we want this to be the case for typical configurations regardless of lockdown. johannes

Re: [PATCH v6 5/6] lib/dlock-list: Enable faster lookup with hashing

2017-10-05 Thread Davidlohr Bueso
On Thu, 05 Oct 2017, Jan Kara wrote: OK, this makes sense but do you have any particular user in mind? In particular I'm not sure how big advantage this API brings over an existing one in include/linux/list_bl.h. Sure it's a tradeoff between bitlock / spinlock but is there a user where it matter

Re: Extreme time jitter with suspend/resume cycles

2017-10-05 Thread Thomas Gleixner
On Wed, 4 Oct 2017, John Stultz wrote: > On Wed, Oct 4, 2017 at 9:11 AM, Gabriel Beddingfield > wrote: > > TL;DR: the "delta_delta" hack[1 and 2] in kernel/time/timekeeping.c > > and drivers/rtc/class.c undermines the NTP system. It's not > > appropriate to use if sub-second precision is availabl

Re: [lockdep] b09be676e0 BUG: unable to handle kernel NULL pointer dereference at 000001f2

2017-10-05 Thread Tetsuo Handa
Josh Poimboeuf wrote: > On Wed, Oct 04, 2017 at 06:44:50AM +0900, Tetsuo Handa wrote: > > Josh Poimboeuf wrote: > > > On Tue, Oct 03, 2017 at 11:28:15AM -0500, Josh Poimboeuf wrote: > > > > There are two bugs: > > > > > > > > 1) Somebody -- presumably lockdep -- is corrupting the stack. Need the

Re: [PATCH v9 04/12] RISC-V: Init and Halt Code

2017-10-05 Thread Mark Rutland
On Tue, Sep 26, 2017 at 06:56:30PM -0700, Palmer Dabbelt wrote: > +/* > + * This is particularly ugly: it appears we can't actually get the definition > + * of task_struct here, but we need access to the CPU this task is running > on. > + * Instead of using C we're using asm-offsets.h to get the c

Re: Extreme time jitter with suspend/resume cycles

2017-10-05 Thread Thomas Gleixner
On Wed, 4 Oct 2017, Gabriel Beddingfield wrote: > On Wed, Oct 4, 2017 at 11:22 AM, Thomas Gleixner wrote: > Long story short: you can't always have your low-power clock be your > monotonic/sched clock. sched_clock and the clocksource for timekeeping, which feeds monotonic are not required to be t

Draft manpage explaining kernel lockdown

2017-10-05 Thread David Howells
Hi Ard, Michael, Attached is a draft for a manual page (kernel_lockdown.7) that I intend to point at from messages emitted when the kernel prohibits something because the kernel is in 'lockdown' mode, typically triggered by EFI secure boot. Let me know what you think. David --- .\" .\" Copyright

Re: [RFT PATCH v2] gpiolib: allow gpio irqchip to map irqs dynamically

2017-10-05 Thread Marc Zyngier
On 21/09/17 12:41, Linus Walleij wrote: > On Fri, Sep 15, 2017 at 10:26 AM, Jerome Brunet wrote: > >> I initially planned to do some refcounting in the gpio layer but that would >> make >> no sense, as you pointed out, the irq could be shared. This refcounting would >> only make sense at the irq

[PATCH RESEND] sched/fair: Force balancing on nohz balance if local group has capacity

2017-10-05 Thread Brendan Jackman
The "goto force_balance" here is intended to mitigate the fact that avg_load calculations can result in bad placement decisions when priority is asymmetrical. From the original commit (fab476228ba3 "sched: Force balancing on newidle balance if local group has capacity") that adds it: Under cer

Re: [PATCH v2 0/4] KVM: LAPIC: Rework lapic timer to behave more like real-hardware

2017-10-05 Thread Wanpeng Li
2017-09-29 9:04 GMT+08:00 Wanpeng Li : > The issue is reported in xen community. > > Anthony PERARD pointed out: > > https://www.mail-archive.com/xen-devel@lists.xen.org/msg117283.html# > > | When developing PVH for OVMF, I've used the lapic timer. It turns out that > the > | way it is used by O

[PATCH 03/20] ARM: dts: sun4i: Change framebuffer node names to avoid warnings

2017-10-05 Thread Maxime Ripard
The simple-framebuffer nodes have a unit address, but no reg property which generates a warning when compiling it with DTC. Change the simple-framebuffer node names so that there is no warnings on this anymore. Signed-off-by: Maxime Ripard --- arch/arm/boot/dts/sun4i-a10.dtsi | 8 1 fi

[PATCH RESEND v2] sched/fair: Sync task util before slow-path wakeup

2017-10-05 Thread Brendan Jackman
We use task_util in find_idlest_group via capacity_spare_wake. This task_util is updated in wake_cap. However wake_cap is not the only reason for ending up in find_idlest_group - we could have been sent there by wake_wide. So explicitly sync the task util with prev_cpu when we are about to head to

[PATCH 02/20] ARM: dts: axp209: Rename usb_power_supply node to avoid warnings

2017-10-05 Thread Maxime Ripard
The USB power supply node in the AXP209 DTSI is using underscores in its node name, which is generating a warning. Change those underscores for hyphens. Signed-off-by: Maxime Ripard --- arch/arm/boot/dts/axp209.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boo

[PATCH 04/20] ARM: dts: sun4i: Change clock node names to avoid warnings

2017-10-05 Thread Maxime Ripard
Our oscillators clock names have a unit address, but no reg property, which generates a warning in DTC. Change these names to remove those unit addresses. Signed-off-by: Maxime Ripard --- arch/arm/boot/dts/sun4i-a10.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arc

Re: [PATCH V9 13/15] mmc: block: Add CQE and blk-mq support

2017-10-05 Thread Bartlomiej Zolnierkiewicz
On Wednesday, October 04, 2017 11:48:27 AM Ulf Hansson wrote: > [...] > > >> MQ code that doesn't perform and therefore we cannot switch seamlessly > >> to MQ. > > > > I think that switching seamlessly to blk-mq in short/medium-term > > is not possible (SCSI tried and failed to do so). The changes

[PATCH 06/20] ARM: dts: sun4i: Provide default muxing for relevant controllers

2017-10-05 Thread Maxime Ripard
Some controllers have only one muxing option in the SoC. In such a case, we can just move the muxing into the DTSI, and remove it from the DTS. Signed-off-by: Maxime Ripard --- arch/arm/boot/dts/sun4i-a10-a1000.dts| 6 -- arch/arm/boot/dts/sun4i-a10-ba10-tvbox.dts | 6 ---

[PATCH v3 3/3] KVM: LAPIC: Apply change to TDCR right away to the timer

2017-10-05 Thread Wanpeng Li
From: Wanpeng Li The description in the Intel SDM of how the divide configuration register is used: "The APIC timer frequency will be the processor's bus clock or core crystal clock frequency divided by the value specified in the divide configuration register." Observation of baremetal shown tha

<    3   4   5   6   7   8   9   10   >