Re: ARC-setup: Use seq_putc() in show_cpuinfo()

2016-10-17 Thread Vineet Gupta
On 10/17/2016 10:19 AM, SF Markus Elfring wrote: >>> A single character (line break) should be put into a sequence. >>> Thus use the corresponding function "seq_putc". >> Perhaps reword the changelog to say that seqc_putc is more efficient than >> seqc_printf to output a single char. >> I mean _pri

Re: [PATCH] crypto: ccm - avoid scatterlist for MAC encryption

2016-10-17 Thread Ard Biesheuvel
On 17 October 2016 at 18:08, Andy Lutomirski wrote: > On Mon, Oct 17, 2016 at 12:37 AM, Ard Biesheuvel > wrote: >> On 17 October 2016 at 08:28, Johannes Berg wrote: >>> On Sat, 2016-10-15 at 18:16 +0100, Ard Biesheuvel wrote: The CCM code goes out of its way to perform the CTR encryption of

Re: [REVIEW][PATCH] mm: Add a user_ns owner to mm_struct and fix ptrace_may_access

2016-10-17 Thread Jann Horn
On Mon, Oct 17, 2016 at 11:39:49AM -0500, Eric W. Biederman wrote: > > During exec dumpable is cleared if the file that is being executed is > not readable by the user executing the file. A bug in > ptrace_may_access allows reading the file if the executable happens to > enter into a subordinate

Re: [PATCH v3 net-next 2/2] net: deprecate eth_change_mtu, remove usage

2016-10-17 Thread Jakub Kicinski
On Mon, 17 Oct 2016 13:15:13 -0400 (EDT), David Miller wrote: > From: Jakub Kicinski > Date: Mon, 17 Oct 2016 18:00:27 +0100 > > > On Mon, 17 Oct 2016 12:49:54 -0400 (EDT), David Miller wrote: > >> From: Jakub Kicinski > >> Date: Mon, 17 Oct 2016 17:20:06 +0100 > >> > >> > Please correct me

Hello Dear

2016-10-17 Thread Rachael Dweck
Hello Dear I have been looking for an opportunity to contact you,i hope you are fine, sorry for the inconveniences but i really need to discuss an important matter with you like a good friend. My name is Rachael Dweck (female). thanks -

Re: [PATCH v3 net-next 2/2] net: deprecate eth_change_mtu, remove usage

2016-10-17 Thread David Miller
From: Jakub Kicinski Date: Mon, 17 Oct 2016 18:20:49 +0100 > Hm. I must be missing something really obvious. I just booted > net-next an hour ago and couldn't set MTU to anything larger than 1500 > on either nfp or igb. As far as I can read the code it will set the > max_mtu to 1500 in setup_e

Re: [PATCH] Input: psmouse - cleanup Focaltech code

2016-10-17 Thread Mathias Gottschlag
The changes look good to me. However, what wording do other drivers use? I've seen "TouchPad" with a capital P before. If this patch breaks scripts searching for that string anyways, that's an opportunity to unify the capitalization there. It probably doesn't matter, though... I don't see a problem

Re: [PATCH 08/10] perf, tools: Expand PMU events by prefix match

2016-10-17 Thread Jiri Olsa
On Mon, Oct 17, 2016 at 09:56:42AM -0700, Andi Kleen wrote: > > so there's a special treatment for uncore events, > > what if user says 'uncore_box/..' then? > > It should work. There's nothing special for uncore later, this > is just for convenience so that I have less to type. really.. 'uncor

Re: please, where has xconfig KConf option documentation gone with linux 4.8's Qt5 / Qt4 xconfig ?

2016-10-17 Thread Randy Dunlap
Boris or Thiago, Any comments, suggestions, or patches about this? thanks. On 10/15/16 04:09, Jason Vas Dias wrote: > Aha, thanks! I never would have known this without being told - > there is no visible indication that the symbol info pane exists > at all until one tries to drag the lower ri

Re: [PATCH 07/10] perf, tools: Collapse identically named events in perf stat

2016-10-17 Thread Jiri Olsa
On Mon, Oct 17, 2016 at 09:30:17AM -0700, Andi Kleen wrote: > > this leads to my next question: why this merging should be default? > > It's the right default for uncore, and it doesn't do anything for > non uncore because these usually don't have duplicated event aliases > over different PMUs. I

Re: [PATCH v2] DAX: enable iostat for read/write

2016-10-17 Thread Ross Zwisler
On Mon, Oct 17, 2016 at 11:18:58AM -0600, Toshi Kani wrote: > DAX IO path does not support iostat, but its metadata IO path does. > Therefore, iostat shows metadata IO statistics only, which has been > confusing to users. > > Add iostat support to the DAX read/write path. > > Note, iostat still d

Re: [PATCH resend 4.9] hw_random: Don't use a stack buffer in add_early_randomness()

2016-10-17 Thread Andy Lutomirski
On Mon, Oct 17, 2016 at 10:17 AM, Stephan Mueller wrote: > Am Montag, 17. Oktober 2016, 10:06:27 CEST schrieb Andy Lutomirski: > > Hi Andy, > >> diff --git a/drivers/char/hw_random/core.c b/drivers/char/hw_random/core.c >> index 9203f2d130c0..340f96e44642 100644 >> --- a/drivers/char/hw_random/cor

Re: [PATCH v2 1/2] watchdog: Introduce arch_watchdog_nmi_enable and arch_watchdog_nmi_disable

