[PATCH] mtd: nand: gpio: make nCE GPIO optional

2017-02-10 Thread Christophe Leroy
On some hardware, the nCE signal is wired to the ChipSelect associated to bus address of the NAND, so it is automatically driven during the memory access and it is not managed by a GPIO. Signed-off-by: Christophe Leroy --- drivers/mtd/nand/gpio.c | 18 -- 1 file changed, 12 inser

Re: [PATCH 07/11] iommu/arm-smmu: Make use of the iommu_register interface

2017-02-10 Thread Robin Murphy
On 09/02/17 11:32, Joerg Roedel wrote: > From: Joerg Roedel > > Also add the smmu devices to sysfs. > > Signed-off-by: Joerg Roedel > --- > drivers/iommu/arm-smmu-v3.c | 22 +- > drivers/iommu/arm-smmu.c| 30 ++ > 2 files changed, 51 insertio

Re: [PATCH v2 2/2] drm/fb_helper: implement ioctl FBIO_WAITFORVSYNC

2017-02-10 Thread Ville Syrjälä
On Thu, Feb 02, 2017 at 11:31:57AM +0100, Maxime Ripard wrote: > From: Stefan Christ > > Implement legacy framebuffer ioctl FBIO_WAITFORVSYNC in the generic > framebuffer emulation driver. Legacy framebuffer users like non kms/drm > based OpenGL(ES)/EGL implementations may require the ioctl to >

[PATCH v3 3/4] [media] s5p-mfc: Set colorspace in VIDIO_{G,TRY}_FMT

2017-02-10 Thread Thibault Saunier
The media documentation says that the V4L2_COLORSPACE_SMPTE170M colorspace should be used for SDTV and V4L2_COLORSPACE_REC709 for HDTV but the driver didn't set the colorimetry, also respect usespace setting. Use 576p display resolution as a threshold to set this. Signed-off-by: Thibault Saunier

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

2017-02-10 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 gatewa

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

2017-02-10 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 IOC

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

2017-02-10 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 compro

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

2017-02-10 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. Signed-of

[PATCH v3 4/4] [media] s5p-mfc: Check and set 'v4l2_pix_format:field' field in try_fmt

2017-02-10 Thread Thibault Saunier
It is required by the standard that the field order is set by the driver. Signed-off-by: Thibault Saunier --- Changes in v3: - Do not check values in the g_fmt functions as Andrzej explained in previous review Changes in v2: - Fix a silly build error that slipped in while rebasing the patches

Re: [PATCH 1/2] Staging: media: bcm2048: fixed 20+ warings/errors

2017-02-10 Thread Greg KH
On Fri, Feb 10, 2017 at 11:37:04AM +0200, Ran Algawi wrote: > Fixed a coding style issues, and two major erros about complex macros > and an error where the driver used a decimal number insted of an octal > number when using a warning. Only do one thing-per-patch please. thanks, gre gk-h

Re: [RFC][PATCH 03/21] ring-buffer: Add TIME_EXTEND_ABS ring buffer type

2017-02-10 Thread Steven Rostedt
On Fri, 10 Feb 2017 15:04:51 +0900 Namhyung Kim wrote: > > > > But it would require comments and/or a wrapper to define it so people > > in the future know what it is doing. > > What about > > event->type_len >= RINGBUF_TYPE_TIME_EXTEND > > ? I think it's easier to understand what it

Re: [Patch] Staging: media: bcm2048: fixed errors and warnings

