[PATCH 0/2] fix the traced mt-exec deadlock

2017-02-13 Thread Oleg Nesterov
Hello, Lets finally fix this problem, it was reported several times. I still think that in the longer term we should (try to) rework the security hooks and (partially) revert this change, but this is not trivial and we need something backportable anyway. Eric, Jann, we already discussed this

Re: [tip:locking/core] refcount_t: Introduce a special purpose refcount type

2017-02-13 Thread Peter Zijlstra
On Fri, Feb 10, 2017 at 12:31:15AM -0800, tip-bot for Peter Zijlstra wrote: > Commit-ID: f405df5de3170c00e5c54f8b7cf4766044a032ba > Gitweb: http://git.kernel.org/tip/f405df5de3170c00e5c54f8b7cf4766044a032ba > Author: Peter Zijlstra > AuthorDate: Mon, 14 Nov 2016

Re: [PATCH 3/8] pinctrl: sunxi: Add A64 R_PIO controller support

2017-02-13 Thread Linus Walleij
On Wed, Feb 8, 2017 at 11:00 AM, Icenowy Zheng wrote: > The A64 has a R_PIO pin controller, similar to the one found on the H3 SoC. > Add support for the pins controlled by the R_PIO controller. > > Signed-off-by: Icenowy Zheng I'd be happy to merge patches

Re: [PATCH 1/2] sched/deadline: Replenishment timer should fire in the next period

2017-02-13 Thread Steven Rostedt
On Mon, 13 Feb 2017 12:10:25 +0100 Peter Zijlstra wrote: > I've interpreted this as: > > Reviewed-by: Luca Abeni > > Holler if you disagree. You can add mine too. I put in a lot of trace_printk()s and it all appears to be exactly as Daniel

Re: [PATCH v11 6/9] mmc: cavium: Add MMC PCI driver for ThunderX SOCs

2017-02-13 Thread Jan Glauber
On Sun, Feb 12, 2017 at 09:09:29AM +0800, kbuild test robot wrote: > Hi Jan, > > [auto build test ERROR on linus/master] > [also build test ERROR on v4.10-rc7 next-20170210] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the system] > > url: >

Re: [PATCH] sched/deadline: Remove redundant code replenishing runtime

2017-02-13 Thread Juri Lelli
[+Luca] On 13/02/17 13:29, Byungchul Park wrote: > On Mon, Feb 13, 2017 at 11:30:09AM +0900, Byungchul Park wrote: > > On Fri, Feb 10, 2017 at 01:39:33PM +, Juri Lelli wrote: > > > Hi, > > > > > > On 10/02/17 18:11, Byungchul Park wrote: > > > > For a task passing its deadline while !rq, it

Re: Linux 4.10-rc8

2017-02-13 Thread Pavel Machek
On Sun 2017-02-12 14:40:04, Linus Torvalds wrote: > On Sun, Feb 12, 2017 at 2:32 PM, Pavel Machek wrote: > > > > I see some crazy stuff with v4.10-rc7 (and older). > > > > Like machine not booting from cold boot, needing ctrl-alt-del at boot, > > otherwise it changs during PCI

Re: IB/hfi1: Use memdup_user() rather than duplicating its implementation in hfi1_user_sdma_process_request()

2017-02-13 Thread Dan Carpenter
On Mon, Feb 13, 2017 at 12:12:51PM +0100, SF Markus Elfring wrote: > >> Thanks for the patch, but this one is already taken care of along > >> with other similar uses of kmalloc/copy: > >> > >> http://marc.info/?l=linux-rdma=148656088729538=2 > >> > > > > Michael's patch doesn't change

Re: [PATCH v2 2/3] xen/privcmd: Add IOCTL_PRIVCMD_DM_OP

2017-02-13 Thread Boris Ostrovsky
How about something like (with rather arbitrary values) #define PRIVCMD_DMOP_MAX_NUM_BUFFERS 16 #define PRIVCMD_DMOP_MAX_TOT_BUFFER_SZ 4096 and make them part of the interface (i.e. put them into privcmd.h)? Given that the values are arbitrary, I think it may be better to make

[PATCH 1/2] exec: don't wait for zombie threads with cred_guard_mutex held

2017-02-13 Thread Oleg Nesterov
de_thread() waits for other threads with ->cred_guard_mutex held and this is really bad because the time is not bounded, debugger can delay the exit and this lock has a lot of users (mostly abusers imo) in fs/proc and more. And this leads to deadlock if debugger tries to take the same mutex:

Re: [PATCH V6 1/5] LIB: Indirect ISA/LPC port IO introduced

2017-02-13 Thread zhichang.yuan
Hi, Alex, On 2017/2/1 3:37, Alexander Graf wrote: > > > On 31/01/2017 14:32, John Garry wrote: >> On 30/01/2017 17:12, Alexander Graf wrote: >>> On 01/24/2017 08:05 AM, zhichang.yuan wrote: Low-pin-count interface is integrated into some SoCs. The accesses to those peripherals

[PATCHv6 2/3] staging: wlan-ng: move else if statement to a single line

2017-02-13 Thread Maksymilian Piechota
move else if statement to a single line Signed-off-by: Maksymilian Piechota --- drivers/staging/wlan-ng/prism2mgmt.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/staging/wlan-ng/prism2mgmt.c b/drivers/staging/wlan-ng/prism2mgmt.c

Re: [PATCH 0/2] [media] exynos-gsc: Fix support for NV21 and NV61 formats