2016-10-17 Thread Don Zickus
On Thu, Oct 13, 2016 at 01:38:01PM -0700, Babu Moger wrote: > Currently we do not have a way to enable/disable arch specific > watchdog handlers if it was implemented by any of the architectures. > > This patch introduces new functions arch_watchdog_nmi_enable and > arch_watchdog_nmi_disable which

Re: [PATCH] sd: assign appropriate log level

2016-10-17 Thread Daniel Walker
On 10/17/2016 10:19 AM, James Bottomley wrote: On Mon, 2016-10-17 at 09:51 -0700, David Singleton wrote: From: Shikhar Dogra Reduce chatter on console for usb hotplug. KERN_ERR is too high severity for these messages, moving them to KERN_WARNING It's an error because we have several USB to ID

[PATCH] xfs: remove redundant assignment of ifp

2016-10-17 Thread Colin King
From: Colin Ian King Remove redundant ifp = ifp statement, it does nothing. Found with static analysis by CoverityScan. Signed-off-by: Colin Ian King --- fs/xfs/libxfs/xfs_bmap.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/xfs/libxfs/xfs_bmap.c b/fs/xfs/libxfs/xfs_bm

Re: [PATCH] xfs: remove redundant assignment of ifp

2016-10-17 Thread Darrick J. Wong
On Mon, Oct 17, 2016 at 06:32:40PM +0100, Colin King wrote: > From: Colin Ian King > > Remove redundant ifp = ifp statement, it does nothing. Found with > static analysis by CoverityScan. > > Signed-off-by: Colin Ian King Reviewed-by: Darrick J. Wong > --- > fs/xfs/libxfs/xfs_bmap.c | 2 +-

Re: [REVIEW][PATCH] mm: Add a user_ns owner to mm_struct and fix ptrace_may_access

2016-10-17 Thread Eric W. Biederman
Jann Horn writes: > On Mon, Oct 17, 2016 at 11:39:49AM -0500, Eric W. Biederman wrote: >> >> During exec dumpable is cleared if the file that is being executed is >> not readable by the user executing the file. A bug in >> ptrace_may_access allows reading the file if the executable happens to >

Re: [PATCH v2] console: Don't prefer first registered if DT specifies stdout-path

2016-10-17 Thread Andreas Schwab
On Okt 17 2016, Paul Burton wrote: > Could you share the device tree from your system? This is the contents of chosen/linux,stdout-path on the systems I have: chosen/linux,stdout-path "/pci@f000/ATY,SnowyParent@10/ATY,Snowy_A@0" chosen/linux,stdout-path

Re: ARC-setup: Use seq_putc() in show_cpuinfo()

2016-10-17 Thread SF Markus Elfring
>>> Perhaps reword the changelog to say that seqc_putc is more efficient than >>> seqc_printf to output a single char. >>> I mean _printf is not wrong but not as efficient ? >> I came along source files for a few other software modules with similar >> change possibilities. >> Unfortunately, the cor

Re: [PATCH v2] mm: vmalloc: Replace purge_lock spinlock with atomic refcount

2016-10-17 Thread Joel Fernandes
Hi Nick, On Sun, Oct 16, 2016 at 9:00 PM, Nicholas Piggin wrote: > On Sat, 15 Oct 2016 03:42:42 -0700 > Joel Fernandes wrote: > >> The purge_lock spinlock causes high latencies with non RT kernel. This has >> been >> reported multiple times on lkml [1] [2] and affects applications like audio. >

Re: [PATCH] Input: psmouse - cleanup Focaltech code

2016-10-17 Thread Dmitry Tunin
Other drivers, like Synaptics use the same as I suggested. They even shorten in psmouse-base.c psmouse_protocol.name to SynPS/2. You used FocaltechPS/2. I think it is OK. I do not think it is a big problem of changing the naming. It happens a lot. I still have the hardware and I am still thankful

Re: [PATCH] thread_info: include for THREAD_INFO_IN_TASK (WAS: [PATCH 2/3] sched/preempt: include asm/current.h)

2016-10-17 Thread Mark Rutland
On Mon, Oct 17, 2016 at 03:48:13PM +0100, Mark Rutland wrote: > On Fri, Oct 14, 2016 at 11:42:25AM +0100, Mark Rutland wrote: > > When CONFIG_THREAD_INFO_IN_TASK is selected, the current_thread_info() > > macro relies on current having been defined prior to its use. However, > > not all users of cu

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Jiri Olsa
On Mon, Oct 17, 2016 at 09:27:54AM -0700, Andi Kleen wrote: > On Mon, Oct 17, 2016 at 01:44:43PM +0200, Jiri Olsa wrote: > > On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > > > From: Andi Kleen > > > > > > Add support for parsing the DividedBy header in the JSON event lists and > >

Re: [PATCH v2] DAX: enable iostat for read/write

2016-10-17 Thread Kani, Toshimitsu
On Mon, 2016-10-17 at 11:29 -0600, Ross Zwisler wrote: > On Mon, Oct 17, 2016 at 11:18:58AM -0600, Toshi Kani wrote: > > > > DAX IO path does not support iostat, but its metadata IO path does. > > Therefore, iostat shows metadata IO statistics only, which has been > > confusing to users. > > > >

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Jiri Olsa
On Mon, Oct 17, 2016 at 07:43:12PM +0200, Jiri Olsa wrote: > On Mon, Oct 17, 2016 at 09:27:54AM -0700, Andi Kleen wrote: > > On Mon, Oct 17, 2016 at 01:44:43PM +0200, Jiri Olsa wrote: > > > On Thu, Oct 13, 2016 at 02:15:31PM -0700, Andi Kleen wrote: > > > > From: Andi Kleen > > > > > > > > Add su

