Re: [PATCH] net: stmmac: make const array route_possibilities static

2017-07-13 Thread David Miller
From: Colin King Date: Tue, 11 Jul 2017 12:18:48 +0100 > From: Colin Ian King > > Don't populate array route_possibilities on the stack but make it > static const. Makes the object code a little smaller by 85 bytes: > > Before: >text data bss dec hex filename >990

[PATCH v5 10/14] drm: mgag200: remove dead code and pointless local lut storage

2017-07-13 Thread Peter Rosin
The redundant fb helpers .load_lut, .gamma_set and .gamma_get are no longer used. Remove the dead code and hook up the crtc .gamma_set to use the crtc gamma_store directly instead of duplicating that info locally. Acked-by: Daniel Vetter Signed-off-by: Peter Rosin --- drivers/gpu/drm/mgag200/mg

Re: [PATCH] dccp: make const array error_code static

2017-07-13 Thread David Miller
From: Colin King Date: Thu, 13 Jul 2017 12:22:24 +0100 > From: Colin Ian King > > Don't populate array error_code on the stack but make it static. Makes > the object code smaller by almost 250 bytes: > > Before: >text data bss dec hex filename > 10366 983

[PATCH v5 08/14] drm: gma500: remove dead code and pointless local lut storage

2017-07-13 Thread Peter Rosin
The redundant fb helpers .gamma_set and .gamma_get are no longer used. Remove the dead code and hook up the crtc .gamma_set to use the crtc gamma_store directly instead of duplicating that info locally. Acked-by: Daniel Vetter Signed-off-by: Peter Rosin --- drivers/gpu/drm/gma500/framebuffer.c

[PATCH v5 07/14] drm: cirrus: remove dead code and pointless local lut storage

2017-07-13 Thread Peter Rosin
The redundant fb helpers .load_lut, .gamma_set and .gamma_get are no longer used. Remove the dead code and hook up the crtc .gamma_set to use the crtc gamma_store directly instead of duplicating that info locally. Acked-by: Daniel Vetter Signed-off-by: Peter Rosin --- drivers/gpu/drm/cirrus/cir

Re: [PATCH] net: broadcom: bnx2x: make a couple of const arrays static

2017-07-13 Thread David Miller
From: Colin King Date: Tue, 11 Jul 2017 11:52:23 +0100 > From: Colin Ian King > > Don't populate various tables on the stack but make them static const. > Makes the object code smaller by nearly 200 bytes: > > Before: >text data bss dec hex filename > 113468 1

[PATCH v5 05/14] drm: armada: remove dead empty functions

2017-07-13 Thread Peter Rosin
The redundant fb helpers .gamma_set and .gamma_get are no longer used. Remove the dead code. Acked-by: Daniel Vetter Signed-off-by: Peter Rosin --- drivers/gpu/drm/armada/armada_crtc.c | 10 -- drivers/gpu/drm/armada/armada_crtc.h | 2 -- drivers/gpu/drm/armada/armada_fbdev.c | 2 --

[PATCH v5 00/14] improve the fb_setcmap helper

2017-07-13 Thread Peter Rosin
Hi! While trying to get CLUT support for the atmel_hlcdc driver, and specifically for the emulated fbdev interface, I received some push-back that my feeble in-driver attempts should be solved by the core. This is my attempt to do it right. I have obviously not tested all of this with more than a

[PATCH v5 03/14] drm/fb-helper: separate the fb_setcmap helper into atomic and legacy paths

2017-07-13 Thread Peter Rosin
The legacy path implements setcmap in terms of crtc .gamma_set. The atomic path implements setcmap by directly updating the crtc gamma_lut property. This has a couple of benefits: - it makes the redundant fb helpers .load_lut, .gamma_set and .gamma_get completely obsolete. They are now unused a

Re: [PATCH v1] staging: rts5208: Change fixed function names with "%s: ", __func__

2017-07-13 Thread Greg KH
On Thu, Jul 13, 2017 at 09:30:20PM +0530, Gaurav Pathak wrote: > Adding public mailing list address and Joe. > > Hi Gerg, > > I am trying my best to avoid such silly mistakes, but this is first > time I am sending patch to a maintainer. Please disregard my > mistakes. > > I will send it to the

Re: [RFC PATCH 1/1] mm/mremap: add MREMAP_MIRROR flag for existing mirroring functionality