2017-02-13 Thread Sylwester Nawrocki
Hi Javier, On 02/13/2017 01:53 PM, Javier Martinez Canillas wrote: > Any comments on this series? The patches look good to me, I will Ack the patches in case Mauro wants to apply them directly. Alternatively I will add them to my tree for v4.12 after the merge window. -- Thanks, Sylwester

[RFC simple allocator v2 1/2] Create Simple Allocator module

2017-02-13 Thread Benjamin Gaignard
This is the core of simple allocator module. It aim to offert one common ioctl to allocate specific memory. version 2: - rebased on 4.10-rc7 Signed-off-by: Benjamin Gaignard --- Documentation/simple-allocator.txt | 81 +++ drivers/Kconfig

[RFC simple allocator v2 0/2] Simple allocator

2017-02-13 Thread Benjamin Gaignard
version 2: - rebase code on 4.10-rc7 - fix bug in CMA allocator - do more tests with wayland dmabuf protocol: https://git.linaro.org/people/benjamin.gaignard/simple_allocator.git The goal of this RFC is to understand if a common ioctl for specific memory regions allocations is needed/welcome.

[BUGFIX PATCH 3/3] kprobes/arm: Fix the return address of multiple kretprobes

2017-02-13 Thread Masami Hiramatsu
This is arm port of commit 737480a0d525 ("kprobes/x86: Fix the return address of multiple kretprobes"). Fix the return address of subsequent kretprobes when multiple kretprobes are set on the same function. For example: # cd /sys/kernel/debug/tracing # echo "r:event1 sys_symlink" >

[BUGFIX PATCH 2/3] kprobes/arm: Skip single-stepping in recursing path if possible

2017-02-13 Thread Masami Hiramatsu
Kprobes/arm skips single-stepping (moreover handling the event) if the conditional instruction must not be executed. This also apply the rule when we hit the recursing kprobe, so that kprobe does not count nmissed up in that case. Signed-off-by: Masami Hiramatsu ---

[tip:locking/urgent] futex: Move futex_init() to core_initcall

2017-02-13 Thread tip-bot for Yang Yang
Commit-ID: 25f71d1c3e98ef0e52371746220d66458eac75bc Gitweb: http://git.kernel.org/tip/25f71d1c3e98ef0e52371746220d66458eac75bc Author: Yang Yang AuthorDate: Fri, 30 Dec 2016 16:17:55 +0800 Committer: Thomas Gleixner CommitDate: Mon, 13 Feb

Re: [PATCHv6 07/37] filemap: allocate huge page in page_cache_read(), if allowed

2017-02-13 Thread Kirill A. Shutemov
On Thu, Feb 09, 2017 at 01:18:35PM -0800, Matthew Wilcox wrote: > On Thu, Jan 26, 2017 at 02:57:49PM +0300, Kirill A. Shutemov wrote: > > Later we can add logic to accumulate information from shadow entires to > > return to caller (average eviction time?). > > I would say minimum rather than

[PATCH v28 17/21] richacl: Add richacl xattr handler

2017-02-13 Thread Andreas Gruenbacher
Add richacl xattr handler implementing the xattr operations based on the get_richacl and set_richacl inode operations. Signed-off-by: Andreas Gruenbacher Reviewed-by: Jeff Layton --- fs/richacl.c | 22

[PATCH v28 21/21] ext4: Add richacl feature flag

2017-02-13 Thread Andreas Gruenbacher
From: "Aneesh Kumar K.V" This feature flag selects richacl instead of POSIX ACL support on the filesystem. When this feature is off, the "acl" and "noacl" mount options control whether POSIX ACLs are enabled. When it is on, richacls are automatically enabled

[PATCH v2 3/3] perf: replace _SC_NPROCESSORS_CONF with max_present_cpu in cpu_topology_map

2017-02-13 Thread Jan Stancek
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 the perf tool." 2. size of cpu_topology_map (perf_env.cpu[]) is allocated

[PATCH v28 18/21] vfs: Add richacl permission checking

2017-02-13 Thread Andreas Gruenbacher
Hook the richacl permission checking function into the vfs. Signed-off-by: Andreas Gruenbacher Cc: Jeff Layton --- fs/namei.c | 70 ++-- fs/richacl.c | 4 ++-- 2 files changed, 70 insertions(+),

[PATCH v2 2/3] perf: make build_cpu_topology skip offline/absent CPUs

2017-02-13 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 NULL and we get NULL ptr deref

Re: [PATCH v27 02/21] vfs: Add MAY_CREATE_FILE and MAY_CREATE_DIR permission flags

2017-02-13 Thread Andreas Gruenbacher
Miklos, On Fri, Dec 2, 2016 at 10:22 AM, Miklos Szeredi wrote: > On Tue, Oct 11, 2016 at 2:50 PM, Andreas Gruenbacher > wrote: >> Richacls distinguish between creating non-directories and directories. To >> support that, add an isdir parameter to

[PATCH v28 16/21] richacl: xattr mapping functions

2017-02-13 Thread Andreas Gruenbacher
Map between "system.richacl" xattrs and the in-kernel representation. Signed-off-by: Andreas Gruenbacher Reviewed-by: Jeff Layton --- fs/Makefile| 2 +- fs/richacl_xattr.c | 161

[PATCH v28 12/21] richacl: Update the file masks in chmod()

2017-02-13 Thread Andreas Gruenbacher
Doing a chmod() sets the file mode, which includes the file permission bits. When a file has a richacl, the permissions that the richacl grants need to be limited to what the new file permission bits allow. This is done by setting the file masks in the richacl to what the file permission bits