Re: [patch] drivers/zram: Don't disable preemption in zcomp_stream_get/put()

2016-10-17 Thread Mike Galbraith
On Mon, 2016-10-17 at 19:18 +0200, Mike Galbraith wrote: > BTW, 4.8 either needs the btrfs deadlock fix (0ccd05285e7f) or the LTP > testcase has to be hacked to not test btrfs. It also fails the first > time it's run in 4.8/4.8-rt, doesn't do that in master/tip-rt. Belay that, the first run fail

Re: [PATCH 1/2] i2c: bcm-iproc: constify i2c_adapter_quirks structures

2016-10-17 Thread Ray Jui
Hi Julia, On 10/15/2016 10:32 AM, Julia Lawall wrote: Check for i2c_adapter_quirks structures that are only stored in the quirks field of an i2c_adapter structure. This field is declared const, so i2c_adapter_quirks structures that have this property can be declared as const also. The semantic

Re: Regression: usb serial gadget on sama5d3 broken

2016-10-17 Thread Peter Rosin
On 2016-10-17 16:54, Nicolas Ferre wrote: > Le 17/10/2016 à 14:53, Peter Rosin a écrit : >> Hi! >> >> I'm suffering from a regression while using the usb gadget port on the >> sama5d3 to get terminal access to the device in question >> (CONFIG_USB_G_SERIAL). >> >> I get this message when I try to

Re: [PATCH] DAX: enable iostat for read/write

