Re: [PATCH] usb: dwc3: host: inherit dma configuration from parent dev

2016-09-02 Thread Arnd Bergmann
On Friday, September 2, 2016 12:55:33 PM CEST Robin Murphy wrote: > > Huh? There's only no DMA description in DT if the device can be assumed > to be happy with the defaults. Anything else should be using > "dma-ranges", "dma-coherent", etc. to describe non-default integration > aspects. For

[tip:smp/hotplug 8/8] ERROR: "__cpuhp_state_remove_instance" [drivers/net/virtio_net.ko] undefined!

2016-09-02 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git smp/hotplug head: 6dc465457fac9a49462bebbb870beea5d536b7c2 commit: 6dc465457fac9a49462bebbb870beea5d536b7c2 [8/8] net/virtio-net: Convert to hotplug state machine config: x86_64-allmodconfig (attached as .config) compiler:

Re: [PATCHv2 1/4] staging: android: ion: Drop heap type masks

2016-09-02 Thread Brian Starkey
Hi Laura, On Thu, Sep 01, 2016 at 03:40:41PM -0700, Laura Abbott wrote: There is no advantage to having heap types be a mask. The ion client has long since dropped the mask. Drop the notion of heap type masks as well. I know this is the same patch you sent last time, so sorry for not

Re: [PATCH] trivial treewide: Convert dev_set_uevent_suppress argument to bool

2016-09-02 Thread Bart Van Assche
On 09/01/16 17:51, Joe Perches wrote: > On Fri, 2016-09-02 at 00:47 +, Bart Van Assche wrote: >> On 09/01/16 13:11, Joe Perches wrote: >>> >>> Assigning an int to a bitfield:1 can lose precision. >>> Change the caller argument uses from 1/0 to true/false. >> Hello Joe, > > Hi Bart. > >> Can

[PATCH 1/5] irqtime: No need for preempt-safe accessors

2016-09-02 Thread Frederic Weisbecker
We can safely use the preempt-unsafe accessors for irqtime when we flush its counters to kcpustat as IRQs are disabled at this time. Cc: Rik van Riel Cc: Paolo Bonzini Cc: Wanpeng Li Cc: Mike Galbraith Cc: Peter

[PATCH 3/5] u64_stats: Introduce IRQs disabled helpers

2016-09-02 Thread Frederic Weisbecker
Introduce light versions of u64_stats helpers for context where either preempt or IRQs are disabled. This way we can make this library usable by scheduler irqtime accounting which currenty implement its ad-hoc version. Cc: Rik van Riel Cc: Paolo Bonzini Cc:

[PATCH 2/5] irqtime: Remove needless IRQs disablement on kcpustat update

2016-09-02 Thread Frederic Weisbecker
The callers of the functions performing irqtime kcpustat updates have IRQS disabled, no need to disable them again. Cc: Rik van Riel Cc: Paolo Bonzini Cc: Wanpeng Li Cc: Mike Galbraith Cc: Peter Zijlstra

Re: [PATCH 1/6] irqchip: mips-gic: Add context saving for MIPS_REMOTEPROC

2016-09-02 Thread Matt Redfearn
Hi Marc, Thanks for the review! On 02/09/16 11:54, Marc Zyngier wrote: Hi Matt, On 02/09/16 10:59, Matt Redfearn wrote: The MIPS remote processor driver allows non-Linux firmware to take control of and execute on one of the systems VPEs. If that VPE is brought back under Linux, it is

Re: [PATCH 3/4] KVM: x86: introduce get_kvmclock_ns

2016-09-02 Thread Roman Kagan
On Thu, Sep 01, 2016 at 05:26:14PM +0200, Paolo Bonzini wrote: > Introduce a function that reads the exact nanoseconds value that is > provided to the guest in kvmclock. This crystallizes the notion of > kvmclock as a thin veneer over a stable TSC, that the guest will > (hopefully) convert with

Re: [PATCH 3/5] u64_stats: Introduce IRQs disabled helpers