2017-07-13 Thread Andrea Arcangeli
On Thu, Jul 13, 2017 at 09:01:54AM -0700, Mike Kravetz wrote: > Sent a patch (in separate e-mail thread) to return EINVAL for private > mappings. The way old_len == 0 behaves for MAP_PRIVATE seems more sane to me than the alternative of copying pagetables for anon pages (as behaving the way that w

Re: [RFC V2 4/6] cpufreq: Use transition_delay_us for legacy governors as well

2017-07-13 Thread Rafael J. Wysocki
On Thu, Jul 13, 2017 at 7:40 AM, Viresh Kumar wrote: > The policy->transition_delay_us field is used only by the schedutil > governor currently, and this field describes how fast the driver wants > the cpufreq governor to change CPUs frequency. It should rather be a > common thing across all gover

Re: [PATCH] libiscsi: Fix use-after-free race during iscsi_session_teardown

2017-07-13 Thread kbuild test robot
Hi Khazhismel, [auto build test WARNING on mkp-scsi/for-next] [also build test WARNING on v4.12 next-20170713] [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/Khazhismel-Kumykov/libiscsi-Fix-use

Re: [PATCH 1/3] mfd: Add support for FTDI FT232H devices

2017-07-13 Thread Anatolij Gustschin
On Wed, 12 Jul 2017 10:50:03 +0200 Johan Hovold jo...@kernel.org wrote: ... >IIRC we should be able read from the EEPROM, and I would at least expect >there to be a way to retrieve the current mode as well. I've just send a patch for ftdi_sio. Thanks, Anatolij

[GIT PULL] Kbuild updates for v4.13 (2nd)

2017-07-13 Thread Masahiro Yamada
Hi Linus, This is a late Kbuild pull request for v4.13. I needed to rebase this after arch changes were pulled in. I think you can merge this cleanly. Please pull! The following changes since commit 2b976203417cf033079e0be30cae5f41d88e385e: Merge branch 'x86-urgent-for-linus' of git://git.ke

[PULL] Documentation fixes

2017-07-13 Thread Jonathan Corbet
The following changes since commit 1cb566ba5634d7593b8b2a0a5c83f1c9e14b2e09: scripts/kernel-doc: handle DECLARE_HASHTABLE (2017-07-03 06:57:09 -0600) are available in the git repository at: git://git.lwn.net/linux.git tags/4.13-fixes for you to fetch changes up to 51e988f4092428e3d2c9f141fb

Re: [PATCH] nvmet: preserve controller serial number between reboots

2017-07-13 Thread Sagi Grimberg
It seems weird that a subsystem has a serial. But that's actually how NVMe defines them. Where is that wording? Which mean we first need to fix our code to generate a serial number per subsystem, and on top of that the patch from Johannes seems perfectly reasonable. If that is indeed the

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-13 Thread Theodore Ts'o
On Thu, Jul 13, 2017 at 07:11:36AM -0500, Eric W. Biederman wrote: > The concise summary: > > Today we have the xattr security.capable that holds a set of > capabilities that an application gains when executed. AKA setuid root exec > without actually being setuid root. > > User namespaces have t

Re: [PATCH V4] PCI: handle CRS returned by device after FLR

2017-07-13 Thread Sinan Kaya
On 7/13/2017 12:29 PM, Keith Busch wrote: >> When used, DRS and FRS allow an improved behavior over the CRS mechanism, >> and eliminate >> its associated periodic polling time of up to 1 second following a reset." > That wording is just confusing. It looks to me the 1 second polling is > to be use

Re: [PATCH 4.9 00/25] 4.9.38-stable review

2017-07-13 Thread Sumit Semwal
Hi Greg, On 13 July 2017 at 21:10, Greg Kroah-Hartman wrote: > This is the start of the stable review cycle for the 4.9.38 release. > There are 25 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > >

Re: [PATCHv3 3/3] ARM: drm: Intel FPGA VIP Frame Buffer II drm driver

2017-07-13 Thread kbuild test robot
Hi Hean, [auto build test WARNING on drm/drm-next] [also build test WARNING on v4.12 next-20170713] [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/hean-loong-ong-intel-com/Intel-FPGA-VIP-Frame

Re: [PATCH 4/4] perf annotate browser: Support the toggle number of samples

2017-07-13 Thread Taeung Song
On 07/13/2017 05:16 AM, Arnaldo Carvalho de Melo wrote: Em Wed, Jul 12, 2017 at 07:14:19AM +0900, Taeung Song escreveu: Cc: Namhyung Kim Cc: Milian Wolff Cc: Jiri Olsa Signed-off-by: Taeung Song Humm, this is cycling thru different things to show on a column, I think 'toggle' would best

[PATCH] scsi: qedf: Limit number of CQs

2017-07-13 Thread Thomas Bogendoerfer
From: Thomas Bogendoerfer FCOE offloading on qedf devices fails with: [qed_sp_fcoe_func_start:150(sp-0-3b:00.02)]Cannot satisfy CQ amount. CQs requested 8, CQs available 6. Aborting function start [qed_fcoe_start:821()]Failed to start fcoe [__qedf_probe:3041]:6: Cannot start FCo

Re: [RFC v5 34/38] procfs: display the protection-key number associated with a vma

2017-07-13 Thread Ram Pai
On Thu, Jul 13, 2017 at 07:07:48AM -0700, Dave Hansen wrote: > On 07/13/2017 01:03 AM, Ram Pai wrote: > > On Tue, Jul 11, 2017 at 11:13:56AM -0700, Dave Hansen wrote: > >> On 07/05/2017 02:22 PM, Ram Pai wrote: > >>> +#ifdef CONFIG_PPC64_MEMORY_PROTECTION_KEYS > >>> +void arch_show_smap(struct seq_

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-13 Thread Stefan Berger
On 07/13/2017 12:40 PM, Theodore Ts'o wrote: On Thu, Jul 13, 2017 at 07:11:36AM -0500, Eric W. Biederman wrote: The concise summary: Today we have the xattr security.capable that holds a set of capabilities that an application gains when executed. AKA setuid root exec without actually being se

Re: [PATCH] lustre: check copy_from_iter/copy_to_iter return code

2017-07-13 Thread James Simmons
> We now get a helpful warning for code that calls copy_{from,to}_iter > without checking the return value, introduced by commit aa28de275a24 > ("iov_iter/hardening: move object size checks to inlined part"). > > drivers/staging/lustre/lnet/klnds/o2iblnd/o2iblnd_cb.c: In function > 'kiblnd_send'

Re: [PATCH v4 3/3] KVM: nVMX: Emulate EPTP switching for the L1 hypervisor

2017-07-13 Thread Bandan Das
David Hildenbrand writes: + /* + * If the (L2) guest does a vmfunc to the currently + * active ept pointer, we don't have to do anything else + */ + if (vmcs12->ept_pointer != address) { + if (address >> cpuid_maxphyaddr(vcpu) || +

Re: [PATCH] drm/udl: Make page_flip asynchronous

2017-07-13 Thread Daniel Vetter
On Thu, Jul 13, 2017 at 6:25 PM, Stéphane Marchesin wrote: >> Can't we roll this driver over to the atomic helpers instead? There >> you get nonblocking pretty much for free ... I'm not sure extending >> the old modeset code has all that much benefit really. > > This code certainly has value by it

Re: [PATCH 3/4] perf annotate: Support --show-nr-samples option

2017-07-13 Thread Taeung Song
On 07/13/2017 05:13 AM, Arnaldo Carvalho de Melo wrote: Em Wed, Jul 12, 2017 at 07:14:14AM +0900, Taeung Song escreveu: Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. Additionally h->sum is properly renamed h->nr_samples. Please first do a patch renami

Re: [PATCH] nvmet: preserve controller serial number between reboots

2017-07-13 Thread Christoph Hellwig
On Thu, Jul 13, 2017 at 07:38:27PM +0300, Sagi Grimberg wrote: > >>> It seems weird that a subsystem has a serial. >> >> But that's actually how NVMe defines them. > > Where is that wording? Right there in the SN field description: "Serial Number (SN): Contains the serial number for the NVM subsy

Re: [PATCH V4] PCI: handle CRS returned by device after FLR

2017-07-13 Thread Keith Busch
On Thu, Jul 13, 2017 at 12:42:44PM -0400, Sinan Kaya wrote: > On 7/13/2017 12:29 PM, Keith Busch wrote: > > That wording is just confusing. It looks to me the 1 second polling is > > to be used following a reset if CRS is not implemented. > > > > > > https://pcisig.com/sites/default/files/speci

Re: [PATCH v5 1/2] clk: gate: expose clk_gate_ops::is_enabled

2017-07-13 Thread Florian Fainelli
On 07/13/2017 07:02 AM, gabriel.fernan...@st.com wrote: > From: Gabriel Fernandez > > This patch exposes clk_gate_ops::is_enabled as functions > that can be directly called and assigned in places like this so > we don't need wrapper functions that do nothing besides forward > the call. > > Signe

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-13 Thread Eric W. Biederman
Theodore Ts'o writes: > On Thu, Jul 13, 2017 at 07:11:36AM -0500, Eric W. Biederman wrote: >> The concise summary: >> >> Today we have the xattr security.capable that holds a set of >> capabilities that an application gains when executed. AKA setuid root exec >> without actually being setuid ro

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-13 Thread Stefan Berger
On 07/13/2017 01:14 PM, Eric W. Biederman wrote: Theodore Ts'o writes: On Thu, Jul 13, 2017 at 07:11:36AM -0500, Eric W. Biederman wrote: The concise summary: Today we have the xattr security.capable that holds a set of capabilities that an application gains when executed. AKA setuid root e

[PATCH 6/6] perf/x86/uncore: fix missing marker for skx_uncore_cha_extra_regs

2017-07-13 Thread kan . liang
From: Stephane Eranian This skx_uncore_cha_extra_regs array was missing an end-marker. Signed-off-by: Stephane Eranian Signed-off-by: Kan Liang --- arch/x86/events/intel/uncore_snbep.c | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/x86/events/intel/uncore_snbep.c b/arch/x86/events/

[PATCH 4/6] perf/x86/uncore: remove invalid Skylake server CHA filter field

2017-07-13 Thread kan . liang
From: Kan Liang There is no field c6 and link for CHA BOX FILTER. Signed-off-by: Kan Liang --- arch/x86/events/intel/uncore_snbep.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/arch/x86/events/intel/uncore_snbep.c b/arch/x86/events/intel/uncore_snbep.c index a30bf97..2401d06 100644 -

[PATCH 3/6] perf/x86/uncore: fix Skylake server CHA LLC_LOOKUP event umask

2017-07-13 Thread kan . liang
From: Kan Liang Correct the umask for LLC_LOOKUP.LOCAL and LLC_LOOKUP.REMOTE events Signed-off-by: Kan Liang --- arch/x86/events/intel/uncore_snbep.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/events/intel/uncore_snbep.c b/arch/x86/events/intel/uncore_snb

[PATCH 0/6] several fixes for SKX uncore

2017-07-13 Thread kan . liang
From: Kan Liang This series fixes the following issues: - wrong event_mask and event_ext_mask for UPI PMU - wrong PCU event format - wrong umask for LLC_LOOKUP.LOCAL and LLC_LOOKUP.REMOTE events - invalid CHA BOX FILTER - missing event filters support for TOR_INSERTS and TOR_OCCUPANCY

[PATCH 5/6] perf/x86/uncore: fix SKX CHA event extra regs

2017-07-13 Thread kan . liang
From: Stephane Eranian This patch adds two missing event extra regs for Skylake Server CHA PMU: - TOR_INSERTS - TOR_OCCUPANCY Were missing support for all the filters, including opcode matchers. Signed-off-by: Stephane Eranian Signed-off-by: Kan Liang --- arch/x86/events/intel/uncore_snbe

Re: [PATCH] gpio: altera-a10sr: constify gpio_chip structure

2017-07-13 Thread Thor Thayer
On 07/11/2017 05:15 PM, Gustavo A. R. Silva wrote: This structure is only used to copy into another structure, so declare it as const. Signed-off-by: Gustavo A. R. Silva --- drivers/gpio/gpio-altera-a10sr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpio/

[PATCH 1/6] perf/x86/uncore: fix Skylake UPI PMU event masks

2017-07-13 Thread kan . liang
From: Stephane Eranian This patch fixes the event_mask and event_ext_mask for the Intel Skylake Server UPI PMU. Bit 21 is not used as a filter. The extended umask is from bit 32 to bit 55. Correct both umasks. Signed-off-by: Stephane Eranian Signed-off-by: Kan Liang --- arch/x86/events/intel/

[PATCH 2/6] perf/x86/uncore: fix Skylake server PCU PMU event format

2017-07-13 Thread kan . liang
From: Kan Liang PCU event format for SKX are different from snbep. Introduce a new format group for SKX PCU. Signed-off-by: Kan Liang --- arch/x86/events/intel/uncore_snbep.c | 22 +- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/arch/x86/events/intel/uncore

Re: [PATCH v1] xen/grant-table: log the lack of grants

2017-07-13 Thread Wengang
Hi, Anyone can you please review this patch? thanks, wengang On 07/07/2017 11:23 AM, Wengang Wang wrote: log a message when we enter this situation: 1) we already allocated the max number of available grants from hypervisor and 2) we still need more (but the request fails because of 1)). So

Re: [Intel-gfx] [PATCH RESEND] drm/i915: Fix pipe/transcoder enum mismatches

2017-07-13 Thread Ville Syrjälä
On Thu, Jul 13, 2017 at 09:23:11AM -0700, Stéphane Marchesin wrote: > On Thu, Jul 13, 2017 at 3:13 AM, Ville Syrjälä > wrote: > > On Wed, Jul 12, 2017 at 07:28:14PM -0700, Stéphane Marchesin wrote: > >> On Fri, May 5, 2017 at 10:40 AM, Ville Syrjälä > >> wrote: > >> > > >> > On Fri, May 05, 2017

Re: [RFC 0/2] arm-smmu-v3 tlbi-on-map option

2017-07-13 Thread Michael S. Tsirkin
On Thu, Jul 13, 2017 at 10:29:42AM +0100, Jean-Philippe Brucker wrote: > On 12/07/17 23:07, Michael S. Tsirkin wrote: > [...] > > I think using hardware support for nesting is the right final > > solution. It will take some time though. Given this, what should > > we do meanwhile? > > > > Assuming

[PATCH v2 0/9] perf annotate: Fix --show-total-period and support --show-nr-samples

2017-07-13 Thread Taeung Song
Hello, Currently the --show-total-period option of perf-annotate is different from perf-report's. It has two problem like below: (Reported by Namhyung Kim and Milian Wolff) 1) Wrong column i.e. 'Percent' (even though using --show-total-period) 2) Show number of samples, not period So fix th

[PATCH v2 1/9] perf annotate: Introduce struct sym_hist_entry

2017-07-13 Thread Taeung Song
struct sym_hist has addr[] but it should have not only number of samples but also the sample period. So use new struct symhist_entry for that. Cc: Namhyung Kim Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/perf/ui/browsers/annotate.c | 6 +++--- tools/perf/ui/gtk/annotate.c | 4 ++-

[PATCH v2 4/9] perf annotate: Show the proper header when using --show-total-period

2017-07-13 Thread Taeung Song
Currently a first column is only "Percent", so fix it to show correct column name based on given options. (e.g. if using --show-total-period, show "Event count" as a first column) Reported-by: Milian Wolff Cc: Namhyung Kim Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/perf/util/annotate.

[PATCH v2 3/9] perf annotate: Fix wrong --show-total-period option showing number of samples

2017-07-13 Thread Taeung Song
Currently the --show-total-period option of perf-annotate is different from perf-report's. For example, perf-report ordinarily shows period and number of samples. # Overhead SamplesPeriod Command Shared Object Symbol # ... ...

[PATCH v2 8/9] perf annotate browser: Support the toggle number of samples with a 'e' hotkey

2017-07-13 Thread Taeung Song
Cc: Namhyung Kim Cc: Milian Wolff Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/perf/ui/browsers/annotate.c | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui/browsers/annotate.c index 34b3189..19173b1

[PATCH v2 9/9] perf annotate: Use the sample period when calculating the percentage

2017-07-13 Thread Taeung Song
Currently the percentages of perf-annotate are calculated with number of samples, not the sample period. So fix it to correspond with perf-report using the sample period for the calculation. Cc: Namhyung Kim Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/perf/util/annotate.c | 11 ++---

[PATCH v2 7/9] perf annotate: Support --show-nr-samples option

2017-07-13 Thread Taeung Song
Add --show-nr-samples option to perf-annotate so that it corresponds with perf-report. Cc: Namhyung Kim Cc: Milian Wolff Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/perf/builtin-annotate.c | 2 ++ tools/perf/util/annotate.c| 6 +- 2 files changed, 7 insertions(+), 1 deletion(-)

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-13 Thread Eric W. Biederman
Stefan Berger writes: > On 07/13/2017 12:40 PM, Theodore Ts'o wrote: >> On Thu, Jul 13, 2017 at 07:11:36AM -0500, Eric W. Biederman wrote: >>> The concise summary: >>> >>> Today we have the xattr security.capable that holds a set of >>> capabilities that an application gains when executed. AKA s

[PATCH v2 6/9] perf annotate browser: Show the proper header when using --show-total-period

2017-07-13 Thread Taeung Song
Currently a first column is only "Percent", so fix it to show correct column name based on given options. (e.g. if using --show-total-period or a 't' hotkey, show "Event count" as a first column) Reported-by: Milian Wolff Cc: Namhyung Kim Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/per

[PATCH v2 2/9] perf annotate: Properly rename 'sum' to 'total_samples' in struct sym_hist

2017-07-13 Thread Taeung Song
Cc: Namhyung Kim Cc: Jiri Olsa Signed-off-by: Taeung Song --- tools/perf/ui/gtk/annotate.c | 2 +- tools/perf/util/annotate.c | 22 +++--- tools/perf/util/annotate.h | 2 +- 3 files changed, 13 insertions(+), 13 deletions(-) diff --git a/tools/perf/ui/gtk/annotate.c b/too

[PATCH v2 5/9] perf anntoate browser: Fix the toggle total period view to show period, not number of samples

2017-07-13 Thread Taeung Song
Currently the toggle total period view on the annotate TUI shows the number of samples, not period like below. So fix the toggle total period view on the annotate TUI like below. $ perf annotate --show-total-period Before: │Disassembly of section .text: │ │

Re: [PATCH V1]: clk: qcom: support qpnp clock divider configuration

2017-07-13 Thread Stephen Boyd
On 07/13, Tirupathi Reddy wrote: > This patch adds a new device driver for configuring the clkdiv module > presents on the Qualcomm QPNP PMIC. The driver configures the frequency > of clkdiv outputs. The driver provides callback functions for different > clock operations and register the clkdiv mod

Re: [PATCH v3 01/18] clk: ingenic: Use const pointer to clk_ops in struct

2017-07-13 Thread Stephen Boyd
On 07/13, Ralf Baechle wrote: > On Thu, Jul 13, 2017 at 12:07:25PM +0200, Paul Cercueil wrote: > > > > Sorry I forgot, did you want an ack for these clk patches or for > > > me to take them through clk tree. If it's the ack case, > > > > > > Acked-by: Stephen Boyd > > > > > > for patches 1 thro

[PATCH][net-next] svcrdma: fix an incorrect check on -E2BIG and -EINVAL

2017-07-13 Thread Colin King
From: Colin Ian King The current check will always be true and will always jump to err1, this looks dubious to me. I believe && should be used instead of ||. Detected by CoverityScan, CID#1450120 ("Logically Dead Code") Fixes: 107c1d0a991a ("svcrdma: Avoid Send Queue overflow") Signed-off-by: C

Re: [PATCH v2 07/22] fpga: intel: pcie: parse feature list and create platform device for features.

2017-07-13 Thread Alan Tull
On Sun, Jun 25, 2017 at 8:52 PM, Wu Hao wrote: > From: Xiao Guangrong > > Device Feature List structure creates a link list of feature headers > within the MMIO space to provide an extensible way of adding features. > > The Intel FPGA PCIe driver walks through the feature headers to enumerate > f

Re: [PATCH][net-next] svcrdma: fix an incorrect check on -E2BIG and -EINVAL

2017-07-13 Thread Chuck Lever
> On Jul 13, 2017, at 1:51 PM, Colin King wrote: > > From: Colin Ian King > > The current check will always be true and will always jump to > err1, this looks dubious to me. I believe && should be used > instead of ||. > > Detected by CoverityScan, CID#1450120 ("Logically Dead Code") > > Fix

Re: [PATCH] Documentation: ABI: mtd: describe "offset" more precisely

2017-07-13 Thread Brian Norris
On Sun, Jun 25, 2017 at 01:11:54PM +0200, Rafał Miłecki wrote: > From: Rafał Miłecki > > So far Linux supported only two levels of MTD devices so we didn't need > a very precise description for this sysfs file. With commit > 97519dc52b44a ("mtd: partitions: add support for subpartitions") there >

Re: [kernel-hardening] Re: [RFC PATCH 6/6] arm64: add VMAP_STACK and detect out-of-bounds SP

2017-07-13 Thread Mark Rutland
On Thu, Jul 13, 2017 at 05:10:50PM +0100, Mark Rutland wrote: > On Thu, Jul 13, 2017 at 12:49:48PM +0100, Ard Biesheuvel wrote: > > On 13 July 2017 at 11:49, Mark Rutland wrote: > > > On Thu, Jul 13, 2017 at 07:58:50AM +0100, Ard Biesheuvel wrote: > > >> On 12 July 2017 at 23:33, Mark Rutland wro

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-13 Thread Eric W. Biederman
Stefan Berger writes: > On 07/13/2017 01:14 PM, Eric W. Biederman wrote: >> Theodore Ts'o writes: >> >>> On Thu, Jul 13, 2017 at 07:11:36AM -0500, Eric W. Biederman wrote: The concise summary: Today we have the xattr security.capable that holds a set of capabilities that an a

Re: [PATCH v12 8/8] virtio-balloon: VIRTIO_BALLOON_F_CMD_VQ

2017-07-13 Thread Michael S. Tsirkin
On Thu, Jul 13, 2017 at 04:46:29PM +0800, Wei Wang wrote: > On 07/13/2017 08:22 AM, Michael S. Tsirkin wrote: > > On Wed, Jul 12, 2017 at 08:40:21PM +0800, Wei Wang wrote: > > > Add a new vq, cmdq, to handle requests between the device and driver. > > > > > > This patch implements two commands sen

Re: [PATCH] Revert "x86/uaccess: Add stack frame output operand in get_user() inline asm"

2017-07-13 Thread Josh Poimboeuf
On Wed, Jul 12, 2017 at 04:22:13PM -0700, Matthias Kaehlcke wrote: > El Wed, Jul 12, 2017 at 05:36:30PM -0500 Josh Poimboeuf ha dit: > > > On Wed, Jul 12, 2017 at 05:35:47PM -0500, Josh Poimboeuf wrote: > > > On Wed, Jul 12, 2017 at 03:20:40PM -0700, Matthias Kaehlcke wrote: > > > > > This is admi

[PATCH] sctp: don't dereference ptr before leaving _sctp_walk_{params,errors}()

2017-07-13 Thread Alexander Potapenko
If the iterator (|pos.p| or |err|) has already reached the end of chunk, we shouldn't access iterator->length. This bug has been detected by KMSAN. For the following pair of system calls: socket(PF_INET6, SOCK_STREAM, 0x84 /* IPPROTO_??? */) = 3 sendto(3, "A", 1, MSG_OOB, {sa_family=AF_INET6,

[GIT PULL] MTD updates for 4.13-rc1

2017-07-13 Thread Brian Norris
/for-linus-20170713 for you to fetch changes up to 7d84120b5ba61912a5333f5fe2c4e8f35ef9514f: Documentation: ABI: mtd: describe "offset" more precisely (2017-07-13 10:54:45 -0700) MTD updates for v4.13-rc1: Gener

Re: [RFC PATCH 1/1] mm/mremap: add MREMAP_MIRROR flag for existing mirroring functionality

2017-07-13 Thread Mike Kravetz
On 07/13/2017 09:30 AM, Andrea Arcangeli wrote: > On Thu, Jul 13, 2017 at 09:01:54AM -0700, Mike Kravetz wrote: >> Sent a patch (in separate e-mail thread) to return EINVAL for private >> mappings. > > The way old_len == 0 behaves for MAP_PRIVATE seems more sane to me > than the alternative of cop

Re: [PATCH v7 3/3] net/cxgb4: Use new PCI_DEV_FLAGS_NO_RELAXED_ORDERING flag

2017-07-13 Thread Alexander Duyck
On Thu, Jul 13, 2017 at 7:21 AM, Ding Tianhong wrote: > From: Casey Leedom > > cxgb4 Ethernet driver now queries PCIe configuration space to determine > if it can send TLPs to it with the Relaxed Ordering Attribute set. > > Remove the enable_pcie_relaxed_ordering() to avoid enable PCIe Capability

Re: [PATCH] sctp: don't dereference ptr before leaving _sctp_walk_{params,errors}()

2017-07-13 Thread Alexander Potapenko
On Thu, Jul 13, 2017 at 8:10 PM, Alexander Potapenko wrote: > If the iterator (|pos.p| or |err|) has already reached the end of > chunk, we shouldn't access iterator->length. > > This bug has been detected by KMSAN. For the following pair of system > calls: > > socket(PF_INET6, SOCK_STREAM, 0x84

Re: [PATCH v7 3/3] net/cxgb4: Use new PCI_DEV_FLAGS_NO_RELAXED_ORDERING flag

2017-07-13 Thread Alexander Duyck
On Thu, Jul 13, 2017 at 11:14 AM, Alexander Duyck wrote: > On Thu, Jul 13, 2017 at 7:21 AM, Ding Tianhong > wrote: >> From: Casey Leedom >> >> cxgb4 Ethernet driver now queries PCIe configuration space to determine >> if it can send TLPs to it with the Relaxed Ordering Attribute set. >> >> Remo

Re: [alsa-devel] [PATCH] ASoC: fsl_asrc: constify snd_soc_dai_ops structure

2017-07-13 Thread Joe Perches
On Thu, 2017-07-13 at 10:18 -0500, Gustavo A. R. Silva wrote: > Hi Mark, > > Quoting Mark Brown : > > > On Thu, Jul 13, 2017 at 09:32:41AM +0200, Takashi Iwai wrote: > > > > > please stop posting in this style. It's really annoying to see > > > spontaneously popping-up almost same patch for mor

Re: [PATCH] sctp: don't dereference ptr before leaving _sctp_walk_{params,errors}()

2017-07-13 Thread Alexander Potapenko
On Thu, Jul 13, 2017 at 8:14 PM, Alexander Potapenko wrote: > On Thu, Jul 13, 2017 at 8:10 PM, Alexander Potapenko > wrote: >> If the iterator (|pos.p| or |err|) has already reached the end of >> chunk, we shouldn't access iterator->length. >> >> This bug has been detected by KMSAN. For the foll

Re: [GIT PULL] RTC for 4.13

2017-07-13 Thread Alexandre Belloni
Me again... On 13/07/2017 at 11:22:39 +0200, Alexandre Belloni wrote: > On 13/07/2017 at 11:01:51 +0200, Alexandre Belloni wrote: > > Hi Linus, > > > > Here is the pull-request for the RTC subsystem for 4.13. > > > > And I forgot to add that you will see a small Add/Add merge conflict in > tool

Re: Decreasing time for `rsa_init`

2017-07-13 Thread Paul Menzel
Dear Stephan, Am Mittwoch, den 12.07.2017, 19:38 +0200 schrieb Paul Menzel: > On 07/12/17 19:28, Stephan Müller wrote: > > Am Mittwoch, 12. Juli 2017, 12:59:58 CEST schrieb Paul Menzel: > > > Building CRYPTO_RSA not as module, but into the Linux kernel, > > > `rsa_init()` takes 130 ms on an ASRo

[PATCH] perf: xgene: Remove unnecessary managed resources cleanup

2017-07-13 Thread Tai Nguyen
Managed resources in the driver should be automatically cleaned up on driver detach. It's unnecessary to manually free/unmmap these resources. One of the manual cleanup causes static checkers to complain. The bug is reported by Dan Carpenter in [1] [1] https://www.spinics.net/lists/arm-kernel/msg

Re: [RFC PATCH v1 00/11] Create fast idle path for short idle periods

2017-07-13 Thread Peter Zijlstra
On Thu, Jul 13, 2017 at 11:13:28PM +0800, Li, Aubrey wrote: > On 2017/7/13 22:53, Peter Zijlstra wrote: > > Fixing C-state selection by creating an alternative idle path sounds so > > very wrong. > > This only happens on the arch which has multiple hardware idle cstates, like > Intel's processor.

Re: [kernel-hardening] Re: [RFC PATCH 6/6] arm64: add VMAP_STACK and detect out-of-bounds SP

2017-07-13 Thread Ard Biesheuvel
On 13 July 2017 at 18:55, Mark Rutland wrote: > On Thu, Jul 13, 2017 at 05:10:50PM +0100, Mark Rutland wrote: >> On Thu, Jul 13, 2017 at 12:49:48PM +0100, Ard Biesheuvel wrote: >> > On 13 July 2017 at 11:49, Mark Rutland wrote: >> > > On Thu, Jul 13, 2017 at 07:58:50AM +0100, Ard Biesheuvel wrote

Re: [PATCH 02/17] pci: Add a generic, weakly-linked pcibios_align_resource

2017-07-13 Thread Palmer Dabbelt
On Wed, 12 Jul 2017 15:50:42 PDT (-0700), helg...@kernel.org wrote: > On Tue, Jul 11, 2017 at 06:31:15PM -0700, Palmer Dabbelt wrote: >> Multiple architectures define this as trivial function, and I'm adding >> another one as part of the RISC-V port. This adds a __weak version of >> pcibios_align_

Re: [PATCH] sctp: don't dereference ptr before leaving _sctp_walk_{params,errors}()

2017-07-13 Thread David Miller
From: Alexander Potapenko Date: Thu, 13 Jul 2017 20:10:34 +0200 > diff --git a/include/net/sctp/sctp.h b/include/net/sctp/sctp.h > index a9519a06a23b..f13632ee33f0 100644 > --- a/include/net/sctp/sctp.h > +++ b/include/net/sctp/sctp.h > @@ -469,6 +469,7 @@ _sctp_walk_params((pos), (chunk), > nto

Re: [PATCH 13/21] x86/intel_rdt/cqm: Add cpus file support

2017-07-13 Thread Shivappa Vikas
On Thu, 6 Jul 2017, Thomas Gleixner wrote: On Thu, 6 Jul 2017, Shivappa Vikas wrote: On Sun, 2 Jul 2017, Thomas Gleixner wrote: + /* Check whether cpus belong to parent ctrl group */ + cpumask_andnot(tmpmask, newmask, &pr->cpu_mask); + if (cpumask_weight(tmpmask)) { +

[GIT PULL] VFIO updates for v4.13-rc1

2017-07-13 Thread Alex Williamson
The following changes since commit 32c1431eea4881a6b17bd7c639315010aeefa452: Linux 4.12-rc5 (2017-06-11 16:48:20 -0700) are available in the git repository at: git://github.com/awilliam/linux-vfio.git tags/vfio-v4.13-rc1 for you to fetch changes up to 7f56c30bd0a232822aca38d288da475613bdff9

[PATCH] x86/platform/uv/BAU: Fix congested_response_us not taking effect

2017-07-13 Thread Justin Ernst
Bug fix for the BAU tunable congested_cycles not being set to the user defined value. Instead of referencing a global variable when deciding on BAU shutdown, a node will reference its own tunable set value ( cong_response_us). This results in the user set tunable value congested_response_us taking

RE: [PATCH 4.4 29/57] RDMA/uverbs: Check port number supplied by user verbs cmds

2017-07-13 Thread Ismail, Mustafa
> -Original Message- > From: Greg Kroah-Hartman [mailto:gre...@linuxfoundation.org] > Sent: Thursday, July 13, 2017 11:26 AM > To: Ismail, Mustafa > Cc: linux-kernel@vger.kernel.org; linux-r...@vger.kernel.org; > sta...@vger.kernel.org; Yevgeny Kliteynik ; > Tziporet Koren ; Alex Polak > ;

Re: [PATCH] Revert "x86/uaccess: Add stack frame output operand in get_user() inline asm"

2017-07-13 Thread Matthias Kaehlcke
El Thu, Jul 13, 2017 at 01:00:01PM -0500 Josh Poimboeuf ha dit: > On Wed, Jul 12, 2017 at 04:22:13PM -0700, Matthias Kaehlcke wrote: > > El Wed, Jul 12, 2017 at 05:36:30PM -0500 Josh Poimboeuf ha dit: > > > > > On Wed, Jul 12, 2017 at 05:35:47PM -0500, Josh Poimboeuf wrote: > > > > On Wed, Jul 12

Re: [PATCH v2 2/2] docs: disable KASLR when debugging kernel

2017-07-13 Thread Jonathan Corbet
On Thu, 13 Jul 2017 08:53:59 +0200 Jan Kiszka wrote: > FWIW: Acked-by: Jan Kiszka > > Who's taking this? Jon? I can take them once the merge window closes. Thanks, jon

[PATCH 0/3] Extend BGMAC driver for Stingray SoC

2017-07-13 Thread Abhishek Shah
The patchset extends Broadcom BGMAC driver for Broadcom Stingray SoC. This patchset is based on Linux-4.12 and tested on NS2 and Stingray. Abhishek Shah (3): net: ethernet: bgmac: Remove unnecessary 'return' from platform_bgmac_idm_write net: ethernet: bgmac: Make IDM register space optio

[PATCH 1/3] net: ethernet: bgmac: Remove unnecessary 'return' from platform_bgmac_idm_write

2017-07-13 Thread Abhishek Shah
Return type for idm register write callback should be void as 'writel' API is used for write operation. However, there no need to have 'return' in this function. Signed-off-by: Abhishek Shah Reviewed-by: Oza Oza Reviewed-by: Ray Jui Reviewed-by: Scott Branden --- drivers/net/ethernet/broadcom

[PATCH 3/3] Documentation: devicetree: net: optional idm regs for bgmac

2017-07-13 Thread Abhishek Shah
Specifying IDM register space in DT is not mendatory for SoCs where firmware takes care of IDM operations. This patch updates BGMAC driver's DT binding documentation indicating the same. Signed-off-by: Abhishek Shah Reviewed-by: Ray Jui Reviewed-by: Oza Oza Reviewed-by: Scott Branden --- Docu

[PATCH 2/3] net: ethernet: bgmac: Make IDM register space optional

2017-07-13 Thread Abhishek Shah
IDM operations are usually one time ops and should be done in firmware itself. Driver is not supposed to touch IDM registers. However, for some SoCs', driver is performing IDM read/writes. So this patch masks IDM operations in case firmware is taking care of IDM operations. Signed-off-by: Abhishe

[GIT PULL] platform-drivers-x86 for 4.13-2

2017-07-13 Thread Darren Hart
Hi Linus, I'm following your advice from: https://www.spinics.net/lists/kernel/msg2492566.html and including a cherry-picked patch from my previous 'fixes' pull request, rather than merging the 'fixes' branch into the development branches. I'm going to keep an eye on these types of commits going

Re: [PATCH] mm/mremap: Fail map duplication attempts for private mappings

2017-07-13 Thread Vlastimil Babka
[+CC linux-api] On 07/13/2017 05:58 PM, Mike Kravetz wrote: > mremap will create a 'duplicate' mapping if old_size == 0 is > specified. Such duplicate mappings make no sense for private > mappings. If duplication is attempted for a private mapping, > mremap creates a separate private mapping unr

Re: [PATCH v5 2/6] clk: sunxi-ng: Add sun4i/sun7i CCU driver

2017-07-13 Thread Priit Laes
On Sun, Jul 09, 2017 at 10:25:23PM +1000, Jonathan Liu wrote: > Hi Priit, > > On 5 July 2017 at 06:04, Priit Laes wrote: > > Introduce a clock controller driver for sun4i A10 and sun7i A20 > > series SoCs. > > > > Signed-off-by: Priit Laes > > --- > > drivers/clk/sunxi-ng/Kconfig |

Re: [PATCH v2] xattr: Enable security.capability in user namespaces

2017-07-13 Thread Theodore Ts'o
On Thu, Jul 13, 2017 at 12:39:10PM -0500, Eric W. Biederman wrote: > > Can you define what 'scalable' means for you in this context? > > From what I can see sharing a filesystem between multiple containers > > doesn't 'scale well' for virtualizing the xattrs primarily because of > > size limitation

Re: [RFC 0/2] arm-smmu-v3 tlbi-on-map option

2017-07-13 Thread Jean-Philippe Brucker
On 13/07/17 18:44, Michael S. Tsirkin wrote: > On Thu, Jul 13, 2017 at 10:29:42AM +0100, Jean-Philippe Brucker wrote: >> On 12/07/17 23:07, Michael S. Tsirkin wrote: >> [...] >>> I think using hardware support for nesting is the right final >>> solution. It will take some time though. Given this, w

[PATCH] mtd: gpmi-nand: do not fail setting ONFI timing mode if available

2017-07-13 Thread Miquel Raynal
GPMI NFC driver fails to apply timing mode if the ->onfi_get_features() does not return the mode that was previously applied. We can assume that a nand chip supports a timing as long as it is read from the ONFI parameter page. Reading back a different mode than the one previously applied does not

Re: [linux-sunxi] [PATCH v5 2/6] clk: sunxi-ng: Add sun4i/sun7i CCU driver

2017-07-13 Thread Priit Laes
On Mon, Jul 10, 2017 at 11:45:32AM +0200, Olliver Schinagl wrote: > Hi Pleas, > > again, but this time with content :) > > On 04-07-17 22:04, Priit Laes wrote: > >Introduce a clock controller driver for sun4i A10 and sun7i A20 > >series SoCs. [ ... ] > >+++ b/drivers/clk/sunxi-ng/Kconfig > >@@

Re: Potential scheduler regression

2017-07-13 Thread Ben Guthro
On Tue, Jul 11, 2017 at 5:55 AM, Greg KH wrote: > On Tue, Jul 11, 2017 at 10:30:14AM +0200, Ingo Molnar wrote: >> >> * Ben Guthro wrote: >> >> > > If people have experience with these in the "enterprise" distros, or any >> > > other >> > > tree, and want to provide me with backported, and tested

Re: [PATCH] Revert "x86/uaccess: Add stack frame output operand in get_user() inline asm"

2017-07-13 Thread Josh Poimboeuf
On Thu, Jul 13, 2017 at 11:47:48AM -0700, Matthias Kaehlcke wrote: > > What happens if you try the below patch instead of the revert? Any > > chance the offending instruction goes away? > > > > diff --git a/arch/x86/include/asm/uaccess.h b/arch/x86/include/asm/uaccess.h > > index 11433f9..beac907

<    1   2   3   4   5   6   7   8   9   >