Re: [PATCH v3 3/4] ARM64: dts: meson-gxbb: Add Meson GXBB PWM Controller nodes

2016-08-28 Thread Martin Blumenstingl
uot;; > + reg = <0x0 0x086c0 0x0 0x10>; > + #pwm-cells = <3>; > + status = "disabled"; > + }; I have successfully tested pwm_e on my meson-gxbb-vega-s95-meta clone (see [0] for the corresponding patches), thus: Tested-by: Martin Blumenstingl [0] https://github.com/xdarklight/linux/tree/meson-gxbb-integration-4.8-20160828

[PATCH 2/4] barrier.h: Move smp_mb__after_unlock_lock to barrier.h

2016-08-28 Thread Manfred Spraul
spin_unlock() + spin_lock() together do not form a full memory barrier: (everything initialized to 0) CPU1: a=1; spin_unlock(&b); spin_lock(&c); + smp_mb__after_unlock_lock(); r1=d; CPU2: d=1; smp_mb(); r2=a; Without the smp_mb__after_unlock_lock(), r1==0 && r2==0 would be possible

[PATCH][RFC v5] timekeeping: Ignore the bogus sleep time if pm_trace is enabled

2016-08-28 Thread Chen Yu
Previously we encountered some memory overflow issues due to the bogus sleep time brought by inconsistent rtc, which is triggered when pm_trace is enabled, and we have fixed it in recent kernel. However it's improper in the first place to call __timekeeping_inject_sleeptime() in case that pm_trace

Re: [PATCH v3 1/4] pwm: Add support for Meson PWM Controller

2016-08-28 Thread Martin Blumenstingl
5-meta clone, which uses pwm_e to generate the LPO clock for the wifi module (see [0] for the corresponding patches). So thank you Neil, and: Tested-by: Martin Blumenstingl [0] https://github.com/xdarklight/linux/tree/meson-gxbb-integration-4.8-20160828

Re: [PATCH][v8] PM / hibernate: Verify the consistent of e820 memory map by md5 value

2016-08-28 Thread Pavel Machek
On Mon 2016-08-29 00:35:40, Chen Yu wrote: > On some platforms, there is occasional panic triggered when trying to > resume from hibernation, a typical panic looks like: > > "BUG: unable to handle kernel paging request at 880085894000 > IP: [] load_image_lzo+0x8c2/0xe70" > > This is because e

[PATCH 1/1] ceph: do not modify fi->frag in need_reset_readdir()

2016-08-28 Thread Nicolas Iooss
Commit f3c4ebe65ea1 ("ceph: using hash value to compose dentry offset") modified "if (fpos_frag(new_pos) != fi->frag)" to "if (fi->frag |= fpos_frag(new_pos))" in need_reset_readdir(), thus replacing a comparison operator with an assignment one. This looks like a typo which is reported by clang wh

Re: [PATCH 1/3] toshiba_acpi: Cleanup variable declaration

2016-08-28 Thread Darren Hart
On Tue, Aug 16, 2016 at 12:06:15PM -0600, Azael Avalos wrote: > This patch moves all the multiple line variable declaration to a > single line declaration (except variables being initialized) > following the reverse tree order, to conform to the practices > of the kernel. Azael, Apologies for the

Re: [PATCH] qla2xxx: fix spelling mistake "retyring" -> "retrying"

2016-08-28 Thread Himanshu Madhani
On 8/28/16, 4:24 AM, "Colin King" wrote: >From: Colin Ian King > >Trivial fix to spelling mistakes in ql_dbg messages. > >Signed-off-by: Colin Ian King >--- > drivers/scsi/qla2xxx/qla_os.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > >diff --git a/drivers/scsi/qla2xxx/qla_os.

Re: [PATCH 2/3] toshiba_acpi: Change error checking logic from TCI functions

2016-08-28 Thread Darren Hart
On Tue, Aug 16, 2016 at 12:06:16PM -0600, Azael Avalos wrote: > Currently the success/error checking logic is intermixed, making the > code a bit cumbersome to understand. > > This patch changes the affected functions to first check for errors > and take appropriate actions, then check for the sup

