Re: [PATCH v9 7/7] vfio/type1: return MSI geometry through VFIO_IOMMU_GET_INFO capability chains

2016-05-10 Thread Eric Auger
On 05/10/2016 01:03 AM, Alex Williamson wrote: > On Wed, 4 May 2016 14:06:19 +0200 > Eric Auger wrote: > >> Hi Alex, >> On 05/04/2016 01:54 PM, Eric Auger wrote: >>> This patch allows the user-space to retrieve the MSI geometry. The >>> implementation is based on capability chains, now also added

[PATCH] KVM: x86: make hwapic_isr_update and hwapic_irr_update look the same

2016-05-10 Thread Paolo Bonzini
Neither APICv nor AVIC actually need the first argument of hwapic_isr_update, but the vCPU makes more sense than passing the pointer to the whole virtual machine! In fact in the APICv case it's just because the vCPU is used implicitly, through the loaded VMCS. The second argument instead is named

Re: [PATCH 1/3] memory-hotplug: add move_pfn_range()

2016-05-10 Thread Yasuaki Ishimatsu
On Mon, 9 May 2016 12:53:37 -0500 Reza Arbab wrote: > Add move_pfn_range(), a wrapper to call move_pfn_range_left() or > move_pfn_range_right(). > > No functional change. This will be utilized by a later patch. > > Signed-off-by: Reza Arbab Looks good to me. Reviewed-by: Yasuaki Ishimatsu

Re: Getting rid of dynamic TASK_SIZE (on x86, at least)

2016-05-10 Thread Andy Lutomirski
On Tue, May 10, 2016 at 9:30 AM, Cyrill Gorcunov wrote: > On Tue, May 10, 2016 at 09:07:49AM -0700, Andy Lutomirski wrote: >> Hi all- >> >> I'm trying to get rid of x86's dynamic TASK_SIZE and just redefine it >> to TASK_SIZE_MAX. So far, these are the TASK_SIZE users that actually >> seem to car

Re: [PATCH 1/1] Staging: comedi: comedi_fops.c: Fixed coding style issue

2016-05-10 Thread Ian Abbott
On 10/05/16 16:51, Shyam Saini wrote: Fixed following checkpatch.pl warnings WARNING: Prefer WRITE_ONCE(, ) over ACCESS_ONCE() = WARNING: Prefer READ_ONCE() over ACCESS_ONCE() Signed-off-by: Shyam Saini --- drivers/staging/comedi/comedi_fops.c | 12 ++-- 1 file changed, 6 insertion

Re: [PATCH v5 2/5] ARM: davinci: da8xx: Add CFGCHIP syscon platform declaration.

2016-05-10 Thread David Lechner
On 05/10/2016 06:26 AM, Sergei Shtylyov wrote: On 5/10/2016 2:46 AM, David Lechner wrote: The CFGCHIP registers are used by a number of devices, so using a syscon device to share them. The first consumer of this will by the phy-da8xx-usb driver. Signed-off-by: David Lechner [...] diff --git

Re: [PATCH v2 00/23] ata: sata_dwc_460ex: make it working again

2016-05-10 Thread Andy Shevchenko
On Tue, 2016-05-10 at 12:30 -0400, Tejun Heo wrote: > Hello, > > On Tue, May 10, 2016 at 11:34:40AM +0530, Vinod Koul wrote: > > > > > > > > slave-dma [1], branch topic/dw. But I think Vinod can tell us > > > which > > > tag/branch will be immutable. Vinod? > > Please use branch topic/dw. I will

[PATCH net] drivers: net: Don't print unpopulated net_device name

2016-05-10 Thread Harvey Hunt
For ethernet devices, net_device.name will be eth%d before register_netdev() is called. Don't print the net_device name until the format string is replaced. Cc: Robert Jarzmik Cc: Barry Song Cc: Marcel Ziswiler Cc: net...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Harvey H

Re: workqueue: race in mod_delayed_work_on?

2016-05-10 Thread Tejun Heo
Hello, On Tue, May 10, 2016 at 07:28:08PM +0300, Konstantin Khlebnikov wrote: > On 10.05.2016 11:21, Konstantin Khlebnikov wrote: > >I've got plenty warnings, bugs and oops around trivial use of > >mod_delayed_work in drivers/infiniband/core/addr.c > > Looks like problem in mod_delayed_work_on w

RE: [PATCH 2/3] perf/x86/mbm: Fix mbm counting for RMID reuse

2016-05-10 Thread Luck, Tony
>> (3) Also we may not want to count at every sched_in and sched_out >> because the MSR reads involve quite a bit of overhead. > > Every single other PMU driver just does this; why are you special? They just have to read a register. We have to write the IA32_EM_EVT_SEL MSR and then read fro

Re: [PATCH v9 7/7] vfio/type1: return MSI geometry through VFIO_IOMMU_GET_INFO capability chains

2016-05-10 Thread Eric Auger
Hi Alex, On 05/10/2016 12:49 AM, Alex Williamson wrote: > On Wed, 4 May 2016 11:54:18 + > Eric Auger wrote: > >> This patch allows the user-space to retrieve the MSI geometry. The >> implementation is based on capability chains, now also added to >> VFIO_IOMMU_GET_INFO. >> >> The returned in

[PATCH v4 3/6] ext4: Add alignment check for DAX mount

2016-05-10 Thread Toshi Kani
When a partition is not aligned by 4KB, mount -o dax succeeds, but any read/write access to the filesystem fails, except for metadata update. Call bdev_dax_supported() to perform proper precondition checks which includes this partition alignment check. Reported-by: Micah Parrish Signed-off-by: T

