Re: [PATCH v2 1/2] tiocsti-restrict : Add owner user namespace to tty_struct

2017-04-23 Thread Matt Brown
On 04/23/2017 01:02 PM, Jann Horn wrote: On Sun, Apr 23, 2017 at 9:24 AM, Matt Brown wrote: This patch adds struct user_namespace *owner_user_ns to the tty_struct. Then it is set to current_user_ns() in the alloc_tty_struct function. This is done to facilitate capability checks against the

Re: [PATCH] dell-laptop: Adds support for keyboard backlight timeout AC settings

2017-04-23 Thread Arcadiy Ivanov
I've tested the patch over 4.10.10 on Fedora 25 and it works both on and off AC power on Dell Precision 7510. Thanks a lot! On 2017-04-23 15:40, Pali Rohár wrote: When changing keyboard backlight state on new Dell laptops, firmware expects a new timeout AC value filled in Set New State SMBIOS

[PATCH] staging: rtl8192u: Fix sparse warning

2017-04-23 Thread Marcos Paulo de Souza
Fix the following sparse warning: rtl8192u/ieee80211/ieee80211_rx.c:1489:45: warning: incorrect type in assignment (different base types) rtl8192u/ieee80211/ieee80211_rx.c:1489:45:expected restricted __le16 rtl8192u/ieee80211/ieee80211_rx.c:1489:45:got unsigned short [unsigned]

Re: [PATCH v2 1/2] tiocsti-restrict : Add owner user namespace to tty_struct

2017-04-23 Thread Jann Horn
On Sun, Apr 23, 2017 at 10:23 PM, Matt Brown wrote: > On 04/23/2017 01:02 PM, Jann Horn wrote: >> >> On Sun, Apr 23, 2017 at 9:24 AM, Matt Brown wrote: >>> >>> This patch adds struct user_namespace *owner_user_ns to the tty_struct. >>> Then it is set to current_user_ns() in the alloc_tty_struct

Re: [PATCH 1/1] drivers:net:ethernet:adi:bfin_mac: Use FIELD_SIZEOF defined kernel macro

2017-04-23 Thread Geert Uytterhoeven
Hi Karim, On Sun, Apr 23, 2017 at 8:02 PM, Karim Eshapa wrote: > Use FIELD_SIZEOF defined kernel macro kernel.h > > Signed-off-by: Karim Eshapa > --- > drivers/net/ethernet/adi/bfin_mac.c | 14 +- > 1 file changed, 9 insertions(+), 5 deletions(-) > > diff --git

[PATCH 1/2] [media] vb2: Fix an off by one error in 'vb2_plane_vaddr'

2017-04-23 Thread Christophe JAILLET
We should ensure that 'plane_no' is '< vb->num_planes' as done in 'vb2_plane_cookie' just a few lines below. Signed-off-by: Christophe JAILLET --- drivers/media/v4l2-core/videobuf2-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 2/2] [media] vb2: Fix error handling in '__vb2_buf_mem_alloc'

2017-04-23 Thread Christophe JAILLET
'call_ptr_memop' can return NULL, so we must test its return value with 'IS_ERR_OR_NULL'. Otherwise, the test 'if (mem_priv)' is meaningless. Signed-off-by: Christophe JAILLET --- Note that error checking after 'call_ptr_memop' calls is not consistent in this file. I guess that 'IS_ERR_OR_NULL'

Re: [PATCH v2 08/17] fs: retrofit old error reporting API onto new infrastructure

2017-04-23 Thread NeilBrown
On Fri, Apr 21 2017, Jeff Layton wrote: > On Tue, 2017-04-18 at 08:56 +1000, NeilBrown wrote: >> On Wed, Apr 12 2017, Jeff Layton wrote: >> >> > On Thu, 2017-04-13 at 08:14 +1000, NeilBrown wrote: >> > > >> > > I suspect that the filemap_check_wb_error() will need to be moved >> > > into some

[PATCH] V4L2 SDR: Add Real U8 format (V4L2_SDR_FMT_RU8)

2017-04-23 Thread Bertold Van den Bergh
This patch adds support for the Real U8 format to the V4L2 SDR framework. This will be used for a piece of hardware we are developing. Signed-off-by: Bertold Van den Bergh --- drivers/media/v4l2-core/v4l2-ioctl.c | 1 + include/uapi/linux/videodev2.h | 1 + 2 files changed, 2

Re: [PATCH 0/5] lightnvm: pblk extra patches for 4.12

2017-04-23 Thread Jens Axboe
On Sun, Apr 23 2017, Matias Bjørling wrote: > On 04/22/2017 01:32 AM, Javier González wrote: > >Hi Matias, > > > >This is a couple of bug fixes for the pblk release patch. > > > >Thanks, > >Javier > > > >Javier González (5): > > lightnvm: pblk: fix race condition on line retry > > lightnvm:

[PATCH] scsi: mvumi: remove code handling zero scsi_sg_count(scmd) case

2017-04-23 Thread Alexey Khoroshilov
As Christoph Hellwig noted, SCSI commands that transfer data always have a SG entry. The patch removes dead code in mvumi_make_sgl(), mvumi_complete_cmd() and mvumi_timed_out() that handle zero scsi_sg_count(scmd) case. Also the patch adds pci_unmap_sg() on failure path in mvumi_make_sgl().

Re: [PATCH RFC] ptr_ring: add ptr_ring_unconsume