undefined reference to `printk'

2016-08-28 Thread kbuild test robot
Hi Peter, It's probably a bug fix that unveils the link errors. tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: af56ff27eba54fceee5f5643e79bf6531f2e1739 commit: 63a3f603413ffe82ad775f2d62a5afff87fd94a0 timeconst.pl: Eliminate Perl warning date: 3 year

[PATCH 1/1] ARM: add printf format attribute to early_print()

2016-08-28 Thread Nicolas Iooss
Adding such an attribute is helpful to detect errors related to printf formats at compile-time. Signed-off-by: Nicolas Iooss --- arch/arm/include/asm/setup.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/include/asm/setup.h b/arch/arm/include/asm/setup.h index 3613

Re: [PATCH] toshiba-wmi: Fix loading the driver on non Toshiba laptops

2016-08-28 Thread Darren Hart
On Thu, Aug 25, 2016 at 12:50:55PM -0600, Azael Avalos wrote: > Bug 150611 uncovered that the WMI ID used by the toshiba-wmi driver > is not Toshiba specific, and as such, the driver was being loaded > on non Toshiba laptops too. > > This patch adds a DMI matching list checking for TOSHIBA as the

Re: Linux 4.8: Reported regressions as of Sunday, 2016-08-28

2016-08-28 Thread Linus Torvalds
On Sun, Aug 28, 2016 at 8:23 AM, Thorsten Leemhuis wrote: > > Desc: irqdomain: Don't set type when mapping an IRQ breaks nexus7 gpio buttons > Repo: 2016-07-30 https://marc.info/?l=linux-kernel&m=146985356305280&w=2 > Stat: 2016-08-12 https://marc.info/?l=linux-kernel&m=147093069326172&w=2 > Note:

Re: [PATCH 3/3] toshiba_acpi: Fix typo in *_cooling_method_set function

2016-08-28 Thread Darren Hart
On Tue, Aug 16, 2016 at 12:06:17PM -0600, Azael Avalos wrote: > This patch simply fixes a typo in the error string printed in such > function. > > Signed-off-by: Azael Avalos Thanks, queued to for-next. -- Darren Hart Intel Open Source Technology Center

Re: [PATCH] toshiba_acpi: Fix the kbd_backlight_mode_show printed value

2016-08-28 Thread Darren Hart
On Tue, Aug 16, 2016 at 12:07:55PM -0600, Azael Avalos wrote: > The mentioned function prints the value in decimal, but the > documentation and its parent function available_kbd_modes_show > are showing the value as hexadecimal. > > This patch simply changes the value printed from decimal to hex,

[PATCH 0/6] KVM: PPC: e500: Fine-tuning for two function implementations

2016-08-28 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 28 Aug 2016 19:01:02 +0200 Several update suggestions were taken into account from static source code analysis. Markus Elfring (6): Use kmalloc_array() in kvm_vcpu_ioctl_config_tlb() Less function calls in kvm_vcpu_ioctl_config_tlb() after error detection De

[PATCH 1/6] KVM: PPC: e500: Use kmalloc_array() in kvm_vcpu_ioctl_config_tlb()

2016-08-28 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 28 Aug 2016 16:30:07 +0200 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". This issue was detected by using the Coccinelle s

[PATCH 2/6] KVM: PPC: e500: Less function calls in kvm_vcpu_ioctl_config_tlb() after error detection

2016-08-28 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 28 Aug 2016 17:34:46 +0200 The kfree() function was called in two cases by the kvm_vcpu_ioctl_config_tlb() function during error handling even if the passed data structure element contained a null pointer. * Split a condition check for memory allocation failures.

Re: checkkpatch (in)sanity ?

2016-08-28 Thread Joe Perches
On Sat, 2016-08-27 at 22:47 -0400, Levin, Alexander wrote: > Would you agree that by default we shouldn't show anything that's > not an error/defect? Not particularly, no. > That doesn't deal with newlines people add to hide the 80 character stuff, > nor it > deals with the "harder to read" par

[PATCH 4/6] KVM: PPC: e500: Replace kzalloc() calls by kcalloc() in two functions

2016-08-28 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 28 Aug 2016 18:30:38 +0200 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kcalloc". Suggested-by: Paolo Bonzini This issue was detected

[PATCH 5/6] KVM: PPC: e500: Use kmalloc_array() in kvmppc_e500_tlb_init()

2016-08-28 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 28 Aug 2016 18:40:08 +0200 * A multiplication for the size determination of a memory allocation indicated that an array data structure should be processed. Thus use the corresponding function "kmalloc_array". * Replace the specification of a data structure by

[PATCH 6/6] KVM: PPC: e500: Rename jump labels in kvmppc_e500_tlb_init()

2016-08-28 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 28 Aug 2016 18:45:26 +0200 Adjust jump labels according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- arch/powerpc/kvm/e500_mmu.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/arch/powerpc/kvm/e500

[PATCH 3/6] KVM: PPC: e500: Delete an unnecessary initialisation in kvm_vcpu_ioctl_config_tlb()

2016-08-28 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 28 Aug 2016 17:37:10 +0200 The local variable "g2h_bitmap" will be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- arch/powerpc/kvm/e500_mmu.c | 2 +- 1 file changed, 1 insertion(

Re: constification and cocci / kernel build test robot ?

2016-08-28 Thread Joe Perches
On Sun, 2016-08-28 at 15:13 +0200, Julia Lawall wrote: > [Adding Kees, in case it's of interest] > Below is the list of types of top-level initialized structures and the > number that are const.  For quicker reading, here are some that are > sometimes const (numerator), but not always (denominator

[PATCH 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call

2016-08-28 Thread Nicolas Iooss
In sst_prepare_and_post_msg(), when a response is received in "block", the following code gets executed: *data = kzalloc(block->size, GFP_KERNEL); memcpy(data, (void *) block->data, block->size); The memcpy() call overwrites the content of the *data pointer instead of filling the newly-al

[PATCH 2/2] power: bq24735-charger: Assume not charging when charger is missing

2016-08-28 Thread Paul Kocialkowski
When the charger is missing (disconnected), it is safe to assume that the charger chip is no charging. This is especially relevant when a status GPIO is present and the charger is getting disconnected. bq24735_charger_is_charging will be triggered due to the interrupt then, it will attempt to read

[PATCH] i40e: avoid potential null pointer dereference when assigning len

2016-08-28 Thread Colin King
From: Colin Ian King There is a sanitcy check for desc being null in the first line of function i40evf_debug_aq. However, before that, aq_desc is cast from desc, and aq_desc is being dereferenced on the assignment of len, so this could be a potential null pointer deference. Fix this by moving t

[PATCH 4/4] ARM: tegra: nyan-blaze: Include compatible revisions for proper detection

2016-08-28 Thread Paul Kocialkowski
Depthcharge (the payload used with cros devices) will attempt to detect boards using their revision. This includes all the known revisions for the nyan-blaze board so that the dtb can be selected preferably. Signed-off-by: Paul Kocialkowski --- arch/arm/boot/dts/tegra124-nyan-blaze.dts | 8 +

[PATCH 1/4] ARM: tegra: nyan: Use proper IRQ type definitions

2016-08-28 Thread Paul Kocialkowski
This switches a few interrupt definitions that were using GPIO_ACTIVE_HIGH as IRQ type, which is invalid. Signed-off-by: Paul Kocialkowski --- arch/arm/boot/dts/tegra124-nyan.dtsi | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/arm/boot/dts/tegra124-nyan.dtsi b/arch

[PATCH 1/2] power: bq24735-charger: Request status GPIO with initial input setup

2016-08-28 Thread Paul Kocialkowski
This requests the status GPIO with initial input setup. it is required to read the GPIO status at probe time and thus correctly avoid sending i2c messages when AC is not plugged. When requesting the GPIO without initial input setup, it always reads 0 which causes probe to fail as it assumes the ch

[PATCH 3/4] ARM: tegra: nyan-big: Include compatible revisions for proper detection

2016-08-28 Thread Paul Kocialkowski
Depthcharge (the payload used with cros devices) will attempt to detect boards using their revision. This includes all the known revisions for the nyan-big board so that the dtb can be selected preferably. Signed-off-by: Paul Kocialkowski --- arch/arm/boot/dts/tegra124-nyan-big.dts | 7 ++-

[PATCH 2/4] ARM: tegra: nyan: Use external control for bq24735 charger

2016-08-28 Thread Paul Kocialkowski
Nyan boards come with an embedded controller that controls when to enable and disable the charge. Thus, it should not be left up to the kernel to handle that. Using the ti,external-control property allows specifying this use-case. Signed-off-by: Paul Kocialkowski --- arch/arm/boot/dts/tegra124-

drm: WARNING in ioremap_wc

2016-08-28 Thread Dmitry Vyukov
Hello, The following program triggers WARNING in ioremap_wc: [ cut here ] LoadPin: kernel-module denied obj="/memfd: (deleted)" pid=12061 cmdline="/tmp/syz-executor" WARNING: CPU: 1 PID: 12056 at arch/x86/mm/ioremap.c:121[< none >] __ioremap_caller+0x348/0x6b0 arch/

Re: [PATCH 5/6] KVM: PPC: e500: Use kmalloc_array() in kvmppc_e500_tlb_init()

2016-08-28 Thread Julia Lawall
On Sun, 28 Aug 2016, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sun, 28 Aug 2016 18:40:08 +0200 > > * A multiplication for the size determination of a memory allocation > indicated that an array data structure should be processed. > Thus use the corresponding function "kmalloc_

Re: constification and cocci / kernel build test robot ?

2016-08-28 Thread Julia Lawall
On Sun, 28 Aug 2016, Joe Perches wrote: > On Sun, 2016-08-28 at 15:13 +0200, Julia Lawall wrote: > > [Adding Kees, in case it's of interest] > > > Below is the list of types of top-level initialized structures and the > > number that are const.  For quicker reading, here are some that are > > so

Re: [PATCH 6/6] KVM: PPC: e500: Rename jump labels in kvmppc_e500_tlb_init()

2016-08-28 Thread Julia Lawall
On Sun, 28 Aug 2016, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sun, 28 Aug 2016 18:45:26 +0200 > > Adjust jump labels according to the current Linux coding style convention. > > Signed-off-by: Markus Elfring > --- > arch/powerpc/kvm/e500_mmu.c | 9 - > 1 file changed, 4

Misuses of ** ? (was Re: [PATCH 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call)

2016-08-28 Thread Joe Perches
On Sun, 2016-08-28 at 19:39 +0200, Nicolas Iooss wrote: > In sst_prepare_and_post_msg(), when a response is received in "block", > the following code gets executed: > > *data = kzalloc(block->size, GFP_KERNEL); > memcpy(data, (void *) block->data, block->size); Yuck, thanks. Julia, Dan,

Re: checkkpatch (in)sanity ?

2016-08-28 Thread Greg KH
On Sun, Aug 28, 2016 at 10:15:57AM -0700, Joe Perches wrote: > On Sat, 2016-08-27 at 22:47 -0400, Levin, Alexander wrote: > > > > By default you should only get the most critical warnings we have in the > > > > kernel like missing S-O-B or corrupt patch. > > > I don't think so, but if you do, add a

[PATCH] media: usb: dvb-usb: fix DIBUSB_MB usage of dib3000mc functions

2016-08-28 Thread Randy Dunlap
From: Randy Dunlap The problem is that this driver uses a "common" driver supplement which calls a few dib3000mc*() functions but that driver is not "select"ed by DVB_USB_DIBUSB_MB. We can fix the build errors by selecting DVB_DIB3000MC (at the expense of around 8 KB on x86_64) just to add a few

Re: [PATCH 2/4] barrier.h: Move smp_mb__after_unlock_lock to barrier.h

2016-08-28 Thread Manfred Spraul
On 08/28/2016 03:43 PM, Paul E. McKenney wrote: Without the smp_mb__after_unlock_lock(), other CPUs can observe the write to d without seeing the write to a. Signed-off-by: Manfred Spraul With the upgraded commit log, I am OK with the patch below. Done. However, others will probably want t

[PATCH 2/4 v3] spinlock.h: Move smp_mb__after_unlock_lock to spinlock.h

2016-08-28 Thread Manfred Spraul
v3: If smp_mb__after_unlock_lock() is in barrier.h, then for arm64, kernel/rcu/tree.c doesn't compile because barrier.h is not included in kernel/rcu/tree.c (v2 was: add example from Paul, something that can happen on real HW) spin_unlock() + spin_lock() together do not form a full memory barrier

perf: WARNING in perf_event_read

2016-08-28 Thread Dmitry Vyukov
Hello, I've got the following WARNING while running syzkaller fuzzer: [ cut here ] WARNING: CPU: 3 PID: 23874 at kernel/events/core.c:3554 CPU: 3 PID: 23874 Comm: syz-executor Not tainted 4.8.0-rc3+ #31 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/0

Re: [tip:perf/core] perf/core: Check return value of the perf_event_read() IPI

2016-08-28 Thread Jiri Olsa
On Mon, Aug 22, 2016 at 12:38:23PM +0200, Jiri Olsa wrote: > On Mon, Aug 22, 2016 at 10:29:32AM +0200, Jiri Olsa wrote: > > On Mon, Aug 22, 2016 at 09:17:37AM +0200, Jiri Olsa wrote: > > > On Sun, Aug 21, 2016 at 02:10:07PM +0200, Vegard Nossum wrote: > > > > > > SNIP > > > > > > > [] ? __fget+0x

Re: chipidea: udc: kernel panic in isr_setup_status_phase

2016-08-28 Thread Clemens Gruber
On Sat, Aug 27, 2016 at 01:21:52AM +0800, Peter Chen wrote: > The gadget triggers UI interrupt due to host sends packet. > > I really can't understand that, why host does not send bus reset > before sending packet (eg, GET_DESCRIPTOR)? It violates USB spec. > > Are you sure the first interrupt is

Re: [PATCH 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call

2016-08-28 Thread Joe Perches
On Sun, 2016-08-28 at 19:39 +0200, Nicolas Iooss wrote: > In sst_prepare_and_post_msg(), when a response is received in "block", > the following code gets executed: > > *data = kzalloc(block->size, GFP_KERNEL); > memcpy(data, (void *) block->data, block->size); > > The memcpy() call overw

[PATCH] x86/topology: Fallback to SMT level only once

2016-08-28 Thread Jiri Olsa
Jirka, Peter and Jean-Pierre reported performance drop on some cpus after making cpu offline and online again. The reason is the kernel logic that falls back to SMT level topology if more than one node is detected within CPU package. During the system boot this logic cuts out the DIE topology leve

Re: [PATCH] selftests/x86: fix spelling mistake "preseve" -> "preserve"

2016-08-28 Thread Andy Lutomirski
On Sun, Aug 28, 2016 at 3:51 AM, Colin King wrote: > From: Colin Ian King > > Trivial fix to spelling mistakes in printf messages. Acked-by: Andy Lutomirski

Re: [PATCH 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call

2016-08-28 Thread Nicolas Iooss
On 28/08/16 20:17, Joe Perches wrote: > On Sun, 2016-08-28 at 19:39 +0200, Nicolas Iooss wrote: >> In sst_prepare_and_post_msg(), when a response is received in "block", >> the following code gets executed: >> >> *data = kzalloc(block->size, GFP_KERNEL); >> memcpy(data, (void *) block->data

Re: [PATCH 1/5] IA64-IRQ: Use kmalloc_array() in sn_irq_lh_init()

2016-08-28 Thread Joe Perches
On Sun, 2016-08-28 at 11:28 +0200, Julia Lawall wrote: > I do think that there is some value in doing similar things in a uniform > way, using meaningful names, even if in a particular case it doesn't help > performance or reduce code size.  Even duplicating code could be OK if it > is not in a cr

sound: division by 0 in snd_hrtimer_callback

2016-08-28 Thread Dmitry Vyukov
Hello, The following program triggers divide error in snd_hrtimer_callback: divide error: [#1] SMP DEBUG_PAGEALLOC KASAN Dumping ftrace buffer: (ftrace buffer empty) Modules linked in: CPU: 3 PID: 17469 Comm: syz-executor Not tainted 4.8.0-rc3+ #33 Hardware name: QEMU Standard PC (i440FX

drm: GPF in drm_legacy_lock_free

2016-08-28 Thread Dmitry Vyukov
Hello, The following program trigger GPF in drm_legacy_lock_free: general protection fault: [#1] SMP DEBUG_PAGEALLOC KASAN Modules linked in: CPU: 2 PID: 3379 Comm: syz-executor Not tainted 4.8.0-rc3+ #35 Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Bochs 01/01/2011 task:

Re: [PATCH] tpm: fix a race condition in tpm2_unseal_trusted()

2016-08-28 Thread Jason Gunthorpe
On Sun, Aug 28, 2016 at 08:36:52AM +0200, Jarkko Sakkinen wrote: > > @@ -576,7 +576,8 @@ static int tpm2_load(struct tpm_chip *chip, > goto out; > } > > - rc = tpm_transmit_cmd(chip, buf.data, PAGE_SIZE, "loading blob"); > + rc = tpm_transmit_cmd(chip, buf.data, PAGE

Re: Misuses of ** ? (was Re: [PATCH 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call)

2016-08-28 Thread Nicolas Iooss
On 28/08/16 19:50, Joe Perches wrote: > On Sun, 2016-08-28 at 19:39 +0200, Nicolas Iooss wrote: >> In sst_prepare_and_post_msg(), when a response is received in "block", >> the following code gets executed: >> >> *data = kzalloc(block->size, GFP_KERNEL); >> memcpy(data, (void *) block->data

drm: GPF in drm_context_switch_complete

2016-08-28 Thread Dmitry Vyukov
Hello, The following program triggers GPF in drm_context_switch_complete: kasan: GPF could be caused by NULL-ptr deref or user memory access general protection fault: [#1] SMP DEBUG_PAGEALLOC KASAN Dumping ftrace buffer: (ftrace buffer empty) Modules linked in: CPU: 1 PID: 1965 Comm: syz-

[PATCH v2 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call

2016-08-28 Thread Nicolas Iooss
In sst_prepare_and_post_msg(), when a response is received in "block", the following code gets executed: *data = kzalloc(block->size, GFP_KERNEL); memcpy(data, (void *) block->data, block->size); The memcpy() call overwrites the content of the *data pointer instead of filling the newly-al

Re: sound: division by 0 in snd_hrtimer_callback

2016-08-28 Thread Vegard Nossum
On 28 August 2016 at 20:39, Dmitry Vyukov wrote: > Hello, > > The following program triggers divide error in snd_hrtimer_callback: > > divide error: [#1] SMP DEBUG_PAGEALLOC KASAN > Dumping ftrace buffer: >(ftrace buffer empty) > Modules linked in: > CPU: 3 PID: 17469 Comm: syz-executor N

Re: perf: WARNING in perf_event_read

2016-08-28 Thread Vegard Nossum
On 28 August 2016 at 20:03, Dmitry Vyukov wrote: > Hello, > > I've got the following WARNING while running syzkaller fuzzer: > > [ cut here ] > WARNING: CPU: 3 PID: 23874 at kernel/events/core.c:3554 Looks like the same as this, with patch from Jiri Olsa: https://lkml.org

Re: Misuses of ** ? (was Re: [PATCH 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call)

2016-08-28 Thread Julia Lawall
On Sun, 28 Aug 2016, Nicolas Iooss wrote: > On 28/08/16 19:50, Joe Perches wrote: > > On Sun, 2016-08-28 at 19:39 +0200, Nicolas Iooss wrote: > >> In sst_prepare_and_post_msg(), when a response is received in "block", > >> the following code gets executed: > >> > >> *data = kzalloc(block->si

Re: [Ksummit-discuss] checkkpatch (in)sanity ?

2016-08-28 Thread Joe Perches
On Sun, 2016-08-28 at 11:59 +0200, Julia Lawall wrote: > On Sun, 28 Aug 2016, Alexey Dobriyan wrote: [] > > The problem is that c-h.pl generates noise in the commit history and > > makes git-blame less useful than it can be. > > Could it be that this is a problem with git blame, rather than with >

PROBLEM: DWC3 USB 3.0 not working on Odroid-XU4 with Exynos 5422

2016-08-28 Thread Michael Niewöhner
[1.] One line summary of the problem: DWC3 USB 3.0 not working on Odroid-XU4 with Exynos 5422 [2.] Full description of the problem/report: No usb 3.0 devices are being detected when attached while USB 2.0 devices work on the same port. USB 3.0 works after applying patches [9.1] and [9.2], but see

Re: [RFC 1/3] x86/vdso: create vdso file, use it for mapping

2016-08-28 Thread Cyrill Gorcunov
On Thu, Aug 25, 2016 at 06:21:08PM +0300, Dmitry Safonov wrote: > I added here a new in-kernel fs with ramfs-like options. > Created vdso file in this fs (yet for testing, only 64-bit vdso). > Mapped this file to process's mm on setup_additional_pages. > Just for testing purpose it's done only for

Re: [PATCH v2] arch: all: include: asm: bitops: Use bool instead of int for all bit test functions

2016-08-28 Thread kbuild test robot
ons/20160828-230301 config: m68k-allmodconfig (attached as .config) compiler: m68k-linux-gcc (GCC) 4.9.0 reproduce: wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config

Re: Misuses of ** ? (was Re: [PATCH 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call)

2016-08-28 Thread Joe Perches
On Sun, 2016-08-28 at 21:38 +0200, Julia Lawall wrote: > On Sun, 28 Aug 2016, Nicolas Iooss wrote: > > On 28/08/16 19:50, Joe Perches wrote: > > > On Sun, 2016-08-28 at 19:39 +0200, Nicolas Iooss wrote: > > >> In sst_prepare_and_post_msg(), when a response is received in "block", > > >> the followi

[PATCH] scsi: constify sr_pm_ops structure

2016-08-28 Thread Julia Lawall
sr_pm_ops, of type struct dev_pm_ops, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/scsi/sr.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/sr.c b/drivers/scsi/sr.c index ed17934..bed2b

Re: [Ksummit-discuss] checkkpatch (in)sanity ?

2016-08-28 Thread Jiri Kosina
On Sun, 28 Aug 2016, Joe Perches wrote: > Are there other existing tools for blame history viewing? fugitive vim plugin has 'Gblame' command, which I personally find rather useful, and given the text-oriented nature could potentially be useful for your needs. -- Jiri Kosina SUSE Labs

Re: [patch 2/2] i2c: mux: mellanox: add driver

2016-08-28 Thread kbuild test robot
to record what (public, well-known) commit your patch series was built on] [Check https://git-scm.com/docs/git-format-patch for more information] url: https://github.com/0day-ci/linux/commits/vadimp-mellanox-com/i2c-add-master-driver-for-mellanox-systems/20160828-225625 base: ht

[PATCH] watchdog: iTCO_wdt: constify iTCO_wdt_pm structure

2016-08-28 Thread Julia Lawall
iTCO_wdt_pm, of type struct dev_pm_ops, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/watchdog/iTCO_wdt.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/watchdog/iTCO_wdt.c b/drivers/watchdog

[PATCH] nfsd: constify reply_cache_stats_operations structure

2016-08-28 Thread Julia Lawall
reply_cache_stats_operations, of type struct file_operations, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- fs/nfsd/nfsctl.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/nfsd/nfsctl.c b/fs/nfsd/nfsctl.c

[git pull] drm fixes for rc4 or 5

2016-08-28 Thread Dave Airlie
Hi, I probably missed -rc4 by minutes, but maybe pointless GPL enforcement debates are distracting you enough! Anyways, a bunch of fixes covering i915, amdgpu, one tegra and some core DRM ones. Nothing too strange at this point. Dave. The following changes since commit fa8410b355251fd30341662

Re: ARM,SoC: About the use DT-defined properties by 3rd-party drivers

2016-08-28 Thread Timur Tabi
On Wed, Aug 24, 2016 at 9:29 AM, Sebastian Frias wrote: > > If this is really not possible, it forces the SoC manufacturer to expose > those properties in a different way, thus wasting a (seemingly) perfectly > fine way of doing so: the DT and its documentation. When you submit a new driver upstr

[PATCH 02/13] perf/core: Extend perf_sample_regs_intr() to include perf_arch_regs update

2016-08-28 Thread Madhavan Srinivasan
Extend perf_sample_regs_intr() to support the updates needed for perf_arch_reg structure and perf_arch_regs_mask. Also add code to init the arch_regs_mask to zero incase of regs_user in perf_sample_regs_user(). Ideally this should be done in perf_sample_data_init, but due to commit 2565711fb7d7 ("p

[PATCH 00/13] Add support for perf_arch_regs

2016-08-28 Thread Madhavan Srinivasan
Patchset to extend PERF_SAMPLE_REGS_INTR to include platform specific PMU registers. Patchset applies cleanly on tip:perf/core branch It's a perennial request from hardware folks to be able to see the raw values of the pmu registers. Partly it's so that they can verify perf is doing what they wan

[PATCH 01/13] perf/core: Add perf_arch_regs and mask to perf_regs structure

2016-08-28 Thread Madhavan Srinivasan
It's a perennial request from hardware folks to be able to see the raw values of the pmu registers. Partly it's so that they can verify perf is doing what they want, and some of it is that they're interested in some of the more obscure info that isn't plumbed out through other perf interfaces. Ove

[PATCH 03/13] perf/core: Update perf_*_sample() to include perf_arch_regs

2016-08-28 Thread Madhavan Srinivasan
perf_prepare_sample is extended to include the perf_arch_regs_mask in the sample header size calculation. Update perf_output_sample() to dump the perf_arch_regs_mask to sample output. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Stephan

[PATCH 06/13] powerpc/perf: Add support for perf_arch_regs in powerpc

2016-08-28 Thread Madhavan Srinivasan
Patch defines struct perf_arch_regs{} for powerpc and update the per-cpu perf pmu structure to include perf_arch_regs bits. perf_arch_reg_value(), perf_get_arch_reg() and perf_get_arch_regs_mask() are implemented to return proper values for powerpc. Finally adds code to call the processor specific

[PATCH 09/13] powerpc/perf: Add support for perf_arch_regs for PPC970 processor

2016-08-28 Thread Madhavan Srinivasan
Add code to define support functions and registers mask for PPC970 processor. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Stephane Eranian Cc: Russell King Cc: Catalin Marinas Cc: Will Deacon Cc: Benjamin Herrenschmidt Cc: Michael

[PATCH 04/13] perf/core: Extend perf_output_sample_regs() to include perf_arch_regs

2016-08-28 Thread Madhavan Srinivasan
Extend perf_output_sample_regs() to take in perf_regs structure as a parameter instead of pt_regs. Add code to check for arch_regs_mask and dump the arch registers to the output sample. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Steph

[PATCH 05/13] powerpc/perf: Define enums for perf_arch_regs registers

2016-08-28 Thread Madhavan Srinivasan
Patch creates a perf_event_powerpc_arch_regs enum and macros to include some of the powerpc pmu registers. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Stephane Eranian Cc: Russell King Cc: Catalin Marinas Cc: Will Deacon Cc: Benjam

[PATCH 12/13] tool/perf: Add perf_arch_reg mask and arch_reg_names structure

2016-08-28 Thread Madhavan Srinivasan
Add arch_reg_names structure and define perf_arch_reg_name() function to aid the printing of arch_regs values. Also, extend regs_dump__printf() to include perf_arch_regs_mask to enable printing support. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Arnaldo Carvalho d

[PATCH 10/13] tool/perf: Add support for perf_arch_regs

2016-08-28 Thread Madhavan Srinivasan
Update the structure regs_dump with the arch_regs_mask variable. Update perf_evsel__parse_sample() and perf_event__sample_event_size() to include arch_regs_mask variable. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Stephane Eranian Cc

[PATCH 08/13] powerpc/perf: Add support for perf_arch_regs for newer Power processor

2016-08-28 Thread Madhavan Srinivasan
Add code to define support functions and registers mask for Power8 and later processor. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Stephane Eranian Cc: Russell King Cc: Catalin Marinas Cc: Will Deacon Cc: Benjamin Herrenschmidt C

[PATCH 07/13] powerpc/perf: Add support for perf_arch_regs for Power7 processor

2016-08-28 Thread Madhavan Srinivasan
Add code to define support functions and registers mask for Power7 processor. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Stephane Eranian Cc: Russell King Cc: Catalin Marinas Cc: Will Deacon Cc: Benjamin Herrenschmidt Cc: Michael

[PATCH 13/13] powerpc/perf: Add support to dump only arch_regs

2016-08-28 Thread Madhavan Srinivasan
perf tool provides us an option to selective dump intr_regs. Add arch_regs option to it. Cc: Thomas Gleixner Cc: Ingo Molnar Cc: Peter Zijlstra Cc: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Stephane Eranian Cc: Russell King Cc: Catalin Marinas Cc: Will Deacon Cc: Benjamin Herrenschmidt

[PATCH 11/13] tools/perf: Fix the mask in regs_dump__printf and print_sample_iregs

2016-08-28 Thread Madhavan Srinivasan
When decoding the perf_regs mask in regs_dump__printf(), we loop through the mask using find_first_bit and find_next_bit functions. "mask" is of type "u64", but sent as a "unsigned long *" to lib functions along with sizeof(). While the exisitng code works fine in most of the case, the logic is br

Re: [PATCH v2] raid6: fix the input of raid6 algorithm

2016-08-28 Thread H. Peter Anvin
On August 27, 2016 11:51:04 PM PDT, liuzhengy...@kylinos.cn wrote: >From: ZhengYuan Liu > >To test and choose the best algorithm for raid6, disks number >and disks data must be offered. These input depend on page >size and gfmul table at current time. It would cause the disk >number less than 4 wh

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

2016-08-28 Thread Rask Ingemann Lambertsen
On Sun, Aug 28, 2016 at 07:21:06PM +0800, Randy Li wrote: > + s5m8767: s5m8767_pmic@66 { > + compatible = "samsung,s5m8767-pmic"; > + reg = <0x66>; You shouldn't use underscores in node names. https://www.mail-archive.com/linux-kernel%40vger.kernel.org/msg1122967.html

Re: [PATCH 01/13] perf/core: Add perf_arch_regs and mask to perf_regs structure

2016-08-28 Thread kbuild test robot
Hi Madhavan, [auto build test ERROR on tip/perf/core] [also build test ERROR on v4.8-rc3 next-20160825] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] [Suggest to use git(>=2.9.0) format-patch --base= (or --base=auto for convenience) to record

Re: [Ksummit-discuss] checkkpatch (in)sanity ?

2016-08-28 Thread Dennis Kaarsemaker
On zo, 2016-08-28 at 12:52 -0700, Joe Perches wrote: > On Sun, 2016-08-28 at 11:59 +0200, Julia Lawall wrote: > > > > On Sun, 28 Aug 2016, Alexey Dobriyan wrote: > [] > > > > > > > > The problem is that c-h.pl generates noise in the commit history > > > and > > > makes git-blame less useful than

Re: [git pull] drm fixes for rc4 or 5

2016-08-28 Thread Linus Torvalds
On Sun, Aug 28, 2016 at 2:00 PM, Dave Airlie wrote: > > I probably missed -rc4 by minutes, but maybe pointless GPL enforcement > debates are distracting you enough! Hey, they must be good for _something_. Anyway, the real problem was that this was in my spam-box. Which I've learnt to check relig

[PATCH] staging: lustre: constify lmv_proc_target_fops structure

2016-08-28 Thread Julia Lawall
lmv_proc_target_fops, of type struct file_operations, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/staging/lustre/lustre/lmv/lmv_internal.h |2 +- drivers/staging/lustre/lustre/lmv/lproc_lmv.c|2 +- 2 files cha

Re: [git pull] drm fixes for rc4 or 5

2016-08-28 Thread Linus Torvalds
On Sun, Aug 28, 2016 at 2:31 PM, Linus Torvalds wrote: > > Anyway, the real problem was that this was in my spam-box. Which I've > learnt to check religiously, so I noticed almost immediately. Btw, on a totally unrelated issue: you make thes pull points tags (good), but they are just plain simple

Re: Misuses of ** ? (was Re: [PATCH 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call)

2016-08-28 Thread Julia Lawall
On Sun, 28 Aug 2016, Joe Perches wrote: > On Sun, 2016-08-28 at 21:38 +0200, Julia Lawall wrote: > > On Sun, 28 Aug 2016, Nicolas Iooss wrote: > > > On 28/08/16 19:50, Joe Perches wrote: > > > > On Sun, 2016-08-28 at 19:39 +0200, Nicolas Iooss wrote: > > > >> In sst_prepare_and_post_msg(), when

[PATCH] staging/lustre: constify sops structure

2016-08-28 Thread Julia Lawall
sops, of type struct seq_operations, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/staging/lustre/lustre/ptlrpc/lproc_ptlrpc.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lu

Re: [PATCH] iio: hid-sensors: avoid unused function warning

2016-08-28 Thread Srinivas Pandruvada
On Fri, 2016-08-26 at 17:29 +0200, Arnd Bergmann wrote: A small rework of the PM code in this driver introduced a harmless > warning when CONFIG_PM_SLEEP is not set: > > drivers/iio/common/hid-sensors/hid-sensor-trigger.c:212:12: error: > 'hid_sensor_resume' defined but not used [-Werror=unused-fu

Re: Misuses of ** ? (was Re: [PATCH 1/1] ASoC: Intel: Atom: add a missing star in a memcpy call)

2016-08-28 Thread Joe Perches
On Sun, 2016-08-28 at 23:40 +0200, Julia Lawall wrote: > On Sun, 28 Aug 2016, Joe Perches wrote: > > On Sun, 2016-08-28 at 21:38 +0200, Julia Lawall wrote: > > > On Sun, 28 Aug 2016, Nicolas Iooss wrote: > > > > On 28/08/16 19:50, Joe Perches wrote: > > > > > On Sun, 2016-08-28 at 19:39 +0200, Nico

[PATCH] staging: lustre: constify lprocfs_generic_fops structure

2016-08-28 Thread Julia Lawall
lprocfs_generic_fops, of type struct file_operations, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/staging/lustre/lustre/obdclass/lprocfs_status.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/driv

Re: [Ksummit-discuss] checkkpatch (in)sanity ?

2016-08-28 Thread Joe Perches
On Sun, 2016-08-28 at 23:24 +0200, Dennis Kaarsemaker wrote: > > There are some that want an ncurses only version of git blame > > that could use arrow-key style navigation for historical commit > > line-ranges. > > > > git gui blame kind of works, but it's not ncurses/text based. > > git-cola ki

Hello Beautiful

2016-08-28 Thread Jones
My name is Jones, from the US. I'm in Syria right now fighting ISIS. I want to get to know you better, if I may be so bold. I consider myself an easy-going man, and I am currently looking for a relationship in which I feel loved. Please tell me more about yourself, if you don't mind. Hope to he

[PATCH] staging: lustre: obdclass: constify obd_psdev_fops structure

2016-08-28 Thread Julia Lawall
obd_psdev_fops, of type struct file_operations, is never modified, so declare it as const. Done with the help of Coccinelle. Signed-off-by: Julia Lawall --- drivers/staging/lustre/lustre/obdclass/linux/linux-module.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/driver

Linux 4.8-rc4

2016-08-28 Thread Linus Torvalds
Another week, another -rc. Everything looks normal, and it's been a bit quieter than rc3 too, so hopefully we're well into the "it's calming down" phase. Although with the usual timing-related fluctuation (different maintainers stagger their pulls differently), it's hard to tell a trend yet. Rega

<    1   2   3   4   >