[PATCH 5/5] perf/probe: Change MAX_CMDLEN

2017-02-02 Thread Ravi Bangoria
There are many SDT markers in powerpc whose uprobe definition goes beyond current MAX_CMDLEN, especially when target filename is long and sdt marker has long list of arguments. For example, definition of sdt marker method__compile__end: 8@17 8@9 8@10 -4@8 8@7 -4@6 8@5 -4@4 1@37(28) from file

[PATCH 3/5] perf/sdt/x86: Move OP parser to tools/perf/arch/x86/

2017-02-02 Thread Ravi Bangoria
SDT marker argument is in N@OP format. N is the size of argument and OP is the actual assembly operand. OP is arch dependent component and hence it's parsing logic also should be placed under tools/perf/arch/. Signed-off-by: Ravi Bangoria --- tools/perf/arch/x86/util/perf_regs.c | 93 ++

[PATCH 4/5] perf/sdt/powerpc: Add argument support

2017-02-02 Thread Ravi Bangoria
SDT marker argument is in N@OP format. Here OP is arch dependent component. Add powerpc logic to parse OP and convert it to uprobe compatible format. Signed-off-by: Ravi Bangoria --- tools/perf/arch/powerpc/util/perf_regs.c | 115 +++ 1 file changed, 115 insertions(+)

[PATCH 2/5] perf/sdt/x86: Add renaming logic for rNN and other registers

2017-02-02 Thread Ravi Bangoria
'perf probe' is failing for sdt markers whose arguments has rNN (with postfix b/w/d), %rsp, %esp, %sil etc. registers. Add renaming logic for these registers. Signed-off-by: Ravi Bangoria --- tools/perf/arch/x86/util/perf_regs.c | 44 ++-- 1 file changed, 32 inser

Re: Cherryview wake up events