[PATCH v28 07/21] richacl: Permission check algorithm

2017-02-13 Thread Andreas Gruenbacher
A richacl roughly grants a requested access if the NFSv4 acl in the richacl grants the requested permissions according to the NFSv4 permission check algorithm and the file mask that applies to the process includes the requested permissions. Signed-off-by: Andreas Gruenbacher

Re: [PATCH v11 6/9] mmc: cavium: Add MMC PCI driver for ThunderX SOCs

2017-02-13 Thread Ulf Hansson
On 13 February 2017 at 16:24, Jan Glauber wrote: > On Sun, Feb 12, 2017 at 09:09:29AM +0800, kbuild test robot wrote: >> Hi Jan, >> >> [auto build test ERROR on linus/master] >> [also build test ERROR on v4.10-rc7 next-20170210] >> [if your patch is applied to the

[PATCH v28 02/21] vfs: Add MAY_CREATE_FILE and MAY_CREATE_DIR permission flags

2017-02-13 Thread Andreas Gruenbacher
Richacls distinguish between creating non-directories and directories. To support that, add an isdir parameter to may_create(). When checking inode_permission() for create permission, pass in MAY_CREATE_FILE or MAY_CREATE_DIR as an additional mask flag. Add may_replace() to allow checking for

[PATCH v28 14/21] richacl: Create-time inheritance

2017-02-13 Thread Andreas Gruenbacher
When a new file is created, it can inherit an acl from its parent directory; this is similar to how default acls work in POSIX ACLs. As with POSIX ACLs, if a file inherits an acl from its parent directory, the intersection between the create mode and the permissions granted by the inherited acl

[PATCH v28 15/21] richacl: Automatic Inheritance

2017-02-13 Thread Andreas Gruenbacher
Automatic Inheritance (AI) allows changes to the acl of a directory to propagate down to children. This is mostly implemented in user space: when a process changes the permissions of a directory and Automatic Inheritance is enabled for that directory, the process must propagate those changes to

Re: [patch net-next] net: qcom/emac: fix a sizeof() typo

2017-02-13 Thread Timur Tabi
Dan Carpenter wrote: We had intended to say "sizeof(u32)" but the "u" is missing. Fortunately, sizeof(32) is also 4, so the original code still works. Fixes: c4e7beea2192 ("net: qcom/emac: add ethtool support for reading hardware registers") Signed-off-by: Dan Carpenter

Re: [PATCH v2 8/9] sched: Don't reinvent the wheel but use existing llist API

2017-02-13 Thread Oleg Nesterov
On 02/13, Peter Zijlstra wrote: > > On Mon, Feb 13, 2017 at 04:21:08PM +0900, Byungchul Park wrote: > > + llist_for_each_entry(p, llist, wake_entry) > > + ttwu_do_activate(rq, p, p->sched_remote_wakeup ? WF_MIGRATED : > > 0, ); > > I think this suffers the exact same problem the

Re: linux-next: build failure after merge of the scsi tree

2017-02-13 Thread Jens Axboe
On 02/12/2017 05:32 PM, Stephen Rothwell wrote: > Hi all, > > On Mon, 6 Feb 2017 16:04:51 +1100 Stephen Rothwell > wrote: >> >> After merging the scsi tree, today's linux-next build (powerpc >> ppc64_defconfig) failed like this: >> >> In file included from

Re: [PATCHv6 08/37] filemap: handle huge pages in do_generic_file_read()

2017-02-13 Thread Matthew Wilcox
On Thu, Jan 26, 2017 at 02:57:50PM +0300, Kirill A. Shutemov wrote: > Most of work happans on head page. Only when we need to do copy data to > userspace we find relevant subpage. > > We are still limited by PAGE_SIZE per iteration. Lifting this limitation > would require some more work. Now

Re: [PATCH V5 3/4] Move stack parameters for sed_ioctl to prevent oversized stack with CONFIG_KASAN

2017-02-13 Thread Scott Bauer
On Mon, Feb 13, 2017 at 04:30:36PM +, David Laight wrote: > From: Scott Bauer Sent: 13 February 2017 16:11 > > When CONFIG_KASAN is enabled, compilation fails: > > > > block/sed-opal.c: In function 'sed_ioctl': > > block/sed-opal.c:2447:1: error: the frame size of 2256 bytes is larger than >

[PATCH v3 1/3] xen/privcmd: return -ENOTTY for unimplemented IOCTLs

2017-02-13 Thread Paul Durrant
The code sets the default return code to -ENOSYS but then overrides this to -EINVAL in the switch() statement's default case, which is clearly silly. This patch removes the override and sets the default return code to -ENOTTY, which is the conventional return for an unimplemented ioctl.

[PATCH v3 2/3] xen/privcmd: Add IOCTL_PRIVCMD_DM_OP

2017-02-13 Thread Paul Durrant
Recently a new dm_op[1] hypercall was added to Xen to provide a mechanism for restricting device emulators (such as QEMU) to a limited set of hypervisor operations, and being able to audit those operations in the kernel of the domain in which they run. This patch adds IOCTL_PRIVCMD_DM_OP as

[PATCH v3 0/3] xen/privcmd: support for dm_op and restriction

2017-02-13 Thread Paul Durrant
This patch series follows on from my recent Xen series [1], to provide support in privcmd for de-privileging of device emulators. [1] https://lists.xen.org/archives/html/xen-devel/2017-01/msg02558.html Paul Durrant (3): xen/privcmd: return -ENOTTY for unimplemented IOCTLs xen/privcmd: Add

[PATCH v3 3/3] xen/privcmd: add IOCTL_PRIVCMD_RESTRICT

2017-02-13 Thread Paul Durrant
The purpose if this ioctl is to allow a user of privcmd to restrict its operation such that it will no longer service arbitrary hypercalls via IOCTL_PRIVCMD_HYPERCALL, and will check for a matching domid when servicing IOCTL_PRIVCMD_DM_OP. The aim of this is to limit the attack surface for a

[PATCH 2/2] MIPS: loongson1: Add PWM support for Loongson1 board

2017-02-13 Thread Yang Ling
The patch adds PWM support for Loongson1 board. Signed-off-by: Yang Ling --- arch/mips/configs/loongson1b_defconfig | 2 ++ arch/mips/configs/loongson1c_defconfig | 2 ++ arch/mips/include/asm/mach-loongson32/platform.h | 1 +

Re: [PATCH v27 03/21] vfs: Add MAY_DELETE_SELF and MAY_DELETE_CHILD permission flags

2017-02-13 Thread Andreas Gruenbacher
On Fri, Dec 2, 2016 at 10:57 AM, Miklos Szeredi wrote: > On Tue, Oct 11, 2016 at 2:50 PM, Andreas Gruenbacher > wrote: >> Normally, deleting a file requires MAY_WRITE access to the parent >> directory. With richacls, a file may be deleted with

Re: [PATCH v27 03/21] vfs: Add MAY_DELETE_SELF and MAY_DELETE_CHILD permission flags

2017-02-13 Thread Andreas Gruenbacher
On Tue, Dec 6, 2016 at 10:25 PM, Miklos Szeredi wrote: > On Tue, Dec 6, 2016 at 10:13 PM, Jeremy Allison wrote: >> On Tue, Dec 06, 2016 at 03:15:29PM -0500, J. Bruce Fields wrote: >>> On Fri, Dec 02, 2016 at 10:57:42AM +0100, Miklos Szeredi wrote: >>> > On Tue,

Re: [PATCH 1/3 staging-next] android: Collect statistics from lowmemorykiller

2017-02-13 Thread peter enderborg
On 02/10/2017 10:15 AM, Michal Hocko wrote: > On Fri 10-02-17 10:05:34, peter enderborg wrote: >> On 02/10/2017 08:59 AM, Michal Hocko wrote: > [...] >>> The approach was wrong from the day 1. Abusing slab shrinkers >>> is just a bad place to stick this logic. This all belongs to the >>>

[PATCH v28 10/21] vfs: Add get_richacl and set_richacl inode operations

2017-02-13 Thread Andreas Gruenbacher
These operations are similar to the get_acl and set_acl operations for POSIX ACLs. The distinction between access and default ACLs doesn't exist for richacls. Signed-off-by: Andreas Gruenbacher Reviewed-by: Steve French --- include/linux/fs.h

Re: [PATCH 0/3 staging-next] android: Lowmemmorykiller task tree

2017-02-13 Thread peter enderborg
On 02/10/2017 11:27 AM, Michal Hocko wrote: > [I have only now see this cover - it answers some of the questions I've > had to specific patches. It would be really great if you could use git > send-email to post patch series - it just does the right thing(tm)] > > On Thu 09-02-17 14:21:40, peter

Re: [linux-sunxi] [PATCH 1/8] dt-bindings: fix for Allwinner H5 pinctrl's compatible

2017-02-13 Thread Chen-Yu Tsai
On Wed, Feb 8, 2017 at 6:00 PM, Icenowy Zheng wrote: > The compatible for Allwinner H5 pin controller is wrong written as > allwinner,sun50i-h5-r-pinctrl, however, it's really a generic pinctrl > rather than a "r" one. > > Fix this compatible string. > > Signed-off-by: Icenowy

[PATCH 1/2] usb: gadget: udc: avoid use of freed pointer

2017-02-13 Thread Gustavo A. R. Silva
Rewrite udc_free_dma_chain() function to avoid use of pointer after free. Addresses-Coverity-ID: 1091172 Reviewed-by: Greg Kroah-Hartman Signed-off-by: Gustavo A. R. Silva --- drivers/usb/gadget/udc/amd5536udc.c | 20 +++- 1

Re: [PATCH] tools:perf:scripting-engines: Fix compile error with some perl5 versions

2017-02-13 Thread Arnaldo Carvalho de Melo
Em Mon, Feb 13, 2017 at 09:19:18AM +0100, Jiri Olsa escreveu: > On Sun, Feb 12, 2017 at 10:46:55AM +0800, Wang YanQing wrote: > > Fix below compile error: > > CC util/scripting-engines/trace-event-perl.o > > In file included from /usr/lib/perl5/5.22.2/i686-linux/CORE/perl.h:5673:0, > >

Re: [PATCH] ARM: dts: exynos: Use thermal fuse value for thermal zone 0 on Exynos5420

2017-02-13 Thread Bartlomiej Zolnierkiewicz
On Saturday, February 11, 2017 10:14:56 PM Krzysztof Kozlowski wrote: > In Odroid XU3 Lite board, the temperature levels reported for thermal > zone 0 were weird. In warm room: > /sys/class/thermal/thermal_zone0/temp:32000 > /sys/class/thermal/thermal_zone1/temp:51000 >

[PATCH V5 1/4] block: sed-opal: change ioctl to take user pointer instead of unsinged long

2017-02-13 Thread Scott Bauer
Signed-off-by: Scott Bauer --- block/sed-opal.c | 6 -- drivers/nvme/host/core.c | 3 ++- include/linux/sed-opal.h | 4 ++-- 3 files changed, 8 insertions(+), 5 deletions(-) diff --git a/block/sed-opal.c b/block/sed-opal.c index bf1406e..2448d4a 100644 ---

Re: loop: divide error in transfer_xor

2017-02-13 Thread Dmitry Vyukov
On Wed, Feb 8, 2017 at 11:39 AM, Ming Lei wrote: > On Wed, Feb 8, 2017 at 5:32 PM, Dmitry Vyukov wrote: >> Hello, >> >> The following program triggers divide error in transfer_xor: >>

Re: [PATCH 1/2] exec: don't wait for zombie threads with cred_guard_mutex held

2017-02-13 Thread kbuild test robot
Hi Oleg, [auto build test ERROR on linus/master] [also build test ERROR on v4.10-rc8 next-20170213] [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/Oleg-Nesterov/fix-the-traced-mt-exec-deadlock

[PATCH] staging: fsl-mc: Add missing header

2017-02-13 Thread Bogdan Purcareata
Compiling the fsl-mc bus driver will yield a couple of static analysis errors: warning: symbol 'fsl_mc_msi_domain_alloc_irqs' was not declared warning: symbol 'fsl_mc_msi_domain_free_irqs' was not declared. warning: symbol 'its_fsl_mc_msi_init' was not declared. warning: symbol

Re: [PATCH] usb: misc: usbtest: remove redundant check on retval < 0

2017-02-13 Thread Colin Ian King
On 13/02/17 10:45, Felipe Balbi wrote: > > Hi, > > Colin King writes: >> From: Colin Ian King >> >> The check for retval being less than zero is always true since >> retval equal to -EPIPE at that point. Replace the existing >> conditional

Re: [linux-sunxi] Re: [PATCH 2/8] ARM: sun4i: Add display blocks for the sun4i dtsi.

2017-02-13 Thread Priit Laes
On Mon, 2017-02-13 at 17:20 +0800, Chen-Yu Tsai wrote: > On Mon, Feb 13, 2017 at 5:16 PM, Maxime Ripard > wrote: > > Hi, > > > > On Sat, Feb 11, 2017 at 07:43:59PM +0200, Priit Laes wrote: > > > Added basic display pipeline consisting of tcon, display backend >

Re: [PATCH 1/2] exec: don't wait for zombie threads with cred_guard_mutex held

2017-02-13 Thread Oleg Nesterov
On 02/14, kbuild test robot wrote: > > Hi Oleg, > > [auto build test ERROR on linus/master] > [also build test ERROR on v4.10-rc8 next-20170213] > [if your patch is applied to the wrong git tree, please drop us a note to > help improve the system] > > url: > ht

Re: [PATCH linux v7 4/6] hwmon: occ: Add callbacks for parsing P8 OCC datastructures

2017-02-13 Thread Guenter Roeck
On Mon, Feb 13, 2017 at 11:47:22AM +1030, Andrew Jeffery wrote: > On Fri, 2017-02-10 at 16:01 +1030, Joel Stanley wrote: > > > On Wed, Feb 8, 2017 at 9:40 AM,   wrote: > > > > > From: "Edward A. James" > > > > > > Add functions to parse the data

Re: [PATCH RFC v2 tip/core/rcu] Maintain special bits at bottom of ->dynticks counter

2017-02-13 Thread Paul E. McKenney
On Mon, Feb 13, 2017 at 01:21:15PM +0100, Peter Zijlstra wrote: > On Thu, Feb 09, 2017 at 03:51:03PM -0800, Paul E. McKenney wrote: > > Currently, IPIs are used to force other CPUs to invalidate their TLBs > > in response to a kernel virtual-memory mapping change. This works, but > >

[PATCH v28 04/21] vfs: Add permission flags for setting file attributes

2017-02-13 Thread Andreas Gruenbacher
Richacls support permissions that allow to take ownership of a file, change the file permissions, and set the file timestamps. Support that by introducing new permission mask flags and by checking for those mask flags in setattr_prepare(). Signed-off-by: Andreas Gruenbacher

Re: [PATCH 2/2] sched/deadline: Throttle a constrained deadline task activated after the deadline

2017-02-13 Thread Daniel Bristot de Oliveira
On 02/13/2017 04:33 PM, Steven Rostedt wrote: >> +static inline bool dl_is_constrained(struct sched_dl_entity *dl_se) >> +{ >> +return dl_se->dl_runtime < dl_se->dl_period; >> +} >> + > Is it ever appropriate for a dl task to have runtime == period? What > purpose would that serve? Just run

[PATCH] pty: show associative slave of ptmx in fdinfo

2017-02-13 Thread Masatake YAMATO
This patch adds "tty-index" field to /proc/PID/fdinfo/N if N specifies /dev/ptmx. The field shows the index of associative slave pts. Though a minor number is given for each pts instance, ptmx is not. It means there is no way in user-space to know the association between file descriptors for

Re: [PATCH v8 3/3] dmaengine: pl330: Don't require irq-safe runtime PM

2017-02-13 Thread Vinod Koul
On Mon, Feb 13, 2017 at 04:32:32PM +0100, Ulf Hansson wrote: > [...] > > >> Although, I don't know of other examples, besides the runtime PM use > >> case, where non-atomic channel prepare/unprepare would make sense. Do > >> you? > > > > The primary ask for that has been to enable runtime_pm for

Re: [PATCH 1/2] usb: gadget: udc: avoid use of freed pointer

2017-02-13 Thread Michal Nazarewicz
On Mon, Feb 13 2017, Gustavo A. R. Silva wrote: > Rewrite udc_free_dma_chain() function to avoid use of pointer after free. > > Addresses-Coverity-ID: 1091172 > Reviewed-by: Greg Kroah-Hartman > Signed-off-by: Gustavo A. R. Silva Acked-by:

[BUGFIX PATCH 0/3] kprobes/arm: Improve kprobes implementation on arm

2017-02-13 Thread Masami Hiramatsu
Hi, Here are patches which improve kprobe on arm implementation. This includes some improves ported from x86 for multiple kretprobes on same function and recursing kprobes on FIQ (NMI) path. Also, I've fixed a bug(?) on recursing path. - [1/3]: Port an improvement (and fix) for recursing kprobe

Re: [PATCH] netlink: move nla_put_{u8,u16,u32} out of line

2017-02-13 Thread Arnd Bergmann
On Fri, Feb 10, 2017 at 2:24 PM, Arnd Bergmann wrote: > On Thu, Feb 9, 2017 at 6:00 PM, Arnd Bergmann wrote: >> To reduce this risk, -fsanitize-address-use-after-scope is now split out >> into a separate Kconfig option, which cannot be selected at the same time >>

[PATCH 2/2] usb: gadget: udc: remove unnecessary variable and update function prototype

2017-02-13 Thread Gustavo A. R. Silva
Remove unnecessary variable and update function prototype. Reviewed-by: Greg Kroah-Hartman Signed-off-by: Gustavo A. R. Silva --- drivers/usb/gadget/udc/amd5536udc.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git

[PATCH V5 2/4] uapi: sed-opal fix IOW for activate lsp to use correct struct

2017-02-13 Thread Scott Bauer
the IOW for the IOC_OPAL_ACTIVATE_LSP took the wrong strcure which would give us the wrong size when using _IOC_SIZE, switch it to the right structure. Fixes: 058f8a2 ("Include: Uapi: Add user ABI for Sed/Opal") Signed-off-by: Scott Bauer ---

SED Opal Fixups

2017-02-13 Thread Scott Bauer
So we have a few patches here, they're pretty small. First patch changes the sed-opal ioctl function parameters to take a void __user* instead of an unsigned long, this required a small cast in the nvme driver. Patch 2 is a UAPI fixup for the IOW to make an ioctl the right size. Patch 3 fixes a

[PATCH V5 3/4] Move stack parameters for sed_ioctl to prevent oversized stack with CONFIG_KASAN

2017-02-13 Thread Scott Bauer
When CONFIG_KASAN is enabled, compilation fails: block/sed-opal.c: In function 'sed_ioctl': block/sed-opal.c:2447:1: error: the frame size of 2256 bytes is larger than 2048 bytes [-Werror=frame-larger-than=] Moved all the ioctl structures off the stack and dynamically activate using _IOC_SIZE()

Re: [PATCH] building libtraceevent with clang

2017-02-13 Thread Arnaldo Carvalho de Melo
Em Mon, Feb 13, 2017 at 01:24:55PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Fri, Feb 10, 2017 at 02:14:08PM -0500, Steven Rostedt escreveu: > > On Fri, 10 Feb 2017 14:03:17 -0300 > > Arnaldo Carvalho de Melo wrote: > > > + case OLD_RINGBUF_TYPE_TIME_EXTEND: > > > +

Re: [PATCH v2] reset: Add i.MX7 SRC reset driver

2017-02-13 Thread Philipp Zabel
Hi Andrey, thank you for the driver, I have a few comments below. On Mon, 2017-02-13 at 07:33 -0800, Andrey Smirnov wrote: > This driver exposes various reset faculties, impelented by System Reset s/impelented/implemented/ > Controller IP block, as a reset driver. Currently only PCIE related >

Re: [PATCH V5 3/4] Move stack parameters for sed_ioctl to prevent oversized stack with CONFIG_KASAN

2017-02-13 Thread Arnd Bergmann
On Mon, Feb 13, 2017 at 5:29 PM, Scott Bauer wrote: > On Mon, Feb 13, 2017 at 04:30:36PM +, David Laight wrote: >> From: Scott Bauer Sent: 13 February 2017 16:11 >> > When CONFIG_KASAN is enabled, compilation fails: >> > >> > block/sed-opal.c: In function 'sed_ioctl':

[PATCH v28 08/21] richacl: Compute maximum file masks from an acl

2017-02-13 Thread Andreas Gruenbacher
Compute upper bound owner, group, and other file masks with as few permissions as possible without denying any permissions that the NFSv4 acl in a richacl grants. This algorithm is used when a file inherits an acl at create time and when an acl is set via a mechanism that does not provide file

[PATCH v28 05/21] richacl: In-memory representation and helper functions

2017-02-13 Thread Andreas Gruenbacher
A richacl consists of an NFSv4 acl and an owner, group, and other mask. These three masks correspond to the owner, group, and other file permission bits, but they contain NFSv4 permissions instead of POSIX permissions. Each entry in the NFSv4 acl applies to the file owner (OWNER@), the owning

Re: [PATCH v2] usb: gadget: udc: remove pointer dereference after free

2017-02-13 Thread Michal Nazarewicz
On Sat, Feb 11 2017, Gustavo A. R. Silva wrote: > Remove pointer dereference after free and set pointer to NULL after free. > > Addresses-Coverity-ID: 1091173 > Signed-off-by: Gustavo A. R. Silva Acked-by: Michal Nazarewicz > --- > Changes in v2: >

14 Bug submissions/code questions

2017-02-13 Thread Allen-Bond, Michael Rafael
The Bugzilla URLs are all together at the bottom as well. 1. fs/inode.c 804Lfind_inode_fast structure inode data structure The data structure is initialized as NULL, is there any guarantee that the head of the list it points to after hlist_for_each_entry() won't be a

Re: [PATCH] thermal: exynos: Remove parsing unused samsung,tmu_cal_mode property

2017-02-13 Thread Bartlomiej Zolnierkiewicz
On Saturday, February 11, 2017 10:12:00 PM Krzysztof Kozlowski wrote: > The property samsung,tmu_cal_mode is not used and not used. We can > safely remove it. > > Signed-off-by: Krzysztof Kozlowski Acked-by: Bartlomiej Zolnierkiewicz Best regards,

Re: [PATCH 2/2] sched/deadline: Throttle a constrained deadline task activated after the deadline

2017-02-13 Thread Daniel Bristot de Oliveira
On 02/13/2017 04:46 PM, Daniel Bristot de Oliveira wrote: > On 02/13/2017 04:33 PM, Steven Rostedt wrote: >>> +static inline bool dl_is_constrained(struct sched_dl_entity *dl_se) >>> +{ >>> + return dl_se->dl_runtime < dl_se->dl_period; >>> +} >>> + >> Is it ever appropriate for a dl task to

RE: [PATCH V5 3/4] Move stack parameters for sed_ioctl to prevent oversized stack with CONFIG_KASAN

2017-02-13 Thread David Laight
From: Scott Bauer Sent: 13 February 2017 16:11 > When CONFIG_KASAN is enabled, compilation fails: > > block/sed-opal.c: In function 'sed_ioctl': > block/sed-opal.c:2447:1: error: the frame size of 2256 bytes is larger than > 2048 bytes [-Werror=frame- > larger-than=] > > Moved all the ioctl

Re: [PATCH] block/loop: fix race between I/O and set_status

2017-02-13 Thread Jens Axboe
On 02/10/2017 08:40 PM, Ming Lei wrote: > Inside set_status, transfer need to setup again, so > we have to drain IO before the transition, otherwise > oops may be triggered like the following: > > divide error: [#1] SMP KASAN > CPU: 0 PID: 2935 Comm: loop7 Not tainted 4.10.0-rc7+

Re: [PATCH] usb: misc: usbtest: remove redundant check on retval < 0

2017-02-13 Thread Alan Stern
On Mon, 13 Feb 2017, Felipe Balbi wrote: > Hi, > > Colin King writes: > > From: Colin Ian King > > > > The check for retval being less than zero is always true since > > retval equal to -EPIPE at that point. Replace the existing > >

[PATCH] drm: mali-dp: add atomic_print_state for planes

2017-02-13 Thread Mihail Atanassov
Print all the extra fields of malidp_plane_state. Signed-off-by: Mihail Atanassov --- drivers/gpu/drm/arm/malidp_planes.c | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers/gpu/drm/arm/malidp_planes.c b/drivers/gpu/drm/arm/malidp_planes.c index

Re: [patch net-next] net: qcom/emac: fix a sizeof() typo

2017-02-13 Thread Timur Tabi
walter harms wrote: The question is: why is a simple calculation const*const separated into a function ? This is a callback function. That's just how it's defined. It's rare, but there are drivers that use the parameter, like this one:

[PATCH v28 06/21] richacl: Permission mapping functions

2017-02-13 Thread Andreas Gruenbacher
We need to map from POSIX permissions to NFSv4 permissions when a chmod() is done, from NFSv4 permissions to POSIX permissions when an acl is set (which implicitly sets the file permission bits), and from the MAY_READ/MAY_WRITE/MAY_EXEC/MAY_APPEND flags to NFSv4 permissions when doing an access

Re: [PATCH] x86/vm86: fix unused variable warning if THP is disabled.

2017-02-13 Thread Thomas Gleixner
On Mon, 13 Feb 2017, Kirill A. Shutemov wrote: > GCC complaines on unused variable 'vma' in mark_screen_rdonly() if THP > is disalbed: > > arch/x86/kernel/vm86_32.c: In function ‘mark_screen_rdonly’: > arch/x86/kernel/vm86_32.c:180:26: warning: unused variable ‘vma’ > [-Wunused-variable] >

Re: [PATCHv6 08/37] filemap: handle huge pages in do_generic_file_read()

2017-02-13 Thread Matthew Wilcox
On Mon, Feb 13, 2017 at 06:33:42PM +0300, Kirill A. Shutemov wrote: > No. pagecache_get_page() returns subpage. See description of the first > patch. Your description says: > We also change interface for page-cache lookup function: > > - functions that lookup for pages[1] would return

Re: [PATCH 1/2] exec: don't wait for zombie threads with cred_guard_mutex held

2017-02-13 Thread kbuild test robot
Hi Oleg, [auto build test ERROR on linus/master] [also build test ERROR on v4.10-rc8 next-20170213] [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/Oleg-Nesterov/fix-the-traced-mt-exec-deadlock

Re: [PATCH 2/2] usb: gadget: udc: remove unnecessary variable and update function prototype

2017-02-13 Thread Michal Nazarewicz
On Mon, Feb 13 2017, Gustavo A. R. Silva wrote: > Remove unnecessary variable and update function prototype. > > Reviewed-by: Greg Kroah-Hartman > Signed-off-by: Gustavo A. R. Silva Acked-by: Michal Nazarewicz > --- >

Re: [PATCHv6 08/37] filemap: handle huge pages in do_generic_file_read()

2017-02-13 Thread Matthew Wilcox
On Mon, Feb 13, 2017 at 08:01:17AM -0800, Matthew Wilcox wrote: > On Mon, Feb 13, 2017 at 06:33:42PM +0300, Kirill A. Shutemov wrote: > > No. pagecache_get_page() returns subpage. See description of the first > > patch. Oh, I re-read patch 1 and it made sense now. I missed the bit where

[PATCH V5 4/4] Maintainers: Modify SED list from nvme to block

2017-02-13 Thread Scott Bauer
Signed-off-by: Scott Bauer --- MAINTAINERS | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/MAINTAINERS b/MAINTAINERS index e325373..b983b25 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11094,7 +11094,7 @@ SECURE ENCRYPTING DEVICE (SED) OPAL DRIVER M:

Re: [PATCH] building libtraceevent with clang

2017-02-13 Thread Arnaldo Carvalho de Melo
Em Fri, Feb 10, 2017 at 02:14:08PM -0500, Steven Rostedt escreveu: > On Fri, 10 Feb 2017 14:03:17 -0300 > Arnaldo Carvalho de Melo wrote: > > + case OLD_RINGBUF_TYPE_TIME_EXTEND: > > + extend = read_4(kbuf, ptr); > > + extend <<= TS_SHIFT; > > +

Re: [PATCH V5 1/4] block: sed-opal: change ioctl to take user pointer instead of unsinged long

2017-02-13 Thread Scott Bauer
esOn Mon, Feb 13, 2017 at 09:11:09AM -0700, Scott Bauer wrote: > Signed-off-by: Scott Bauer > --- > block/sed-opal.c | 6 -- > drivers/nvme/host/core.c | 3 ++- > include/linux/sed-opal.h | 4 ++-- > 3 files changed, 8 insertions(+), 5 deletions(-) > > diff

[PATCH RFC] iopoll: allow for poll_timeout to back-off

2017-02-13 Thread Nicholas Mc Guire
mpile test this but actually the key issue is to get feedback on the concept rather than if the patch is usable in the below form. Patch was compile tested with: x86_64_defconfig Patch is against 4.10-rc7 (localversion-next is next-20170213) include/linux/iopoll.h | 12 1 file c

Re: 4.10-rc1: thinkpad x60: who ate my cpu?

2017-02-13 Thread lkml
On Mon, Feb 13, 2017 at 12:25:28PM +0100, Thomas Gleixner wrote: > On Mon, 13 Feb 2017, l...@pengaru.com wrote: > > > On Mon, Feb 13, 2017 at 10:42:36AM +0100, Pavel Machek wrote: > > > Hi! > > > > > > On Mon 2017-02-13 09:48:41, Thomas Gleixner wrote: > > > > On Mon, 13 Feb 2017, Pavel Machek

Re: [PATCH v8 1/3] dmaengine: Add new device_{set,release}_slave callbacks

2017-02-13 Thread Marek Szyprowski
Hi Vinod, On 2017-02-13 02:42, Vinod Koul wrote: On Fri, Feb 10, 2017 at 01:07:41PM +0100, Marek Szyprowski wrote: Hi Vinod, On 2017-02-10 05:34, Vinod Koul wrote: On Thu, Feb 09, 2017 at 03:22:49PM +0100, Marek Szyprowski wrote: Add two new callbacks to DMA engine device. They will used to

Re: [patch net-next] net: qcom/emac: fix a sizeof() typo

2017-02-13 Thread walter harms
Am 13.02.2017 12:00, schrieb Dan Carpenter: > We had intended to say "sizeof(u32)" but the "u" is missing. > Fortunately, sizeof(32) is also 4, so the original code still works. > > Fixes: c4e7beea2192 ("net: qcom/emac: add ethtool support for reading > hardware registers") > Signed-off-by:

Re: [PATCH] lz4: fix performance regressions

2017-02-13 Thread Sven Schmidt
On Sun, Feb 12, 2017 at 10:41:17PM +0100, Willy Tarreau wrote: > On Sun, Feb 12, 2017 at 04:20:00PM +0100, Sven Schmidt wrote: > > On Sun, Feb 12, 2017 at 02:05:08PM +0100, Willy Tarreau wrote: > > > Hi Sven, > > > > > > On Sun, Feb 12, 2017 at 12:16:18PM +0100, Sven Schmidt wrote: > > > > Fix

Re: [PATCH] ARM: dts: exynos: Use thermal fuse value for thermal zone 0 on Exynos5420

2017-02-13 Thread Krzysztof Kozlowski
On Mon, Feb 13, 2017 at 1:38 PM, Anand Moon wrote: > Hi Krzysztof, > > > > On 12 February 2017 at 01:44, Krzysztof Kozlowski wrote: >> In Odroid XU3 Lite board, the temperature levels reported for thermal >> zone 0 were weird. In warm room: >>

<    1   2   3   4   5   6   7   8   9   10   >