2016-09-02 Thread Paolo Bonzini
On 02/09/2016 16:03, Frederic Weisbecker wrote: > static inline unsigned int u64_stats_fetch_begin(const struct u64_stats_sync > *syncp) > { > -#if BITS_PER_LONG==32 && defined(CONFIG_SMP) > - return read_seqcount_begin(>seq); > -#else > -#if BITS_PER_LONG==32 > +#if BITS_PER_LONG==32 &&

[PATCH v2 2/2] crypto: qat - fix resource release omissions

2016-09-02 Thread Quentin Lambert
In certain cases qat_uclo_parse_uof_obj used to return with an error code before releasing all resources. This patch add a jump to the appropriate label ensuring that the resources are properly released before returning. This issue was found with Hector. Signed-off-by: Quentin Lambert

Re: [PATCH v3 2/4] dt-bindings: arm: Add Sierra Wireless modules bindings

2016-09-02 Thread Rob Herring
On Tue, Aug 23, 2016 at 01:39:04PM +0200, Neil Armstrong wrote: > Signed-off-by: Neil Armstrong > --- > Documentation/devicetree/bindings/arm/swir.txt | 12 > 1 file changed, 12 insertions(+) > create mode 100644

Re: [PATCH ] drivers/base: cacheinfo: remove warning in resume

2016-09-02 Thread Sudeep Holla
On 02/09/16 13:58, Sumit Gupta wrote: Hi Sudeep, Thank you for your comments. I understand the warning we get but the patch is completely wrong. One it removes the feature of adding/removing the cache devices on cpu hotplug events. Have you tested your patch with simple cpu hotplug and seen

RE: [tip:irq/urgent] genirq/msi: Make sure PCI MSIs are activated early

2016-09-02 Thread Bharat Kumar Gogada
Thanks Marc and Thomas for addressing the issue. > -Original Message- > From: tip tree robot [mailto:tip...@zytor.com] > Sent: Tuesday, August 09, 2016 12:59 PM > To: linux-tip-comm...@vger.kernel.org > Cc: Bharat Kumar Gogada ; bhelg...@google.com; > h...@zytor.com;

[PATCH] staging: rtl8712: fixed comment style and space indent

2016-09-02 Thread Louie Lu
fixed comment style and space indent report from checkpatch in rtl8712/rtl871x_ioctl_linux.c: * WARNING: Statements should start on a tabstop * WARNING: Block comments use * on subsequent lines * WARNING: suspect code indent for conditional statements (16, 32) Signed-off-by: Louie Lu

Re: [PATHC v2 5/9] ima: on soft reboot, save the measurement list

2016-09-02 Thread Mimi Zohar
Hi Dave, On Thu, 2016-09-01 at 09:57 +0800, Dave Young wrote: > On 08/30/16 at 06:40pm, Mimi Zohar wrote: > > + * Called during kexec_file_load so that IMA can add a segment to the kexec > > + * image for the measurement list for the next kernel. > > + */ > > +void ima_add_kexec_buffer(struct

Re: [PATCH 24/26] dt-bindings: Document img,boston-clock binding

2016-09-02 Thread Paul Burton
On 02/09/16 13:54, Rob Herring wrote: > On Tue, Aug 30, 2016 at 04:53:01PM +0100, Paul Burton wrote: >> On 26/08/16 18:44, Stephen Boyd wrote: >>> On 08/26, Paul Burton wrote: diff --git a/Documentation/devicetree/bindings/clock/img,boston-clock.txt

Re: [PATCH 3/4] kvm: add stub for arch specific debugfs support

2016-09-02 Thread Paolo Bonzini
On 31/08/2016 19:05, Luiz Capitulino wrote: > kvm_arch_create_vm_debugfs() allows arch specific code to > create entries in the VM's directory in debugfs. x86 will > implement support for this in the next commit. > > Signed-off-by: Luiz Capitulino > --- >

Re: [PATCH 2/4] kvm: kvm_create_vm_debugfs(): cleanup on error

2016-09-02 Thread Paolo Bonzini
On 31/08/2016 19:05, Luiz Capitulino wrote: > Memory and debugfs entries are leaked on error. Fix it. > > Signed-off-by: Luiz Capitulino > --- > virt/kvm/kvm_main.c | 10 +++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/virt/kvm/kvm_main.c

Re: [PATCH V4] leds: trigger: Introduce an USB port trigger

2016-09-02 Thread Alan Stern
On Fri, 2 Sep 2016, Jacek Anaszewski wrote: > >>> I'm pretty sure noone ever planned to have more than 1 trigger > >>> assigned to a single LED. I just realized there will be a problem with > >>> proposed solution: sysfs files conflict. ... > >> Currently we support only triggers dedicated to

Re: [git pull] drm fixes for rc5

2016-09-02 Thread Linus Torvalds
On Thu, Sep 1, 2016 at 10:59 PM, Dave Airlie wrote: > > I've tried using a signed tag, let's see if works. Worked fine. But your email was once again marked as spam. Google hates you, and your email habits. Linus

[PATCH] serial: earlycon: Extend earlycon command line option to support 64-bit addresses

2016-09-02 Thread Alexander Sverdlin
earlycon implementation used "unsigned long" internally, but there are systems (ARM with LPAE) where sizeof(unsigned long) == 4 and uart is mapped beyond 4GiB address range. Switch to resource_size_t internally and replace obsoleted simple_strtoul() with kstrtoull(). Signed-off-by: Alexander

Re: [PATCH 2/2] ARM: dts: am57xx-beagle-x15: Add support for rev B1

2016-09-02 Thread Nishanth Menon
+ x15 list ( see https://patchwork.kernel.org/patch/9310617/) On 09/02/2016 08:52 AM, Robert Nelson wrote: On Fri, Sep 2, 2016 at 5:41 AM, Sekhar Nori wrote: + Robert Nelson On Friday 02 September 2016 02:36 PM, Nishanth Menon wrote: I understand that there are existing users

Re: [PATCH RESEND] genirq: Machine-parsable version of /proc/interrupts

2016-09-02 Thread Thomas Gleixner
On Tue, 26 Jul 2016, Craig Gallek wrote: > /* > * Core internal functions to deal with irq descriptors > @@ -92,6 +93,7 @@ struct irq_desc { > int parent_irq; > struct module *owner; > const char *name; > + struct kobject

Re: [PATCH v4 2/2] pci:aer: add support aer interrupt with none MSI/MSI-X/INTx mode

2016-09-02 Thread Rob Herring
On Wed, Aug 31, 2016 at 02:37:22PM +0800, Po Liu wrote: > On some platforms, root port doesn't support MSI/MSI-X/INTx in RC mode. > When chip support the aer interrupt with none MSI/MSI-X/INTx mode, > maybe there is interrupt line for aer pme etc. Search the interrupt > number in the fdt file.

Re: [PATCH v4 3/4] drm/i915: Use new CRC debugfs API

2016-09-02 Thread Emil Velikov
Hi Tomeu, IMHO it would be better to split out the refactoring into preparatory patch. It brings a minor change which (not 100% sure on that) should not cause issues but is worth pointing out. On 5 August 2016 at 11:45, Tomeu Vizoso wrote: > +static int

Re: [PATCH v2 4/4] pwm: sunxi: Add H3 support

2016-09-02 Thread Rob Herring
On Wed, Aug 31, 2016 at 05:25:20PM +0900, Milo Kim wrote: > H3 PWM controller has same register layout as sun4i driver, so it works > by adding H3 specific data. > > Cc: Thierry Reding > Cc: Rob Herring > Cc: Maxime Ripard

Re: [PATCH v5 3/4] tracing: Add trace_irqsoff tracepoints

2016-09-02 Thread Steven Rostedt
On Fri, 2 Sep 2016 18:07:30 +0530 Binoy Jayan wrote: > This work is based on work by Daniel Wagner. A few tracepoints are added > at the end of the critical section. With the hist trigger in place, the > hist trigger plots may be generated, with per-cpu breakdown of

Re: [PATCH 1/3] ALSA: timer: fix NULL pointer dereference in read()/ioctl() race

2016-09-02 Thread Takashi Iwai
On Fri, 02 Sep 2016 14:34:47 +0200, Vegard Nossum wrote: > > On 08/29/2016 09:14 AM, Vegard Nossum wrote: > > On 08/29/2016 09:02 AM, Takashi Iwai wrote: > >> On Mon, 29 Aug 2016 00:33:49 +0200, > >> Vegard Nossum wrote: > >>> @@ -1602,15 +1602,25 @@ static int snd_timer_user_tselect(struct file

Re: [Question] about patch: don't use [delayed_]work_pending()

2016-09-02 Thread Tejun Heo
Hello, On Fri, Sep 02, 2016 at 09:17:04AM +0800, qiaozhou wrote: > > > I don't know whether it's meaningful to still check pending work here, or > > > it's not suggested to use pm_qos_update_request in this early boot up > > > phase. > > > Could you help to share some opinions? (I can fix this

Re: [PATCH RESEND] genirq: Machine-parsable version of /proc/interrupts

2016-09-02 Thread Thomas Gleixner
On Wed, 10 Aug 2016, Craig Gallek wrote: > On Tue, Jul 26, 2016 at 10:25 AM, Craig Gallek wrote: > > From: Craig Gallek > > > > Add struct kobject to struct irq_desc to allow for easy export > > to sysfs. This allows for much simpler userspace-parsing

Re: [Patch v9] driver/clk/clk-si5338: Add common clock framework driver for si5338

2016-09-02 Thread Rob Herring
On Fri, Aug 26, 2016 at 02:45:49PM -0700, York Sun wrote: > From: York Sun > > SI5338 is a programmable clock generator. It has 4 sets of inputs, > PLL, multisynth and dividers to make 4 outputs. This driver splits > them into multiple clocks to comply with common clock

Re: [PART2 PATCH v7 00/12] iommu/AMD: Introduce IOMMU AVIC support

2016-09-02 Thread Joerg Roedel
Hi Paolo, On Fri, Sep 02, 2016 at 12:46:28PM +0200, Paolo Bonzini wrote: > Joerg, if there's no other issues, could you apply the first 9 patches > to a branch based on 4.8-rc1 or similar, so that I can pull it into the > KVM tree? Sure, I was actually waiting for your Acked-By to put all the

Re: [PATCH 2/5] irqtime: Remove needless IRQs disablement on kcpustat update

2016-09-02 Thread Paolo Bonzini
On 02/09/2016 16:03, Frederic Weisbecker wrote: > The callers of the functions performing irqtime kcpustat updates have > IRQS disabled, no need to disable them again. They do, but perhaps this should be annotated through some sparse magic. It's starting to be hairy, with the requirement

Re: [PATCH v10 1/2] printk: Make printk() completely async

2016-09-02 Thread Petr Mladek
On Fri 2016-09-02 16:58:08, Sergey Senozhatsky wrote: > On (09/01/16 10:58), Petr Mladek wrote: > > On Wed 2016-08-31 21:52:24, Sergey Senozhatsky wrote: > > > a console_unlock() doing > > > wake_up_process(printk_kthread) would make it better. > > > > I am not sure what you mean by this. > > I

Re: [PATCH 1/2] genirq: Generic chip: add irq_unmap_generic_chip

2016-09-02 Thread Thomas Gleixner
On Mon, 1 Aug 2016, Sebastian Frias wrote: > NOTE: While the proposed unmap() function attempts to undo as much things > as done by the map() function, I did not find a way to undo the following: > > a) irq_gc_init_mask_cache(gc, dgc->gc_flags) You can't undo that. Because that represents the

Re: [PATCH 12/18] arm64: ilp32: add sys_ilp32.c and a separate table (in entry.S) to use it

2016-09-02 Thread Yury Norov
On Fri, Sep 02, 2016 at 02:55:34PM +0200, Arnd Bergmann wrote: > On Friday, September 2, 2016 6:46:19 PM CEST Bamvor Jian Zhang wrote: > > diff --git a/arch/arm64/include/uapi/asm/unistd.h > > b/arch/arm64/include/uapi/asm/unistd.h > > index 043d17a..78bea1d 100644 > > ---

Re: [Patch v4 01/12] microblaze: irqchip: Move intc driver to irqchip

2016-09-02 Thread Jason Cooper
Hi Michal, Zubair, On Fri, Sep 02, 2016 at 12:27:54PM +0200, Michal Simek wrote: > On 2.9.2016 12:06, Zubair Lutfullah Kakakhel wrote: > > On 09/02/2016 07:25 AM, Michal Simek wrote: ... > >> Also there is another copy of this driver in the tree which was using > >> old ppc405 and ppc440 xilinx

Re: [PATCH v2 1/2] perf: add container identifier entry in perf sample data

2016-09-02 Thread Peter Zijlstra
On Fri, Sep 02, 2016 at 07:25:31PM +0530, Hari Bathini wrote: > >I'm thinking this value is mostly the same for tasks, just like COMM and > > I think so, too. Namespaces aren't changed that often for tasks... > > >MMAP. Could we therefore not emit (sideband) events whenever a task > >changes

Re: [PATCH 11/26] dt-bindings: Document mti,mips-cpc binding

2016-09-02 Thread Paul Burton
On 02/09/16 13:34, Rob Herring wrote: > On Fri, Aug 26, 2016 at 04:37:10PM +0100, Paul Burton wrote: >> Document a binding for the MIPS Cluster Power Controller (CPC) which >> simply allows the device tree to specify where the CPC registers should >> be mapped. >> >> Signed-off-by: Paul Burton

Re: [PATCH v3 03/22] usb: ulpi: Support device discovery via device properties

2016-09-02 Thread Heikki Krogerus
Hi, On Wed, Aug 31, 2016 at 05:40:17PM -0700, Stephen Boyd wrote: > @@ -174,14 +219,37 @@ static int ulpi_register(struct device *dev, struct > ulpi *ulpi) > ulpi->id.product = ulpi_read(ulpi, ULPI_PRODUCT_ID_LOW); > ulpi->id.product |= ulpi_read(ulpi, ULPI_PRODUCT_ID_HIGH) << 8; >

Re: [PATCH] usb: dwc3: host: inherit dma configuration from parent dev

2016-09-02 Thread Felipe Balbi
Hi, Felipe Balbi writes: > Hi, > > Russell King - ARM Linux writes: >> On Fri, Sep 02, 2016 at 12:43:39PM +0200, Arnd Bergmann wrote: >>> On Thursday, September 1, 2016 5:14:28 PM CEST Leo Li wrote: >>> > >>> > Hi Felipe and Arnd, >>> > >>> > It has

[PATCH] MIPS: Fix detection of unsupported highmem with cache aliases

2016-09-02 Thread Paul Burton
The paging_init() function contains code which detects that highmem is in use but unsupported due to dcache aliasing. However this code was ineffective because it was being run before the caches are probed, meaning that cpu_has_dc_aliases would always evaluate to false (unless a platform overrides

[PATCH] net: ti: cpmac: Fix compiler warning due to type confusion

2016-09-02 Thread Paul Burton
cpmac_start_xmit() used the max() macro on skb->len (an unsigned int) and ETH_ZLEN (a signed int literal). This led to the following compiler warning: In file included from include/linux/list.h:8:0, from include/linux/module.h:9, from

Re: [PATCH 2/2] arm64: dts: rockchip: add eMMC's power domain support for rk3399

2016-09-02 Thread Ziyuan Xu
Hi Ulf, On 2016年09月02日 18:24, Ulf Hansson wrote: On 1 September 2016 at 23:50, Doug Anderson wrote: >Hi, > >On Thu, Sep 1, 2016 at 6:45 AM, Ulf Hansson wrote: >>I was reading the discussion regarding this change and browsing the DT

[PATCH] net: lantiq_etop: Remove unused 'i' variable

2016-09-02 Thread Paul Burton
Commit e7f4dc3536a4 ("mdio: Move allocation of interrupts into core") removed the only use of the 'i' variable from ltq_etop_mdio_init() but left the variable declaration behind, leading to the following compiler warning: drivers/net/ethernet/lantiq_etop.c: In function 'ltq_etop_mdio_init':

Re: [PATCH 6/6] Documentation: devicetree: dwc2: Deprecate g-tx-fifo-size

2016-09-02 Thread Rob Herring
On Mon, Aug 29, 2016 at 01:39:03PM -0700, John Youn wrote: > This property is not needed because the periodic fifos are not > configurable. So it was incorrect to add this property in the first > place. > > Signed-off-by: John Youn > --- >

[PATCH 1/2][RESEND] crypto: qat - introduces a variable to handle error codes

2016-09-02 Thread Quentin Lambert
Most error code used to jump to a label that lead to a "return -EFAULT" statement. This patch introduces a variable that stores the error code so that other error branches can use the same label to exit. Signed-off-by: Quentin Lambert ---

[PATCH 0/2][RESEND] add omitted release in qat_common

2016-09-02 Thread Quentin Lambert
The first patch introduces a variable to handle different error codes and be able to reuse the same clean up code. The second add an omitted release by jumping to the clean code having set the returned value to the proper error code. --- drivers/crypto/qat/qat_common/qat_uclo.c | 16

[PATCH 2/2] crypto: qat - fix resource release omissions

2016-09-02 Thread Quentin Lambert
In certain cases qat_uclo_parse_uof_obj used to return with an error code before releasing all resources. This patch add a jump to the appropriate label ensuring that the resources are properly released before returning. This issue was found with Hector. Signed-off-by: Quentin Lambert

Re: [PATCH v4 2/4] drm: Add API for capturing frame CRCs

2016-09-02 Thread Emil Velikov
Hi Tomeu, On 5 August 2016 at 11:45, Tomeu Vizoso wrote: > +#ifdef CONFIG_DEBUG_FS > + spin_lock_init(>crc.lock); > + init_waitqueue_head(>crc.wq); > + crtc->crc.source = kstrdup("auto", GFP_KERNEL); Pedantic: kstrdup() can never fail ? > +#endif >

Re: [PATCH 2/2] ARM: dts: am57xx-beagle-x15: Add support for rev B1

2016-09-02 Thread Tony Lindgren
* Robert Nelson [160902 06:52]: > On Fri, Sep 2, 2016 at 5:41 AM, Sekhar Nori wrote: > > Instead, it seems to be easier for maintenance and safer overall if the > > older version has a file of its own which can be kept alone. > > > > Also, how about

Re: [PATCH V2 1/5] Documentation: Add support for TI System Control Interface (TI-SCI) protocol

2016-09-02 Thread Rob Herring
On Tue, Aug 30, 2016 at 08:06:43AM -0500, Nishanth Menon wrote: > Texas Instrument's System Control Interface (TI-SCI) Message Protocol > is used in Texas Instrument's System on Chip (SoC) such as those in > newer SoCs in the keystone processor family starting with K2G. > > This message protocol

[PATCH] MIPS: Malta: Cleanup DMA coherence #ifdefs

2016-09-02 Thread Paul Burton
DMA coherence is not user-selectable in Kconfig, and Malta selects CONFIG_DMA_MAYBE_COHERENT which in turn selects CONFIG_DMA_NONCOHERENT. Remove #ifdefs on CONFIG_DMA_COHERENT which is not set for Malta. This removes a significant amount of code from bonito_quirks_setup(), but the code is

Re: [PATCH 3/3] staging: rtl8712: fixed comment style and space indent

2016-09-02 Thread Greg KH
On Fri, Sep 02, 2016 at 09:04:29PM +0800, Louie Lu wrote: > 2016-09-02 20:55 GMT+08:00 Greg KH : > > On Fri, Sep 02, 2016 at 06:16:10PM +0800, Louie Lu wrote: > >> fixed comment style and space indent report from checkpatch: > >> > >> * WARNING: Statements should start

Re: [PATCH v5 2/4] tracing: Add cpu as a key field in histogram

2016-09-02 Thread Steven Rostedt
On Fri, 2 Sep 2016 18:07:29 +0530 Binoy Jayan wrote: > The field 'cpu' although part of the set of generic fields, is not made > part of the key fields when mentioned in the trigger command. This hack > suggested by Daniel marks it as one of the key fields and make it

Re: [PATCH] generic: Add the exception case checking routine for ppi interrupt

2016-09-02 Thread Thomas Gleixner
On Thu, 1 Sep 2016, Marc Zyngier wrote: > On 01/09/16 09:15, majun (F) wrote: > Well, this issue goes way beyond the hack you wanted to add to the > generic code, and it should probably be addressed in the GIC code > itself, as an implementation specific workaround. Without knowing the > details

Re: [PATCH v5 4/4] tracing: Histogram for delayed hrtimer offsets

2016-09-02 Thread Steven Rostedt
[ Added Cc to hrtimer maintainer ] On Fri, 2 Sep 2016 18:07:31 +0530 Binoy Jayan wrote: > Generate a histogram of the latencies of delayed timer offsets in > nanoseconds. It shows the latency captured due to a delayed timer expire > event. It happens for example when a

Re: [PATCH 2/2] sched/wait: avoid abort_exclusive_wait() in __wait_on_bit_lock()

2016-09-02 Thread Peter Zijlstra
On Fri, Sep 02, 2016 at 02:06:43PM +0200, Oleg Nesterov wrote: > Yes, I considered this option, but to me the addtional finish_wait() > looks simpler. its all relative, this stuff always makes my head hurt one way or the other ;-) > And, if you agree with this change I will try to change

Re: [PATCH 4/4] kvm: x86: export TSC offset to user-space

2016-09-02 Thread Stefan Hajnoczi
On Wed, Aug 31, 2016 at 01:05:45PM -0400, Luiz Capitulino wrote: > We need to retrieve a VM's TSC offset in order to use > the host's TSC to merge host and guest traces. This is > explained in detail in this thread: > > [Qemu-devel] [RFC] host and guest kernel trace merging >

Re: [PATCH v11 3/4] tee: add OP-TEE driver

2016-09-02 Thread Andrew F. Davis
On 09/02/2016 05:49 AM, Jens Wiklander wrote: > On Thu, Sep 01, 2016 at 01:06:04PM -0500, Andrew F. Davis wrote: >> On 09/01/2016 04:22 AM, Jens Wiklander wrote: >>> On Wed, Aug 31, 2016 at 11:40:20AM -0500, Andrew F. Davis wrote: On 08/31/2016 08:50 AM, Jens Wiklander wrote: > On Tue,

[PATCH 5/5] irqtime: Consolidate irqtime flushing code

2016-09-02 Thread Frederic Weisbecker
The code performing irqtime nsecs stats flushing to kcpustat is roughly the same for hardirq and softirq. So lets consolidate that common code. Cc: Rik van Riel Cc: Paolo Bonzini Cc: Wanpeng Li Cc: Mike Galbraith Cc:

Re: [PATCH] usb: dwc3: host: inherit dma configuration from parent dev

2016-09-02 Thread Alan Stern
On Fri, 2 Sep 2016, Felipe Balbi wrote: > Hi, > > Russell King - ARM Linux writes: > > On Fri, Sep 02, 2016 at 12:43:39PM +0200, Arnd Bergmann wrote: > >> On Thursday, September 1, 2016 5:14:28 PM CEST Leo Li wrote: > >> > > >> > Hi Felipe and Arnd, > >> > > >> > It has

Re: [PATCH] iscsi-target: fix spelling mistake "Unsolicitied" -> "Unsolicited"

2016-09-02 Thread Bart Van Assche
On 09/02/2016 07:32 AM, Colin King wrote: Trivial fix to spelling mistakes in pr_debug message and comments Reviewed-by: Bart Van Assche

Re: [PATCH 1/3] arm64: dts: rockchip: add the PCIe PHY for RK3399

2016-09-02 Thread Heiko Stübner
Am Freitag, 19. August 2016, 11:24:27 schrieb Shawn Lin: > This patch adds PCIe node for RK3399 to support > PCIe controller. > > Signed-off-by: Shawn Lin with both the pcie-controller and -phy now being in maintainer trees, I've applied all 3 dts patches to my dts64

RE: [PATCH] x86/AMD: Fix Socket ID for LLC topology for AMD Fam17h systems

2016-09-02 Thread Thomas Gleixner
On Fri, 2 Sep 2016, Ghannam, Yazen wrote: > > > The Socket ID is ApicId[bits] on Fam17h systems. > > > > > > Change substraction to logical AND when extracting socket_id > > > from c->apicid. > > > > So Fam17h will only ever have 2 sockets, right? > > > > This is the decoding of the ApicId

[PATCH 4/5] irqtime: Consolidate accounting synchronization with u64_stats API

2016-09-02 Thread Frederic Weisbecker
The irqtime accounting currently implement its own ad hoc implementation of u64_stats API. Lets rather consolidate it with the appropriate library. Cc: Rik van Riel Cc: Paolo Bonzini Cc: Wanpeng Li Cc: Mike Galbraith

[PATCH 0/5] sched/cputime: irqtime cleanups

2016-09-02 Thread Frederic Weisbecker
This series contains a few optimizations against irq disabling, and the rest is consolidation. git://git.kernel.org/pub/scm/linux/kernel/git/frederic/linux-dynticks.git sched/irqtime HEAD: 63024a0947091e0a20bafa33b685ae33c275 Thanks, Frederic --- Frederic Weisbecker (5):

[PATCH 0/2] add omitted release in qat_common

2016-09-02 Thread Quentin Lambert
The first patch introduces a variable to handle different error codes and be able to reuse the same clean up code. The second add an omitted release by jumping to the clean code having set the returned value to the proper error code. --- drivers/crypto/qat/qat_common/qat_uclo.c | 16

[PATCH 2/2] crypto: qat - fix resource release omissions

2016-09-02 Thread Quentin Lambert
This issue was found with Hector. Signed-off-by: Quentin Lambert --- drivers/crypto/qat/qat_common/qat_uclo.c |3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) --- a/drivers/crypto/qat/qat_common/qat_uclo.c +++ b/drivers/crypto/qat/qat_common/qat_uclo.c @@

[PATCH 1/2] crypto: qat - introduces a variable to handle error codes

2016-09-02 Thread Quentin Lambert
Most error code used to jump to a label that lead to a "return -EFAULT" statement. This patch introduces a variable that stores the error code so that other error branches can use the same label to exit. Signed-off-by: Quentin Lambert ---

Re: [PATCH 1/2] sdhci-of-arasan: Add device tree parameter fails-without-test-cd bit

2016-09-02 Thread Rob Herring
On Mon, Aug 29, 2016 at 06:20:56PM -0500, Zach Brown wrote: > The sdhci controller on xilinx zynq devices will not function unless > the CD bit is provided. http://www.xilinx.com/support/answers/61064.html > In cases where it is impossible to provide the CD bit in hardware, > setting the

Re: [writeback] 8bc4ad9498: INFO: suspicious RCU usage. ]

2016-09-02 Thread Tejun Heo
(cc'ing Paul, hi!) Hello, On Thu, Sep 01, 2016 at 02:13:34PM -0600, Jens Axboe wrote: > On 09/01/2016 04:21 AM, kernel test robot wrote: > > [7.323356] cdrom: Uniform CD-ROM driver Revision: 3.20 > > [7.334239] > > [7.337256] === > > [7.340532] [ INFO:

Re: [PATCH net-next 3/3] net: dsa: mv88e6xxx: make global2 code optional

2016-09-02 Thread Andrew Lunn
On Fri, Sep 02, 2016 at 08:08:19AM -0400, Vivien Didelot wrote: > Since not every chip has a Global2 set of registers, make its support > optional, in which case the related functions will return -EOPNOTSUPP. > > This also allows to reduce the size of the mv88e6xxx driver for devices > such as

Re: [PATCH v3 4/8] gpio: dt-bindings: Add documentation for Aspeed GPIO controllers

2016-09-02 Thread Rob Herring
On Tue, Aug 30, 2016 at 05:24:23PM +0930, Andrew Jeffery wrote: > Signed-off-by: Andrew Jeffery > Acked-by: Joel Stanley > --- > .../devicetree/bindings/gpio/gpio-aspeed.txt | 36 > ++ > 1 file changed, 36 insertions(+) > create mode

Re: [PATCH] fs/proc/kcore.c: Omit kernel text area for hardened usercopy feature

2016-09-02 Thread Andi Kleen
On Fri, Sep 02, 2016 at 02:25:45PM +0200, Jiri Olsa wrote: > One of the bullets for hardened usercopy feature is: > - object must not overlap with kernel text > > which is what we expose via /proc/kcore. We can hit > this check and crash the system very easily just by > reading the text area in

Re: [Patch v4 01/12] microblaze: irqchip: Move intc driver to irqchip

2016-09-02 Thread Marc Zyngier
On 02/09/16 11:06, Zubair Lutfullah Kakakhel wrote: > Hi, > > On 09/02/2016 07:25 AM, Michal Simek wrote: >> On 1.9.2016 18:50, Zubair Lutfullah Kakakhel wrote: >>> The Xilinx AXI Interrupt Controller IP block is used by the MIPS >>> based xilfpga platform. >>> >>> Move the interrupt controller

Re: [PATCH 1/4] KVM: x86: always fill in vcpu->arch.hv_clock

2016-09-02 Thread Paolo Bonzini
On 02/09/2016 15:31, Roman Kagan wrote: > > > > - if (!vcpu->pv_time_enabled) > > - return 0; > > Strictly speaking, you only need .hv_clock updated if either kvmclock or > tsc_ref_page is enabled, so you may want to still skip the calculations > otherwise. Yeah, but that's

Re: [PATCH 1/4] kvm: kvm_destroy_vm_debugfs(): check debugs_stat_data pointer

2016-09-02 Thread Paolo Bonzini
On 31/08/2016 19:05, Luiz Capitulino wrote: > This make it possible to call kvm_destroy_vm_debugfs() from > kvm_create_vm_debugfs() in error conditions. > > Signed-off-by: Luiz Capitulino > --- > virt/kvm/kvm_main.c | 8 +--- > 1 file changed, 5 insertions(+), 3

Re: [PATCH 2/2] ARM: dts: am57xx-beagle-x15: Add support for rev B1

2016-09-02 Thread Robert Nelson
On Fri, Sep 2, 2016 at 5:41 AM, Sekhar Nori wrote: > + Robert Nelson > > On Friday 02 September 2016 02:36 PM, Nishanth Menon wrote: > > I understand that there are existing users of A2 boards and so we simply > cannot remove support for those boards (at least yet). > > But given

Re: [PATCH] mm, thp: fix leaking mapped pte in __collapse_huge_page_swapin()

2016-09-02 Thread Rik van Riel
On Fri, 2016-09-02 at 15:44 +0300, Ebru Akagunduz wrote: > Currently, khugepaged does not let swapin, if there is no > enough young pages in a THP. The problem is when a THP does > not have enough young page, khugepaged leaks mapped ptes. > > This patch prohibits leaking mapped ptes. > >

Re: [RFC PATCH v2 05/20] x86: Add the Secure Memory Encryption cpu feature

2016-09-02 Thread Borislav Petkov
On Mon, Aug 22, 2016 at 05:36:22PM -0500, Tom Lendacky wrote: > Update the cpu features to include identifying and reporting on the > Secure Memory Encryption feature. > > Signed-off-by: Tom Lendacky > --- > arch/x86/include/asm/cpufeature.h|7 +-- >

[PATCH] ASoC: fsl_esai: fix spelling mistake "Transmition" -> "Transmission"

2016-09-02 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistakes in dev_dbg messages Signed-off-by: Colin Ian King --- sound/soc/fsl/fsl_esai.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/sound/soc/fsl/fsl_esai.c

Re: [PATCH 4/4] kvm: x86: export TSC offset to user-space

2016-09-02 Thread Steven Rostedt
On Fri, 2 Sep 2016 09:43:01 -0400 Stefan Hajnoczi wrote: > Can TSC offset changes occur at runtime? > > One example is vcpu hotplug where the tracing tool would need to fetch > the new vcpu's TSC offset after tracing has already started. > > Another example is if QEMU or

Re: [PATCH v4 1/3] nvmem: rockchip-efuse: update compatible strings for Rockchip efuse

2016-09-02 Thread Rob Herring
On Mon, Aug 29, 2016 at 02:50:08AM -0700, Finley Xiao wrote: > Rk3399-efuse is organized as 32bits by 32 one-time programmable electrical > fuses. The efuse of earlier SoCs are organized as 32bits by 8 one-time > programmable electrical fuses with random access interface. > > Add different device

Re: [PATCH 1/4] Documentation: mmc: sdhci-of-arasan: Add clk_syscon as an optional one

2016-09-02 Thread Rob Herring
On Mon, Aug 29, 2016 at 04:02:56PM +0800, Shawn Lin wrote: > We introduced soc-ctl-syscon to do several things, for instance, update > baseclk or update clkmul, etc. In odrder to access this physical block, > we need to explicitly enable its clock. Currently we don't control this > clock as we

Re: [PART2 PATCH v7 00/12] iommu/AMD: Introduce IOMMU AVIC support

2016-09-02 Thread Suravee Suthikulpanit
Thanks All. Please let me know if you need anything else from my side. Suravee On 9/2/16 21:05, Joerg Roedel wrote: Hi Paolo, On Fri, Sep 02, 2016 at 12:46:28PM +0200, Paolo Bonzini wrote: Joerg, if there's no other issues, could you apply the first 9 patches to a branch based on 4.8-rc1 or

Re: [PATCH 1/2] IB/hfi1: Fix a parameter of find_first_bit.

2016-09-02 Thread Doug Ledford
On 8/28/2016 2:06 AM, Leon Romanovsky wrote: > On Fri, Aug 26, 2016 at 03:34:48PM -0400, Doug Ledford wrote: >> On 8/26/2016 3:29 PM, Leon Romanovsky wrote: >>> On Fri, Aug 26, 2016 at 02:01:55PM -0400, Doug Ledford wrote: On 8/26/2016 9:35 AM, Doug Ledford wrote: > On 8/26/2016 12:49 AM,

[PATCH v2 0/2] add omitted release in qat_common

2016-09-02 Thread Quentin Lambert
The first patch introduces a variable to handle different error codes and be able to reuse the same clean up code. The second add an omitted release by jumping to the clean code having set the returned value to the proper error code. -changes since v1 I failed to send the first version properly

[PATCH v2 1/2] crypto: qat - introduces a variable to handle error codes

2016-09-02 Thread Quentin Lambert
Most error code used to jump to a label that lead to a "return -EFAULT" statement. This patch introduces a variable that stores the error code so that other error branches can use the same label to exit. Signed-off-by: Quentin Lambert ---

Re: [PATCH] Force processes to non-realtime before mm_exit

2016-09-02 Thread Thomas Gleixner
On Thu, 14 Jul 2016, Peter Zijlstra wrote: > On Fri, Jun 03, 2016 at 04:18:44PM -0700, Brian Silverman wrote: > > Without this, a realtime process which has called mlockall exiting > > causes large latencies for other realtime processes at the same or > > lower priorities. This seems like a fairly

Re: [PATCH v3 1/2] regulator: pwm: Add support for a fixed delay after duty cycle changes

2016-09-02 Thread Rob Herring
On Fri, Sep 02, 2016 at 10:11:51AM -0500, Rob Herring wrote: > On Tue, Aug 30, 2016 at 09:21:15PM -0700, Douglas Anderson wrote: > > From: Matthias Kaehlcke > > > > A change of the duty cycle doesn't necessarily cause an immediate switch > > to the target voltage. On many PWM

Re: [PATCH v3 1/2] regulator: pwm: Add support for a fixed delay after duty cycle changes

2016-09-02 Thread Rob Herring
On Tue, Aug 30, 2016 at 09:21:15PM -0700, Douglas Anderson wrote: > From: Matthias Kaehlcke > > A change of the duty cycle doesn't necessarily cause an immediate switch > to the target voltage. On many PWM regulators there is a fixed "settle > time" (irrespective of the jump

Re: [Question] about patch: don't use [delayed_]work_pending()

2016-09-02 Thread Tejun Heo
On Fri, Sep 02, 2016 at 09:50:07AM -0400, Tejun Heo wrote: > Hello, > > On Fri, Sep 02, 2016 at 09:17:04AM +0800, qiaozhou wrote: > > > > I don't know whether it's meaningful to still check pending work here, > > > > or > > > > it's not suggested to use pm_qos_update_request in this early boot

Re: Block-level access

2016-09-02 Thread Alex Austin
My access is almost purely sequential and primarily writing, so read-ahead doesn't help me. What's problematic with pread/pwrite is the lack of error channel from media errors. BSG looks very interesting. I'll look further into that today. On Thu, Sep 1, 2016 at 5:16 PM, Bart Van Assche

Re: [PATCH 1/3] Documentation: dt: Add TI-SCI PM Domains

2016-09-02 Thread Rob Herring
On Fri, Aug 19, 2016 at 06:56:51PM -0500, Nishanth Menon wrote: > From: Dave Gerlach > > Add a generic power domain implementation, TI SCI PM Domains, that > will hook into the genpd framework and allow each PD, which will be > created one per device, to be managed over the

[PATCH] iscsi-target: fix spelling mistake "Unsolicitied" -> "Unsolicited"

2016-09-02 Thread Colin King
From: Colin Ian King Trivial fix to spelling mistakes in pr_debug message and comments Signed-off-by: Colin Ian King --- drivers/target/iscsi/iscsi_target.c | 2 +- drivers/target/iscsi/iscsi_target_login.c | 4 ++-- 2 files changed, 3

Re: [PATCH 1/2] ARM: dts: am57xx-beagle-x15: Remove pinmux configurations

2016-09-02 Thread Nishanth Menon
On 09/02/2016 05:18 AM, Sekhar Nori wrote: On Friday 02 September 2016 02:35 PM, Nishanth Menon wrote: pinmuxing for DRA7x/AM57x family of processors need to be done in IO isolation as part of initial bootloader executed from SRAM. This is done as part of iodelay configuration sequence and is

[PATCH] MIPS: Malta: Fix IOCU disable switch read for MIPS64

2016-09-02 Thread Paul Burton
Malta boards used with CPU emulators feature a switch to disable use of an IOCU. Software has to check this switch & ignore any present IOCU if the switch is closed. The read used to do this was unsafe for 64 bit kernels, as it simply casted the address 0xbf403000 to a pointer & dereferenced it.

Re: [writeback] 8bc4ad9498: INFO: suspicious RCU usage. ]

2016-09-02 Thread Paul E. McKenney
On Fri, Sep 02, 2016 at 10:56:22AM -0400, Tejun Heo wrote: > (cc'ing Paul, hi!) > > Hello, > > On Thu, Sep 01, 2016 at 02:13:34PM -0600, Jens Axboe wrote: > > On 09/01/2016 04:21 AM, kernel test robot wrote: > > > [7.323356] cdrom: Uniform CD-ROM driver Revision: 3.20 > > > [7.334239] >

  1   2   3   4   5   6   7   8   9   10   >