2017-04-23 Thread Michael S. Tsirkin
On Tue, Apr 18, 2017 at 11:07:42AM +0800, Jason Wang wrote: > > > On 2017年04月17日 07:19, Michael S. Tsirkin wrote: > > Applications that consume a batch of entries in one go > > can benefit from ability to return some of them back > > into the ring. > > > > Add an API for that - assuming there's

get_zone_device_page() in get_page() and page_cache_get_speculative()

2017-04-23 Thread Kirill A. Shutemov
On Thu, Apr 20, 2017 at 02:46:51PM -0700, Dan Williams wrote: > On Sat, Mar 18, 2017 at 2:52 AM, tip-bot for Kirill A. Shutemov > wrote: > > Commit-ID: 2947ba054a4dabbd82848728d765346886050029 > > Gitweb: > > http://git.kernel.org/tip/2947ba054a4dabbd82848728d765346886050029 > > Author:

Re: [PATCH v4 net-next] mdio_bus: Issue GPIO RESET to PHYs.

2017-04-23 Thread Andrew Lunn
On Fri, Apr 21, 2017 at 03:31:09PM +0200, Lars-Peter Clausen wrote: > On 04/21/2017 03:15 PM, Roger Quadros wrote: > > diff --git a/Documentation/devicetree/bindings/net/mdio.txt > > b/Documentation/devicetree/bindings/net/mdio.txt > > new file mode 100644 > > index 000..4ffbbac > > ---

[PATCH v3 0/2] tiocsti-restrict : make TIOCSTI ioctl require CAP_SYS_ADMIN

2017-04-23 Thread Matt Brown
This patchset introduces the tiocsti_restrict sysctl, whose default is controlled via CONFIG_SECURITY_TIOCSTI_RESTRICT. When activated, this control restricts all TIOCSTI ioctl calls from non CAP_SYS_ADMIN users. This patch was inspired from GRKERNSEC_HARDEN_TTY. This patch would have prevented

[PATCH v3 1/2] tiocsti-restrict : Add owner user namespace to tty_struct

2017-04-23 Thread Matt Brown
This patch adds struct user_namespace *owner_user_ns to the tty_struct. Then it is set to current_user_ns() in the alloc_tty_struct function. This is done to facilitate capability checks against the original user namespace that allocated the tty. E.g. ns_capable(tty->owner_user_ns,CAP_SYS_ADMIN)

[PATCH v3 2/2] tiocsti-restrict : make TIOCSTI ioctl require CAP_SYS_ADMIN

2017-04-23 Thread Matt Brown
This introduces the tiocsti_restrict sysctl, whose default is controlled via CONFIG_SECURITY_TIOCSTI_RESTRICT. When activated, this control restricts all TIOCSTI ioctl calls from non CAP_SYS_ADMIN users. This patch was inspired from GRKERNSEC_HARDEN_TTY. This patch would have prevented

Re: [PATCH] usb: core: Warn if an URB's transfer_buffer is on stack

2017-04-23 Thread Alan Stern
On Sun, 23 Apr 2017, Florian Fainelli wrote: > > In that case, it would be better to move the warning to a central place > > where it will always get triggered, such as map_urb_for_dma(). As it > > is, the patch will only issue a warning for callers of usb_bulk_msg(), > > usb_interrupt_msg(), or

Re: [PATCH] ore: fix spelling mistake: "Multples" -> "multiples"

2017-04-23 Thread Joe Perches
On Sun, 2017-04-23 at 12:37 +0300, Boaz Harrosh wrote: > On 04/22/2017 03:48 PM, Colin King wrote: > > trivial fix to spelling mistake in ORE_ERR message and make word all > > lower case. [] > > diff --git a/fs/exofs/ore.c b/fs/exofs/ore.c [] > > @@ -68,7 +68,7 @@ int ore_verify_layout(unsigned

Build failures in -next due to 'linux/io.h: Add pci_remap_cfgspace() interface'

2017-04-23 Thread Guenter Roeck
Hi, -next fails to build for several architectures due to commit 'linux/io.h: Add pci_remap_cfgspace() interface'. Error message is include/linux/io.h: In function ‘pci_remap_cfgspace’: include/linux/io.h:108:9: error: implicit declaration of function ‘ioremap_nocache’ Affected architectures

Linux 4.11-rc8

2017-04-23 Thread Linus Torvalds
So originally I was just planning on releasing the final 4.11 today, but while we didn't have a *lot* of changes the last week, we had a couple of really annoying ones, so I'm doing another rc release instead. I did get fixes for the issues that popped up, so I could have released 4.11 as-is, but

Build failures in -next due to 'drm/amdgpu: add NGG parameters'

2017-04-23 Thread Guenter Roeck
Hi, alpha:allmodconfig (and possibly others) fail to build in -next with the following error. drivers/gpu/drm/amd/amdgpu/amdgpu.h:1006:2: error: expected identifier before '(' token drivers/gpu/drm/amd/amdgpu/amdgpu.h:1011:28: error: 'NGG_BUF_MAX' undeclared here Bisect points

RE:drivers:net:ethernet:adi:bfin_mac: Use FIELD_SIZEOF defined kernel macro

2017-04-23 Thread Karim Eshapa
On Sun, 23 Apr 2017 22:56:38 +0200, Geert Uytterhoeven: >IMHO this makes the code less safe and less future-proof. >What if the type of info is ever changed? >There's no safety check to validate that the FIELD_SIZEOF() operates on the >same data as the strlcpy() destination. Really make sense :)

patch series moving compat syscalls from fs/compat.c

2017-04-23 Thread Stephen Rothwell
Hi Al, I couldn't easily find this patch series posted anywhere (I didn't try too hard) so I sill just comment here. I first noticed them in the vfs tree in linux-next today. Overall, I like what they day, but when I first created kernel/compat.c (in 2002), Linus did not want the compat code