[PATCH v4 0/6] Add alignment check for DAX mount

2016-05-10 Thread Toshi Kani
When a partition is not aligned by 4KB, mount -o dax succeeds, but any read/write access to the filesystem fails, except for metadata update. Add alignment check to ext4, ext2, and xfs. - Patch 1-2 add bdev_dax_supported() which performs all the checks necessary for dax mount. - Patch 3-5 chang

Re: [PATCH v5 01/13] x86/xsaves: Define and use user_xstate_size for xstate size in signal context

2016-05-10 Thread Yu-cheng Yu
On Tue, May 10, 2016 at 06:29:00PM +0200, Borislav Petkov wrote: > Also, please snip the mail text you're quoting if you're not going to > refer to it. Like I just did. Ok :-)

Re: [PATCH] mmc: tegra: Disable UHS-I modes for tegra30

2016-05-10 Thread Stephen Warren
On 05/10/2016 10:13 AM, Jon Hunter wrote: On 09/05/16 16:15, Jon Hunter wrote: Support for SD cards is not working on the Tegra30 Beaver board and on boot the following error message is seen if an SD card is present: mmc0: error -110 whilst initialising SD card In addition to this, Tegra30

[PATCH v4 1/6] block: Add vfs_msg() interface

2016-05-10 Thread Toshi Kani
In preparation of moving DAX capability checks to the block layer from filesystem code, add a VFS message interface that aligns with filesystem's message format. For instance, a vfs_msg() message followed by XFS messages in case of a dax mount error may look like: VFS (pmem0p1): error: unaligne

[PATCH v4 4/6] ext2: Add alignment check for DAX mount

2016-05-10 Thread Toshi Kani
When a partition is not aligned by 4KB, mount -o dax succeeds, but any read/write access to the filesystem fails, except for metadata update. Call bdev_dax_supported() to perform proper precondition checks which includes this partition alignment check. Signed-off-by: Toshi Kani Reviewed-by: Jan

[PATCH v4 6/6] block: Update blkdev_dax_capable() for consistency

2016-05-10 Thread Toshi Kani
blkdev_dax_capable() is similar to bdev_dax_supported(), but needs to remain as a separate interface for checking dax capability of a raw block device. Rename and relocate blkdev_dax_capable() to keep them maintained consistently, and call bdev_direct_access() for the dax capability check. There

Re: workqueue: race in mod_delayed_work_on?

2016-05-10 Thread Konstantin Khlebnikov
On 10.05.2016 11:21, Konstantin Khlebnikov wrote: I've got plenty warnings, bugs and oops around trivial use of mod_delayed_work in drivers/infiniband/core/addr.c Looks like problem in mod_delayed_work_on was hidden because add_timer is equal to mod_timer but Sasha accidentally backported 874

[PATCH v4 5/6] xfs: Add alignment check for DAX mount

2016-05-10 Thread Toshi Kani
When a partition is not aligned by 4KB, mount -o dax succeeds, but any read/write access to the filesystem fails, except for metadata update. Call bdev_dax_supported() to perform proper precondition checks which includes this partition alignment check. Signed-off-by: Toshi Kani Reviewed-by: Chri

Re: [PATCH 1/3] Input - soc_button_array: use gpio_is_valid()