2017-02-10 Thread kbuild test robot
Hi Ran, [auto build test ERROR on linuxtv-media/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: https://github.com/0day-ci/linux/commits/Ran-Algawi/Staging-media-bcm2048-fixed

Re: [PATCHv3] arm: ftrace: Adds support for CONFIG_DYNAMIC_FTRACE_WITH_REGS

2017-02-10 Thread Russell King - ARM Linux
On Fri, Feb 10, 2017 at 12:03:06PM +, Abel Vesa wrote: > The only problem I don't have a solution for at this point is OLD_LR (or > previous LR as it is called in this patch). If you want the context at function entry, then you need to save the registers as they were at that point. The stacki

Re: [PATCH v4] drivers/misc: Add Aspeed LPC control driver

2017-02-10 Thread Greg KH
On Wed, Feb 08, 2017 at 10:42:47AM +1100, Cyril Bur wrote: > In order to manage server systems, there is typically another processor > known as a BMC (Baseboard Management Controller) which is responsible > for powering the server and other various elements, sometimes fans, > often the system flash

Re: [PATCH] x86/mm/ptdump: Fix soft lockup in page table walker.

2017-02-10 Thread Mark Rutland
Hi, On Fri, Feb 10, 2017 at 04:56:19PM +0300, Andrey Ryabinin wrote: > On 02/10/2017 04:02 PM, Dmitry Vyukov wrote: > > On Fri, Feb 10, 2017 at 1:15 PM, Andrey Ryabinin > > wrote: > >> On 02/10/2017 02:18 PM, Thomas Gleixner wrote: > >>> On Fri, 10 Feb 2017, Dmitry Vyukov wrote: > diff --git a/a

Re: [PATCH v3] fork: free vmapped stacks in cache when cpus are offline

2017-02-10 Thread Hoeun Ryu
On Fri, Feb 10, 2017 at 9:05 PM, Michal Hocko wrote: > On Fri 10-02-17 17:32:07, Hoeun Ryu wrote: > [...] >> +static int free_vm_stack_cache(unsigned int cpu) >> +{ >> + struct vm_struct **cached_vm_stacks = per_cpu_ptr(cached_stacks, cpu); >> + int i; >> + >> + for (i = 0; i < NR_CACH

Re: [PATCH v5 0/2] firmware: add driver data API

2017-02-10 Thread Greg KH
On Mon, Feb 06, 2017 at 05:08:52PM -0800, Luis R. Rodriguez wrote: > Greg, > > This v5 drops the driver porting examples in light of the fact simple > cases do not yet benefit from use of the newer API. The API is purposely > introduced bare bones so new features are easier to review. I am not go

Re: [PATCH v2 6/9] arm, arm64: factorize common cpu capacity default code

2017-02-10 Thread Greg KH
On Thu, Feb 09, 2017 at 09:25:22AM +, Juri Lelli wrote: > arm and arm64 share lot of code relative to parsing CPU capacity > information from DT, using that information for appropriate scaling and > exposing a sysfs interface for chaging such values at runtime. > > Factorize such code in a com

Re: [PATCH 08/11] iommu/msm: Make use of iommu_device_register interface

2017-02-10 Thread Robin Murphy
On 09/02/17 11:32, Joerg Roedel wrote: > From: Joerg Roedel > > Register the MSM IOMMUs to the iommu core and add sysfs > entries for that driver. > > Signed-off-by: Joerg Roedel > --- > drivers/iommu/msm_iommu.c | 73 > +++ > drivers/iommu/msm_iomm

Re: [PATCH 00/13] Add HD44780 Character LCD support

2017-02-10 Thread Greg Kroah-Hartman
On Mon, Feb 06, 2017 at 03:38:02PM +0100, Geert Uytterhoeven wrote: > Hi all, > > Character LCDs are currently typically driven from userspace, either > directly through bit-banging, or via some kind of serial (UART/i2c) > interface. > > This patch series adds kernel support for character L

Re: timers: Make flags output in the timer_start tracepoint useful

2017-02-10 Thread Thomas Gleixner
On Fri, 10 Feb 2017, Steven Rostedt wrote: > On Fri, 10 Feb 2017 14:25:03 +0100 (CET) > Thomas Gleixner wrote: > > > The timer flags in the timer_start trace event contain lots of useful > > information, but the meaning is not clear in the trace output because its > > just printed as a hex value

Re: [PATCH] x86/mm/ptdump: Fix soft lockup in page table walker.

2017-02-10 Thread Andrey Ryabinin
On 02/10/2017 05:29 PM, Mark Rutland wrote: > Hi, > > On Fri, Feb 10, 2017 at 04:56:19PM +0300, Andrey Ryabinin wrote: >> On 02/10/2017 04:02 PM, Dmitry Vyukov wrote: >>> On Fri, Feb 10, 2017 at 1:15 PM, Andrey Ryabinin >>> wrote: On 02/10/2017 02:18 PM, Thomas Gleixner wrote: > On Fri

Re: [PATCH v3] fork: free vmapped stacks in cache when cpus are offline

2017-02-10 Thread Michal Hocko
On Fri 10-02-17 23:31:41, Hoeun Ryu wrote: > On Fri, Feb 10, 2017 at 9:05 PM, Michal Hocko wrote: > > On Fri 10-02-17 17:32:07, Hoeun Ryu wrote: [...] > >> static unsigned long *alloc_thread_stack_node(struct task_struct *tsk, > >> int node) > >> @@ -456,6 +474,11 @@ void __init fork_init(void)

Re: [PATCH v4 2/2] procfs/tasks: add a simple per-task procfs hidepid= field

2017-02-10 Thread Lafcadio Wluiki
On Sat, Jan 21, 2017 at 1:53 AM, Andy Lutomirski wrote: > I agree that the kernel change to do it per task is very simple. But > this is an unfortunate slippery slope. What if you want to block off > everything in /proc that isn't associated with a PID? What if you > want to suppress /sys acce

Re: [PATCH 2/2] x86/vdso: Add VCLOCK_HVCLOCK vDSO clock read method

2017-02-10 Thread Vitaly Kuznetsov
Stephen Hemminger writes: > Why not use existing seqlock's? > To be honest I don't quite understand how we could use it -- the sequence locking here is done against the page updated by the hypersior, we're not creating new structures (so I don't understand how we could use struct seqcount which

Re: [PATCH 24/32] tools include: Add a __fallthrough statement

2017-02-10 Thread Arnaldo Carvalho de Melo
Em Thu, Feb 09, 2017 at 07:02:06PM -0800, Joe Perches escreveu: > On Thu, 2017-02-09 at 22:39 -0300, Arnaldo Carvalho de Melo wrote: > > From: Arnaldo Carvalho de Melo > > > > For cases where implicit fall through case labels are intended, > > to let us inform that to gcc >= 7: > > I believe thi

Re: [PATCH] x86/mm/ptdump: Fix soft lockup in page table walker.

2017-02-10 Thread Mark Rutland
On Fri, Feb 10, 2017 at 05:38:20PM +0300, Andrey Ryabinin wrote: > On 02/10/2017 05:29 PM, Mark Rutland wrote: > > On Fri, Feb 10, 2017 at 04:56:19PM +0300, Andrey Ryabinin wrote: > >> On 02/10/2017 04:02 PM, Dmitry Vyukov wrote: > >>> On Fri, Feb 10, 2017 at 1:15 PM, Andrey Ryabinin > >>> wrote:

[PATCH 1/1] dma: imx-sdma: add 1ms delay to ensure SDMA channel is stopped

2017-02-10 Thread jiada_wang
From: Jiada Wang sdma_disable_channel() cannot ensure dma is stopped to access module's FIFOs. Maybe SDMA core is running and accessing BD when disable of corresponding channel, this may cause sometimes even after call of .sdma_disable_channel(), SDMA core still be running and accessing module's

[GIT PULL] MMC fixes for v.4.10 rc8

2017-02-10 Thread Ulf Hansson
Hi Linus, Here's one mmc fix intended for v4.10 rc8 (or final if no new rc). Based on v4.10-rc7. Details are as usual found in the signed tag. Please pull this in! Kind regards Ulf Hansson The following changes since commit d5adbfcd5f7bcc6fa58a41c5c5ada0e5c826ce2c: Linux 4.10-rc7 (2017-02-0

Re: [PATCH 05/13] vmbus: remove per channel state

2017-02-10 Thread Greg KH
On Sun, Feb 05, 2017 at 05:20:35PM -0700, k...@exchange.microsoft.com wrote: > From: Stephen Hemminger > > The netvsc no longer needs per channel state hook to track receive buffer. > > Signed-off-by: Stephen Hemminger > Signed-off-by: K. Y. Srinivasan > --- > include/linux/hyperv.h | 14 --

Re: [PATCH 1/1] Ddrivers: hv: Turn off write permission on the hypercall page

2017-02-10 Thread Greg KH
On Wed, Feb 08, 2017 at 06:30:56PM -0700, k...@exchange.microsoft.com wrote: > From: K. Y. Srinivasan > > The hypercall page only needs to be executable but currently it is setup to > be writable as well. Fix the issue. > > Signed-off-by: K. Y. Srinivasan I had to hand-edit the subject: line,

[GIT PULL] irqchip: Fixes for v4.10

2017-02-10 Thread Jason Cooper
Thomas, This is a couple of fixes that have unintentionally had plenty of time in linux-next. :-/ Please pull. thx, Jason. The following changes since commit 7ce7d89f48834cefece7804d38fc5d85382edf77: Linux 4.10-rc1 (2016-12-25 16:13:08 -0800) are available in the git repository at: gi

Re: timers: Make flags output in the timer_start tracepoint useful

2017-02-10 Thread Steven Rostedt
On Fri, 10 Feb 2017 15:37:11 +0100 (CET) Thomas Gleixner wrote: > > > --- a/include/trace/events/timer.h > > > +++ b/include/trace/events/timer.h > > > @@ -36,6 +36,13 @@ DEFINE_EVENT(timer_class, timer_init, > > > TP_ARGS(timer) > > > ); > > > > > > +#define decode_timer_flags(flags)

Re: [PATCH v2 1/4] MicroSemi Switchtec management interface driver

2017-02-10 Thread Greg Kroah-Hartman
On Thu, Feb 02, 2017 at 11:06:00AM -0700, Logan Gunthorpe wrote: > + cdev = &stdev->cdev; > + cdev_init(cdev, &switchtec_fops); > + cdev->owner = THIS_MODULE; > + cdev->kobj.parent = &dev->kobj; Minor nit, the kobject in a cdev is unlike any other kobject you have ever seen, don't

Re: [PATCH] staging: r8712u: remove unnecessary le32_to_cpu

2017-02-10 Thread Larry Finger
On 02/10/2017 08:08 AM, Greg KH wrote: On Thu, Feb 09, 2017 at 08:51:55PM -0700, Perry Hooker wrote: This patch fixes the following sparse warning: drivers/staging/rtl8712/usb_ops_linux.c:212:33: warning: cast to restricted __le32 Signed-off-by: Perry Hooker --- drivers/staging/rtl8712/usb_o

[GIT PULL] sound fixes for 4.10

2017-02-10 Thread Takashi Iwai
Linus, please pull sound fixes for v4.10 from: git://git.kernel.org/pub/scm/linux/kernel/git/tiwai/sound.git tags/sound-4.10 The topmost commit is af677166cf63c179dc2485053166e02c4aea01eb sound fixes for 4.10 here is the last-

Re: [PATCH] x86/mm/ptdump: Fix soft lockup in page table walker.

2017-02-10 Thread Mark Rutland
On Fri, Feb 10, 2017 at 02:02:19PM +0100, Dmitry Vyukov wrote: > On Fri, Feb 10, 2017 at 1:15 PM, Andrey Ryabinin > wrote: > > > > > > On 02/10/2017 02:18 PM, Thomas Gleixner wrote: > >> On Fri, 10 Feb 2017, Dmitry Vyukov wrote: > >>> Could we default this rodata check to n under KASAN? Or at lea

Re: [PATCH v2 1/4] MicroSemi Switchtec management interface driver

2017-02-10 Thread Greg Kroah-Hartman
On Thu, Feb 02, 2017 at 11:06:00AM -0700, Logan Gunthorpe wrote: > Microsemi's "Switchtec" line of PCI switch devices is already well > supported by the kernel with standard PCI switch drivers. However, the > Switchtec device advertises a special management endpoint with a separate > PCI function a

Re: [PATCH v2 3/4] switchtec: Add sysfs attributes to the Switchtec driver

2017-02-10 Thread Greg Kroah-Hartman
On Thu, Feb 02, 2017 at 11:06:02AM -0700, Logan Gunthorpe wrote: > This patch adds a few read-only sysfs attributes which provide > some device information that is exposed from the devices. Primarily > component and device names and versions. These are documented in > Documentation/ABI/testing/sysf

Re: [PATCH] staging: r8712u: remove unnecessary le32_to_cpu

2017-02-10 Thread Greg KH
On Fri, Feb 10, 2017 at 08:52:12AM -0600, Larry Finger wrote: > On 02/10/2017 08:08 AM, Greg KH wrote: > > On Thu, Feb 09, 2017 at 08:51:55PM -0700, Perry Hooker wrote: > > > This patch fixes the following sparse warning: > > > drivers/staging/rtl8712/usb_ops_linux.c:212:33: warning: cast to > > >

[PATCH RESEND v7 1/1] usb: xhci: plat: Enable async suspend/resume

2017-02-10 Thread Robert Foss
From: Andrew Bresticker USB host controllers can take a significant amount of time to suspend and resume, adding several hundred miliseconds to the kernel resume time. Since the XHCI controller has no outside dependencies (other than clocks, which are suspended late/resumed early), allow it to su

Re: [PATCH v2 0/4] New Microsemi PCI Switch Management Driver

2017-02-10 Thread Greg Kroah-Hartman
On Thu, Feb 02, 2017 at 11:05:59AM -0700, Logan Gunthorpe wrote: > Changes since v1: > > * Rebased onto 4.10-rc6 (cleanly) > * Split the patch into a few more easily digestible patches (as > suggested by Greg Kroah-Hartman) > * Folded switchtec.c into switchtec.h (per Greg) > * Fixed a bunch of

[PATCH RESEND v7 0/1] usb: xhci: plat: Enable runtime PM

2017-02-10 Thread Robert Foss
This series enables runtime PM and asynchronous resume/suspend support for xhci-plat devices. Changes since v1: - Added Signed-off-by: Robert Foss - Added proper metadata tags to series Changes since v2: - Added missing changelog to cover-letter - Added error checking to pm_runtime_get_sync() c

Re: [PATCH 0/8] Sync upstream binder code with android-4.9 tree

2017-02-10 Thread Greg Kroah-Hartman
On Fri, Feb 03, 2017 at 02:40:44PM -0800, John Stultz wrote: > Hey All, > With the android-4.9 tree being shared recently, I noticed > there were some new binder changes that hadn't yet made it > upstream (though an earlier version of the patchset was > submitted a bit back). > > Anyway, I wante

Re: [PATCH v3] fork: free vmapped stacks in cache when cpus are offline

2017-02-10 Thread Hoeun Ryu
On Fri, Feb 10, 2017 at 11:41 PM, Michal Hocko wrote: > On Fri 10-02-17 23:31:41, Hoeun Ryu wrote: >> On Fri, Feb 10, 2017 at 9:05 PM, Michal Hocko wrote: >> > On Fri 10-02-17 17:32:07, Hoeun Ryu wrote: > [...] >> >> static unsigned long *alloc_thread_stack_node(struct task_struct *tsk, >> >> i

Re: [PATCH v4] net: ethernet: faraday: To support device tree usage.

2017-02-10 Thread Rob Herring
On Wed, Feb 8, 2017 at 5:59 AM, Greentime Hu wrote: > On Sat, Jan 28, 2017 at 6:17 AM, Rob Herring wrote: >> >> On Wed, Jan 25, 2017 at 10:09:20PM +0100, Arnd Bergmann wrote: >> > On Wed, Jan 25, 2017 at 6:34 PM, David Miller wrote: >> > > From: Greentime Hu >> > > Date: Tue, 24 Jan 2017 16:46:

Re: [PATCH] staging: r8712u: remove unnecessary le32_to_cpu

2017-02-10 Thread Larry Finger
On 02/10/2017 08:58 AM, Greg KH wrote: On Fri, Feb 10, 2017 at 08:52:12AM -0600, Larry Finger wrote: On 02/10/2017 08:08 AM, Greg KH wrote: On Thu, Feb 09, 2017 at 08:51:55PM -0700, Perry Hooker wrote: This patch fixes the following sparse warning: drivers/staging/rtl8712/usb_ops_linux.c:212:3

[PULL REQUEST] i2c for 4.10

2017-02-10 Thread Wolfram Sang
Linus, here are two bugfixes (proper IO mapping and use of mutex) for a driver feature we introduced in this cycle. Please pull. Thanks, Wolfram The following changes since commit d5adbfcd5f7bcc6fa58a41c5c5ada0e5c826ce2c: Linux 4.10-rc7 (2017-02-05 15:10:58 -0800) are available in the g

Re: [PATCH v3 1/4] [media] exynos-gsc: Use 576p instead 720p as a threshold for colorspaces

2017-02-10 Thread Hans Verkuil
On 02/10/2017 03:10 PM, Thibault Saunier wrote: > From: Javier Martinez Canillas > > The media documentation says that the V4L2_COLORSPACE_SMPTE170M colorspace > should be used for SDTV and V4L2_COLORSPACE_REC709 for HDTV. But drivers > don't agree on the display resolution that should be used as

Re: [PATCH v3 3/4] [media] s5p-mfc: Set colorspace in VIDIO_{G,TRY}_FMT

2017-02-10 Thread Hans Verkuil
On 02/10/2017 03:10 PM, Thibault Saunier wrote: > The media documentation says that the V4L2_COLORSPACE_SMPTE170M colorspace > should be used for SDTV and V4L2_COLORSPACE_REC709 for HDTV but the driver > didn't set the colorimetry, also respect usespace setting. > > Use 576p display resolution as

Re: [PATCH] ARM: dts: Odroid XU4: fix USB3.0 ports

2017-02-10 Thread Richard Genoud
2017-02-10 14:34 GMT+01:00 Krzysztof Kozlowski : > On Thu, Feb 09, 2017 at 11:26:48AM +0100, Richard Genoud wrote: >> I did some tests with XU3 and XU4, playing with USB2 and USB3 quirks >> (snps,dis_u{2,3}_susphy_quirk) >> >> kernel for the tests: next-20170206 >> DTBs: exynos5422-odroidxu3-lite.d

RE: [PATCH v3 2/3] USB3/DWC3: Add property "snps, incr-burst-type-adjustment" for INCR burst type

2017-02-10 Thread Jerry Huang
> >> >> Why do you need the first field? Is the 2nd field used if the 1st is 0? > >> >> If not, then just use the presence of the property to enable or not. > >> > The first field is one switch. > >> > When it is 1, means undefined length INCR burst type enabled, we > >> > can use > >> any length l

Re: [PATCHv6 11/37] HACK: readahead: alloc huge pages, if allowed

2017-02-10 Thread Matthew Wilcox
On Thu, Feb 09, 2017 at 05:23:31PM -0700, Andreas Dilger wrote: > On Feb 9, 2017, at 4:34 PM, Matthew Wilcox wrote: > > Well ... what if we made readahead 2 hugepages in size for inodes which > > are using huge pages? That's only 8x our current readahead window, and > > if you're asking for hugep

Re: [PATCH] mtd: nand: gpio: make nCE GPIO optional

2017-02-10 Thread Marek Vasut
On 02/10/2017 03:01 PM, Christophe Leroy wrote: > On some hardware, Can you be more specific or is that confidential ? Anyway, that's just my curiosity, what hardware is done like that. > the nCE signal is wired to the ChipSelect associated > to bus address of the NAND, so it is automatically dri

RE: [PATCH 1/1] Drivers: hv: Fix the bug in generating the guest ID

2017-02-10 Thread KY Srinivasan
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Friday, February 10, 2017 6:40 AM > To: KY Srinivasan > Cc: o...@aepfle.de; jasow...@redhat.com; linux-kernel@vger.kernel.org; > a...@canonical.com; de...@linuxdriverproject.org; > leann.ogasaw...@canonical.

Re: [PATCH] usb: typec: fix ptr_ret.cocci warnings

2017-02-10 Thread Heikki Krogerus
On Tue, Jan 31, 2017 at 01:38:34AM +0800, kbuild test robot wrote: > drivers/usb/typec/typec.c:1249:1-3: WARNING: PTR_ERR_OR_ZERO can be used > > > Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR > > Generated by: scripts/coccinelle/api/ptr_ret.cocci > > CC: Heikki Krogerus > Signed

Re: [PATCH 06/11] iommu: Add iommu_device_set_fwnode() interface

2017-02-10 Thread Joerg Roedel
Hi Robin, On Fri, Feb 10, 2017 at 02:16:54PM +, Robin Murphy wrote: > > +static inline void iommu_device_set_fwnode(struct iommu_device *iommu, > > + struct fwnode_handle *fwnode) > > +{ > > + iommu->fwnode = fwnode; > > +} > > Would it make sense to sim

[PATCH] atmel_serial: Use the fractional divider when possible

2017-02-10 Thread Romain Izard
The fractional baud rate generator is available when using the asynchronous mode of Atmel USART controllers. It makes it possible to use higher baudrates, in exchange for a less precise clock with a variable duty cycle. The existing code restricts its use to the normal mode of the USART controller

Re: timers: Make flags output in the timer_start tracepoint useful

2017-02-10 Thread Thomas Gleixner
On Fri, 10 Feb 2017, Steven Rostedt wrote: > On Fri, 10 Feb 2017 15:37:11 +0100 (CET) > Thomas Gleixner wrote: > > > > > --- a/include/trace/events/timer.h > > > > +++ b/include/trace/events/timer.h > > > > @@ -36,6 +36,13 @@ DEFINE_EVENT(timer_class, timer_init, > > > > TP_ARGS(timer) >

RE: [PATCH v4 3/3] USB3/DWC3: Enable undefined length INCR burst type

2017-02-10 Thread Jerry Huang
> >> -- > >> 1.7.9.5 > > Hi, Balbi and all guys, > > Any comment for these patches? Can they be accepted? > > Rob had comments which you didn't reply yet. I cannot take this patchset > yet ;-) > Balbi, I look into his mail again, which was based v3, and I replied it. He had different understandin

Re: [PATCH v3] fork: free vmapped stacks in cache when cpus are offline

2017-02-10 Thread Thomas Gleixner
On Fri, 10 Feb 2017, Michal Hocko wrote: > On Fri 10-02-17 23:31:41, Hoeun Ryu wrote: > > On Fri, Feb 10, 2017 at 9:05 PM, Michal Hocko wrote: > > > On Fri 10-02-17 17:32:07, Hoeun Ryu wrote: > [...] > > >> static unsigned long *alloc_thread_stack_node(struct task_struct *tsk, > > >> int node) >

Re: [PATCH 07/11] iommu/arm-smmu: Make use of the iommu_register interface

2017-02-10 Thread Joerg Roedel
On Fri, Feb 10, 2017 at 02:20:34PM +, Robin Murphy wrote: > > @@ -1795,8 +1798,10 @@ static int arm_smmu_add_device(struct device *dev) > > } > > > > group = iommu_group_get_for_dev(dev); > > - if (!IS_ERR(group)) > > + if (!IS_ERR(group)) { > > iommu_group_put(group);

Re: [PATCH v4] eeprom/at24: use device_property_*() functions instead of of_get_property()

2017-02-10 Thread Wolfram Sang
On Thu, Feb 09, 2017 at 11:36:08AM -0600, Ben Gardner wrote: > Allow the at24 driver to get configuration information from both OF and > ACPI by using the more generic device_property functions. > This change was inspired by the at25.c driver. > > I have a custom board with a ST M24C02 EEPROM atta

Re: [PATCH v3] fork: free vmapped stacks in cache when cpus are offline

2017-02-10 Thread Thomas Gleixner
On Sat, 11 Feb 2017, Hoeun Ryu wrote: > On Fri, Feb 10, 2017 at 11:41 PM, Michal Hocko wrote: > > On Fri 10-02-17 23:31:41, Hoeun Ryu wrote: > >> On Fri, Feb 10, 2017 at 9:05 PM, Michal Hocko wrote: > >> > On Fri 10-02-17 17:32:07, Hoeun Ryu wrote: > > [...] > >> >> static unsigned long *allo

Re: [PATCH] ipc/mqueue: use unsigned int for retval

2017-02-10 Thread Waiman Long
On 02/10/2017 02:36 AM, Nicholas Mc Guire wrote: > The retval variable is assigned by bitops only and thus there is no > reason to use a signed type. Further matching it with the return > type of the function also makes static code checkers happy. > > Signed-off-by: Nicholas Mc Guire > --- > > Fo

Re: [PATCH 01/11] iommu: Rename iommu_get_instance()

2017-02-10 Thread Joerg Roedel
Hi Robin, On Fri, Feb 10, 2017 at 02:12:54PM +, Robin Murphy wrote: > I'm really liking this series! Superficially it doesn't seem to break > anything on my Juno, but I'll give it a more thorough workout soon. Great, thanks for testing! Any problems in this series should show up on boot anywa

[PATCH] staging: android: ion: fix coding style issue

2017-02-10 Thread Youngdo, Lee
Replaced sizeof(struct foo) into sizeof(*ptr), found by checkpatch.pl. Signed-off-by: Youngdo, Lee --- drivers/staging/android/ion/ion_cma_heap.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/android/ion/ion_cma_heap.c b/drivers/staging/android/ion/io

net/llc: bug in llc_pdu_init_as_xid_cmd/skb_over_panic

2017-02-10 Thread Andrey Konovalov
Hi, I've got the following error report while fuzzing the kernel with syzkaller. On commit 926af6273fc683cd98cd0ce7bf0d04a02eed6742. A reproducer and .config are attached kernel BUG at net/core/skbuff.c:105! invalid opcode: [#1] SMP KASAN Dumping ftrace buffer: (ftrace buffer empty) Mod

[PATCH 3/3] mfd: wm831x: Add device tree binding document

2017-02-10 Thread Charles Keepax
This is just a very simple initial binding, supporting only registering the chip and the GPIOs. More features will be added as support for device tree in the driver is expanded. Signed-off-by: Charles Keepax --- Documentation/devicetree/bindings/mfd/wm831x.txt | 31 1 fi

Re: [PATCH 1/3] ath10k: remove ath10k_vif_to_arvif()

2017-02-10 Thread Adrian Chadd
On 9 February 2017 at 23:37, Joe Perches wrote: > On Thu, 2017-02-09 at 23:14 -0800, Adrian Chadd wrote: > >> If there >> were accessors for the skb data / len fields (like we do for mbufs) >> then porting the code would've involved about 5,000 less changed >> lines. > > What generic mechanisms wo

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

2017-02-10 Thread Rob Herring
On Mon, Feb 06, 2017 at 07:08:36AM -0800, Andrey Smirnov wrote: > This driver exposes various reset faculties, impelented by System Reset > Controller IP block, as a reset driver. Currently only PCIE related > reset lines are implemented. > > Cc: Lucas Stach > Cc: Rob Herring > Cc: Mark Rutland

[PATCH v2] media: s5p_mfc print buf pointer in hex constistently

2017-02-10 Thread Shuah Khan
Fix s5p_mfc_set_dec_frame_buffer_v6() to print buffer pointer in hex to be consistent with the rest of the messages in the routine. Signed-off-by: Shuah Khan --- Fixed commit log. No code changes. Thanks for the catch. drivers/media/platform/s5p-mfc/s5p_mfc_opr_v6.c | 2 +- 1 file changed, 1 i

[PATCH V2] timers: Make flags output in the timer_start tracepoint useful

2017-02-10 Thread Thomas Gleixner
The timer flags in the timer_start trace event contain lots of useful information, but the meaning is not clear in the trace output. Making tools rely on the bit positions is bad as they might change over time. Decode the flags in the print out. Tools can retrieve the bits and their meaning from t

RE: [RFC] mm/hugetlb: use mem policy when allocating surplus huge pages

2017-02-10 Thread Andrejczuk, Grzegorz
On Mike Kravetz, February 9, 2017 8:32 PM wrote: > I believe another way of stating the problem is as follows: > > At mmap(MAP_HUGETLB) time a reservation for the number of huge pages > is made. If surplus huge pages need to be (and can be) allocated to > satisfy the reservation, they will be allo

timers: Make flags output in the timer_start tracepoint useful

2017-02-10 Thread Thomas Gleixner
The timer flags in the timer_start trace event contain lots of useful information, but the meaning is not clear in the trace output because its just printed as a hex value. Making tools rely on the bit positions is bad as they might change over time. Decode the flags in the printout. Tools can ret

Re: [PATCH v4] drivers/misc: Add Aspeed LPC control driver

2017-02-10 Thread Patrick Williams
On Fri, Feb 10, 2017 at 03:30:12PM +0100, Greg KH wrote: > On Wed, Feb 08, 2017 at 10:42:47AM +1100, Cyril Bur wrote: > > Signed-off-by: Cyril Bur > > --- > > Without some other reviewed-by: or at least tested-by lines here, I'm > not going to take this. Go poke your fellow ppc people to do some

Re: [PATCH] atmel_serial: Use the fractional divider when possible

2017-02-10 Thread Ludovic Desroches
On Fri, Feb 10, 2017 at 04:24:46PM +0100, Romain Izard wrote: > The fractional baud rate generator is available when using the > asynchronous mode of Atmel USART controllers. It makes it possible to > use higher baudrates, in exchange for a less precise clock with a > variable duty cycle. > > The

Re: [PATCH v2 1/2] drm/cma-helper: Add multi buffer support for cma fbdev

2017-02-10 Thread Maxime Ripard
Hi Daniel, On Thu, Feb 09, 2017 at 06:04:38PM +0100, Daniel Vetter wrote: > On Thu, Feb 02, 2017 at 11:31:56AM +0100, Maxime Ripard wrote: > > From: Xinliang Liu > > > > This patch add a config to support to create multi buffer for cma fbdev. > > Such as double buffer and triple buffer. > > > >

[PATCH v2] x86/paravirt: Don't make vcpu_is_preempted() a callee-save function

2017-02-10 Thread Waiman Long
It was found when running fio sequential write test with a XFS ramdisk on a VM running on a 2-socket x86-64 system, the %CPU times as reported by perf were as follows: 69.75% 0.59% fio [k] down_write 69.15% 0.01% fio [k] call_rwsem_down_write_failed 67.12% 1.12% fio [k] rwsem_down_writ

Re: [PATCHSET for-4.11] cgroup: implement cgroup v2 thread mode

2017-02-10 Thread Tejun Heo
On Thu, Feb 09, 2017 at 05:07:16AM -0800, Paul Turner wrote: > What are the motivations that you see for forcing this all onto one > mount-point via .threads sub-tree tags? So, you wanted rgroup but with /proc interface? I'm afraid it's too late for that. Thanks. -- tejun

Re: [PATCH 1/1] Drivers: hv: Fix the bug in generating the guest ID

2017-02-10 Thread Greg KH
On Sat, Feb 04, 2017 at 04:24:40PM +, KY Srinivasan wrote: > > > > -Original Message- > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Saturday, February 4, 2017 7:58 AM > > To: KY Srinivasan > > Cc: linux-kernel@vger.kernel.org; de...@linuxdriverproject.org; > > o...@

Re: [PATCH 08/11] iommu/msm: Make use of iommu_device_register interface

2017-02-10 Thread Joerg Roedel
On Fri, Feb 10, 2017 at 02:35:39PM +, Robin Murphy wrote: > On 09/02/17 11:32, Joerg Roedel wrote: > > + ret = iommu_device_sysfs_add(&iommu->iommu, iommu->dev, NULL, > > +"msm-smmu.%pa", &ioaddr); > > + if (ret) { > > + pr_err("Could not add msm-sm

Re: [PATCH] Make EN2 pin optional in the TRF7970A driver

2017-02-10 Thread Rob Herring
On Tue, Feb 07, 2017 at 06:22:04AM +0100, Heiko Schocher wrote: > From: Guan Ben > > Make the EN2 pin optional. This is useful for boards, > which have this pin fix wired, for example to ground. > > Signed-off-by: Guan Ben > Signed-off-by: Mark Jonas > Signed-off-by: Heiko Schocher > > --- >

Re: [PATCH 06/11] iommu: Add iommu_device_set_fwnode() interface

2017-02-10 Thread Robin Murphy
On 10/02/17 15:22, Joerg Roedel wrote: > Hi Robin, > > On Fri, Feb 10, 2017 at 02:16:54PM +, Robin Murphy wrote: >>> +static inline void iommu_device_set_fwnode(struct iommu_device *iommu, >>> + struct fwnode_handle *fwnode) >>> +{ >>> + iommu->fwnode = f

Re: [PATCH] x86, locking: Inline *_unlock_bh & *_unlock_irqrestore

2017-02-10 Thread Thomas Gleixner
On Thu, 2 Feb 2017, Waiman Long wrote: > For spinlock and read-write locks, both *_unlock() and *_unlock_irq() > functions are inlined if !PREEMPT. Whereas the *_unlock_bh() and > *unlock_irqrestore() are not inlined as that will increase the size > of the kernel binary. > > Given the fact that t

Re: [PATCH v2] Btrfs: add another missing end_page_writeback on submit_extent_page failure

2017-02-10 Thread David Sterba
On Thu, Feb 09, 2017 at 05:24:33PM +0900, Takafumi Kubota wrote: > If btrfs_bio_alloc fails in submit_extent_page, submit_extent_page returns > without clearing the writeback bit of the failed page. > > __extent_writepage_io, that is a caller of submit_extent_page, > does not clear the remaining w

Re: [PATCH V3 2/2] Move stack parameters for sed_ioctl to prevent oversized stack with CONFIG_KASAN

2017-02-10 Thread Scott Bauer
On Fri, Feb 10, 2017 at 09:01:23AM +0100, Arnd Bergmann wrote: > On Thursday, February 9, 2017 10:20:01 AM CET Scott Bauer wrote: > > 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

Re: [PATCH V2 2/2] tracing/probe: Show subsystem name in messages

2017-02-10 Thread Namhyung Kim
On Fri, Feb 10, 2017 at 10:23:06PM +0900, Masami Hiramatsu wrote: > Show "trace_probe:", "trace_kprobe:" and "trace_uprobe:" > headers for each warning/error/info message. This will > help people to notice that kprobe/uprobe events caused > those messages. > > Signed-off-by: Masami Hiramatsu > --

Re: [PATCHSET for-4.11] cgroup: implement cgroup v2 thread mode

2017-02-10 Thread Tejun Heo
Hello, On Thu, Feb 09, 2017 at 11:29:09AM +0100, Peter Zijlstra wrote: > Uhm, no. They would see the exact same hierarchy, seeing how there is > only one tree. They would have different view of it maybe, but I don't > see how that matters, nor do you explain. Sure, the base hierarchy is the same

Re: [PATCH V2] timers: Make flags output in the timer_start tracepoint useful

2017-02-10 Thread Steven Rostedt
On Fri, 10 Feb 2017 16:41:15 +0100 (CET) Thomas Gleixner wrote: > The timer flags in the timer_start trace event contain lots of useful > information, but the meaning is not clear in the trace output. Making tools > rely on the bit positions is bad as they might change over time. > > Decode the

Re: [PATCH 06/11] iommu: Add iommu_device_set_fwnode() interface

2017-02-10 Thread Joerg Roedel
On Fri, Feb 10, 2017 at 04:03:07PM +, Robin Murphy wrote: > Yeah, on reflection explicit initialisation is certainly easier to read > than a bunch of arguments handled implicitly by register(), but then > from that angle, even more clear would be to simply have the drivers > write the relevant

Re: [WIP PATCHSET 1/4] blk-mq: pass bio to blk_mq_sched_get_rq_priv

2017-02-10 Thread Jens Axboe
On 02/07/2017 10:24 AM, Paolo Valente wrote: > bio is used in bfq-mq's get_rq_priv, to get the request group. We could > pass directly the group here, but I thought that passing the bio was > more general, giving the possibility to get other pieces of information > if needed. I applied this one, t

Sed-opal Fixes

2017-02-10 Thread Scott Bauer
changes from v3->v4: Changed manual kzalloc + copy from user to memdup_user. This small series fixes a small ABI issue when using the _IOC_SIZE with the Activate Locking SP ioctl. I had put the wrong structure in the IOW macro in the uapi header which caused issues when trying to copy in the cont

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

2017-02-10 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 --- include/uapi/linux/sed-opal.h | 2 +- 1 file changed

[PATCH V4 2/2] Move stack parameters for sed_ioctl to prevent oversized stack with CONFIG_KASAN

2017-02-10 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: [WIP PATCHSET 0/4] WIP branch for bfq-mq

2017-02-10 Thread Bart Van Assche
On Tue, 2017-02-07 at 18:24 +0100, Paolo Valente wrote: > [1] https://github.com/Algodev-github/bfq-mq Hello Paolo, That branch includes two changes of the version suffix (EXTRAVERSION in Makefile). Please don't do that but set CONFIG_LOCALVERSION in .config to add a suffix to the kernel version

Re: [PATCH v2 0/4] New Microsemi PCI Switch Management Driver

2017-02-10 Thread Jens Axboe
On Thu, Feb 2, 2017 at 11:05 AM, Logan Gunthorpe wrote: > Changes since v1: > > * Rebased onto 4.10-rc6 (cleanly) > * Split the patch into a few more easily digestible patches (as > suggested by Greg Kroah-Hartman) > * Folded switchtec.c into switchtec.h (per Greg) > * Fixed a bunch of 32bit bui

Re: [PATCH V2 1/2] tracing/probes: Fix a warning message to show correct maximum length

2017-02-10 Thread Steven Rostedt
On Fri, 10 Feb 2017 22:21:55 +0900 Masami Hiramatsu wrote: > Since tracing/*probe_events will accept a probe definition > up to 4096 - 2 ('\n' and '\0') bytes, it must show 4094 instead > of 4096 in warning message. Actually, during the testing I found that we don't need the '\n'. echo

[PATCH] staging: bcm2835-audio: fix trailing statements

2017-02-10 Thread AbdAllah-MEZITI
Trailing statements should be on next line. Signed-off-by: AbdAllah-MEZITI --- drivers/staging/bcm2835-audio/bcm2835-vchiq.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/bcm2835-audio/bcm2835-vchiq.c b/drivers/staging/bcm2835-audio/bcm2835-vchiq.c index

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

2017-02-10 Thread Boris Ostrovsky
On 02/10/2017 09:24 AM, Paul Durrant wrote: > +static long privcmd_ioctl_dm_op(void __user *udata) > +{ > + struct privcmd_dm_op kdata; > + struct privcmd_dm_op_buf *kbufs; > + unsigned int nr_pages = 0; > + struct page **pages = NULL; > + struct xen_dm_op_buf *xbufs = NULL; > +

<    1   2   3   4   5   6   7   8   >