2017-02-02 Thread Johannes Stezenbach
On Thu, Feb 02, 2017 at 12:31:22PM +0200, Mika Westerberg wrote: > On Thu, Feb 02, 2017 at 10:52:00AM +0100, Johannes Stezenbach wrote: > > OperationRegion (GPOP, GeneralPurposeIo, Zero, 0x0100) > > Field (GPOP, ByteAcc, NoLock, Preserve) > > { > >

[PATCH 0/5] perf/sdt: Argument support for x86 and powepc

2017-02-02 Thread Ravi Bangoria
The v5 patchset for sdt marker argument support for x86 [1] has couple of issues. For example, it still has x86 specific code in general code. It lacks support for rNN (with size postfix b/w/d), %rsp, %esp, %sil etc. registers and such sdt markers are failing at 'perf probe'. It also fails to conv

Re: [PATCH v3 0/2] iov_iter: allow iov_iter_get_pages_alloc to allocate more pages per call

2017-02-02 Thread Al Viro
On Thu, Feb 02, 2017 at 02:56:51AM -0800, Christoph Hellwig wrote: > > I really wonder if this is the right approach. Most of the users of > > iov_iter_get_pages()/iov_iter_get_pages_alloc() look like they want > > something like > > iov_iter_for_each_page(iter, size, f, data) > > with int (*f

Re: [PATCH] staging: lustre: shut up clang warnings on CLASSERT()

2017-02-02 Thread Arnd Bergmann
On Feb 2, 2017 11:43 AM, "Greg Kroah-Hartman" wrote: > On Thu, Feb 02, 2017 at 11:40:32AM +0100, Arnd Bergmann wrote: > > I've done a semi-automated patch to replace CLASSERT() with the respective > > BUILD_BUG_ON() now, but that patch is quite large. > > Should be easy to script, I missed that th

Re: [PATCH 2/2] cpufreq: brcmstb-avs-cpufreq: properly retrieve P-state upon suspend

2017-02-02 Thread Rafael J. Wysocki
On Thu, Feb 2, 2017 at 4:33 AM, Viresh Kumar wrote: > On 02-02-17, 00:44, Rafael J. Wysocki wrote: >> On Tuesday, January 31, 2017 10:53:01 AM Markus Mayer wrote: >> > On 5 January 2017 at 20:11, Viresh Kumar wrote: >> > > On 19-12-16, 12:10, Markus Mayer wrote: >> > >> From: Markus Mayer >> > >

Re: [PATCH 1/2 v3] mm: vmscan: do not pass reclaimed slab to vmpressure

2017-02-02 Thread vinayak menon
On Thu, Feb 2, 2017 at 4:18 PM, Michal Hocko wrote: > On Thu 02-02-17 11:44:22, Michal Hocko wrote: >> On Tue 31-01-17 14:32:08, Vinayak Menon wrote: >> > During global reclaim, the nr_reclaimed passed to vmpressure >> > includes the pages reclaimed from slab. But the corresponding >> > scanned sl

Re: [PATCH 5/6] DT bindings documentation for Broadcom IPROC USB Device controller.

2017-02-02 Thread Raviteja Garimella
Hi Rob, On Wed, Feb 1, 2017 at 9:43 PM, Rob Herring wrote: > On Mon, Jan 30, 2017 at 01:26:12PM +0530, Raviteja Garimella wrote: >> The device node is used for UDCs integrated into Broadcom's >> iProc family of SoCs'. The UDC is based on Synopsys Designware >> Cores AHB Subsystem USB Device Contr

Re: [PATCH 1/2 v3] mm: vmscan: do not pass reclaimed slab to vmpressure

2017-02-02 Thread vinayak menon
On Thu, Feb 2, 2017 at 4:14 PM, Michal Hocko wrote: > > We usually refer to the culprit comment as > Fixes: 6b4f7799c6a5 ("mm: vmscan: invoke slab shrinkers from shrink_zone()") > Thanks for pointing that out Michal. I see that added to the version of patch in mmotm. > To unsubscribe, send a mes

Re: [PATCH v3 3/4] driver property: constify property arrays values

2017-02-02 Thread Rafael J. Wysocki
On Wednesday, February 01, 2017 03:30:03 PM Joe Perches wrote: > On Wed, 2017-02-01 at 09:31 -0800, Dmitry Torokhov wrote: > > Data that is fed into property arrays should not be modified, so let's mark > > relevant pointers as const. This will allow us making source arrays as > > const/__initconst

Re: [PATCH] perf: fix topology test on systems with sparse CPUs

2017-02-02 Thread Jiri Olsa
On Tue, Jan 31, 2017 at 05:03:51PM +0100, Jan Stancek wrote: > On 01/30/2017 07:49 PM, Jiri Olsa wrote: > > so basically we're changing from avail to online cpus > > > > have you checked all the users of this FEATURE > > if such change is ok? > > Jiri, > > It wasn't OK as there are other users w

Re: [PATCH] perf: fix topology test on systems with sparse CPUs

2017-02-02 Thread Jiri Olsa
On Tue, Jan 31, 2017 at 05:03:51PM +0100, Jan Stancek wrote: SNIP > > There are 2 problems wrt. cpu_topology_map on systems with sparse CPUs: > > 1. offline/absent CPUs will have their socket_id and core_id set to -1 >which triggers: >"socket_id number is too big.You may need to upgrade

[PATCH] staging: lustre: remove CLASSERT macro

2017-02-02 Thread Arnd Bergmann
lustre uses a fake switch() statement as a compile-time assert, but unfortunately each use of that causes a warning when building with clang: drivers/staging/lustre/lnet/klnds/socklnd/socklnd.c:2907:2: warning: no case matching constant switch condition '42' drivers/staging/lustre/lnet/klnds/soc

Re: Cherryview wake up events

2017-02-02 Thread Mika Westerberg
On Thu, Feb 02, 2017 at 12:12:22PM +0100, Johannes Stezenbach wrote: > On Thu, Feb 02, 2017 at 12:31:22PM +0200, Mika Westerberg wrote: > > On Thu, Feb 02, 2017 at 10:52:00AM +0100, Johannes Stezenbach wrote: > > > OperationRegion (GPOP, GeneralPurposeIo, Zero, 0x0100) > > >

[PATCH] [media] staging: bcm2835: mark all symbols as 'static'

2017-02-02 Thread Arnd Bergmann
I got a link error in allyesconfig: drivers/staging/media/platform/bcm2835/bcm2835-camera.o: In function `vidioc_enum_framesizes': bcm2835-camera.c:(.text.vidioc_enum_framesizes+0x0): multiple definition of `vidioc_enum_framesizes' drivers/media/platform/vivid/vivid-vid-cap.o:vivid-vid-cap.c:(.t

[PATCH] ext4: fix up huge_fault rename

2017-02-02 Thread Arnd Bergmann
It looks like a search+replace accidentally change one function pointer that should have remained as before, as shown by this new warning: fs/ext4/file.c:279:1: error: 'ext4_dax_huge_fault' defined but not used [-Werror=unused-function] This reverts the one line of the changeset. Fixes: 80a8418

[PATCH] ARM: orion5x: fix Makefile for linkstation-lschl.dtb

2017-02-02 Thread Arnd Bergmann
The rename of orion5x-lschl.dts needs to be reflected in the Makefile: make[3]: *** No rule to make target 'arch/arm/boot/dts/orion5x-lschl.dtb', needed by '__build'. Fixes: 6cfd3cd8d836 ("ARM: dts: orion5x-lschl: More consistent naming on linkstation series") Signed-off-by: Arnd Bergmann ---

Re: ibmvtpm byteswapping inconsistency

2017-02-02 Thread Michal Suchánek
On Wed, 1 Feb 2017 23:40:33 -0500 Vicky wrote: > > On Jan 26, 2017, at 5:58 PM, Ashley Lai > > wrote: > > > > Adding Vicky from IBM. > > > > > > On 01/26/2017 04:05 PM, Jason Gunthorpe wrote: > >> On Thu, Jan 26, 2017 at 09:22:48PM +0100, Michal Such??nek wrote: > >> > >>> This is repeat

Re: [PATCH] ARM: orion5x: fix Makefile for linkstation-lschl.dtb

2017-02-02 Thread Roger Shimizu
Dear Arnd, On Thu, Feb 2, 2017 at 8:38 PM, Arnd Bergmann wrote: > The rename of orion5x-lschl.dts needs to be reflected in the Makefile: Indeed. Sorry that I forgot to do it. And thanks for your patch! Cheers, -- Roger Shimizu, GMT +9 Tokyo PGP/GPG: 4096R/6C6ACD6417B3ACB1

Re: [PATCH v4 13/15] livepatch: change to a per-task consistency model

2017-02-02 Thread Petr Mladek
On Thu 2017-01-19 09:46:21, Josh Poimboeuf wrote: > Change livepatch to use a basic per-task consistency model. This is the > foundation which will eventually enable us to patch those ~10% of > security patches which change function or data semantics. This is the > biggest remaining piece needed

Re: [PATCH v4 1/3] ACPI / bus: Export acpi_of_modalias equiv of of_modalias_node

2017-02-02 Thread Rafael J. Wysocki
On Friday, January 27, 2017 01:30:13 PM Dan O'Donovan wrote: > From: Crestez Dan Leonard > > When using devicetree stuff like i2c_client.name or spi_device.modalias > is initialized to the first DT compatible id with the vendor prefix > stripped. Since some drivers rely on this try to replicate i

Re: [PATCH v4 13/15] livepatch: change to a per-task consistency model

2017-02-02 Thread Petr Mladek
IMPORTANT: Please, forget this version. It is few days old and incomplete and probably wrong. I am sorry for confusion. Best Regards, Petr

Please get back to me

2017-02-02 Thread Qatif Group of Companies
Dear Friend, I would like to discuss a very important issue with you. I am writing to find out if this is your valid email. Please, let me know if this email is valid Kind regards Adrien Saif Attorney to Qatif Group of Companies

Re: [PATCH v3 21/24] media: imx: Add MIPI CSI-2 Receiver subdev driver

2017-02-02 Thread Philipp Zabel
On Fri, 2017-01-06 at 18:11 -0800, Steve Longerbeam wrote: > Adds MIPI CSI-2 Receiver subdev driver. This subdev is required > for sensors with a MIPI CSI2 interface. > > Signed-off-by: Steve Longerbeam > --- > drivers/staging/media/imx/Makefile| 1 + > drivers/staging/media/imx/imx-mi

Re: [PATCH v4 13/15] livepatch: change to a per-task consistency model

2017-02-02 Thread Petr Mladek
!!! This is the right version. I am sorry again for the confusion. !!! > Change livepatch to use a basic per-task consistency model. This is the > foundation which will eventually enable us to patch those ~10% of > security patches which change function or data semantics. This is the > biggest r

Re: [PATCH 1/2 v3] mm: vmscan: do not pass reclaimed slab to vmpressure

2017-02-02 Thread Michal Hocko
On Thu 02-02-17 16:55:49, vinayak menon wrote: > On Thu, Feb 2, 2017 at 4:18 PM, Michal Hocko wrote: > > On Thu 02-02-17 11:44:22, Michal Hocko wrote: > >> On Tue 31-01-17 14:32:08, Vinayak Menon wrote: > >> > During global reclaim, the nr_reclaimed passed to vmpressure > >> > includes the pages r

[PATCH] usb: misc: adutux: remove redundant error check on copy_to_user return code

2017-02-02 Thread Colin King
From: Colin Ian King The 2nd check for a non-zero return from copy_to_user is redundant as it is has already been made a few lines earlier. This check was made redundant because of previous fix to the copy_to_user error return check. Detected by CoverityScan, CID#114347 ("Logically Dead Code")

Re: [PATCH V7 3/4] drm/bridge: Add driver for GE B850v3 LVDS/DP++ Bridge

2017-02-02 Thread Peter Senna Tschudin
On 02 February, 2017 02:46 CET, Emil Velikov wrote: > On 1 February 2017 at 11:35, Daniel Vetter wrote: > > On Wed, Feb 01, 2017 at 10:58:43AM +, Peter Senna Tschudin wrote: > >> Hi Archit, > >> > >> On 01 February, 2017 10:44 CET, Archit Taneja > >> wrote: > >> > >> > > >> > > >> > On

Re: [PATCH 1/2] gpio: Rename devm_get_gpiod_from_child()

2017-02-02 Thread Boris Brezillon
On Thu, 2 Feb 2017 11:53:17 +0100 Linus Walleij wrote: > On Mon, Jan 30, 2017 at 4:41 PM, Boris Brezillon > wrote: > > > Rename devm_get_gpiod_from_child() into > > devm_fwnode_get_gpiod_from_child() to reflect the fact that this > > function is operating on a fwnode object. > > > > Signed-off-

Re: [PATCH] initity: try to improve __nocapture annotations

2017-02-02 Thread Arnd Bergmann
On Thu, Feb 2, 2017 at 1:39 AM, Kees Cook wrote: > On Wed, Feb 1, 2017 at 8:11 AM, Arnd Bergmann wrote: >> -void ACPI_INTERNAL_VAR_XFACE >> +void __unverified_nocapture(3) ACPI_INTERNAL_VAR_XFACE >> acpi_debug_print(u32 requested_debug_level, >> u32 line_number, >>

Re: [PATCH] [media] staging: bcm2835: mark all symbols as 'static'

2017-02-02 Thread Arnd Bergmann
On Thu, Feb 2, 2017 at 12:34 PM, Arnd Bergmann wrote: > I got a link error in allyesconfig: > > drivers/staging/media/platform/bcm2835/bcm2835-camera.o: In function > `vidioc_enum_framesizes': > bcm2835-camera.c:(.text.vidioc_enum_framesizes+0x0): multiple definition of > `vidioc_enum_framesizes

Re: [PATCH 2/2] drm: meson: rename driver name to meson-drm

2017-02-02 Thread Daniel Vetter
On Thu, Feb 02, 2017 at 11:50:59AM +0100, Neil Armstrong wrote: > On 02/02/2017 11:45 AM, Daniel Vetter wrote: > > On Thu, Feb 02, 2017 at 10:47:44AM +0100, Neil Armstrong wrote: > >> The platform driver name is currently "meson" which can lead to some > >> confusion, this patch renames it to "meso

Re: [PATCH] staging: rtl8192u: Adding a blank line after declarations

2017-02-02 Thread Greg KH
On Thu, Feb 02, 2017 at 01:21:13AM +0530, simran singhal wrote: > This patch fixes the checkpatch warning by adding a blank line after > declarations. > WARNING: Missing a blank line after declarations > > Signed-off-by: simran singhal > --- > drivers/staging/rtl8192u/ieee80211/ieee80211_softmac

Re: [PATCH] perf: fix topology test on systems with sparse CPUs

2017-02-02 Thread Jan Stancek
> > > When build_cpu_topo() encounters offline/absent CPUs, > > it fails to find any sysfs entries and returns failure. > > This leads to build_cpu_topology() and write_cpu_topology() > > failing as well. > > > > Because HEADER_CPU_TOPOLOGY has not been written, read leaves > > cpu_topology_map N

Re: [RFC PATCH] x86/CPU/AMD: Bring back Compute Unit ID

2017-02-02 Thread Borislav Petkov
On Wed, Feb 01, 2017 at 11:41:50PM +0100, Borislav Petkov wrote: > More playing with this tomorrow. It is late here and brain wants to > sleep now. Ok, did some measurements of our favourite workload with and without those patches on rc6+tip/master. It is a Kaveri laptop so small, only 2 CUs. per

Re: [Intel-gfx] [PATCH] drm/i915: fix i915 running as dom0 under Xen

2017-02-02 Thread Tvrtko Ursulin
On 02/02/2017 09:47, Juergen Gross wrote: Commit 920cf4194954ec ("drm/i915: Introduce an internal allocator for disposable private objects") introduced a regression for the kernel running as Xen dom0: when switching to graphics mode a GPU HANG occurred. Reason seems to be a missing adaption sim

Re: [PATCH] [media] staging: bcm2835: mark all symbols as 'static'

2017-02-02 Thread Arnd Bergmann
On Thu, Feb 2, 2017 at 1:04 PM, Arnd Bergmann wrote: > On Thu, Feb 2, 2017 at 12:34 PM, Arnd Bergmann wrote: >> I got a link error in allyesconfig: >> Fixes: 7b3ad5abf027 ("staging: Import the BCM2835 MMAL-based V4L2 camera >> driver.") >> Signed-off-by: Arnd Bergmann > > Please disregard this

[PATCH 1/2] staging: rts5208: remove redundant retval status check

2017-02-02 Thread Colin King
From: Colin Ian King The retval status checks in the proceeding do loop return out of function ms_read_attritbute_info if there is an error condition, thus we never reach the end of the loop with retval failed status. Therefore, the retval status check at end of the do loop is redundant and can

[PATCH 2/2] staging: rts5208: remove unncessary result set and check, just return SUCCESS

2017-02-02 Thread Colin King
From: Colin Ian King Minor clean up, there is no need to assign result to zero, then check if it is less than zero. Just return SUCCESS. Signed-off-by: Colin Ian King --- drivers/staging/rts5208/rtsx.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rt

Re: Cherryview wake up events

2017-02-02 Thread Mika Westerberg
On Thu, Feb 02, 2017 at 01:35:08PM +0200, Mika Westerberg wrote: > On Thu, Feb 02, 2017 at 12:12:22PM +0100, Johannes Stezenbach wrote: > > On Thu, Feb 02, 2017 at 12:31:22PM +0200, Mika Westerberg wrote: > > > On Thu, Feb 02, 2017 at 10:52:00AM +0100, Johannes Stezenbach wrote: > > > >

[PATCH] staging: bcm2835: don't mark 'bcm2835_v4l2_debug' as static

2017-02-02 Thread Arnd Bergmann
This one unfortunately slipped through my own build testing, my patch caused a new build error: bcm2835-camera.c:53:12: error: static declaration of 'bcm2835_v4l2_debug' follows non-static declaration We want the symbol to be global as it is indeed used in more than one file and declared 'extern

Re: net/tcp: warning in tcp_try_coalesce/skb_try_coalesce

2017-02-02 Thread Eric Dumazet
On Thu, 2017-02-02 at 10:05 +0100, Dmitry Vyukov wrote: > On Tue, Jan 31, 2017 at 3:27 PM, 'Eric Dumazet' via syzkaller > wrote: > > On Tue, Jan 31, 2017 at 5:17 AM, Andrey Konovalov > > wrote: > >> Hi, > >> > >> I've got the following error report while running the syzkaller fuzzer. > >> > >> O

Re: [PATCH] [media] exynos-gsc: Avoid spamming the log on VIDIOC_TRY_FMT

2017-02-02 Thread Javier Martinez Canillas
Hello Shuah, On 02/01/2017 11:25 PM, Shuah Khan wrote: > On 01/24/2017 02:42 PM, Javier Martinez Canillas wrote: >> There isn't an ioctl to enum the supported field orders, so a user-space >> application can call VIDIOC_TRY_FMT using different field orders to know >> if one is supported. For examp

[PATCH] CIFS: use correct format string for size_t

2017-02-02 Thread Arnd Bergmann
This warning is harmless as size_t is always as wide as 'unsigned long': fs/cifs/smb2ops.c:2008:245: error: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'size_t {aka unsigned int}' [-Werror=format=] Using %zu however is the correct format string that we don

Re: [PATCH] [media] staging: bcm2835: mark all symbols as 'static'

2017-02-02 Thread Greg Kroah-Hartman
On Thu, Feb 02, 2017 at 01:11:36PM +0100, Arnd Bergmann wrote: > On Thu, Feb 2, 2017 at 1:04 PM, Arnd Bergmann wrote: > > On Thu, Feb 2, 2017 at 12:34 PM, Arnd Bergmann wrote: > >> I got a link error in allyesconfig: > >> Fixes: 7b3ad5abf027 ("staging: Import the BCM2835 MMAL-based V4L2 camera >

Re: [Intel-gfx] [PATCH] drm/i915: fix i915 running as dom0 under Xen

2017-02-02 Thread Chris Wilson
On Thu, Feb 02, 2017 at 12:11:29PM +, Tvrtko Ursulin wrote: > > On 02/02/2017 09:47, Juergen Gross wrote: > >Commit 920cf4194954ec ("drm/i915: Introduce an internal allocator for > >disposable private objects") introduced a regression for the kernel > >running as Xen dom0: when switching to gr

Re: [RFC v3 00/10] Provide the EL1 physical timer to the VM

2017-02-02 Thread Christoffer Dall
Hi Jintack, On Wed, Feb 01, 2017 at 12:43:00PM -0500, Jintack Lim wrote: > The ARM architecture defines the EL1 physical timer and the virtual timer, > and it is reasonable for an OS to expect to be able to access both. > However, the current KVM implementation does not provide the EL1 physical >

Re: [PATCH] [media] staging: bcm2835: mark all symbols as 'static'

2017-02-02 Thread Arnd Bergmann
On Thu, Feb 2, 2017 at 1:22 PM, Greg Kroah-Hartman wrote: > On Thu, Feb 02, 2017 at 01:11:36PM +0100, Arnd Bergmann wrote: >> On Thu, Feb 2, 2017 at 1:04 PM, Arnd Bergmann wrote: >> > On Thu, Feb 2, 2017 at 12:34 PM, Arnd Bergmann wrote: >> >> I got a link error in allyesconfig: >> >> Fixes: 7b3

Re: [PATCH V7 3/4] drm/bridge: Add driver for GE B850v3 LVDS/DP++ Bridge

2017-02-02 Thread Emil Velikov
On 2 February 2017 at 11:53, Peter Senna Tschudin wrote: > > On 02 February, 2017 02:46 CET, Emil Velikov wrote: > >> On 1 February 2017 at 11:35, Daniel Vetter wrote: >> > On Wed, Feb 01, 2017 at 10:58:43AM +, Peter Senna Tschudin wrote: >> >> Hi Archit, >> >> >> >> On 01 February, 2017 10:

[tip:WIP.sched/core 101/114] drivers/staging/lustre/lnet/libcfs/linux/linux-prim.c:51:37: error: dereferencing pointer to incomplete type 'struct sighand_struct'

2017-02-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.sched/core head: a7fd62a2a53e181668849e0cb7262c69fd40f985 commit: 4062c957804a9f2d7eb7e2bfa5850e3efa55c944 [101/114] sched/headers, signals: Separate out task_struct::signal and task_struct::sighand types and accessors int

Re: [PATCH v8 2/3] drm/panel: Add support for S6E3HA2 panel driver on TM2 board

2017-02-02 Thread Inki Dae
Dear Thierry, 2017년 02월 01일 23:44에 Thierry Reding 이(가) 쓴 글: > On Wed, Feb 01, 2017 at 08:48:30AM +0900, Inki Dae wrote: >> >> >> 2017년 02월 01일 06:31에 Thierry Reding 이(가) 쓴 글: >>> On Tue, Jan 31, 2017 at 10:15:10AM -0800, Eric Anholt wrote: Thierry Reding writes: > [ Unknown signatu

Re: irq_work_run called from armv8pmu_handle_irq

2017-02-02 Thread Will Deacon
On Wed, Feb 01, 2017 at 12:15:01PM -0800, Sodagudi Prasad wrote: > we would like to understand why irq_work_run() is getting called from > armv8pmu_handle_irq() irq handler for arm SoC. > > Is this irq_work frame work designed to run works from ARMPMU irq handlers? > If there are no of drivers usi

[PATCH v4 1/3] x86: Introduce a new constant KERNEL_MAPPING_SIZE

2017-02-02 Thread Baoquan He
In x86, KERNEL_IMAGE_SIZE is used to limit the size of kernel image in running space, but also represents the size of kernel image mapping area. This looks good when kernel virtual address is invariable inside 512M area and kernel image size is not bigger than 512M. Along with the adding of kaslr,

[PATCH v4 2/3] x86/64/KASLR: Change kernel mapping size to 1G unconditionally

2017-02-02 Thread Baoquan He
The current KASLR changes kernel mapping size from 512M to 1G as long as CONFIG_RANDOMIZE_BASE is enabled, though "nokaslr" kernel option is added. This is buggy. When people specify "nokaslr", whether KASLR code compiled in or not, they expect to see no KASLR change at all, including the size of k

[PATCH v4 0/3] x86/64/KASLR: Change kernel mapping size to 1G unconditionally

2017-02-02 Thread Baoquan He
This is v4 post. In the previous 3 versions I tried to detect and determine kernel image mapping size at runtime for x86_64. With this the inconsistency between KASLR code is not compiled in by disabling CONFIG_RANDOMIZE_BASE and KASLR code is compiled in but add 'nokaslr' kernel option can be fix

[PATCH v4 3/3] x86/64/doc: Update the ranges of kernel text and modules mapping

2017-02-02 Thread Baoquan He
The size of kernel mapping range has been increased to 1G unconditionally. Accordingly kernel modules mapping is shrunk to 1G. So update the description in document. Signed-off-by: Baoquan He --- Documentation/x86/x86_64/mm.txt | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --gi

[PATCH 2/2] scatterlist: do not disable IRQs in sg_copy_buffer

2017-02-02 Thread Gilad Ben-Yossef
Commit 50bed2e2862a ("sg: disable interrupts inside sg_copy_buffer") introduced disabling interrupts in sg_copy_buffer() since atomic uses of miter required it due to use of kmap_atomic(). However, as commit 8290e2d2dcbf ("scatterlist: atomic sg_mapping_iter() no longer needs disabled IRQs") ackno

[PATCH 1/2] scatterlist: reorder compound boolean expression

2017-02-02 Thread Gilad Ben-Yossef
Test the cheaper boolean expression with no side effects first. Signed-off-by: Gilad Ben-Yossef --- lib/scatterlist.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/scatterlist.c b/lib/scatterlist.c index 004fc70..393920f 100644 --- a/lib/scatterlist.c +++ b/lib/scatterl

Re: [PATCH 4/4] tools include: Fix include path for uapi/asm-generic/mman.h

2017-02-02 Thread Jiri Olsa
On Wed, Feb 01, 2017 at 10:38:04PM -0800, David Carrillo-Cisneros wrote: > commit f3539c12d819 ("tools include: Add uapi mman.h for each architecture") > > copied include/uapi/linux/mman.h into tools/include/uapi/linux/mman.h > but did not update the include path for uapi/asm-generic/mman.h. Fix i

[PATCH 0/2] scatterlist: clean up

2017-02-02 Thread Gilad Ben-Yossef
Ran into these two clean up / optimization opportunities while chasing an unrelated bug. Gilad Ben-Yossef (2): scatterlist: reorder compound boolean expression scatterlist: do not disable IRQs in sg_copy_buffer lib/scatterlist.c | 6 +- 1 file changed, 1 insertion(+), 5 deletions(-) --

[PATCH] staging/emxx_udc: Fix styling issues

2017-02-02 Thread Zhengyi Shen
Fix line over 80 characters. Statements longer than 80 columns were broken into sensible chunks. Descendants in four different palces were in the same uniform style. Signed-off-by: Zhengyi Shen --- drivers/staging/emxx_udc/emxx_udc.c | 13 - 1 file changed, 8 insertions(+), 5 deletio

Re: [PATCH 0/4] Fixes for perf build and feature detection

2017-02-02 Thread Jiri Olsa
On Wed, Feb 01, 2017 at 10:38:00PM -0800, David Carrillo-Cisneros wrote: > Fixes for minor problems with Makefiles and libs that I found while > building perf. > > David Carrillo-Cisneros (4): > perf tools: pass PYTHON config to feature detection > tools lib traceevent: Robustify do_generate_d

Re: net/tcp: warning in tcp_try_coalesce/skb_try_coalesce

2017-02-02 Thread Dmitry Vyukov
On Thu, Feb 2, 2017 at 1:20 PM, Eric Dumazet wrote: > On Thu, 2017-02-02 at 10:05 +0100, Dmitry Vyukov wrote: >> On Tue, Jan 31, 2017 at 3:27 PM, 'Eric Dumazet' via syzkaller >> wrote: >> > On Tue, Jan 31, 2017 at 5:17 AM, Andrey Konovalov >> > wrote: >> >> Hi, >> >> >> >> I've got the followin

Re: [PATCH v3 0/2] iov_iter: allow iov_iter_get_pages_alloc to allocate more pages per call

2017-02-02 Thread Jeff Layton
On Thu, 2017-02-02 at 11:16 +, Al Viro wrote: > On Thu, Feb 02, 2017 at 02:56:51AM -0800, Christoph Hellwig wrote: > > > I really wonder if this is the right approach. Most of the users of > > > iov_iter_get_pages()/iov_iter_get_pages_alloc() look like they want > > > something like > > > io

Re: [PATCH] perf: fix topology test on systems with sparse CPUs

2017-02-02 Thread Jiri Olsa
On Thu, Feb 02, 2017 at 07:06:43AM -0500, Jan Stancek wrote: > > > > > When build_cpu_topo() encounters offline/absent CPUs, > > > it fails to find any sysfs entries and returns failure. > > > This leads to build_cpu_topology() and write_cpu_topology() > > > failing as well. > > > > > > Because H

Re: [PATCH] ARM: orion5x: fix Makefile for linkstation-lschl.dtb

2017-02-02 Thread Gregory CLEMENT
Hi Arnd, On jeu., févr. 02 2017, Arnd Bergmann wrote: > The rename of orion5x-lschl.dts needs to be reflected in the Makefile: > > make[3]: *** No rule to make target > 'arch/arm/boot/dts/orion5x-lschl.dtb', needed by '__build'. Sorry for that, I did a make dtbs but with the wrong config, (mv

Re: [PATCH 1/4] mm: Fix sparse, use plain integer as NULL pointer

2017-02-02 Thread Kirill A. Shutemov
On Thu, Feb 02, 2017 at 10:37:17AM +1100, Tobin C. Harding wrote: > From: Tobin C Harding > > Patch fixes sparse warning: Using plain integer as NULL pointer. Replaces > assignment of 0 to pointer with NULL assignment. > > Signed-off-by: Tobin C Harding I wrote this part when ARC had pgtable_t

Re: [PATCH] net: phy: dp83867: Port mirroring support in the DP83867 TI's PHY driver

2017-02-02 Thread Andrew Lunn
> The bootstrapping process in the PHY sets this bit. This is wrong since > the board lane layout is not "swapped" Ah, you mean a strapping pin? Resistor to ground/VCC? That is a different matter. It makes it a lot less likely to break some existing board with such a change. > I have thought a b

Re: [PATCH 3.12 000/235] 3.12.70-stable review

2017-02-02 Thread Jiri Slaby
On 01/27/2017, 04:51 PM, Shuah Khan wrote: > On 01/27/2017 03:55 AM, Jiri Slaby wrote: >> This is the start of the stable review cycle for the 3.12.70 release. >> There are 235 patches in this series, all will be posted as a response >> to this one. If anyone has any issues with these being applie

[PATCH] usb: storage: fix infinite wait loop by incrementing loop counter

2017-02-02 Thread Colin King
From: Colin Ian King If jumpshot_get_status continues to return a failed result then the wait loop will spin forever because the waitcount counter is never being incremented and we don't ever timeout. Fix this by incrementing waitcount. Cc: Signed-off-by: Colin Ian King --- drivers/usb/stora

[PATCH 1/2] block: Use for_each_thread() in sys_ioprio_set()/sys_ioprio_get()

2017-02-02 Thread Tetsuo Handa
IOPRIO_WHO_USER case in sys_ioprio_set()/sys_ioprio_get() are using while_each_thread(), which is unsafe under RCU lock according to commit 0c740d0afc3bff0a ("introduce for_each_thread() to replace the buggy while_each_thread()"). Use for_each_thread() which is safe under RCU lock. Link: http://l

[PATCH 2/2] pid: Use for_each_thread() in do_each_pid_thread()

2017-02-02 Thread Tetsuo Handa
while_each_pid_thread() is using while_each_thread(), which is unsafe under RCU lock according to commit 0c740d0afc3bff0a ("introduce for_each_thread() to replace the buggy while_each_thread()"). Use for_each_thread() in do_each_pid_thread() which is safe under RCU lock. Link: http://lkml.kernel.

[PATCH] parisc: ccio-dma: Handle return NULL error from ioremap_nocache

2017-02-02 Thread Arvind Yadav
Here, If ioremap_nocache will fail. It will return NULL. Kernel can run into a NULL-pointer dereference. This error check will avoid NULL pointer dereference. If kzalloc will fail return -ENOMEM instead of '1'. Signed-off-by: Arvind Yadav --- drivers/parisc/ccio-dma.c | 6 +- 1 file changed

Re: [Intel-wired-lan] [PATCH] net: intel: e1000e: use new api ethtool_{get|set}_link_ksettings

2017-02-02 Thread Neftin, Sasha
On 2/1/2017 00:01, Philippe Reynes wrote: Hi Sasha, On 1/31/17, Neftin, Sasha wrote: Philippe, We will look into and try test this patch. I would like ask question. I see that this thread has been started from implementation for e1000 code. Why do you decide shift implementation to e1000e? I

Re: [PATCH nf-next v2] netfilter: allow logging from non-init namespaces

2017-02-02 Thread Pablo Neira Ayuso
On Tue, Jan 31, 2017 at 10:30:06AM +0100, Michal Kubecek wrote: > Commit 69b34fb996b2 ("netfilter: xt_LOG: add net namespace support for > xt_LOG") disabled logging packets using the LOG target from non-init > namespaces. The motivation was to prevent containers from flooding > kernel log of the ho

[PATCH v5 0/8] uapi: export all headers under uapi directories

2017-02-02 Thread Nicolas Dichtel
Here is the v5 of this series. Seems there was too many people cc'ed, so I've removed arch specific ml. Hope it's better now. Patches #1 and #2 are just cleanup: some exported headers were still under a non-uapi directory. Patch #3 is a fix to avoid exporting a file that was not under an uapi dir

[PATCH v5 4/8] Makefile.headersinst: cleanup input files

2017-02-02 Thread Nicolas Dichtel
After the last four patches, all exported headers are under uapi/, thus input-files2 are not needed anymore. The side effect is that input-files1-name is exactly header-y. Note also that input-files3-name is genhdr-y. Signed-off-by: Nicolas Dichtel --- scripts/Makefile.headersinst | 34

[PATCH v5 6/8] uapi: includes linux/types.h before exporting files

2017-02-02 Thread Nicolas Dichtel
Some files will be exported after the next patch. 0-day tests report the following warning/error: ./usr/include/linux/bcache.h:8: include of is preferred over ./usr/include/linux/bcache.h:11: found __[us]{8,16,32,64} type without #include ./usr/include/linux/qrtr.h:8: found __[us]{8,16,32,64}

[PATCH v5 7/8] uapi: export all headers under uapi directories

2017-02-02 Thread Nicolas Dichtel
Regularly, when a new header is created in include/uapi/, the developer forgets to add it in the corresponding Kbuild file. This error is usually detected after the release is out. In fact, all headers under uapi directories should be exported, thus it's useless to have an exhaustive list. After

[PATCH v5 2/8] nios2: put setup.h in uapi

2017-02-02 Thread Nicolas Dichtel
This header file is exported, but from a userland pov, it's just a wrapper to asm-generic/setup.h. Signed-off-by: Nicolas Dichtel Reviewed-by: Tobias Klauser --- arch/nios2/include/uapi/asm/Kbuild | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/nios2/include/uapi/asm/Kbuild b/arch/nio

[PATCH v5 1/8] h8300: put bitsperlong.h in uapi

2017-02-02 Thread Nicolas Dichtel
This header file is exported, thus move it to uapi. Signed-off-by: Nicolas Dichtel --- arch/h8300/include/asm/bitsperlong.h | 14 -- arch/h8300/include/uapi/asm/bitsperlong.h | 14 ++ 2 files changed, 14 insertions(+), 14 deletions(-) delete mode 100644 arch/h8300/i

[PATCH v5 8/8] uapi: export all arch specifics directories

2017-02-02 Thread Nicolas Dichtel
This patch removes the need of subdir-y. Now all files/directories under arch//include/uapi/ are exported. The only change for userland is the layout of the command 'make headers_install_all': directories asm- are replaced by arch-/. Those new directories contains all files/directories of the spec

Re: [PATCH v3] netfilter: nf_ct_helper: warn when not applying default helper assignment

2017-02-02 Thread Pablo Neira Ayuso
On Wed, Feb 01, 2017 at 09:01:54PM +0100, Jiri Kosina wrote: > From: Jiri Kosina > > Commit 3bb398d925 ("netfilter: nf_ct_helper: disable automatic helper > assignment") is causing behavior regressions in firewalls, as traffic > handled by conntrack helpers is now by default not passed through

[PATCH v5 3/8] x86: stop exporting msr-index.h to userland

2017-02-02 Thread Nicolas Dichtel
Even if this file was not in an uapi directory, it was exported because it was listed in the Kbuild file. Fixes: b72e7464e4cf ("x86/uapi: Do not export as part of the user API headers") Suggested-by: Borislav Petkov CC: Ingo Molnar Signed-off-by: Nicolas Dichtel --- arch/x86/include/uapi/asm

Re: [PATCH] ARM: orion5x: fix Makefile for linkstation-lschl.dtb

2017-02-02 Thread Arnd Bergmann
On Thu, Feb 2, 2017 at 2:05 PM, Gregory CLEMENT wrote: > On jeu., févr. 02 2017, Arnd Bergmann wrote: > > Will you applied this patch just after merging tags/mvebu-fixes-4.10-1 ? Ah, I had not realized that. > Do you want I resubmit a PR with your patch? Please add this patch on top of the fi

[PATCH v5 5/8] Makefile.headersinst: remove destination-y option

2017-02-02 Thread Nicolas Dichtel
This option was added in commit c7bb349e7c25 ("kbuild: introduce destination-y for exported headers") but never used in-tree. Signed-off-by: Nicolas Dichtel --- Documentation/kbuild/makefiles.txt | 23 --- scripts/Makefile.headersinst | 2 +- 2 files changed, 5 inserti

[PATCH] x86: Clean up a tiny description mistake in kernel ld script

2017-02-02 Thread Baoquan He
Remove the redundent word "is". Signed-off-by: Baoquan He --- arch/x86/kernel/vmlinux.lds.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/x86/kernel/vmlinux.lds.S b/arch/x86/kernel/vmlinux.lds.S index e79f15f..f8f56b9 100644 --- a/arch/x86/kernel/vmlinux.lds.S +++

Re: [PATCH v4 1/3] ACPI / bus: Export acpi_of_modalias equiv of of_modalias_node

2017-02-02 Thread Dan O'Donovan
On 02/02/2017 11:41 AM, Rafael J. Wysocki wrote: > On Friday, January 27, 2017 01:30:13 PM Dan O'Donovan wrote: >> From: Crestez Dan Leonard >> >> When using devicetree stuff like i2c_client.name or spi_device.modalias >> is initialized to the first DT compatible id with the vendor prefix >> strip

[tip:WIP.sched/core 101/114] arch/sh/kernel/cpu/sh2a/fpu.c:574:2: error: implicit declaration of function 'force_sig'

2017-02-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.sched/core head: 6ddb6fab35cf50127c3c4dc9d5e15da4258a86c5 commit: 397fbcb564288fb6d39400ae1d2f3363ba19c501 [101/114] sched/headers, signals: Separate out task_struct::signal and task_struct::sighand types and accessors int

Re: [PATCH] usb: storage: fix infinite wait loop by incrementing loop counter

2017-02-02 Thread walter harms
Am 02.02.2017 14:19, schrieb Colin King: > From: Colin Ian King > > If jumpshot_get_status continues to return a failed result then the > wait loop will spin forever because the waitcount counter is never > being incremented and we don't ever timeout. Fix this by incrementing > waitcount. > >

Re: [PATCH v2] printk: Correctly handle preemption in console_unlock()

2017-02-02 Thread Sergey Senozhatsky
On (01/25/17 15:08), Petr Mladek wrote: [..] > Fixes: 6b97a20d3a7909 ("printk: set may_schedule for some of > console_trylock() callers") > Suggested-by: Tetsuo Handa > Signed-off-by: Petr Mladek Reviewed-by: Sergey Senozhatsky -ss

[PATCH] MIPS: sync-r4k: Fix KERN_CONT fallout

2017-02-02 Thread Matt Redfearn
Since commit 4bcc595ccd80 ("printk: reinstate KERN_CONT for printing continuation lines") the output of counter synchornisation has been split across lines: [ 0.665181] Synchronize counters for CPU 1: [ 0.678578] done. Fix this by using pr_cont, and replace printk with pr_info. Signed-off-by: Mat

[tip:WIP.sched/core 101/114] arch/ia64/mm/init.c:90:29: error: implicit declaration of function 'rlimit_max'

2017-02-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git WIP.sched/core head: 6ddb6fab35cf50127c3c4dc9d5e15da4258a86c5 commit: 397fbcb564288fb6d39400ae1d2f3363ba19c501 [101/114] sched/headers, signals: Separate out task_struct::signal and task_struct::sighand types and accessors int

Re: [PATCH v5 3/8] x86: stop exporting msr-index.h to userland

2017-02-02 Thread Ingo Molnar
* Nicolas Dichtel wrote: > Even if this file was not in an uapi directory, it was exported because > it was listed in the Kbuild file. > > Fixes: b72e7464e4cf ("x86/uapi: Do not export as part of > the user API headers") > Suggested-by: Borislav Petkov > CC: Ingo Molnar > Signed-off-by: Nic

Re: net/tcp: warning in tcp_try_coalesce/skb_try_coalesce

2017-02-02 Thread Eric Dumazet
On Thu, 2017-02-02 at 13:59 +0100, Dmitry Vyukov wrote: > > Right, I can confirm that this is not happening on net-next tip > (62e13097c46c69dbd7544ab2cd585ccf48f360a4). > I reproduced it on a random month old commit on net-next, though > (23a8ed4a624324dc696c328f09bd502c4a3816f0). Let's consider

Re: [PATCH 1/5] perf/sdt: Show proper hint

2017-02-02 Thread Ingo Molnar
* Ravi Bangoria wrote: > After patch: > $ perf record -e sdt_glib:main__after_check > event syntax error: 'sdt_glib:idle__add' > \___ unknown tracepoint > > Error: File /sys/kernel/debug/tracing/events/sdt_glib/idle__add ... > Hint: SDT event has to be prob

Re: [PATCH] tpm: fix type issues in tpm_getcap()

2017-02-02 Thread Jarkko Sakkinen
On Wed, Feb 01, 2017 at 07:53:47PM +0200, Jarkko Sakkinen wrote: > There are two type issues associated with tpm_getcap(). > > You must not do arithmetic with __be32 or __le32 types because sometimes > it results incorrect results. Calculations must be done only with data > that is in CPU byte ord

<    1   2   3   4   5   6   7   8   >