2016-10-17 Thread Kani, Toshimitsu
On Sat, 2016-10-15 at 18:54 +1100, Dave Chinner wrote: > On Fri, Oct 14, 2016 at 11:25:13AM -0600, Toshi Kani wrote:  : > > +static void dax_iostat_start(struct gendisk *disk, struct iov_iter > > *iter, > > +  unsigned long *start) > > +{ > > + int rw = iov_iter_rw(iter); >

Re: [PATCH v8 1/2] ARM: dts: Add TOPEET itop core board SCP package version

2016-10-17 Thread ayaka
On 10/18/2016 12:27 AM, Krzysztof Kozlowski wrote: On Mon, Sep 19, 2016 at 11:48:22PM +0800, Randy Li wrote: The TOPEET itop is a samsung exnynos 4412 core board, which have two package versions. This patch add the support for SCP version. Currently supported are USB3503A HSIC, USB OTG, eMMC,

Re: [PATCH] lpfc: use %zd format string for size_t

2016-10-17 Thread Martin K. Petersen
> "Arnd" == Arnd Bergmann writes: Arnd> A recent bugfix introduced a harmless warning in the lpfc driver: Arnd> drivers/scsi/lpfc/lpfc_init.c: In function 'lpfc_write_firmware': Arnd> drivers/scsi/lpfc/lpfc_logmsg.h:56:45: error: format '%ld' expects Arnd> argument of type 'long int', but arg

RE: [ANNOUNCE] bridge-utils 1.6 release

2016-10-17 Thread Rosen, Rami
Hi, Alexander, >This link seems to be broken. Seems that it should be: https://www.kernel.org/pub/linux/utils/net/bridge-utils/bridge-utils-1.6.tar.gz Instead of: > > http://www.kernel.org/pub/linux/utils/net/bridge-utils/bridge-utils.1.6.tar.gz Regards, Rami Rosen Intel Corporation

Re: [PATCH] ARM: dts: sunxi: Add cpu-supply for Olimex A20 EVB

2016-10-17 Thread Maxime Ripard
Hi, On Sat, Oct 15, 2016 at 05:23:40PM +0200, Emmanuel Vadot wrote: > sun7i-a20-olimex-som-evb.dts doesn't contain cpu-supply needed for > voltage-scaling with cpufreq-dt so define it. > The default voltages are defined in sun7i-a20.dtsi. > > Signed-off-by: Emmanuel Vadot Applied, thanks! Maxim

Re: [PATCH 07/10] perf, tools: Collapse identically named events in perf stat

2016-10-17 Thread Andi Kleen
On Mon, Oct 17, 2016 at 07:28:36PM +0200, Jiri Olsa wrote: > On Mon, Oct 17, 2016 at 09:30:17AM -0700, Andi Kleen wrote: > > > this leads to my next question: why this merging should be default? > > > > It's the right default for uncore, and it doesn't do anything for > > non uncore because these

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-17 Thread Tejun Heo
Hello, On Sat, Oct 15, 2016 at 08:48:01PM +1100, Michael Ellerman wrote: > > Hmm... it doesn't reproduce it here and can't see how the commit would > > affect this given that it doesn't really change when the kworker > > kthreads are being created. > > Try turning on CONFIG_DEBUG_PER_CPU_MAPS=y ?

Re: [PATCH] mac80211_hwsim: suggest nl80211 instead of wext driver in documentation

2016-10-17 Thread Linus Lüssing
On Mon, Oct 17, 2016 at 11:39:04AM +0200, Johannes Berg wrote: > On Mon, 2016-10-17 at 00:39 +0200, Linus Lüssing wrote: > > For mac80211_hwsim interfaces, suggest to use wpa_supplicant with the > > more modern, netlink based driver instead of wext. > > Makes sense, applied. > > > Actually, I was

Re: Oops on Power8 (was Re: [PATCH v2 1/7] workqueue: make workqueue available early during boot)

2016-10-17 Thread Tejun Heo
Hello, Michael. On Mon, Oct 17, 2016 at 11:24:34PM +1100, Michael Ellerman wrote: > The bad case (where we hit the BUG_ON I added above) is where we are > creating a wq for node 1. > > In wq_calc_node_cpumask() we do: > > cpumask_and(cpumask, attrs->cpumask, wq_numa_possible_cpumask[node])

Re: [GIT PULL] final round of SCSI updates for the 4.8+ merge window

2016-10-17 Thread Martin K. Petersen
> "Finn" == Finn Thain writes: >> Ondrej Zary (3): scsi: g_NCR5380: Stop using scsi_module.c Finn> I believe that patch introduces an .init.text section mismatch, Finn> that I failed to notice when I reviewed it. So you may want Arnd Finn> Bergmann's fix which is commit 8f087c089dae5 ("scsi:

Re: [PATCH v4 08/18] x86/intel_rdt: Pick up L3/L2 RDT parameters from CPUID

2016-10-17 Thread Fenghua Yu
On Mon, Oct 17, 2016 at 07:02:19PM +0200, Thomas Gleixner wrote: > On Mon, 17 Oct 2016, Fenghua Yu wrote: > > On Mon, Oct 17, 2016 at 03:45:32PM +0200, Thomas Gleixner wrote: > > > I wonder whether this is the proper abstraction level. We might as well do > > > the following: > > > > > > rdtresour

[PATCH v1] media: saa7146: Fix for "[BUG] process stuck when closing saa7146 [dvb_ttpci]"

2016-10-17 Thread Andrey Utkin
Release queued DMA buffers when ending streaming, so that videobuf_waiton() doesn't block forever. As reported, this fixes avoids occasional lockup of process reading from video device, which manifests in such log: INFO: task ffmpeg:9864 blocked for more than 120 seconds. Tainted: P

Re: [PATCH 09/10] perf, tools: Support DividedBy header in JSON event list

2016-10-17 Thread Andi Kleen
> > so.. > > - you put 'DividedBy' into JSON event's defition any further > explanation how or why the format we use for event defs will > be used now used to describe ratios > > - then you force perf stat to merge together all 'same' uncore events > to get just one number.. The ratios do

[PATCH v6 1/3] block: Add iocontext priority to request

2016-10-17 Thread Adam Manzanares
Patch adds an association between iocontext ioprio and the ioprio of a request. This is done to enable request based drivers the ability to act on priority information stored in the request. An example being ATA devices that support command priorities. If the ATA driver discovers that the device su

Re: [bug/regression] libhugetlbfs testsuite failures and OOMs eventually kill my system

2016-10-17 Thread Aneesh Kumar K.V
Jan Stancek writes: > Hi Mike, > > Revert of 67961f9db8c4 helps, I let whole suite run for 100 iterations, > there were no issues. > > I cut down reproducer and removed last mmap/write/munmap as that is enough > to reproduce the problem. Then I started introducing some traces into kernel > and n

[PATCH v6 0/3] Enabling ATA Command Priorities

2016-10-17 Thread Adam Manzanares
This patch builds ATA commands with high priority if the iocontext of a process is set to real time. The goal of the patch is to improve tail latencies of workloads that use higher queue depths. This requires setting the iocontext ioprio on the request when it is initialized This patch has been

[PATCH v6 3/3] ata: ATA Command Priority Disabled By Default

2016-10-17 Thread Adam Manzanares
Add a sysfs entry to turn on priority information being passed to a ATA device. By default this feature is turned off. This patch depends on ata: Enabling ATA Command Priorities Signed-off-by: Adam Manzanares --- drivers/ata/libahci.c | 1 + drivers/ata/libata-core.c | 2 +- drivers/ata/l

Re: [PATCH 3.12 00/84] 3.12.65-stable review

2016-10-17 Thread Guenter Roeck
On Mon, Oct 17, 2016 at 09:51:48AM +0200, Jiri Slaby wrote: > This is the start of the stable review cycle for the 3.12.65 release. > There are 84 patches in this series, all will be posted as a response > to this one. If anyone has any issues with these being applied, please > let me know. > > R

[PATCH v6 2/3] ata: Enabling ATA Command Priorities

2016-10-17 Thread Adam Manzanares
This patch checks to see if an ATA device supports NCQ command priorities. If so and the user has specified an iocontext that indicates IO_PRIO_CLASS_RT then we build a tf with a high priority command. This is done to improve the tail latency of commands that are high priority by passing priority

Re: [PATCH] xfs: Fix uninitialized variable in xfs_reflink_reserve_cow_range()

2016-10-17 Thread Darrick J. Wong
On Mon, Oct 17, 2016 at 12:16:44PM +0200, Geert Uytterhoeven wrote: > with gcc 4.1.2: > > fs/xfs/xfs_reflink.c: In function ‘xfs_reflink_reserve_cow_range’: > fs/xfs/xfs_reflink.c:327: warning: ‘error’ may be used uninitialized in > this function > > Indeed, if "count" is zero, the funct

Re: [PATCH resend 4.9] hw_random: Don't use a stack buffer in add_early_randomness()

2016-10-17 Thread Stephan Mueller
Am Montag, 17. Oktober 2016, 10:30:13 CEST schrieb Andy Lutomirski: Hi Andy, > > Sure, but shouldn't that be a separate patch covering the whole hw_crypto > core? I think that you are right -- there are many more cases where a memset(0) is warranted. Do you want to make this change or should I

[PATCH 2/4] power: reset: at91-poweroff: Fix module autoload

2016-10-17 Thread Javier Martinez Canillas
If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding module. Export the module alias information using the MODULE_DEVICE_TABLE() macro. Before this patch: $ modinfo drive

[PATCH 1/4] power: reset: at91-reset: Fix module autoload

2016-10-17 Thread Javier Martinez Canillas
If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding module. Export the module alias information using the MODULE_DEVICE_TABLE() macro. Before this patch: $ modinfo drive

[PATCH 4/4] power: reset: zx-reboot: Fix module autoload

2016-10-17 Thread Javier Martinez Canillas
If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding module. Export the module alias information using the MODULE_DEVICE_TABLE() macro. Before this patch: $ modinfo drive

[PATCH 0/4] power: reset: Fix module autoload for some platform drivers

2016-10-17 Thread Javier Martinez Canillas
Hello Sebastian, I noticed that module autoload won't be working in a some of the reset platform drivers This patch series contains the fixes for these. Best regards, Javier Javier Martinez Canillas (4): power: reset: at91-reset: Fix module autoload power: reset: at91-poweroff: Fix module a

[PATCH 3/4] power: reset: syscon-reboot-mode: Fix module autoload

2016-10-17 Thread Javier Martinez Canillas
If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding module. Export the module alias information using the MODULE_DEVICE_TABLE() macro. Before this patch: $ modinfo drive

Build failure with v4.9-rc1 and GCC trunk -- compiler weirdness

2016-10-17 Thread Will Deacon
Hi all, I'm seeing an arm64 build failure with -rc1 and GCC trunk, although I believe that the new compiler behaviour at the heart of the problem has the potential to affect other architectures and other pieces of kernel code relying on dead-code elimination to remove deliberately undefined functi

Re: [PATCH] aio: fix a use after free (and fix freeze protection of aio writes)

2016-10-17 Thread Jeff Moyer
Hi, Christoph, Christoph Hellwig writes: > diff --git a/fs/aio.c b/fs/aio.c > index 1157e13..bf315cd 100644 > --- a/fs/aio.c > +++ b/fs/aio.c > @@ -1078,6 +1078,17 @@ static void aio_complete(struct kiocb *kiocb, long > res, long res2) > unsigned tail, pos, head; > unsigned long f

Re: [PATCH -v4 5/8] locking/mutex: Add lock handoff to avoid starvation

2016-10-17 Thread Waiman Long
On 10/07/2016 10:52 AM, Peter Zijlstra wrote: /* * Actual trylock that will work on any unlocked state. + * + * When setting the owner field, we must preserve the low flag bits. + * + * Be careful with @handoff, only set that in a wait-loop (where you set + * HANDOFF) to avoid recursive lock

[PATCH 07/14] staging: fsl-mc: dprc: shorten DPRC interrupt name

2016-10-17 Thread Stuart Yoder
improve readibility of the DPRC interrupt name in sysfs by shortening this and just using the device name. There is only one interrupt DPRC used, so no further differentiation is needed. Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/bus/dprc-driver.c | 2 +- 1 file changed, 1 insertion

[PATCH 12/14] staging: fsl-mc: improve comment wording

2016-10-17 Thread Stuart Yoder
improve and clarify miscellaneous comments -when referring to the bus call it "fsl-mc" -reduce the verbosity of some comments for clarity -improve wording of the slightly confusing term "MC object device" Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/bus/fsl-mc-allocator.c | 68 ++

Re: [PATCH] [media] solo6x10: avoid delayed register write

2016-10-17 Thread Andrey Utkin
On Thu, Sep 22, 2016 at 03:03:31AM +0300, Andrey Utkin wrote: > This fixes a lockup at device probing which happens on some solo6010 > hardware samples. This is a regression introduced by commit e1ceb25a1569 > ("[media] SOLO6x10: remove unneeded register locking and barriers") > > The observed loc

Re: [PATCH] [media] tw5864: crop picture width to 704

2016-10-17 Thread Andrey Utkin
On Thu, Sep 22, 2016 at 03:04:20AM +0300, Andrey Utkin wrote: > Previously, width of 720 was used, but it gives 16-pixel wide black bar > at right side of encoded picture. > > Signed-off-by: Andrey Utkin > --- > drivers/media/pci/tw5864/tw5864-reg.h | 8 > drivers/media/pci/tw5864/tw

Re: [PATCH] DAX: enable iostat for read/write

2016-10-17 Thread Dan Williams
On Mon, Oct 17, 2016 at 10:40 AM, Kani, Toshimitsu wrote: > On Sat, 2016-10-15 at 18:54 +1100, Dave Chinner wrote: >> On Fri, Oct 14, 2016 at 11:25:13AM -0600, Toshi Kani wrote: > : >> > +static void dax_iostat_start(struct gendisk *disk, struct iov_iter >> > *iter, >> > + unsigned long *star

Re: [PATCH] aio: fix a use after free (and fix freeze protection of aio writes)

2016-10-17 Thread Christoph Hellwig
On Mon, Oct 17, 2016 at 02:19:47PM -0400, Jeff Moyer wrote: > This ends up being a call to __sb_end_write: > > void __sb_end_write(struct super_block *sb, int level) > { > percpu_up_read(sb->s_writers.rw_sem + level-1); > } > > Nothing guarantees that submission and completion happen on t

Re: [PATCH] arm64: mm: Fix memmap to be initialized for the entire section

2016-10-17 Thread Robert Richter
Mark, Will, any opinion here? Thanks, -Robert On 06.10.16 18:11:14, Robert Richter wrote: > Ard, > > thank you for your answer and you explanation. > > On 06.10.16 11:00:33, Ard Biesheuvel wrote: > > On 6 October 2016 at 10:52, Robert Richter wrote: > > > There is a memory setup problem on Th

Re: [PATCH 06/10] gpio: sam: Document bindings of SAM FPGA GPIO block

2016-10-17 Thread Pantelis Antoniou
Hi Rob, > On Oct 10, 2016, at 23:03 , Rob Herring wrote: > > On Fri, Oct 07, 2016 at 06:18:34PM +0300, Pantelis Antoniou wrote: >> From: Georgi Vlaev >> >> Add device tree bindings document for the GPIO driver of >> Juniper's SAM FPGA. >> >> Signed-off-by: Georgi Vlaev >> [Ported from Junipe

[PATCH 06/14] staging: fsl-mc: irq: shorten name of interrupt name

2016-10-17 Thread Stuart Yoder
For /proc/interrupts readability, platform bus MSIs are named "ITS-pMSI" in the GIC ITS implementation for that bus. Follow a similar naming convention and call fsl-mc bus MSIs "ITS-fMSI". Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/bus/irq-gic-v3-its-fsl-mc-msi.c | 2 +- 1 file chan

Re: [PATCH 08/10] mtd: flash-sam: Bindings for Juniper's SAM FPGA flash

2016-10-17 Thread Pantelis Antoniou
Hi Rob, > On Oct 10, 2016, at 23:07 , Rob Herring wrote: > > gOn Fri, Oct 07, 2016 at 06:18:36PM +0300, Pantelis Antoniou wrote: >> From: Georgi Vlaev >> >> Add binding document for Junipers Flash IP block present >> in the SAM FPGA on PTX series of routers. >> >> Signed-off-by: Georgi Vlaev

[PATCH 05/14] staging: fsl-mc: update MAINTAINERS

2016-10-17 Thread Stuart Yoder
-German has moved on to other things and wished to be removed as a maintainer -cleanup the driver description to use the proper name of the driver (i.e. the fsl-mc bus driver) and remove incorrect references to Freescale Signed-off-by: Stuart Yoder Acked-by: J. German Rivera --- MAINTAINERS

Re: [PATCH v3] time: alarmtimer: Add the trcepoints for alarmtimer

2016-10-17 Thread Steven Rostedt
On Tue, 11 Oct 2016 18:48:16 +0800 Baolin Wang wrote: > For system debugging, we sometimes want to know who sets one > alarm timer, the time of the timer, when the timer started and > fired and so on. Thus adding tracepoints can help us trace the > alarmtimer information. > > For example, when w

[RFC] Vendor events file/dir names

2016-10-17 Thread Arnaldo Carvalho de Melo
Hi, Ingo pointed out to me that in the kernel sources we do not use file/dir names with uppercase chars (look, for instance, at arch/), so I mostly scripted a conversion to lowercase and what I got is at: git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git perf/vendor_events

[PATCH 10/14] staging: fsl-mc: cleanup comment style

2016-10-17 Thread Stuart Yoder
Be consistent and use kernel-preferred multi-line comment style everywhere. Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/bus/dpbp-cmd.h | 3 ++- drivers/staging/fsl-mc/bus/dpbp.c | 3 ++- drivers/staging/fsl-mc/bus/dpcon-cmd.h | 3 ++- drivers/staging/fsl-mc/bus/dpmcp-cmd.h

[PATCH 13/14] staging: fsl-mc: update copyright dates

2016-10-17 Thread Stuart Yoder
Many source files have evolved without copyright date updates. Update the dates to reflect work through 2016. Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/bus/dpcon-cmd.h | 2 +- drivers/staging/fsl-mc/bus/dpmcp.h | 2 +- drivers/staging/fsl-mc/bus/

[PATCH 03/14] staging: fsl-mc: clean up Kconfig description

2016-10-17 Thread Stuart Yoder
Except for copyrights we are avoiding all references to Freescale, which no longer exists as a brand. Cleanup Freescale references and simplify the Kconfig description of the fsl-mc bus. Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/bus/Kconfig | 22 +++--- 1 file chang

[PATCH 02/14] staging: fsl-mc: add missing includes to fsl-mc-private.h

2016-10-17 Thread Stuart Yoder
fsl-mc-private.h references several definitions that it does not explicitly have includes for. Up until now we've gotten lucky due to include ordering that things compile. Add the missing includes. Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/bus/fsl-mc-private.h | 3 +++ 1 file chang

[PATCH 00/14] staging: fsl-mc: cleanup and uprev to MC v10.x

2016-10-17 Thread Stuart Yoder
This patch does miscellaneous minor cleanup and uprevs the supported MC firmware version to 10.x. I believe with this series the TODO items on our list are complete, with the exception of "Add at least one device driver for a DPAA2 object", which I expect to send a patch for very soon. I plan to

[PATCH] x86/vmware: Skip timer_irq_works() check on VMware

2016-10-17 Thread Renat Valiullin
The timer_irq_works() boot check may sometimes fail in a VM, when the Host is overcommitted or when the Guest is running nested. Since the intended check is unnecessary on VMware's virtual hardware, this change by-passes it. Signed-off-by: Renat Valiullin Acked-by: Alok N Kataria --- arch/x86/k

[PATCH 14/14] staging: fsl-mc: uprev binary interface to match MC v10.x

2016-10-17 Thread Stuart Yoder
From: Ioana Ciornei DPAA2 will not support MC firmware versions prior to MC v10.x. Update the MC interface code and drivers to reflect this. -update the object .h files and code that builds commands to include the new command version in the command header -object versions are no longer

[PATCH 09/14] staging: fsl-mc: misc whitespace cleanup

2016-10-17 Thread Stuart Yoder
From: Ioana Ciornei Make whitespace consistent with other fsl-mc source files. -delete extraneous tabs -align start of function arguments properly -for structs and function definitions, separate type and variable name by a single space -remove multiple blank lines in comments Signed-o

Re: [PATCH 2/4] mfd: ptxpmb-ext-cpld: Add documentation for PTXPMB extended CPLD

2016-10-17 Thread Pantelis Antoniou
Hi Rob, > On Oct 10, 2016, at 23:10 , Rob Herring wrote: > > On Fri, Oct 07, 2016 at 06:19:32PM +0300, Pantelis Antoniou wrote: >> From: Georgi Vlaev >> >> Add DT bindings document for the PTXPMB extended CPLD device. >> >> Signed-off-by: Georgi Vlaev >> [Ported from Juniper kernel] >> Signe

[PATCH 04/14] staging: fsl-mc: update Kconfig dependency

2016-10-17 Thread Stuart Yoder
The Kconfig dependency previously included ARM64, which is not strictly correct. Change it to ARCH_LAYERSCAPE which is the SoC platform that includes the DPAA2 hardware. Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/bus/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) dif

[PATCH 01/14] staging: fsl-mc: cleanup: move dpbp and dpcon cmd headers

2016-10-17 Thread Stuart Yoder
The dpbp and dpcon cmd header files should not be in the public include directory but should be private to the bus driver. Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/{include => bus}/dpbp-cmd.h | 0 drivers/staging/fsl-mc/bus/dpbp.c | 3 ++- drivers/staging/fsl-mc/

[PATCH 08/14] staging: fsl-mc: cleanup: convert uintX_t types to uX

2016-10-17 Thread Stuart Yoder
From: Ioana Ciornei Replace all uses of uintX_t types with uX types in order to comply with kernel coding style and resolve checkpatch warnings. Signed-off-by: Ioana Ciornei Signed-off-by: Stuart Yoder --- drivers/staging/fsl-mc/bus/dpmcp.h | 84 +++--- 1 file

[PATCH 11/14] staging: fsl-mc: message cleanup

2016-10-17 Thread Stuart Yoder
Cleanup and clarify messages -use "fsl-mc" prefix for all messages that don't come from dev_* functions -remove unnecessary verbosity to make things clearer -use explicit __func__ arguments to clarify some error messages -improve misc wording for clarity Signed-off-by: Stuart Yoder ---

Re: [PATCH -v4 5/8] locking/mutex: Add lock handoff to avoid starvation

2016-10-17 Thread Waiman Long
On 10/17/2016 02:45 PM, Waiman Long wrote: On 10/07/2016 10:52 AM, Peter Zijlstra wrote: /* * Actual trylock that will work on any unlocked state. + * + * When setting the owner field, we must preserve the low flag bits. + * + * Be careful with @handoff, only set that in a wait-loop (where

Re: [PATCH 04/10] i2c: i2c-mux-i2cs: Add device tree bindings

2016-10-17 Thread Pantelis Antoniou
Hi Rob, > On Oct 10, 2016, at 18:48 , Peter Rosin wrote: > > On 2016-10-07 17:21, Pantelis Antoniou wrote: >> From: Georgi Vlaev >> >> Add binding document for the i2c mux driver of Juniper's I2CS FPGA. >> >> Signed-off-by: Georgi Vlaev >> [Ported from Juniper kernel] >> Signed-off-by: Pante

Re: Regression: usb serial gadget on sama5d3 broken

2016-10-17 Thread Alexandre Belloni
On 17/10/2016 at 19:50:48 +0200, Peter Rosin wrote : > On 2016-10-17 16:54, Nicolas Ferre wrote: > > Le 17/10/2016 à 14:53, Peter Rosin a écrit : > >> Hi! > >> > >> I'm suffering from a regression while using the usb gadget port on the > >> sama5d3 to get terminal access to the device in question

Re: [PATCH 4/4] gpio: ptxpmb-ext-cpld: Document bindings of PTXPMB extended CPLD

2016-10-17 Thread Pantelis Antoniou
Hi Rob, > On Oct 10, 2016, at 23:19 , Rob Herring wrote: > > On Fri, Oct 07, 2016 at 06:19:34PM +0300, Pantelis Antoniou wrote: >> From: Georgi Vlaev >> >> Add device tree bindings document for the GPIO driver of >> Juniper's PTXPMB extended CPLD. >> >> Signed-off-by: Georgi Vlaev >> [Ported

Re: [PATCH 02/10] mfd: dt-bindings: Add bindings for the Juniper I2CS MFD

2016-10-17 Thread Pantelis Antoniou
Hi Rob, > On Oct 10, 2016, at 23:23 , Rob Herring wrote: > > On Fri, Oct 07, 2016 at 06:21:01PM +0300, Pantelis Antoniou wrote: >> From: Georgi Vlaev >> >> Add device tree bindings for the Juniper I2CS MFD driver. >> >> Signed-off-by: Georgi Vlaev >> [Ported from Juniper kernel] >> Signed-of

Re: [PATCH] Add missing space at end of pr_fmt().

2016-10-17 Thread Frank Rowand
On 10/17/16 08:10, frowand.l...@gmail.com wrote: > From: Frank Rowand > > Make pr_fmt() in fdt.c consistent with all other files in drivers/of/ > > Signed-off-by: Frank Rowand Rob, Please ignore due to bad subject (as noted by Joe). I will resend with proper subject. -Frank

Re: [PATCH] make of_device_make_bus_id() static

2016-10-17 Thread Frank Rowand
On 10/17/16 08:50, frowand.l...@gmail.com wrote: > From: Frank Rowand > > of_device_make_bus_id() was changed to non-static by commit c66012253800 > ("of/device: Make of_device_make_bus_id() usable by other code") more than > 6 years ago, but there are no users of it outside of platform.c. Make

[PATCH 1/2] power: supply: max8997_charger: Fix module autoload

2016-10-17 Thread Javier Martinez Canillas
If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding module. Export the module alias information using the MODULE_DEVICE_TABLE() macro. Before this patch: $ modinfo drive

Re: [PATCH 10/10] hwmon: i2cs-fan: Add hwmon dts binding documentation

2016-10-17 Thread Pantelis Antoniou
> On Oct 10, 2016, at 23:29 , Rob Herring wrote: > > On Fri, Oct 07, 2016 at 06:21:09PM +0300, Pantelis Antoniou wrote: >> From: Georgi Vlaev >> >> Adds the I2CS Fan Tray hwmon device tree node documentation. >> >> Signed-off-by: Georgi Vlaev >> [Ported from Juniper kernel] >> Signed-off-by:

[PATCH 0/2] power: supply: Fix module autoload for some platform drivers

2016-10-17 Thread Javier Martinez Canillas
Hello Sebastian, I noticed that module autoload won't be working for some of the power supply platform drivers. This patch series contains fixes for these. Best regards, Javier Javier Martinez Canillas (2): power: supply: max8997_charger: Fix module autoload power: supply: axp288_fuel_gauge

[PATCH 2/2] power: supply: axp288_fuel_gauge: Fix module autoload

2016-10-17 Thread Javier Martinez Canillas
If the driver is built as a module, autoload won't work because the module alias information is not filled. So user-space can't match the registered device with the corresponding module. Export the module alias information using the MODULE_DEVICE_TABLE() macro. Before this patch: $ modinfo drive

Re: [PATCH v2 0/2] spi, i2c: mark device nodes only in case of successful instantiation

2016-10-17 Thread Pantelis Antoniou
Hi Ralf, > On Oct 17, 2016, at 16:59 , Ralf Ramsauer wrote: > > Hi, > > this one fixes initialisation of I2C/SPI nodes. Upon failure during > intialisation, nodes were erroneously populated and never unmarked. > > This lead to the problem that re-loaded drivers will never probe those devices

Re: [PATCH v2 1/2] spi: mark device nodes only in case of successful instantiation

2016-10-17 Thread Pantelis Antoniou
Hi Ralf, > On Oct 17, 2016, at 16:59 , Ralf Ramsauer wrote: > > Instantiated SPI device nodes are marked with OF_POPULATE. This was > introduced in bd6c164. On unloading, loaded device nodes will of course > be unmarked. The problem are nodes that fail during initialisation: If a > node fails, i

Re: [PATCH v2 2/2] i2c: mark device nodes only in case of successful instantiation

2016-10-17 Thread Pantelis Antoniou
Hi Ralf, > On Oct 17, 2016, at 16:59 , Ralf Ramsauer wrote: > > Instantiated I2C device nodes are marked with OF_POPULATE. This was > introduced in 4f001fd. On unloading, loaded device nodes will of course > be unmarked. The problem are nodes that fail during initialisation: If a > node fails, i

Re: [PATCH] pcie: aer: aerdrv: PCIe AER workaround and handling for ASR1K platforms.

2016-10-17 Thread Bjorn Helgaas
[+cc Po] Hi Steve & David, On Mon, Oct 17, 2016 at 09:51:06AM -0700, David Singleton wrote: > From: Steve Shih > > ASR1K FPGAs and ASICs are configured to raise SERR/PERR through PCIe AER. > When an error is raised, it is detected at the root complex, but it is not > detected by the AER driver.

[PATCH] of: make of_device_make_bus_id() static

2016-10-17 Thread frowand . list
From: Frank Rowand of_device_make_bus_id() was changed to non-static by commit c66012253800 ("of/device: Make of_device_make_bus_id() usable by other code") more than 6 years ago, but there are no users of it outside of platform.c. Make the function static again. Signed-off-by: Frank Rowand --

Re: [PATCH] spi: mark device nodes only in case of successful instantiation

2016-10-17 Thread Pantelis Antoniou
Hi Ralf, > On Oct 16, 2016, at 12:55 , Ralf Ramsauer wrote: > > Hi Geert, > > On 10/16/2016 10:49 AM, Geert Uytterhoeven wrote: >> Hi Ralf, >> >> (Cc i2c) >> >> On Fri, Oct 14, 2016 at 9:31 PM, Ralf Ramsauer >> wrote: >>> Instantiated SPI device nodes are marked with OF_POPULATE. This was >>

<    3   4   5   6   7   8   9   10   11   12   >