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

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 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

[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

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 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

[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 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 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 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

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

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 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

[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

[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 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

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

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: [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

[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: [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

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: [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 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: 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 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: [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

[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

[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

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

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 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: 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 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: [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: [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

[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: [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 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 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;

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 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 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: [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

[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: 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

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: [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] 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: 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?

[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: [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

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: [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 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: 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 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

[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

[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 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

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 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 :-)

[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

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

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: 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

[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: [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

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 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: 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/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

[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 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 -v2] x86/hweight: Get rid of the special calling convention

2016-05-10 Thread Borislav Petkov
From: Borislav Petkov Date: Wed, 4 May 2016 18:52:09 +0200 Subject: [PATCH -v2] x86/hweight: Get rid of the special calling convention People complained about ARCH_HWEIGHT_CFLAGS and how it throws a wrench into kcov, lto, etc, experimentation. And its not like we absolutely need it so let's get

Re: [PATCH v2 1/2] Documentation: DT: bindings: Add Broadcom STB PCIe bindings

2016-05-10 Thread Florian Fainelli
On 05/05/2016 02:15 PM, Arnd Bergmann wrote: >> +Required properties: >> +- compatible: must be one of: "brcm,bcm7425-pcie" >> + "brcm,bcm7435-pcie" >> + "brcm,bcm7445-pcie" >> + >> +- reg: specifies the physical base address of the controller regis

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

2016-05-10 Thread Paolo Bonzini
On 10/05/2016 18:24, Borislav Petkov wrote: > Sure but how can one even read that? > > AVIC_PHYSICAL_ID_ENTRY_HOST_PHYSICAL_ID_MASK > > AVIC's physical ID entry's host's physical ID's mask? > > That sucks in any language :-) If you read it backwards, that's the Mask for the hos

Re: [PATCH v5 02/13] x86/xsaves: Rename xstate_size to kernel_xstate_size to explicitly distinguish xstate size in kernel from user space

2016-05-10 Thread Borislav Petkov
On Mon, May 09, 2016 at 01:45:59PM -0700, Yu-cheng Yu wrote: > User space uses standard format xsave area. fpstate in signal frame should > have standard format size. > > To explicitly distinguish between xstate size in kernel space and the one > in user space, we rename xstate_size to kernel_xsta

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

2016-05-10 Thread Cyrill Gorcunov
On Tue, May 10, 2016 at 09:45:34AM -0700, Andy Lutomirski wrote: > 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_

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

2016-05-10 Thread Borislav Petkov
On Tue, May 10, 2016 at 07:00:43PM +0200, Paolo Bonzini wrote: > If you read it backwards, that's the > > Mask > for the host physical id > in entries of > the physical ID table > (an AVIC thing). Note to self: defines in arch/x86/kvm/ should be read backwards. > Quite a mout

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

2016-05-10 Thread Eric Anholt
Robert Foss writes: > 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.

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

2016-05-10 Thread Paolo
Il 10/05/2016 18:12, Jeff Moyer ha scritto: 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 gro

Re: [PATCH v5 02/13] x86/xsaves: Rename xstate_size to kernel_xstate_size to explicitly distinguish xstate size in kernel from user space

2016-05-10 Thread Dave Hansen
On 05/10/2016 10:01 AM, Borislav Petkov wrote: >> >pr_info("x86/fpu: Enabled xstate features 0x%llx, context size is %d >> > bytes, using '%s' format.\n", >> >xfeatures_mask, >> > - xstate_size, >> > + kernel_xstate_size, >> >cpu_has_xsaves ? "compacte

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

2016-05-10 Thread Jon Hunter
On 10/05/16 17:34, Stephen Warren wrote: > On 05/10/2016 10:13 AM, Jon Hunter wrote: [snip] >> Stephen, for your u-boot testing, do you are set the bit in the vendor >> misc register to enable version 3.0 support for sdhci on tegra30? This >> is what the above quirk is doing (and has done so for

Re: [PATCH v6 2/2] kvm: introduce KVM_MAX_VCPU_ID

2016-05-10 Thread Paolo Bonzini
On 09/05/2016 18:23, Cornelia Huck wrote: > On Mon, 09 May 2016 18:13:37 +0200 > Greg Kurz wrote: > >> The KVM_MAX_VCPUS define provides the maximum number of vCPUs per guest, and >> also the upper limit for vCPU ids. This is okay for all archs except PowerPC >> which can have higher ids, depen

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

2016-05-10 Thread Alexander Potapenko
On Tue, May 10, 2016 at 5:39 PM, Andrey Ryabinin wrote: > 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

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