2016-05-10 Thread Benjamin Tissoires
On May 10 2016 or thereabouts, Fabio Estevam wrote: > On Tue, May 10, 2016 at 12:37 PM, Benjamin Tissoires > wrote: > > > @@ -92,7 +93,7 @@ soc_button_device_create(struct platform_device *pdev, > > continue; > > > > gpio = soc_button_lookup_gpio(&pdev->dev

[PATCH v4 2/6] block: Add bdev_dax_supported() for dax mount checks

2016-05-10 Thread Toshi Kani
DAX imposes additional requirements to a device. Add bdev_dax_supported() which performs all the precondition checks necessary for filesystem to mount the device with dax option. Also add a new check to verify if a partition is aligned by 4KB. When a partition is unaligned, any dax read/write acc

Re: Getting rid of dynamic TASK_SIZE (on x86, at least)

2016-05-10 Thread Cyrill Gorcunov
On Tue, May 10, 2016 at 09:07:49AM -0700, Andy Lutomirski wrote: > Hi all- > > I'm trying to get rid of x86's dynamic TASK_SIZE and just redefine it > to TASK_SIZE_MAX. So far, these are the TASK_SIZE users that actually > seem to care about the task in question: > > get_unmapped_area. This is

Re: [PATCH v2 00/23] ata: sata_dwc_460ex: make it working again

2016-05-10 Thread Tejun Heo
Hello, On Tue, May 10, 2016 at 11:34:40AM +0530, Vinod Koul wrote: > > slave-dma [1], branch topic/dw. But I think Vinod can tell us which > > tag/branch will be immutable. Vinod? > > Please use branch topic/dw. I will not rebase this before sending to Linus. Okay, pulled topic/dw into libata/fo

Re: [PATCH v5 01/13] x86/xsaves: Define and use user_xstate_size for xstate size in signal context

2016-05-10 Thread Borislav Petkov
On Tue, May 10, 2016 at 08:59:43AM -0700, Yu-cheng Yu wrote: > This is actually a patch of Fenghua's, but I re-based it. > I will fix it. Then please take a look at Documentation/SubmittingPatches too. Also, please snip the mail text you're quoting if you're not going to refer to it. Like I just

Re: [PATCH RESEND 05/12] sh: DeviceTree support update

2016-05-10 Thread Rich Felker
On Tue, May 10, 2016 at 05:25:36PM +0900, Yoshinori Sato wrote: > On Wed, 04 May 2016 12:10:05 +0900, > Rich Felker wrote: > > > > On Sun, May 01, 2016 at 02:08:29PM +0900, Yoshinori Sato wrote: > > > Changes bellow > > > - FDT setup timing fix. > > > - chosen/bootargs support. > > > - zImage supp

Re: [PART1 V5 07/13] KVM: x86: Detect and Initialize AVIC support

2016-05-10 Thread Borislav Petkov
On Tue, May 10, 2016 at 04:43:05PM +0200, Paolo Bonzini wrote: > That was actually our suggestion. Despite the long names, the lines do > not come out too long because the masks are only used for writing > fields, not very much for reading them (e.g. in complex "if" conditions). Sure but how can

[PATCH] x86/fsgsbase/64: Use TASK_SIZE_MAX for FSBASE/GSBASE upper limits

2016-05-10 Thread Andy Lutomirski
The GSBASE upper limit exists to prevent user code from confusing the paranoid idtentry path. The FSBASE upper limit is just for consistency. There's no need to enforce a smaller limit for 32-bit tasks. Just use TASK_SIZE_MAX. This simplifies the logic and will save a few bytes of code. Signed

Re: Using the mailbox subsystem for plain doorbells?

2016-05-10 Thread Jassi Brar
On Tue, May 10, 2016 at 9:26 PM, Stephen Warren wrote: > On 05/09/2016 09:29 PM, Jassi Brar wrote: > >> Some controllers need a mask/list of destination cpus, to which the >> irq is raised, written to some 'data' register. You too probably need >> to program the destination "id" in the controller?

Re: [PATCH] mmc: tegra: Disable UHS-I modes for tegra30

2016-05-10 Thread Jon Hunter
On 09/05/16 16:15, Jon Hunter wrote: > Support for SD cards is not working on the Tegra30 Beaver board and on > boot the following error message is seen if an SD card is present: > > mmc0: error -110 whilst initialising SD card > > In addition to this, Tegra30 is also randomly hanging during sy

Re: [PATCH BUGFIX] block: add missing group association in bio_split

2016-05-10 Thread Jeff Moyer
Paolo Valente writes: > When a bio is split, the newly created bio must be associated with the > same blkcg as the original bio (if BLK_CGROUP is enabled). If this > operation is not performed, then the new bio is not associated with > any group, and the group of the current task is returned when

Re: [PATCH v9 5/7] vfio/type1: also check IRQ remapping capability at msi domain

2016-05-10 Thread Eric Auger
Hi Alex, On 05/10/2016 12:49 AM, Alex Williamson wrote: > On Wed, 4 May 2016 11:54:16 + > Eric Auger wrote: > >> On x86 IRQ remapping is abstracted by the IOMMU. On ARM this is abstracted >> by the msi controller. vfio_safe_irq_domain allows to check whether >> interrupts are "safe" for a gi

Re: Is it really correct to check for breakpoint in kernel space against ptracer's address space?

2016-05-10 Thread Andy Lutomirski
On Tue, May 10, 2016 at 8:40 AM, Ruslan Kabatsayev wrote: > On Tue, May 10, 2016 at 6:23 PM, Andy Lutomirski wrote: >> Anyway, what kernel are you on? I think I fixed your specific issue in: >> >> commit 27747f8bc355a2808ca9e490ab6866acd85b4c16 >> Author: Andy Lutomirski >> Date: Thu Jul 30 2

[PATCH] tty: serial: msm: Disable restoring Rx interrupts for DMA Mode

2016-05-10 Thread Abhishek Sahu
From: Charanya The Data loss was happening with current QCOM MSM serial driver during large file transfer due to simultaneous enabling of both UART and DMA interrupt. When UART operates in DMA mode, RXLEV (Rx FIFO over watermark) or RXSTALE (stale interrupts) should not be enabled, since these co

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-10 Thread Eric Dumazet
On Tue, 2016-05-10 at 18:03 +0200, Paolo Abeni wrote: > If a single core host is under network flood, i.e. ksoftirqd is > scheduled and it eventually (after processing ~640 packets) will let the > user space process run. The latter will execute a syscall to receive a > packet, which will have to d

Re: [PATCH v1 1/2] dmaengine: slave means at least one of DMA_SLAVE, DMA_CYCLIC

2016-05-10 Thread Vinod Koul
On Tue, May 10, 2016 at 07:01:03PM +0300, Andy Shevchenko wrote: > On Tue, 2016-05-10 at 21:26 +0530, Vinod Koul wrote: > > On Wed, May 04, 2016 at 10:29:39PM +0300, Andy Shevchenko wrote: > > > > > > When check for capabilities recognize slave support by eother > > > DMA_SLAVE or > >

Getting rid of dynamic TASK_SIZE (on x86, at least)

2016-05-10 Thread Andy Lutomirski
Hi all- I'm trying to get rid of x86's dynamic TASK_SIZE and just redefine it to TASK_SIZE_MAX. So far, these are the TASK_SIZE users that actually seem to care about the task in question: get_unmapped_area. This is used by mmap, mremap, exec, uprobe XOL, and maybe some other things. - mmap,

Re: [PATCH v1 1/2] dmaengine: slave means at least one of DMA_SLAVE, DMA_CYCLIC

2016-05-10 Thread Andy Shevchenko
On Tue, 2016-05-10 at 21:26 +0530, Vinod Koul wrote: > On Wed, May 04, 2016 at 10:29:39PM +0300, Andy Shevchenko wrote: > > > > When check for capabilities recognize slave support by eother > > DMA_SLAVE or > ^^^ > typo Indeed, thanks. Do

Re: [PATCH v1 2/2] dmaengine: rename cmd_pause to cmd_suspend

2016-05-10 Thread Vinod Koul
On Tue, May 10, 2016 at 07:00:23PM +0300, Andy Shevchenko wrote: > On Tue, 2016-05-10 at 21:29 +0530, Vinod Koul wrote: > > On Wed, May 04, 2016 at 10:29:40PM +0300, Andy Shevchenko wrote: > > > > > > Rename cmd_pause to cmd_suspend to be clear that latter capability > > > reflects > > > pause AND

Re: [PATCH V3 2/4] net-next: mediatek: fix gigabit and flow control advertisement

2016-05-10 Thread David Miller
From: John Crispin Date: Tue, 10 May 2016 07:08:39 +0200 > @@ -236,7 +247,8 @@ static int mtk_phy_connect(struct mtk_mac *mac) > mac->phy_dev->autoneg = AUTONEG_ENABLE; > mac->phy_dev->speed = 0; > mac->phy_dev->duplex = 0; > - mac->phy_dev->supported &= PHY_BASIC_FEATURES;

[PATCH 2/3] drm/rockchip: fb: add missing header

2016-05-10 Thread John Keeping
This fixes the following sparse warnings: drivers/gpu/drm/rockchip/rockchip_drm_fb.c:32:23: warning: symbol 'rockchip_fb_get_gem_obj' was not declared. Should it be static? drivers/gpu/drm/rockchip/rockchip_drm_fb.c:315:24: warning: symbol 'rockchip_drm_framebuffer_init' was not declared. Should

[PATCH 3/3] drm/rockchip: fix "should it be static?" warnings

2016-05-10 Thread John Keeping
Combined with the previous commit, this fixes all of the sparse warnings in drm/rockchip. Signed-off-by: John Keeping --- drivers/gpu/drm/rockchip/rockchip_drm_drv.c | 4 ++-- drivers/gpu/drm/rockchip/rockchip_drm_fb.c | 6 +++--- drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 4 ++-- drivers/gp

RE: [PATCH v21 00/22] Richacls

2016-05-10 Thread Frank Filz
> On Tue, May 10, 2016 at 06:18:10AM +0200, Volker Lendecke wrote: > > On Tue, May 10, 2016 at 12:02:33AM +0200, Andreas Gruenbacher wrote: > > > What more can I do to finally get this merged? > > > > While I am not the one to comment on kernel specifics, from a pure > > Samba user space perspectiv

[PATCH 1/3] drm/rockchip: dw_hdmi: remove unused #include

2016-05-10 Thread John Keeping
drm_encoder_slave is not used in this file. Signed-off-by: John Keeping --- drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c b/drivers/gpu/drm/rockchip/dw_hdmi-rockchip.c index 801110f65a63..0665fb915579 1

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-10 Thread Paolo Abeni
Hi, On Tue, 2016-05-10 at 07:29 -0700, Eric Dumazet wrote: > On Tue, 2016-05-10 at 16:11 +0200, Paolo Abeni wrote: > > Currently, the softirq loop can be scheduled both inside the ksofirqd kernel > > thread and inside any running process. This makes nearly impossible for the > > process scheduler

Re: [PATCH v5 01/13] x86/xsaves: Define and use user_xstate_size for xstate size in signal context

2016-05-10 Thread Yu-cheng Yu
On Tue, May 10, 2016 at 01:04:41PM +0200, Borislav Petkov wrote: > On Mon, May 09, 2016 at 01:45:58PM -0700, Yu-cheng Yu wrote: > > If "xsaves" is enabled, kernel always uses compacted format of xsave area. > > But user space still uses standard format of xsave area. Thus, xstate size > > in kernel

[PATCH 0/3] drm/rockchip: fix some sparse warnings

2016-05-10 Thread John Keeping
The first patch wasn't found by sparse, but is something I noticed while in the area. The following two fix all of the sparse warnings inside drm/rockchip. John Keeping (3): drm/rockchip: dw_hdmi: remove unused #include drm/rockchip: fb: add missing header drm/rockchip: fix "should it be st

Re: [PATCH] xen/events: Don't move disabled irqs

2016-05-10 Thread Boris Ostrovsky
On 05/10/2016 11:11 AM, Ross Lagerwall wrote: > Commit ff1e22e7a638 ("xen/events: Mask a moving irq") open-coded > irq_move_irq() but left out checking if the IRQ is disabled. This broke > resuming from suspend since it tries to move a (disabled) irq without > holding the IRQ's desc->lock. Fix it b

Re: [PATCH v1 2/2] dmaengine: rename cmd_pause to cmd_suspend

2016-05-10 Thread Andy Shevchenko
On Tue, 2016-05-10 at 21:29 +0530, Vinod Koul wrote: > On Wed, May 04, 2016 at 10:29:40PM +0300, Andy Shevchenko wrote: > > > > Rename cmd_pause to cmd_suspend to be clear that latter capability > > reflects > > pause AND resume. > How does cmd_suspend be any clearer to reflect that channel is pau

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-10 Thread Thomas Gleixner
On Tue, 10 May 2016, Paolo Abeni wrote: Nice patch set and very promising results! > At this point we are not really sure if we should go with this simpler > approach by putting NAPI itself into kthreads or leverage the threadirqs > function by putting the whole interrupt into a thread and signa

Re: Using the mailbox subsystem for plain doorbells?

2016-05-10 Thread Stephen Warren
On 05/09/2016 09:29 PM, Jassi Brar wrote: On Tue, May 10, 2016 at 5:15 AM, Stephen Warren wrote: Jassi, Does the HW described below sound like something that should be represented using the Linux kernel's mailbox subsystem, and related DT bindings? I think the existing drivers/mailbox/pcc.c is

Re: [PATCH 2/2] dt-bindings: rockchip-dw-mshc: add rockchip,default-drv-phase

2016-05-10 Thread Doug Anderson
(again, but not HTML) On Tue, May 10, 2016 at 3:19 AM, Shawn Lin wrote: > 在 2016/5/10 0:31, Doug Anderson 写道: >> >> Hi, >> >> On Mon, May 9, 2016 at 4:12 AM, Shawn Lin >> wrote: 1. Specifying a single number for this property in terms of "degrees" is probably not right. The whole

Re: [PATCH 1/3] Input - soc_button_array: use gpio_is_valid()

2016-05-10 Thread Fabio Estevam
On Tue, May 10, 2016 at 12:37 PM, Benjamin Tissoires wrote: > @@ -92,7 +93,7 @@ soc_button_device_create(struct platform_device *pdev, > continue; > > gpio = soc_button_lookup_gpio(&pdev->dev, info->acpi_index); > - if (gpio < 0) > +

Re: [PATCH v1 2/2] dmaengine: rename cmd_pause to cmd_suspend

2016-05-10 Thread Vinod Koul
On Wed, May 04, 2016 at 10:29:40PM +0300, Andy Shevchenko wrote: > Rename cmd_pause to cmd_suspend to be clear that latter capability reflects > pause AND resume. How does cmd_suspend be any clearer to reflect that channel is paused and not. One can argue this might be related to power management

Re: [PATCH 3/3] blk-throttle: don't parse cgroup path if trace isn't enabled

2016-05-10 Thread Jeff Moyer
Shaohua Li writes: > if trace isn't enabled, parsing cgroup path just wastes cpu > > Signed-off-by: Shaohua Li > --- > block/blk-throttle.c | 5 ++--- > include/linux/blktrace_api.h | 9 + > 2 files changed, 11 insertions(+), 3 deletions(-) > > diff --git a/block/blk-throttle.c

[PATCH 1/1] Staging: comedi: comedi_fops.c: Fixed coding style issue

2016-05-10 Thread Shyam Saini
Fixed following checkpatch.pl warnings WARNING: Prefer WRITE_ONCE(, ) over ACCESS_ONCE() = WARNING: Prefer READ_ONCE() over ACCESS_ONCE() Signed-off-by: Shyam Saini --- drivers/staging/comedi/comedi_fops.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers

[PATCH] iommu/rockchip: fix zap cache during device attach

2016-05-10 Thread John Keeping
rk_iommu_command() takes a struct rk_iommu and iterates over the slave MMUs, so this is doubly wrong in that we're passing in the wrong pointer and talking to MMUs that we shouldn't be. Fixes: cd6438c5f844 ("iommu/rockchip: Reconstruct to support multi slaves") Signed-off-by: John Keeping --- dr

Re: [RFC PATCH 0/2] net: threadable napi poll loop

2016-05-10 Thread David Miller
From: Eric Dumazet Date: Tue, 10 May 2016 07:29:50 -0700 > We already have ksoftirqd to normally cope with the case you are > describing. > > If it is not working as intended, please identify the bugs and fix them, > instead of adding yet another tests in fast path and extra complexity in > the

Re: [PATCH v1 1/2] dmaengine: slave means at least one of DMA_SLAVE, DMA_CYCLIC

2016-05-10 Thread Vinod Koul
On Wed, May 04, 2016 at 10:29:39PM +0300, Andy Shevchenko wrote: > When check for capabilities recognize slave support by eother DMA_SLAVE or ^^^ typo > DMA_CYCLIC bit set. If we don't do that the user can't get a normally worked > DMA su

Re: [PATCH v2 0/3] input: rmi4: Regulator supply support

2016-05-10 Thread Bjorn Andersson
On Mon 09 May 17:36 PDT 2016, Andrew Duggan wrote: > Hi Bjorn, > > On 05/06/2016 09:40 PM, Bjorn Andersson wrote: > >The first version of the regulator support patch suffered from being > >implemented in the transport driver, as a work around for resource > >availability > >racing (EPROBE_DEFER

Re: mount --move and shared namespaces

2016-05-10 Thread Al Viro
On Mon, May 09, 2016 at 06:46:02PM -0400, Phillip Susi wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA512 > > It seems that mount --move does not work on a shared namespace ( now > the default under systemd ), yet you can mount --bind and then umount > the original, which seems to amount t

Re: [PATCH v7 2/2] memory: atmel-ebi: add DT bindings documentation

2016-05-10 Thread Boris Brezillon
On Tue, 10 May 2016 09:52:41 -0500 Rob Herring wrote: > On Tue, May 10, 2016 at 8:08 AM, Jean-Jacques Hiblot > wrote: > > 2016-05-10 14:41 GMT+02:00 Boris Brezillon > > : > >> On Tue, 10 May 2016 12:07:42 +0100 > >> Mark Rutland wrote: > >> > >>> On Tue, May 10, 2016 at 10:04:48AM +0200, B

Re: [RESEND PATCH 0/5] gcov fixes and maybe-uninitialized warnings

2016-05-10 Thread Arnd Bergmann
On Tuesday 10 May 2016 17:14:37 Michal Marek wrote: > On Mon, Apr 25, 2016 at 05:35:26PM +0200, Arnd Bergmann wrote: > > Hi Michal, > > > > This is a resend of a series I originally sent back in February, but > > unfortunately I never heard back from you. Could you apply these > > patches for v4.7

Re: [PATCH 2/3] blktrace: add missed mask name

2016-05-10 Thread Jeff Moyer
Shaohua Li writes: > BLK_TC_NOTIFY is missed in mask_maps, so we can't print out notify or > set mask with 'notify' name. For the text-based interface, yeah. Reviewed-by: Jeff Moyer > > Signed-off-by: Shaohua Li > --- > kernel/trace/blktrace.c | 1 + > 1 file changed, 1 insertion(+) > > dif

Re: [PATCH v1] builddeb: fix missing headers in linux-headers package

2016-05-10 Thread Michal Marek
On Thu, May 05, 2016 at 10:42:12AM -0400, Christopher Covington wrote: > On 03/31/2016 01:31 PM, Azriel Samson wrote: > > The kernel headers package (linux-headers) doesn't include > > header files from other architectures required to build > > out-of-tree modules. > > > > For e.g. on ARM64, opcod

[PATCH 1/3] Input - soc_button_array: use gpio_is_valid()

2016-05-10 Thread Benjamin Tissoires
gpio_keys will later use gpio_is_valid(). To match the actual behavior, we should use it here too. Signed-off-by: Benjamin Tissoires --- drivers/input/misc/soc_button_array.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/input/misc/soc_button_array.c b/drivers/in

Re: [PART1 V5 13/13] svm: Manage vcpu load/unload when enable AVIC

2016-05-10 Thread Paolo Bonzini
On 04/05/2016 21:09, Suravee Suthikulpanit wrote: > From: Suravee Suthikulpanit > > When a vcpu is loaded/unloaded to a physical core, we need to update > host physical APIC ID information in the Physical APIC-ID table > accordingly. > > Also, when vCPU is blocking/un-blocking (due to halt ins

Re: [PATCH] builddeb: include objtool binary in headers package

2016-05-10 Thread Michal Marek
On Wed, May 04, 2016 at 03:35:39PM +0200, Bjørn Mork wrote: > "objtool" is required for building external m dules if "Compile-time > stack metadata validation" is enabled. Otherwise all builds based > on the headers package fail with: > > make[1]: Entering directory '/usr/src/linux-headers-4.6.0

Re: Is it really correct to check for breakpoint in kernel space against ptracer's address space?

2016-05-10 Thread Ruslan Kabatsayev
On Tue, May 10, 2016 at 6:23 PM, Andy Lutomirski wrote: > Anyway, what kernel are you on? I think I fixed your specific issue in: > > commit 27747f8bc355a2808ca9e490ab6866acd85b4c16 > Author: Andy Lutomirski > Date: Thu Jul 30 20:32:42 2015 -0700 > > perf/x86/hw_breakpoints: Fix check for

Re: [Drbd-dev] [PATCH net-next v3] block/drbd: align properly u64 in nl messages

2016-05-10 Thread David Miller
From: Lars Ellenberg Date: Tue, 10 May 2016 11:40:23 +0200 > If we introduce a new config option, > we have to add it to the config scanner (one line), > define min, max, default and scale (four short defines), > and add it to the netlink definition here (one line). > Done, rest of the code is ge

Re: [PATCH v8 7/7] mm: kasan: Initial memory quarantine implementation

2016-05-10 Thread Andrey Ryabinin
2016-03-15 13:10 GMT+03:00 Alexander Potapenko : > > static inline int kasan_module_alloc(void *addr, size_t size) { return 0; } > static inline void kasan_free_shadow(const struct vm_struct *vm) {} > diff --git a/lib/test_kasan.c b/lib/test_kasan.c > index 82169fb..799c98e 100644 > --- a/lib/te

[PATCH 0/3] Input: soc_button_array fixes and question

2016-05-10 Thread Benjamin Tissoires
Hi, This series has been triggered by the Surface 3 I have been given. The way Microsoft follows its own specs is always intriguing. As written in drivers/platform/x86/surfacepro3_button.c, the PNP0C40 ACPI device which should follow the specification doesn't have any GPIO listed (thus the first 2

[PATCH 3/3] Input - soc_button_array: make sure one GPIO is not assigned twice

2016-05-10 Thread Benjamin Tissoires
The Surface 3 declares twice the GPIO as GpioInt and GpioIo in its ACPI table. Given that we do not keep the gpiod around, but the actual number associated to, there is a chance while enumerating the GPIOs that one gets assigned twice. Make sure a previous button has not been mapped already to the

[PATCH 2/3] Input - soc_button_array: bail out earlier if gpiod_count is null

2016-05-10 Thread Benjamin Tissoires
The PNP0C40 device of the Surface 3 doesn't have any GPIO attached to it. Instead of trying to access the GPIO, request the count beforehand and bail out if it is null or if an error is returned. Signed-off-by: Benjamin Tissoires --- drivers/input/misc/soc_button_array.c | 5 + 1 file change

Re: [PATCH v9 4/7] vfio: allow reserved msi iova registration

2016-05-10 Thread Eric Auger
Hi Alex, On 05/10/2016 05:29 PM, Alex Williamson wrote: > On Wed, 4 May 2016 11:54:15 + > Eric Auger wrote: > >> The user is allowed to register a reserved MSI IOVA range by using the >> DMA MAP API and setting the new flag: VFIO_DMA_MAP_FLAG_MSI_RESERVED_IOVA. >> This region is stored in th

Re: [PATCH v2] drm/vc4: Return -EBUSY if there's already a pending flip event.

2016-05-10 Thread Robert Foss
On 2016-05-03 03:22 PM, Eric Anholt wrote: robert.f...@collabora.com writes: From: Robert Foss As per the documentation in drm_crtc.h, atomic_commit should return -EBUSY if an asycnhronous update is requested and there is an earlier update pending. Note: docs cited here are drm_crtc.h, and

Re: [PATCH v9 4/7] vfio: allow reserved msi iova registration

2016-05-10 Thread Alex Williamson
On Wed, 4 May 2016 11:54:15 + Eric Auger wrote: > The user is allowed to register a reserved MSI IOVA range by using the > DMA MAP API and setting the new flag: VFIO_DMA_MAP_FLAG_MSI_RESERVED_IOVA. > This region is stored in the vfio_dma rb tree. At that point the iova > range is not mapped

[PATCH 02/11] irqdomain: Warn if we fail to set the IRQ type

2016-05-10 Thread Jon Hunter
When setting the IRQ type we don't check the return value to see if it is set correctly. Due to this, failures to set the IRQ type have gone unnoticed and because these failures were not catastrophic have not had an impact on the system. Ideally, we should return an error if we fail to set the typ

Re: [GIT PULL] overlayfs fixes for 4.6-rc7

2016-05-10 Thread Al Viro
On Tue, May 03, 2016 at 10:04:03PM +0200, Miklos Szeredi wrote: > Hi Al, > git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git > overlayfs-next > > This fixes two issues with overlayfs. > > Thanks, > Miklos > > --- > Miklos Szeredi (3): > vfs: rename: check backing inode bein

[PATCH 01/11] genirq: Ensure IRQ descriptor is valid when setting-up the IRQ

2016-05-10 Thread Jon Hunter
In the function, setup_irq(), we don't check that the descriptor returned from irq_to_desc() is valid before we start using it. For example chip_bus_lock() called from setup_irq(), assumes that the descriptor pointer is valid and doesn't check before dereferencing it. In many other functions inclu

[PATCH 04/11] irqchip/gic: Don't unnecessarily write the IRQ configuration

2016-05-10 Thread Jon Hunter
If the interrupt configuration matches the current configuration, then don't bother writing the configuration again. Signed-off-by: Jon Hunter Acked-by: Marc Zyngier --- drivers/irqchip/irq-gic-common.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/irqchip/

[PATCH 00/11] Various IRQ and GIC fixes and clean-ups

2016-05-10 Thread Jon Hunter
Most of these patches were originally part of the series to add support for the Tegra210 AGIC [0]. However, given that series has grown and some of the changes to the IRQ core needed some more review/testing, per an offline discussion with Marc Z, I have split out the more trivial fixes and clean-u

[PATCH 03/11] irqchip: Mask the non-type/sense bits when translating an IRQ

2016-05-10 Thread Jon Hunter
The firmware parameter that contains the IRQ sense bits may also contain other data. When return the IRQ type, bits outside of these sense bits should be masked. If these bits are not masked and irq_create_fwspec_mapping() is called to map an IRQ, then the comparison of the type returned from irq_d

Re: sched: tweak select_idle_sibling to look for idle threads

2016-05-10 Thread Mike Galbraith
On Tue, 2016-05-10 at 09:49 +0200, Mike Galbraith wrote: > Only whacking > cfs_rq_runnable_load_avg() with a rock makes schbench -m -t > -a work well. 'Course a rock in its gearbox also > rendered load balancing fairly busted for the general case :) Smaller rock doesn't injure heavy tbench, b

[PATCH 06/11] irqchip/gic: Don't initialise chip if mapping IO space fails

2016-05-10 Thread Jon Hunter
If we fail to map the address space for the GIC distributor or CPU interface, then don't attempt to initialise the chip, just WARN and return. Signed-off-by: Jon Hunter Acked-by: Marc Zyngier --- drivers/irqchip/irq-gic.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git

[PATCH 08/15] perf tools: Rewrite strbuf not to die()

2016-05-10 Thread Arnaldo Carvalho de Melo
From: Masami Hiramatsu Rewrite strbuf implementation not to use die() nor xrealloc(). Instead of die(), now most of the API returns error code or 0 if succeeded. Suggested-by: Arnaldo Carvalho de Melo Signed-off-by: Masami Hiramatsu Cc: Namhyung Kim Cc: Peter Zijlstra Link: http://lkml.kern

Re: Is it really correct to check for breakpoint in kernel space against ptracer's address space?

2016-05-10 Thread Andy Lutomirski
On Tue, May 10, 2016 at 4:50 AM, Ruslan Kabatsayev wrote: > On Tue, May 10, 2016 at 6:33 AM, Andy Lutomirski wrote: >> Is there any reason at all for TASK_SIZE to be different from TASK_SIZE_MAX? > > The distinction originally appears to have been introduced by commit 8492980. Hmm. Anyway, what

[PATCH 02/15] perf stat: Scale values by unit before metrics

2016-05-10 Thread Arnaldo Carvalho de Melo
From: Andi Kleen Scale values by unit before passing them to the metrics printing functions. This is needed for TopDown, because it needs to scale the slots correctly by pipeline width / SMTness. For existing metrics it shouldn't make any difference, as those generally use events that don't hav

Re: [PATCH 1/2] kselftests/ftrace: Detect tracefs mount point

2016-05-10 Thread Shuah Khan
On 05/10/2016 08:53 AM, Namhyung Kim wrote: > Currently ftracetest assumes tracing directory is located under > $DEBUGFS/tracing. But it's possible to mount tracefs directly without > debugfs. > > Signed-off-by: Namhyung Kim Looks good to me. Steve! Ack if you are good with this patch. I can ge

[PATCH 05/11] irqchip/gic: WARN if setting the interrupt type for a PPI fails

2016-05-10 Thread Jon Hunter
Setting the interrupt type for private peripheral interrupts (PPIs) may not be supported by a given GIC because it is IMPLEMENTATION DEFINED whether this is allowed. There is no way to know if setting the type is supported for a given GIC and so the value written is read back to verify it matches t

[PATCH 13/15] perf pmu: Make pmu_formats_string to check return value of strbuf

2016-05-10 Thread Arnaldo Carvalho de Melo
From: Masami Hiramatsu Make pmu_formats_string() to check return value of strbuf APIs so that it can detect errors in it. Signed-off-by: Masami Hiramatsu Cc: Namhyung Kim Cc: Peter Zijlstra Link: http://lkml.kernel.org/r/20160510054744.6158.37810.stgit@devbox Signed-off-by: Arnaldo Carvalho d

Re: [PATCH 2/2] kselftests/ftrace: Add a test case for event pid filtering

2016-05-10 Thread Shuah Khan
On 05/10/2016 08:53 AM, Namhyung Kim wrote: > Check event is filtered by set_event_pid and options/event-fork. > > Signed-off-by: Namhyung Kim Looks good to me. Steve! Ack if you are good with this patch. I can get this into 4.8-rc1 thanks, -- Shuah > --- > .../selftests/ftrace/test.d/event/e

[PATCH V7 01/11] PCI: Provide common functions for ECAM mapping

2016-05-10 Thread Tomasz Nowicki
From: Jayachandran C Add config option PCI_ECAM and file drivers/pci/ecam.c to provide generic functions for accessing memory mapped PCI config space. The API is defined in drivers/pci/ecam.h and is written to replace the API in drivers/pci/host/pci-host-common.h. The file defines a new 'struct

[PATCH 07/15] perf symbols: Fix handling of zero-length symbols.

2016-05-10 Thread Arnaldo Carvalho de Melo
From: Chris Phlipot This change introduces a fix to symbols__find, so that it is able to find symbols of length zero (where start == end). The current code has the following problem: - The current implementation of symbols__find is unable to find any symbols of length zero. - The db-export f

[PATCH V7 11/11] arm64, pci, acpi: Start using ACPI based PCI host controller driver for ARM64.

2016-05-10 Thread Tomasz Nowicki
It is perfectly fine to use ACPI_PCI_HOST_GENERIC for ARM64, so lets get rid of PCI init and RAW ACPI accessor empty stubs and go with full-blown PCI host controller driver. Signed-off-by: Tomasz Nowicki To: Catalin Marinas To: Lorenzo Pieralisi To: Will Deacon To: Arnd Bergmann --- arch/arm

[PATCH V7 06/11] pci, acpi: Provide a way to assign bus domain number.

2016-05-10 Thread Tomasz Nowicki
This patch provides a way to set the ACPI domain number in PCI code. pci_create_root_bus is called with NULL as parent in ACPI. This ends up calling pci_bus_assign_domain_nr with a NULL parent. So we define acpi_pci_bus_domain_nr() which is meant to retrieve the PCI domain number based on 'struct p

[PATCH V7 07/11] pci, acpi: Handle ACPI companion assignment.

2016-05-10 Thread Tomasz Nowicki
This patch provides a way to set the ACPI companion in PCI code. We define acpi_pci_set_companion() to set the ACPI companion pointer and call it from PCI core code. The function is stub for now. Signed-off-by: Jayachandran C Signed-off-by: Tomasz Nowicki --- drivers/pci/probe.c | 2 ++ in

[PATCH V7 04/11] pci: Add new function to unmap IO resources.

2016-05-10 Thread Tomasz Nowicki
It is very useful to release I/O resources so that the same I/O resources can be allocated again (pci_remap_iospace), like in PCI hotplug removal scenario. Therefore this patch implements new pci_unmap_iospace call which unmaps I/O space as the symmetry to pci_remap_iospace. Signed-off-by: Sinan K

[PATCH V7 09/11] arm64, pci, acpi: ACPI support for legacy IRQs parsing and consolidation with DT code.

2016-05-10 Thread Tomasz Nowicki
To enable PCI legacy IRQs on platforms booting with ACPI, arch code should include ACPI specific callbacks that parse and set-up the device IRQ number, equivalent to the DT boot path. Owing to the current ACPI core scan handlers implementation, ACPI PCI legacy IRQs bindings cannot be parsed at devi

[PATCH V7 03/11] pci, of: Move the PCI I/O space management to PCI core code.

2016-05-10 Thread Tomasz Nowicki
No functional changes in this patch. PCI I/O space mapping code does not depend on OF, therefore it can be moved to PCI core code. This way we will be able to use it e.g. in ACPI PCI code. Suggested-by: Lorenzo Pieralisi Signed-off-by: Tomasz Nowicki CC: Arnd Bergmann CC: Liviu Dudau --- dri

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