[PATCH 1/1] power: Fix unchecked return value of devm_kzalloc

2017-04-23 Thread Pan Bian
Function devm_kzalloc() will return a NULL pointer. However, in function isp1704_charger_probe(), the return value of devm_kzalloc() is directly used without validation. This may result in a bad memory access bug. Signed-off-by: Pan Bian --- drivers/power/supply/isp1704_charger.c | 2 ++ 1 file

Build failure in -next due to 'hexagon: switch to RAW_COPY_USER'

2017-04-23 Thread Guenter Roeck
Hi, hexagon:defconfig fails to build in -next with the following build error. In file included from include/linux/uaccess.h:13:0, from include/linux/poll.h:11, from include/linux/ring_buffer.h:7, from include/linux/trace_events.h:5,

Re: [HMM 03/15] mm/unaddressable-memory: new type of ZONE_DEVICE for unaddressable memory

2017-04-23 Thread John Hubbard
On 4/23/17 6:13 AM, Dan Williams wrote: On Sat, Apr 22, 2017 at 11:11 AM, Jerome Glisse wrote: On Fri, Apr 21, 2017 at 10:30:01PM -0700, Dan Williams wrote: On Fri, Apr 21, 2017 at 8:30 PM, Jérôme Glisse wrote: [...] +/* + * Specialize ZONE_DEVICE memory into multiple types each having

[PATCH 2/2] arm64:vdso: Remove ISB from gettimeofday.

2017-04-23 Thread Andrew Pinski
ISB is normally required before mrs CNTVCT if we want the mrs to completed after the loads. In this case it is not. As we are taking the difference and if that difference was going to be negative, we just use the last counter value instead. Signed-off-by: Andrew Pinski ---

[PATCH 1/1] rndis_wlan: add return value validation

2017-04-23 Thread Pan Bian
From: Pan Bian Function create_singlethread_workqueue() will return a NULL pointer if there is no enough memory, and its return value should be validated before using. However, in function rndis_wlan_bind(), its return value is not checked. This may cause NULL dereference bugs. This patch fixes

[PATCH 1/2] arm64:vdso: Rewrite gettimeofday into C.

2017-04-23 Thread Andrew Pinski
This allows the compiler to optimize the divide by 1000. And remove the other divide. On ThunderX, gettimeofday improves by 32%. On ThunderX 2, gettimeofday improves by 18%. Signed-off-by: Andrew Pinski --- arch/arm64/kernel/vdso/Makefile | 13 +- arch/arm64/kernel/vdso/gettimeofday.S

Re: [PATCH v6 2/7] perf/x86/intel: Record branch type

2017-04-23 Thread Jin, Yao
On 4/23/2017 9:55 PM, Jiri Olsa wrote: On Thu, Apr 20, 2017 at 08:07:50PM +0800, Jin Yao wrote: SNIP +#define X86_BR_TYPE_MAP_MAX 16 + +static int +common_branch_type(int type) +{ + int i, mask; + const int branch_map[X86_BR_TYPE_MAP_MAX] = { + PERF_BR_CALL,

[PATCH] drivers: input: joystick: Add PSX(Play Staion 1/2) pad with SPI driver Add PSX(Play Staion 1/2) pad with SPI driver. Pads can be connected directry SPI bus.

2017-04-23 Thread AZO
--- drivers/input/joystick/Kconfig | 9 + drivers/input/joystick/Makefile | 1 + drivers/input/joystick/psxpad-spi.c | 679 3 files changed, 689 insertions(+) create mode 100644 drivers/input/joystick/psxpad-spi.c diff --git

Re: Performance of low-cpu utilisation benchmark regressed severely since 4.6

2017-04-23 Thread Rafael J. Wysocki
On Sun, Apr 23, 2017 at 5:31 PM, Doug Smythies wrote: > On 2017.04.22 14:08 Rafael wrote: >> On Friday, April 21, 2017 11:29:06 PM Doug Smythies wrote: >>> On 2017.04.20 18:18 Rafael wrote: On Thursday, April 20, 2017 07:55:57 AM Doug Smythies wrote: > On 2017.04.19 01:16 Mel Gorman

Re: patch series moving compat syscalls from fs/compat.c

2017-04-23 Thread Al Viro
On Mon, Apr 24, 2017 at 10:31:34AM +1000, Stephen Rothwell wrote: > Hi Al, > > I couldn't easily find this patch series posted anywhere (I didn't try > too hard) so I sill just comment here. I first noticed them in the vfs > tree in linux-next today. > > Overall, I like what they day, but when

Re: [PATCH v3 1/2] tiocsti-restrict : Add owner user namespace to tty_struct

2017-04-23 Thread Serge E. Hallyn
Quoting Matt Brown (m...@nmatt.com): > This patch adds struct user_namespace *owner_user_ns to the tty_struct. > Then it is set to current_user_ns() in the alloc_tty_struct function. > > This is done to facilitate capability checks against the original user > namespace that allocated the tty. >

Re: [PATCH v2 03/11] dt-bindings: soc/fsl: Update reserved memory binding for QBMan

2017-04-23 Thread Scott Wood
On Wed, 2017-04-19 at 16:48 -0400, Roy Pledge wrote: > Updates the QMan and BMan device tree bindings for reserved memory > nodes. This makes the reserved memory allocation compatiable with > the shared-dma-pool usage. > > Signed-off-by: Roy Pledge > --- >  

[PATCH v3] ARM: dts: at91: sama5d2: add m_can nodes

2017-04-23 Thread Wenyou Yang
Add nodes to support the Controller Area Network(M_CAN) on SAMA5D2. The version of M_CAN IP core is 3.1.0 (CREL = 0x31040730). As said in SAMA5D2 datasheet, the CAN clock is recommended to use frequencies of 20, 40 or 80 MHz. To achieve these frequencies, PMC GCLK3 must select the UPLLCK(480 MHz)

Re: Performance of low-cpu utilisation benchmark regressed severely since 4.6

2017-04-23 Thread Srinivas Pandruvada
On Mon, 2017-04-24 at 02:59 +0200, Rafael J. Wysocki wrote: > On Sun, Apr 23, 2017 at 5:31 PM, Doug Smythies > wrote: [...] > > It looks like the cost is mostly related to moving the load from > > > one CPU to > > > another and waiting for the new one to ramp up then. Last time when we analyzed

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

2017-04-23 Thread Stephen Rothwell
Hi all, On Fri, 21 Apr 2017 12:10:14 +1000 Stephen Rothwell wrote: > > After merging the drm-misc tree, today's linux-next build (x86_64 > allmodconfig) failed like this: > > drivers/tee/tee_shm.c:87:2: error: unknown field 'kmap_atomic' specified in > initializer > .kmap_atomic =

[PATCH v2] usb: core: Warn if an URB's transfer_buffer is on stack

2017-04-23 Thread Florian Fainelli
We see a large number of fixes to several drivers to remove the usage of on-stack buffers feeding into USB transfer functions. Make it easier to spot the offenders by adding a warning in usb_hcd_map_urb_for_dma() checking that urb->transfer_buffer is not a stack object. Signed-off-by: Florian

Re: [dm-devel] [PATCH] dm-region-hash: fix strange usage of mempool_alloc.

2017-04-23 Thread NeilBrown
On Fri, Apr 21 2017, Mikulas Patocka wrote: > On Mon, 10 Apr 2017, NeilBrown wrote: > >> mempool_alloc() should only be called with GFP_ATOMIC when >> it is not safe to wait. Passing __GFP_NOFAIL to kmalloc() >> says that it is safe to wait indefinitely. So this code is >> inconsistent. >> >>

Re: [PATCH v2 02/11] soc/fsl/qbman: Use shared-dma-pool for QMan private memory allocations

2017-04-23 Thread Scott Wood
On Wed, 2017-04-19 at 16:48 -0400, Roy Pledge wrote: > @@ -727,10 +730,81 @@ static int fsl_qman_probe(struct platform_device > *pdev) >   qm_channel_caam = QMAN_CHANNEL_CAAM_REV3; >   } >   > - ret = zero_priv_mem(dev, node, fqd_a, fqd_sz); > - WARN_ON(ret); > - if

Re: your mail

2017-04-23 Thread Joonsoo Kim
On Fri, Apr 21, 2017 at 09:16:16AM +0200, Michal Hocko wrote: > On Fri 21-04-17 13:38:28, Joonsoo Kim wrote: > > On Thu, Apr 20, 2017 at 09:28:20AM +0200, Michal Hocko wrote: > > > On Thu 20-04-17 10:27:55, Joonsoo Kim wrote: > > > > On Mon, Apr 17, 2017 at 10:15:15AM +0200, Michal Hocko wrote: >

Re: [PATCH v2 07/11] soc/fsl/qbman: Rework ioremap() calls for ARM/PPC

2017-04-23 Thread Scott Wood
On Wed, 2017-04-19 at 16:48 -0400, Roy Pledge wrote: > Rework ioremap() for PPC and ARM. The PPC devices require a > non-coherent mapping while ARM will work with a non-cachable/write > combine mapping. > > Signed-off-by: Roy Pledge > --- >  drivers/soc/fsl/qbman/bman_portal.c | 16

Re: [PATCH 02/11] blk: make the bioset rescue_workqueue optional.

2017-04-23 Thread NeilBrown
On Fri, Apr 21 2017, Christoph Hellwig wrote: > On Thu, Apr 20, 2017 at 03:38:48PM +1000, NeilBrown wrote: >> This patch converts bioset_create() and >> bioset_create_nobvec() to not create a workqueue so >> alloctions will never trigger punt_bios_to_rescuer(). It >> also introduces

Re: [PATCH] kallsyms: Use bounded strnchr() when parsing string

2017-04-23 Thread Masami Hiramatsu
On Sun, 23 Apr 2017 22:53:43 +0530 "Naveen N. Rao" wrote: > When parsing for the format, we use strchr() to look for > the separator, when we know that the module name can't be longer than > MODULE_NAME_LEN. Enforce the same using strnchr(). Looks good to me :) Reviewed-by: Masami Hiramatsu

Re: [PATCH v5 1/4] printk/nmi: generic solution for safe printk in NMI

2017-04-23 Thread Sergey Senozhatsky
On (04/21/17 14:06), Petr Mladek wrote: [..] > > I agree that this_cpu_read(printk_context) covers slightly more than > > logbuf_lock scope, so we may get positive this_cpu_read(printk_context) > > with unlocked logbuf_lock, but I don't tend to think that it's a big > > problem. > >

Re: [PATCH v3] kbuild: Add support to generate LLVM bitcode files

2017-04-23 Thread Masahiro Yamada
Hi Matthias, 2017-04-22 4:55 GMT+09:00 Matthias Kaehlcke : > Hi Masahiro, > > El Fri, Apr 21, 2017 at 02:02:46PM +0900 Masahiro Yamada ha dit: > >> 2017-04-05 2:27 GMT+09:00 Matthias Kaehlcke : >> > From: Vinícius Tinti >> > >> > Add rules to kbuild in order to generate LLVM bitcode files with

Re: [PATCH 01/11] blk: remove bio_set arg from blk_queue_split()

2017-04-23 Thread NeilBrown
On Sat, Apr 22 2017, Javier González wrote: >> On 20 Apr 2017, at 07.38, NeilBrown wrote: >> >> blk_queue_split() is always called with the last arg being q->bio_split, >> where 'q' is the first arg. >> >> Also blk_queue_split() sometimes uses the passed-in 'bs' and sometimes uses >>

[PATCH v2 0/3] Handle memmap and mem kernel options in boot stage kaslr

2017-04-23 Thread Baoquan He
People reported kernel panic occurs during system boots up with mem boot option. After checking code, several problems are found about memmap= and mem= in boot stage kaslr. *) In commit f28442497b5c ("x86/boot: Fix KASLR and memmap= collision"), only one memmap entry is considered and only

[PATCH v2 1/3] KASLR: Parse all memmap entries in cmdline

2017-04-23 Thread Baoquan He
In commit: f28442497b5c ("x86/boot: Fix KASLR and memmap= collision") ... the memmap= option is parsed so that KASLR can avoid those reserved regions. It uses cmdline_find_option() to get the value if memmap= is specified, however the problem is that cmdline_find_option() can only find the

[PATCH v2 2/3] KASLR: Handle memory limit specified by memmap and mem option

2017-04-23 Thread Baoquan He
Option mem= will limit the max address a system can use and any memory region above the limit will be removed. Furthermore, memmap=nn[KMG] which has no offset specified has the same behaviour as mem=. KASLR needs to consider this when choosing the random position for decompressing the kernel.

[PATCH v2 3/3] Documentation/kernel-parameters.txt: Update 'memmap=' option description

2017-04-23 Thread Baoquan He
In commit: 9710f581bb4c ("x86, mm: Let "memmap=" take more entries one time") ... 'memmap=' was changed to adopt multiple, comma delimited values in a single entry, so update the related description. In the special case of only specifying size value without an offset, like memmap=nn[KMG],

Re: [PATCH v1 1/1] srcu-cbmc: Use /usr/bin/awk instead of /bin/awk

2017-04-23 Thread Paul E. McKenney
On Fri, Apr 21, 2017 at 03:05:49PM -0700, Josh Triplett wrote: > On Fri, Apr 21, 2017 at 09:52:10PM +, Kushwaha, Priyalee wrote: > > Tested 9 distros centos, Debian, Fedora, Gentoo, Opensuse, slackware, > > Ubuntu, poky showed awk at /usr/bin/awk. > > > > Here is another similar patch which

Re: [PATCH 0/4] Handle memmap and mem kernel options in boot stage kaslr

2017-04-23 Thread Baoquan He
On 04/18/17 at 02:51pm, Ingo Molnar wrote: > > * Baoquan He wrote: > > > On 04/18/17 at 11:47am, Ingo Molnar wrote: > > > I ported this series to tip:x86/boot (please post future versions against > > > that), > > > and beyond a trivial conflict with e820entry => e820_entry, it fails to > > >

TREE_SRCU slows hotplug by factor ~16

2017-04-23 Thread Mike Galbraith
Greetings, Running Steven's hotplug stress script in tip w. CLASSIC_SRCU takes 55s in my i4790 box, whereas TREE_SRCU takes over 16m. (Master with the same config does it in 39s.. but then lockdep isn't enabled in master) -Mike

Re: patch series moving compat syscalls from fs/compat.c

2017-04-23 Thread Stephen Rothwell
Hi Al, On Mon, 24 Apr 2017 01:55:30 +0100 Al Viro wrote: > > FWIW, the same kind of stuff had been done before (e.g. compat > variants of readv/writev moved to fs/read_write.c, compat aio syscalls > to fs/aio.c, etc.) with no objections from anybody. OK. I guess I was just suprised by

Re: [PATCH v6 05/15] lockdep: Implement crossrelease feature

2017-04-23 Thread Byungchul Park
On Wed, Apr 19, 2017 at 07:19:54PM +0200, Peter Zijlstra wrote: > On Tue, Mar 14, 2017 at 05:18:52PM +0900, Byungchul Park wrote: > > +/* > > + * Only access local task's data, so irq disable is only required. > > A comment describing what it does; record a hist_lock entry; would be > more

Re: [PATCH 11/11] block: don't check for BIO_MAX_PAGES in blk_bio_segment_split()

2017-04-23 Thread NeilBrown
On Fri, Apr 21 2017, Christoph Hellwig wrote: > On Thu, Apr 20, 2017 at 03:38:50PM +1000, NeilBrown wrote: >> blk_bio_segment_split() makes sure bios have no more than >> BIO_MAX_PAGES entries in the bi_io_vec. >> This was done because bio_clone_bioset() (when given a >> mempool bioset) could not

Re: [PATCH v6 05/15] lockdep: Implement crossrelease feature

2017-04-23 Thread Byungchul Park
On Wed, Apr 19, 2017 at 07:20:19PM +0200, Peter Zijlstra wrote: > On Tue, Mar 14, 2017 at 05:18:52PM +0900, Byungchul Park wrote: > > +config LOCKDEP_CROSSRELEASE > > + bool "Lock debugging: make lockdep work for crosslocks" > > + select PROVE_LOCKING > > depends PROVE_LOCKING > >

RE: [PATCH 1/2 v2] dt-bindings: qoriq-clock: Add coreclk

2017-04-23 Thread Andy Tang
Does anyone give me a clue why this patch set can't be responded after so long time? Thanks, Andy -Original Message- From: Andy Tang Sent: Monday, April 17, 2017 9:37 AM To: 'mturque...@baylibre.com' ; 'sb...@codeaurora.org' Cc: 'robh...@kernel.org' ; 'mark.rutl...@arm.com' ;

Re: [PATCH 11/11] block: don't check for BIO_MAX_PAGES in blk_bio_segment_split()

2017-04-23 Thread NeilBrown
On Fri, Apr 21 2017, Ming Lei wrote: > On Fri, Apr 21, 2017 at 7:34 PM, Christoph Hellwig wrote: >> On Thu, Apr 20, 2017 at 03:38:50PM +1000, NeilBrown wrote: >>> blk_bio_segment_split() makes sure bios have no more than >>> BIO_MAX_PAGES entries in the bi_io_vec. >>> This was done because

Re: [PATCH] Input: i8042 - add Clevo P650RS to the i8042 reset list

2017-04-23 Thread Ed Bordin
Works for me. Thanks for the fix! On 14 April 2017 at 08:51, Dmitry Torokhov wrote: > Clevo P650RS and other similar devices require i8042 to be reset in order > to detect Synaptics touchpad. > > Reported-by: Paweł Bylica > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=190301 >

Re: [PATCH v1 1/1] srcu-cbmc: Use /usr/bin/awk instead of /bin/awk

2017-04-23 Thread Lance Roy
This is fine with me. I didn't notice that /bin/awk was just a symlink on the distro I was using when I wrote the script. Thanks, Lance On Fri, 21 Apr 2017 10:50:42 -0700 "Paul E. McKenney" wrote: > On Sat, Apr 22, 2017 at 10:17:11AM -0700, priyalee.kushw...@intel.com wrote: > > From: Priyalee

Re: Build failure in -next due to 'hexagon: switch to RAW_COPY_USER'

2017-04-23 Thread Al Viro
On Sun, Apr 23, 2017 at 05:35:45PM -0700, Guenter Roeck wrote: > hexagon:defconfig fails to build in -next with the following build error. > > > In file included from include/linux/uaccess.h:13:0, > from include/linux/poll.h:11, > from

Re: [PATCH v3 1/2] tiocsti-restrict : Add owner user namespace to tty_struct

2017-04-23 Thread Matt Brown
On 04/23/2017 09:09 PM, Serge E. Hallyn wrote: Quoting Matt Brown (m...@nmatt.com): This patch adds struct user_namespace *owner_user_ns to the tty_struct. Then it is set to current_user_ns() in the alloc_tty_struct function. This is done to facilitate capability checks against the original

Re: TREE_SRCU slows hotplug by factor ~16

2017-04-23 Thread Paul E. McKenney
On Mon, Apr 24, 2017 at 04:48:09AM +0200, Mike Galbraith wrote: > Greetings, > > Running Steven's hotplug stress script in tip w. CLASSIC_SRCU takes 55s > in my i4790 box, whereas TREE_SRCU takes over 16m. (Master with the > same config does it in 39s.. but then lockdep isn't enabled in master)

linux-next: build failure after merge of the tip tree

2017-04-23 Thread Stephen Rothwell
Hi all, After merging the tip tree, today's linux-next build (arm multi_v7_defconfig) failed like this: kernel/tracepoint.c: In function 'tracepoint_remove_func': kernel/tracepoint.c:253:4: error: implicit declaration of function 'static_key_slow_dec_cpuslocked'

Re: [PATCH v1 1/1] srcu-cbmc: Use /usr/bin/awk instead of /bin/awk

2017-04-23 Thread Paul E. McKenney
On Sun, Apr 23, 2017 at 08:24:11PM -0700, Lance Roy wrote: > This is fine with me. I didn't notice that /bin/awk was just a symlink on the > distro I was using when I wrote the script. Very good! May I please have your Acked-by? Thanx,

[nfs PATCH] NFS: remove error handling for callers of rpc_new_task()

2017-04-23 Thread NeilBrown
Since commit 62b2417e84ba ("sunrpc: don't check for failure from mempool_alloc()") rpc_new_task() cannot fail, so functions which fail only when it failed also cannot fail. This means we no longer need to check for errors from: rpc_run_task() rpc_run_bc_task() __nlm_async_call()

Re: [PATCH v3 1/2] tiocsti-restrict : Add owner user namespace to tty_struct

2017-04-23 Thread Serge E. Hallyn
Quoting Matt Brown (m...@nmatt.com): > On 04/23/2017 09:09 PM, Serge E. Hallyn wrote: > >Quoting Matt Brown (m...@nmatt.com): > >>This patch adds struct user_namespace *owner_user_ns to the tty_struct. > >>Then it is set to current_user_ns() in the alloc_tty_struct function. > >> > >>This is done

Re: [PATCH v3 2/2] tiocsti-restrict : make TIOCSTI ioctl require CAP_SYS_ADMIN

2017-04-23 Thread Serge E. Hallyn
Quoting Matt Brown (m...@nmatt.com): > This introduces the tiocsti_restrict sysctl, whose default is controlled via > CONFIG_SECURITY_TIOCSTI_RESTRICT. When activated, this control restricts > all TIOCSTI ioctl calls from non CAP_SYS_ADMIN users. > > This patch was inspired from

Re: [PATCH] usb: chipidea: Fix missing resume call after suspend

2017-04-23 Thread Peter Chen
On Fri, Apr 21, 2017 at 12:10:53PM +0200, Bernhard Walle wrote: > We have a i.MX53-based hardware (quite similar to the i.MX53 QSB from > Freescale/NXP). I'm reading the /ci_hdrc.0/gadget/suspended sysfs > file to find out whether a PC is connected to the USB gadget. With old > kernel

Re: [PATCH v2 3/3] Documentation/kernel-parameters.txt: Update 'memmap=' option description

2017-04-23 Thread Dou Liyang
At 04/24/2017 10:40 AM, Baoquan He wrote: In commit: 9710f581bb4c ("x86, mm: Let "memmap=" take more entries one time") ... 'memmap=' was changed to adopt multiple, comma delimited values in a single entry, so update the related description. In the special case of only specifying size

Re: [PATCH 1/1] usb: hub: check return value

2017-04-23 Thread Viresh Kumar
On Sun, Apr 23, 2017 at 4:52 PM, Pan Bian wrote: > From: Pan Bian > > Function usb_autopm_get_interface() will return a negative errno on > failure, and function usb_autopm_put_interface() should not be called if > it fails. However, in function usb_remove_device(), the return value of >

Re: [PATCH 1/1] mt7601u: check return value of alloc_skb

2017-04-23 Thread Jakub Kicinski
On Sun, 23 Apr 2017 15:00:23 +0800, Pan Bian wrote: > Function alloc_skb() will return a NULL pointer if there is no enough > memory. However, in function mt7601u_mcu_msg_alloc(), its return value > is not validated before it is used. This patch fixes it. > > Signed-off-by: Pan Bian Acked-by:

Re: [PATCH] hwmon: tmp103: use SIMPLE_DEV_PM_OPS helper macro

2017-04-23 Thread Heiko Schocher
Hello Guenter, Rahul, Am 23.04.2017 um 17:43 schrieb Guenter Roeck: Hi Rahul, On 04/23/2017 07:10 AM, Rahul Bedarkar wrote: Replace ifdefs with SIMPLE_DEV_PM_OPS helper macro. Signed-off-by: Rahul Bedarkar Thanks a lot for your patch. While I in general prefer code that avoids #ifdef, I

Re: [PATCH] dmaengine: ioat: use setup_timer

2017-04-23 Thread Vinod Koul
On Sun, Apr 09, 2017 at 09:41:30AM +0800, Geliang Tang wrote: > Use setup_timer() instead of init_timer() to simplify the code. Applied, thanks -- ~Vinod

Re: [PATCH v7 0/7] Introduce ZONE_CMA

2017-04-23 Thread Bob Liu
On 2017/4/11 11:17, js1...@gmail.com wrote: > From: Joonsoo Kim > > Changed from v6 > o Rebase on next-20170405 > o Add a fix for lowmem mapping on ARM (last patch) > o Re-organize the cover letter > > Changes from v5 > o Rebase on next-20161013 > o Cosmetic change on patch 1 > o Optimize span

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

2017-04-23 Thread Vinod Koul
On Sat, Apr 22, 2017 at 10:41:37AM +0200, Mason wrote: > On 22/04/2017 01:43, Russell King - ARM Linux wrote: > > > Or I drop the offending patch (done) and we get the DMA subsystem fixed > > first. Given how long it's been this way, I doubt there's any hurry to > > get this change in for the

[PATCH 1/2] perf tools: Guard _GNU_SOURCE in pmu-events/jevents.c

2017-04-23 Thread Tony Jones
Guard _GNU_SOURCE (as done in pt-decoder/intel-pt-decoder.c) to prevent possible redefinition error. Signed-off-by: Tony Jones --- tools/perf/pmu-events/jevents.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/perf/pmu-events/jevents.c b/tools/perf/pmu-events/jevents.c index

[PATCH 2/2] perf tools: Add NULL end elements to 'unit_to_pmu' map

2017-04-23 Thread Tony Jones
Add NULL end elements to 'unit_to_pmu' map to prevent compiler warning on some toolchains. Signed-off-by: Tony Jones --- tools/perf/pmu-events/jevents.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/perf/pmu-events/jevents.c b/tools/perf/pmu-events/jevents.c index

Re: [PATCH v3 2/2] modules:capabilities: add a per-task modules autoload restriction

2017-04-23 Thread Rusty Russell
Djalal Harouni writes: > When value is (1), task must have CAP_SYS_MODULE to be able to trigger a > module auto-load operation, or CAP_NET_ADMIN for modules with a > 'netdev-%s' alias. Sorry, the magic 'netdev-' prefix is a crawling horror. To do this properly, you need to hand the capability

Re: [PATCH v6 05/15] lockdep: Implement crossrelease feature

2017-04-23 Thread Byungchul Park
On Wed, Apr 19, 2017 at 05:08:35PM +0200, Peter Zijlstra wrote: > On Tue, Mar 14, 2017 at 05:18:52PM +0900, Byungchul Park wrote: > > +/* > > + * Only access local task's data, so irq disable is only required. > > + */ > > +static int same_context_xhlock(struct hist_lock *xhlock) > > +{ > > +

[PATCH] led: ledtrig-transient: replace timer_list with hrtimer

2017-04-23 Thread David Lin
This patch replaces the kernel timer used by led transient trigger as an one-shot timer with an hrtimer. As Android is moving away from the obsoleted timed_output to ledtrig-transient for the vibrator HAL, ledtrig-transient needs to be able to handle the "duration" property to millisecond

Re: [PATCH] module: Unify the return value type of try_module_get

2017-04-23 Thread Jessica Yu
+++ gfree.w...@foxmail.com [19/04/17 09:47 +0800]: From: Gao Feng The prototypes of try_module_get are different with different macro. When enable module and module unload, it returns bool, but others not. Now unify their return value type as bool. Signed-off-by: Gao Feng Applied, thanks

Re: [PATCH v1 2/8] ASoC: Intel: Skylake: Use recently introduced uuid_le_cmp_p{p}()

2017-04-23 Thread Vinod Koul
On Fri, Apr 21, 2017 at 05:46:39PM +0300, Andy Shevchenko wrote: > Recently introduced helpers take pointers to uuid_{be|le} instead of > reference. Are they in linus's tree, if not it introduces dependency, so we might want to defer after merge window > > Using them makes code less ugly. How

[PATCH v2 05/29] drm/bochs: fix include notation and remove -Iinclude/drm flag

2017-04-23 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/bochs/Makefile | 1 - drivers/gpu/drm/bochs/bochs.h | 4 ++-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git

[PATCH v2 06/29] drm/bridge: fix include notation and remove -Iinclude/drm flag

2017-04-23 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. While we are here, sort the touched parts alphabetically. Signed-off-by: Masahiro Yamada Reviewed-by: Andrzej Hajda --- Changes in v2: - Sort the includes alphabetically (Andrzej Hajda)

[PATCH v2 10/29] drm/msm: fix include notation and remove -Iinclude/drm flag

2017-04-23 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. While we are here, sort the touched parts with public headers first. mdp4_kms.h must declare struct device_node to be self-contained. Signed-off-by: Masahiro Yamada --- Changes in v2: - Sort

[PATCH v2 09/29] drm/mgag200: fix include notation and remove -Iinclude/drm flag

2017-04-23 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: - Sort the touched part alphabetically drivers/gpu/drm/mgag200/Makefile | 1 - drivers/gpu/drm/mgag200/mgag200_ttm.c | 3 ++- 2 files

[PATCH v2 16/29] drm/virtio: fix include notation and remove -Iinclude/drm flag

2017-04-23 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/virtio/Makefile | 2 -- drivers/gpu/drm/virtio/virtgpu_debugfs.c | 2 +- drivers/gpu/drm/virtio/virtgpu_drv.c

[PATCH v2 14/29] drm/tilcdc: fix include notation and remove -Iinclude/drm flag

2017-04-23 Thread Masahiro Yamada
Include instead of relative path from include/drm, then remove the -Iinclude/drm compiler flag. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/tilcdc/Makefile | 1 - drivers/gpu/drm/tilcdc/tilcdc_drv.c | 3 +-- 2 files changed, 1 insertion(+), 3 deletions(-) diff

[PATCH v2 23/29] drm/r128: remove unneeded -Iinclude/drm compiler flag

2017-04-23 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/r128/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/r128/Makefile b/drivers/gpu/drm/r128/Makefile index

Re: [PATCH -mm -v3] mm, swap: Sort swap entries before free

2017-04-23 Thread Minchan Kim
On Fri, Apr 21, 2017 at 08:29:30PM +0800, Huang, Ying wrote: > "Huang, Ying" writes: > > > Minchan Kim writes: > > > >> On Wed, Apr 19, 2017 at 04:14:43PM +0800, Huang, Ying wrote: > >>> Minchan Kim writes: > >>> > >>> > Hi Huang, > >>> > > >>> > On Fri, Apr 07, 2017 at 02:49:01PM +0800,

[PATCH v2 27/29] drm/udl: remove unneeded -Iinclude/drm compiler flag

2017-04-23 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/udl/Makefile | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/gpu/drm/udl/Makefile b/drivers/gpu/drm/udl/Makefile index

[PATCH v2 18/29] drm/gma500: remove unneeded -Iinclude/drm compiler flag

2017-04-23 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/gma500/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/gma500/Makefile b/drivers/gpu/drm/gma500/Makefile

[PATCH v2 20/29] drm/i2c: remove unneeded -Iinclude/drm compiler flag

2017-04-23 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/i2c/Makefile | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/i2c/Makefile b/drivers/gpu/drm/i2c/Makefile index

[PATCH v2 19/29] drm/i810: remove unneeded -Iinclude/drm compiler flag

2017-04-23 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/i810/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/i810/Makefile b/drivers/gpu/drm/i810/Makefile index

[PATCH v2 29/29] drm/via: remove unneeded -Iinclude/drm compiler flag

2017-04-23 Thread Masahiro Yamada
With the include directives under include/drm/ fixed, this flag is no longer needed. Signed-off-by: Masahiro Yamada --- Changes in v2: None drivers/gpu/drm/via/Makefile | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/via/Makefile b/drivers/gpu/drm/via/Makefile index

<    1   2   3   4   5   6   7   >