Re: [PATCH v2] usb: dwc3: gadget: Wait for end transfer complete before free irq

2016-10-14 Thread Baolin Wang
Hi Felipe, On 13 October 2016 at 21:34, Felipe Balbi wrote: > > > Hi, > > Baolin Wang writes: >>> Baolin Wang writes: >> Baolin Wang writes: >> I'm thinking this is a bug in configfs interface of Gadget API, not >> dwc3. The only reason for this to happen would be if we get

Re: [PATCH v2] i2c: i2c-piix4: several coding style fixes

2016-10-14 Thread Uwe Kleine-König
Hello, On Thu, Oct 13, 2016 at 11:39:01PM +0100, Carlos Palminha wrote: > @@ -196,7 +204,7 @@ static int piix4_setup(struct pci_dev *PIIX4_dev, > } else { > pci_read_config_word(PIIX4_dev, SMBBA, &piix4_smba); > piix4_smba &= 0xfff0; > - if(piix4_smba

Commit 'MIPS: Malta: Use syscon-reboot driver to reboot' in -next and changed reset behavior

2016-10-14 Thread Guenter Roeck
Hi Paul, with commit 'MIPS: Malta: Use syscon-reboot driver to reboot' in -next, the value written into the reset register is changed from 0x42 to 0x4d. Is this change on purpose, or a copy-and-paste error from the SEAD3 changes ? Reason for asking is that qemu only accepts a value of 0x42, whi

Re: [Xen-devel] [RFC KERNEL PATCH 0/2] Add Dom0 NVDIMM support for Xen

2016-10-14 Thread Haozhong Zhang
On 10/13/16 20:33 +0100, Andrew Cooper wrote: On 13/10/16 19:59, Dan Williams wrote: On Thu, Oct 13, 2016 at 9:01 AM, Andrew Cooper wrote: On 13/10/16 16:40, Dan Williams wrote: On Thu, Oct 13, 2016 at 2:08 AM, Jan Beulich wrote: [..] I think we can do the similar for Xen, like to lay anoth

Re: [PATCH 2/3] kvm: x86: do not use KVM_REQ_EVENT for APICv interrupt injection

2016-10-14 Thread Yang Zhang
On 2016/9/28 5:20, Paolo Bonzini wrote: Since bf9f6ac8d749 ("KVM: Update Posted-Interrupts Descriptor when vCPU is blocked", 2015-09-18) the posted interrupt descriptor is checked unconditionally for PIR.ON. Therefore we don't need KVM_REQ_EVENT to trigger the scan and, if NMIs or SMIs are not i

[PATCH 1/1] mm/percpu.c: append alignment sanity checkup to avoid memory leakage

2016-10-14 Thread zijun_hu
From: zijun_hu the percpu allocator only works well currently when allocates a power of 2 aligned area, but there aren't any hints for alignment requirement, so memory leakage maybe be caused if allocate other alignment areas the alignment must be a even at least since the LSB of a chunk->map el

pull-request: wireless-drivers 2016-10-14

2016-10-14 Thread Kalle Valo
Hi Dave, first wireless-drivers pull request for 4.9 and this time we have unusually many fixes even before -rc1 is released. Most important here are the wlcore and rtlwifi commits which fix critical regressions, otherwise smaller impact fixes and one new sdio id for ath6kl. Please let me know if

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Johannes Berg
On Thu, 2016-10-13 at 14:49 -0700, Andy Lutomirski wrote: >  > It's failing before that.  With CONFIG_VMAP_STACK=y, the stack may > not be physically contiguous and can't be used for DMA, so putting it > in a scatterlist is bogus in general, and the crypto code mostly > wants a scatterlist. I see,

[patch] irqchip/gicv3: fix a loop timeout

2016-10-14 Thread Dan Carpenter
We expect the loop to exit with "count" set to zero so let's switch it from a post-op to a pre-op count down. Otherwise we don't see the error message when this times out. Fixes: a2c225101234 ("irqchip: gic-v3: Refactor gic_enable_redist to support both enabling and disabling") Signed-off-by: Da

Re: [lkp] [x86] 784d5699ed: kmsg.ip_tables:no_symbol_version_for_copy_from_user

2016-10-14 Thread Mathieu OTHACEHE
Hi Michal, Sorry for late reply. My toolchain in arch linux gcc version 6.2.1 20160830. I used defconfig (x86_64), with CONFIG_MODVERSIONS=y. I'm understanding better what's happening here. Running: nm arch/x86/lib/clear_page_64.o gives, T clear_page 0050 T clear_pa

Re: [PATCH 3/3] KVM: x86: do not scan IRR twice on APICv vmentry

2016-10-14 Thread Yang Zhang
On 2016/9/28 5:20, Paolo Bonzini wrote: Calling apic_find_highest_irr results in IRR being scanned twice, once in vmx_sync_pir_from_irr and once in apic_search_irr. Change vcpu_enter_guest to use sync_pir_from_irr (with a new argument to trigger the RVI write), and let sync_pir_from_irr get the

[patch] power: supply: lp8788: remove an unneeded NULL check

2016-10-14 Thread Dan Carpenter
We checked that "pdata->chg_params" is non-NULL earlier in this function so when we add "i" to it, it's still non-NULL. Signed-off-by: Dan Carpenter diff --git a/drivers/power/supply/lp8788-charger.c b/drivers/power/supply/lp8788-charger.c index 7321b72..509e2b3 100644 --- a/drivers/power/suppl

[patch 1/2] IB/hns: Fix a couple pointer math bugs

2016-10-14 Thread Dan Carpenter
"wqe" is a void pointer so adding sizeof() works. The original code adds sizeof() multiplied by sizeof() so it doesn't work at all. Fixes: 9a4435375cd1 ('IB/hns: Add driver files for hns RoCE driver') Signed-off-by: Dan Carpenter diff --git a/drivers/infiniband/hw/hns/hns_roce_hw_v1.c b/driver

Re: [PATCH 2/3] kvm: x86: do not use KVM_REQ_EVENT for APICv interrupt injection

2016-10-14 Thread Yang Zhang
On 2016/9/28 19:50, Paolo Bonzini wrote: On 28/09/2016 13:40, Wu, Feng wrote: IIUIC, the issue you describe above is that IPI for posted-interrupts may be issued between vcpu->mode = IN_GUEST_MODE; and local_irq_disable(); But if that really happens, we will call kvm_vcpu_kick() in vmx_del

Re: [PATCH v3 02/16] scsi: don't use fc_bsg_job::request and fc_bsg_job::reply directly

2016-10-14 Thread Johannes Thumshirn
On Thu, Oct 13, 2016 at 05:55:11PM +0200, Steffen Maier wrote: > Hm, still behaves for me like I reported for v2: > http://marc.info/?l=linux-scsi&m=147637177902937&w=2 Hi Steffen, Can you please try the following on top of 2/16? diff --git a/drivers/scsi/scsi_transport_fc.c b/drivers/scsi/scsi_

Re: [PATCH 32/41] staging: lustre: llite: restart short read/write for normal IO

2016-10-14 Thread Greg Kroah-Hartman
On Thu, Oct 13, 2016 at 11:45:28PM +0100, James Simmons wrote: > > > On Wed, Oct 12, 2016 at 12:22:35AM +0100, James Simmons wrote: > > > > > > > On Sun, Oct 02, 2016 at 10:28:28PM -0400, James Simmons wrote: > > > > > From: Bobi Jam > > > > > > > > > > If normal IO got short read/write, we'd r

Re: [PATCH v2] usb: dwc3: gadget: Wait for end transfer complete before free irq

2016-10-14 Thread Felipe Balbi
Hi, Baolin Wang writes: > I see what the problem is. Databook tells us we *must* set CMDIOC when > issuing EndTransfer command and we should always wait for Command > Complete IRQ. However, we took a shortcut and just delayed for 100us > after issuing End Transfer

[PATCH] brcmfmac: print name of connect status event

2016-10-14 Thread Rafał Miłecki
From: Rafał Miłecki This simplifies debugging. Format %s (%u) comes from similar debugging message in brcmf_fweh_event_worker. Signed-off-by: Rafał Miłecki --- drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c | 3 ++- drivers/net/wireless/broadcom/brcm80211/brcmfmac/fweh.c | 4 +

Re: [RFC 1/1] drivers: i2c: omap: Add slave support

2016-10-14 Thread Ravikumar
On Thursday 25 August 2016 10:44 PM, Wolfram Sang wrote: Hi, The omap i2c controller (at least on dra7x devices) doesn't have start/stop (STT/STP) support for slave mode so event #5 is not implemented in the driver. I think you can deduce that. If a new {READ|WRITE}_REQUESTED slave event co

Re: [PATCH 3/3] KVM: x86: do not scan IRR twice on APICv vmentry

2016-10-14 Thread Paolo Bonzini
On 14/10/2016 09:32, Yang Zhang wrote: >> >> -void __kvm_apic_update_irr(u32 *pir, void *regs) >> +int __kvm_apic_update_irr(u32 *pir, void *regs) >> { >> -u32 i, pir_val; >> +u32 i, vec; >> +u32 pir_val, irr_val; >> +int max_irr = -1; >> >> -for (i = 0; i <= 7; i++) { >> +

Re: [RFC 1/1] drivers: i2c: omap: Add slave support

2016-10-14 Thread Ravikumar
On Saturday 27 August 2016 07:29 PM, Matthijs van Duin wrote: Greetings, unfortunate souls trying to use the omap-i2c peripheral in slave mode! :-) That would be me :( and greetings to you too :) I recently posted some stuff about exactly that topic on TI's E2E forum, you may want to read th

Re: aarch64 ACPI boot regressed by commit 7ba5f605f3a0 ("arm64/numa: remove the limitation that cpu0 must bind to node0")

2016-10-14 Thread Andrew Jones
On Fri, Oct 14, 2016 at 12:50:29AM +0200, Laszlo Ersek wrote: > (4) Analysis (well, a lame attempt at that, because I have zero > familiarity with this code). Let me quote the patch: > > > commit 7ba5f605f3a0d9495aad539eeb8346d726dfc183 > > Author: Zhen Lei > > Date: Thu Sep 1 14:55:04 2016 +08

Re: [PATCH 2/3] kvm: x86: do not use KVM_REQ_EVENT for APICv interrupt injection

2016-10-14 Thread Paolo Bonzini
On 14/10/2016 09:37, Yang Zhang wrote: > Please ignore my previous reply. It seems you already aware the issue > and get the resolution to fix it.:-) Thanks anyway for confirming! The final version of the patches is on the way. Paolo

Re: [PATCH 2/3] sched/preempt: include asm/current.h

2016-10-14 Thread Heiko Carstens
On Fri, Oct 14, 2016 at 12:25:56AM +0100, Mark Rutland wrote: > Hi, > > On Thu, Oct 13, 2016 at 01:57:11PM +0200, Heiko Carstens wrote: > > The generic preempt code needs to include . Otherwise > > compilation fails if THREAD_INFO_IN_TASK is selected and the generic > > preempt code is used: > >

Re: [PATCH 02/18] [media] RedRat3: Move two assignments in redrat3_transmit_ir()

2016-10-14 Thread Dan Carpenter
I have asked you about six or seven times to only send bug fixes and stop sending clean up patches. You have refused. But now I'm asking you to stop randomly doing things without at least thinking about it for a bit. The original code was correct. regards, dan carpenter

Re: [PATCH 2/3] irqchip: mtk-cirq: Add mediatek mtk-cirq implement

2016-10-14 Thread Youlin Pei
Hi Marc, Thanks for your review. On Thu, 2016-10-13 at 09:28 +0100, Marc Zyngier wrote: > On Thu, 13 Oct 2016 13:06:33 +0800 > Youlin Pei wrote: > > > This commit add the mtk-cirq implement for mt2701. > Can you please expand this a bit? Okay, I will improve it as following: In Mediatek SOCs,

[PATCH V11 1/1] usb:serial: Add Fintek F81532/534 driver

2016-10-14 Thread Ji-Ze Hong (Peter Hong)
This driver is for Fintek F81532/F81534 USB to Serial Ports IC. F81532 spec: https://drive.google.com/file/d/0B8vRwwYO7aMFOTRRMmhWQVNvajQ/view?usp= sharing F81534 spec: https://drive.google.com/file/d/0B8vRwwYO7aMFV29pQWJqbVBNc00/view?usp= sharing Features: 1. F81532 is 1-to-2 & F81534 is 1-to-4

Re: [PATCH 1/3] binding: irqchip: mtk-cirq: Add binding document

2016-10-14 Thread Youlin Pei
Hi Matthias, Thanks for your review. On Thu, 2016-10-13 at 15:00 +0200, Matthias Brugger wrote: > > On 10/13/2016 07:06 AM, Youlin Pei wrote: > > This commit adds the device tree binding document for > > the mediatek cirq. > > > > Signed-off-by: Youlin Pei > > > > --- > > base on v4.8-rc1 > > --

Re: [v4.8-rc1 Regression] sched/fair: Apply more PELT fixes

2016-10-14 Thread Vincent Guittot
On 13 October 2016 at 23:34, Vincent Guittot wrote: > On 13 October 2016 at 20:49, Dietmar Eggemann > wrote: >> On 13/10/16 17:48, Vincent Guittot wrote: >>> On 13 October 2016 at 17:52, Joseph Salisbury >>> wrote: On 10/13/2016 06:58 AM, Vincent Guittot wrote: > Hi, > > On 12

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Johannes Berg
>    1. revert that patch (doing so would need some major adjustments now, >   since it's pretty old and a number of new things were added in the >   meantime) This it will have to be, I guess. >    2. allocate a per-CPU buffer for all the things that we put on the >   stack and use

[PATCH v4 00/12] syscall/tracing: compat syscall support

2016-10-14 Thread Marcin Nowakowski
This patchset adds support syscall event tracing for compat syscalls. Patch 1 removes the unnecessary syscall_nr field from syscall metadata, which was one of the obstacles for adding proper support for compat syscalls. Patch 2 adds a method to distinguish handling of syscalls for compat tasks if

[PATCH v4 01/12] tracing/syscalls: remove syscall_nr from syscall metadata

2016-10-14 Thread Marcin Nowakowski
Some architectures map multiple syscall numbers to a single syscall. This meant that on those platforms, some system calls could not be properly traced using syscall event tracing mechanism, as a different number of a syscall was used for registration to the one used by applications. We can use sys

[PATCH v4 11/12] sparc/tracing: fix compat syscall handling

2016-10-14 Thread Marcin Nowakowski
Add missing arch code - arch_trace_is_compat_syscall and arch_syscall_addr Signed-off-by: Marcin Nowakowski Cc: Steven Rostedt Cc: Ingo Molnar Cc: "David S. Miller" Cc: sparcli...@vger.kernel.org --- arch/sparc/include/asm/ftrace.h | 10 ++ arch/sparc/kernel/Makefile | 1 + arch

[PATCH v4 08/12] arm64/tracing: fix compat syscall handling

2016-10-14 Thread Marcin Nowakowski
Add arch_syscall_addr for arm64 and define NR_compat_syscalls, as the number of compat syscalls for arm64 exceeds the number defined by NR_syscalls. Convert the existing uses of __NR_compat_syscalls to the newly introduced NR_compat_syscalls Signed-off-by: Marcin Nowakowski Cc: Steven Rostedt Cc

Re: [PATCH v3 02/20] remoteproc: core: Add function to dump resource table

2016-10-14 Thread Matt Redfearn
Hi Loic, On 12/10/16 17:00, Loic Pallardy wrote: Firmware can be loaded with a resource table, which details resources needed by coprocessor like carevout memory, virtual device, trace log buffer etc. Until now, no method exists to display resource table content. This function adds the capabil

[PATCH v4 06/12] x86/tracing: fix compat syscall handling

2016-10-14 Thread Marcin Nowakowski
Adapt the arch-specific code to new syscall tracing interface: arch_trace_is_compat_syscall() now only indicates if a syscall is ia32, as x32 syscalls exist in the same syscall table as native 64 bit ones, so should not be treated as compat ones Add arch_trace_syscall_get_nr that removes the x32 bi

[PATCH v4 05/12] syscall/tracing: allow arch to override syscall_get_nr for ftrace

2016-10-14 Thread Marcin Nowakowski
Some architectures modify syscall numbers to indicate ABI used and as a result syscall number as returned by syscall_get_nr does not correspond to the syscall number used inside the kernel. Allow an arch to provide a separate implementation for ftrace that returns the 'real' syscall number from the

[PATCH v4 09/12] powerpc/tracing: fix compat syscall handling

2016-10-14 Thread Marcin Nowakowski
Adapt the code to make use of new syscall handling interface Signed-off-by: Marcin Nowakowski Cc: Steven Rostedt Cc: Ingo Molnar Cc: Benjamin Herrenschmidt Cc: Paul Mackerras Cc: Michael Ellerman Cc: linuxppc-...@lists.ozlabs.org --- arch/powerpc/include/asm/ftrace.h | 13 + arc

[PATCH v4 07/12] s390/tracing: fix compat syscall handling

2016-10-14 Thread Marcin Nowakowski
Add missing arch code - arch_trace_is_compat_syscall and arch_syscall_addr Signed-off-by: Marcin Nowakowski Cc: Steven Rostedt Cc: Ingo Molnar Cc: Martin Schwidefsky Cc: Heiko Carstens Cc: linux-s...@vger.kernel.org --- arch/s390/include/asm/ftrace.h | 11 +++ arch/s390/include/asm/

[PATCH v4 04/12] tracing/syscalls: add compat syscall metadata

2016-10-14 Thread Marcin Nowakowski
Now that compat syscalls are properly distinguished from native calls, we can add metadata for compat syscalls as well. All the macros used to generate the metadata are the same as for standard syscalls, but with a compat_ prefix to distinguish them easily. Signed-off-by: Marcin Nowakowski Cc: St

[PATCH v4 05/12] tracing/syscalls: allow arch to override syscall_get_nr for ftrace

2016-10-14 Thread Marcin Nowakowski
Some architectures modify syscall numbers to indicate ABI used and as a result syscall number as returned by syscall_get_nr does not correspond to the syscall number used inside the kernel. Allow an arch to provide a separate implementation for ftrace that returns the 'real' syscall number from the

[PATCH v4 12/12] parisc/tracing: fix compat syscall handling

2016-10-14 Thread Marcin Nowakowski
Add missing arch code - arch_trace_is_compat_syscall and arch_syscall_addr Signed-off-by: Marcin Nowakowski Cc: Steven Rostedt Cc: Ingo Molnar Cc: "James E.J. Bottomley" Cc: Helge Deller Cc: linux-par...@vger.kernel.org --- arch/parisc/include/asm/ftrace.h | 10 ++ arch/parisc/kernel

Re: [PATCH v2] mm: exclude isolated non-lru pages from NR_ISOLATED_ANON or NR_ISOLATED_FILE.

2016-10-14 Thread Ming Ling
On 四, 10月 13, 2016 at 10:09:37上午 +0200, Michal Hocko wrote: Hello, > On Thu 13-10-16 14:39:09, ming.ling wrote: > > From: Ming Ling > > > > Non-lru pages don't belong to any lru, so counting them to > > NR_ISOLATED_ANON or NR_ISOLATED_FILE doesn't make any sense. > > It may misguide functions suc

[PATCH v4 10/12] tile/tracing: fix compat syscall handling

2016-10-14 Thread Marcin Nowakowski
Add missing arch code - arch_trace_is_compat_syscall and arch_syscall_addr Signed-off-by: Marcin Nowakowski Cc: Steven Rostedt Cc: Ingo Molnar Signed-off-by: Chris Metcalf --- arch/tile/include/asm/ftrace.h | 10 ++ arch/tile/kernel/Makefile | 1 + arch/tile/kernel/ftrace.c

[PATCH v4 02/12] tracing/syscalls: add handling for compat tasks

2016-10-14 Thread Marcin Nowakowski
Extend the syscall tracing subsystem by adding a handler for compat tasks. For some architectures, where compat tasks' syscall numbers have an exclusive set of syscall numbers, this already works since the removal of syscall_nr. Architectures where the same syscall may use a different syscall numbe

[PATCH v4 03/12] tracing/syscalls: drop arch_syscall_match_sym_name

2016-10-14 Thread Marcin Nowakowski
arch_syscall_match_sym_name has been previously added to allow arch to override syscall_match_sym_name method, but only ppc arch implements it and the only difference made is that a prefix is longer. This can be simplified by using a #define specifying the prefix length difference for a given arch.

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Ard Biesheuvel
On 14 October 2016 at 09:28, Johannes Berg wrote: > >>1. revert that patch (doing so would need some major adjustments now, >> since it's pretty old and a number of new things were added in the >> meantime) > > This it will have to be, I guess. > >>2. allocate a per-CPU buffer

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Sergey Senozhatsky
On (10/13/16 14:49), Andy Lutomirski wrote: [..] > > > FAIL: 412cba02 > c900802cba02 || 1 -> (412cba02 > > > >> 39) == 130 > > > > Yeah, we already know that in this function the aad variable is on the > > stack, it explicitly is. > > > > The question, though, is why precisely

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Ard Biesheuvel
On 14 October 2016 at 09:39, Ard Biesheuvel wrote: > On 14 October 2016 at 09:28, Johannes Berg wrote: >> >>>1. revert that patch (doing so would need some major adjustments now, >>> since it's pretty old and a number of new things were added in the >>> meantime) >> >> This it wil

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Johannes Berg
On Fri, 2016-10-14 at 09:41 +0100, Ard Biesheuvel wrote: > > I assume the stack buffer itself is not the problem here, but aad, > > which is allocated on the stack one frame up. > > Do we really need to revert the whole patch to fix that? > > Ah never mind, this is about 'odata'. Apologies, shoul

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Johannes Berg
On Fri, 2016-10-14 at 17:39 +0900, Sergey Senozhatsky wrote: >  > given that we have a known issue shouldn't VMAP_STACK be > disabled for now, or would you rather prefer to mark MAC80211 > as incompatible: "depends on CFG80211 && !VMAP_STACK"? Yeah. It's a bit complicated by the fact that most peo

Re: [bug/regression] libhugetlbfs testsuite failures and OOMs eventually kill my system

2016-10-14 Thread Jan Stancek
On 10/14/2016 01:26 AM, Mike Kravetz wrote: > > Hi Jan, > > Any chance you can get the contents of /sys/kernel/mm/hugepages > before and after the first run of libhugetlbfs testsuite on Power? > Perhaps a script like: > > cd /sys/kernel/mm/hugepages > for f in hugepages-*/*; do > n=`cat $f

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Ard Biesheuvel
On 14 October 2016 at 09:42, Johannes Berg wrote: > On Fri, 2016-10-14 at 09:41 +0100, Ard Biesheuvel wrote: > >> > I assume the stack buffer itself is not the problem here, but aad, >> > which is allocated on the stack one frame up. >> > Do we really need to revert the whole patch to fix that? >>

Re: [PATCH] Coccinelle: misc: Improve the script for more accurate results

2016-10-14 Thread Lars-Peter Clausen
On 10/13/2016 07:01 PM, Vaishali Thakkar wrote: > > > On Thursday 13 October 2016 09:45 PM, Julia Lawall wrote: >> >> >> On Thu, 13 Oct 2016, Vaishali Thakkar wrote: >> >>> Currently because of the left associativity of the operators, >>> pattern IRQF_ONESHOT | flags does not match with the patte

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Johannes Berg
For reference, this was my patch moving the mac80211 buffers to percpu. diff --git a/net/mac80211/aes_ccm.c b/net/mac80211/aes_ccm.c index 7663c28ba353..c3709ddf71e9 100644 --- a/net/mac80211/aes_ccm.c +++ b/net/mac80211/aes_ccm.c @@ -29,6 +29,8 @@ void ieee80211_aes_ccm_encrypt(struct crypto_aead

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Johannes Berg
On Fri, 2016-10-14 at 09:47 +0100, Ard Biesheuvel wrote: >  > Do you have a reference for the sg_set_buf() call on odata? > crypto/ccm.c does not seem to have it (afaict), It's indirect - crypto_ccm_encrypt() calls crypto_ccm_init_crypt() which does it. > and the same problem > does not exist in

Re: [RFC 1/1] drivers: i2c: omap: Add slave support

2016-10-14 Thread Ravikumar
On Monday 29 August 2016 09:13 AM, Matthijs van Duin wrote: On 28 August 2016 at 07:35, Wolfram Sang wrote: Well, I2C is simple, what could go wrong? :/ Actually I2C is elegant and *seems* simple, but in all its asynchronicity there are actually a surprising number of fine details you can tr

Your Partnership in business

2016-10-14 Thread MR.ALEJANDRO FOLMER
-- Dear Sir, I am MR.ALEJANDRO FOLMER. I have a Proposition involving an investment initiative in your Country to discuss with you,It will be of mutual benefit to both of us,and I believe we can handle it together once we have a common understanding and mutual co-operation in the execution of the

[PATCH v2] staging:vt6656:baseband.h: fix function definition argument without identifier name issue

2016-10-14 Thread Mikhail Golubev
Function definitions arguments should also have an identifier name as reported by checkpatch.pl. Signed-off-by: Mikhail Golubev --- Changes in v2: - Make the changelog comment wrapped at 72 columns. drivers/staging/vt6656/baseband.h | 20 ++-- 1 file changed, 10 insertions(+),

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Ard Biesheuvel
On 14 October 2016 at 09:55, Johannes Berg wrote: > On Fri, 2016-10-14 at 09:47 +0100, Ard Biesheuvel wrote: >> >> Do you have a reference for the sg_set_buf() call on odata? >> crypto/ccm.c does not seem to have it (afaict), > > It's indirect - crypto_ccm_encrypt() calls crypto_ccm_init_crypt() >

Re: [PATCH 03/10] perf, tools: Add support for parsing uncore json files

2016-10-14 Thread Jiri Olsa
On Thu, Oct 13, 2016 at 02:15:25PM -0700, Andi Kleen wrote: > From: Andi Kleen > > Handle the Unit field, which is needed to find the right PMU for > an event. We call it "pmu". Handle the ExtSel field. > Handle the Filter field. Then output the fields into the pmu-events > data structures which

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Johannes Berg
On Fri, 2016-10-14 at 10:05 +0100, Ard Biesheuvel wrote: >  > Indeed. And the decrypt path does the same for auth_tag[]. Hadn't gotten that far, due to the BUG_ON() in CONFIG_DEBUG_SG in the encrypt path :) > But that still means there are two separate problems here, one which > affects the WPA c

Re: [PATCH v2] usb: dwc3: gadget: Wait for end transfer complete before free irq

2016-10-14 Thread Baolin Wang
Hi Felipe, On 14 October 2016 at 15:46, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> I see what the problem is. Databook tells us we *must* set CMDIOC >> when >> issuing EndTransfer command and we should always wait for Command >> Complete IRQ. However,

[PATCH v4] usb: dwc3: Wait for control tranfer completed when stopping gadget

2016-10-14 Thread Baolin Wang
When we change the USB function with configfs dynamically, we possibly met this situation: one core is doing the control transfer, another core is trying to unregister the USB gadget from userspace, we must wait for completing this control tranfer, or it will hang the controller to set the DEVCTRLH

[PATCH v2 3/6] arm: stmf32: Enable SYSCON

2016-10-14 Thread gabriel.fernandez
From: Gabriel Fernandez The clock drivers needs to disable the power domain write protection using syscon/regmap to enable RTC clock. Signed-off-by: Gabriel Fernandez --- arch/arm/configs/stm32_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/arm/configs/stm32_defconfig b/arc

[PATCH v2 6/6] ARM: dts: stm32f429: Add QSPI clock

2016-10-14 Thread gabriel.fernandez
From: Gabriel Fernandez This patch adds the QSPI clock for stm32f469 discovery board. Signed-off-by: Gabriel Fernandez --- arch/arm/boot/dts/stm32f469-disco.dts | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm/boot/dts/stm32f469-disco.dts b/arch/arm/boot/dts/stm32f469-disco.dt

[PATCH v2 0/6] STM32F4 Add RTC & QSPI clocks

2016-10-14 Thread gabriel.fernandez
From: Gabriel Fernandez v2: - rename compatible property "st,stm32f46xx-rcc" into "st,stm32f469-rcc" - cosmetic: remove bad copy/paste This patch-set introduce RTC and QSPI clocks for STM32F4 socs RTC clock has 3 parents clock oscillators (lsi/lse/hse_rtc) example to use rtc clock:

[PATCH v2 2/6] ARM: dts: stm32f429: add LSI and LSE clocks

2016-10-14 Thread gabriel.fernandez
From: Gabriel Fernandez This patch adds lsi / lse oscillators. These clocks can be use by RTC clocks. The clock drivers needs to disable the power domain write protection using syscon / regmap to enable these clocks. Signed-off-by: Gabriel Fernandez --- arch/arm/boot/dts/stm32f429.dtsi | 18 ++

[PATCH v2 1/6] clk: stm32f4: Add LSI & LSE clocks

2016-10-14 Thread gabriel.fernandez
From: Gabriel Fernandez This patch introduces the support of the LSI & LSE clocks. The clock drivers needs to disable the power domain write protection using syscon/regmap to enable these clocks. Signed-off-by: Gabriel Fernandez --- drivers/clk/clk-stm32f4.c | 138 +

[PATCH v2 5/6] clk: stm32f469: Add QSPI clock

2016-10-14 Thread gabriel.fernandez
From: Gabriel Fernandez This patch adds the QSPI clock for stm32f469 discovery board. The gate mapping is a little bit different from stm32f429 soc. Signed-off-by: Gabriel Fernandez --- .../devicetree/bindings/clock/st,stm32-rcc.txt | 4 +- drivers/clk/clk-stm32f4.c

[PATCH v2 4/6] clk: stm32f4: Add RTC clock

2016-10-14 Thread gabriel.fernandez
From: Gabriel Fernandez This patch introduces the support of the RTC clock. RTC clock can have 3 sources: lsi, lse and hse_rtc. Signed-off-by: Gabriel Fernandez --- drivers/clk/clk-stm32f4.c | 135 +- 1 file changed, 134 insertions(+), 1 deletion(-)

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Ard Biesheuvel
On 14 October 2016 at 10:10, Johannes Berg wrote: > On Fri, 2016-10-14 at 10:05 +0100, Ard Biesheuvel wrote: >> >> Indeed. And the decrypt path does the same for auth_tag[]. > > Hadn't gotten that far, due to the BUG_ON() in CONFIG_DEBUG_SG in the > encrypt path :) > >> But that still means there

Re: [patch 1/2] IB/hns: Fix a couple pointer math bugs

2016-10-14 Thread oulijun
在 2016/10/14 15:28, Dan Carpenter 写道: > "wqe" is a void pointer so adding sizeof() works. The original code > adds sizeof() multiplied by sizeof() so it doesn't work at all. > > Fixes: 9a4435375cd1 ('IB/hns: Add driver files for hns RoCE driver') > Signed-off-by: Dan Carpenter > > diff --git a/

[PATCH] fs: pipe, fix undefined behaviour

2016-10-14 Thread Jiri Slaby
echo | ./program where ./program contains fcntl(0, F_SETPIPE_SZ, 0) this is triggered: UBSAN: Undefined behaviour in ../include/linux/log2.h:63:13 shift exponent 64 is too large for 64-bit type 'long unsigned int' CPU: 3 PID: 4978 Comm: syz-executor Not tainted 4.8.1-0-syzkaller #1 Hardware name: Q

[PATCH] fs: fcntl, avoid undefined behaviour

2016-10-14 Thread Jiri Slaby
fcntl(0, F_SETOWN, 0x8000) triggers: UBSAN: Undefined behaviour in fs/fcntl.c:118:7 negation of -2147483648 cannot be represented in type 'int': CPU: 1 PID: 18261 Comm: syz-executor Not tainted 4.8.1-0-syzkaller #1 ... Call Trace: ... [] ? f_setown+0x1d8/0x200 [] ? SyS_fcntl+0x999/0xf30 [] ?

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Johannes Berg
On Fri, 2016-10-14 at 10:21 +0100, Ard Biesheuvel wrote: > It is annotated with a TODO, though :-) > > 38320c70d282b (Herbert Xu   2008-01-28 19:35:05 > -0800  41) >  * TODO: Use spare space in skb for this where possible. I saw that, but I don't think generally there will be spare s

[PATCH v7 0/5] ARM: dts: imx6q: Add Engicam i.CoreM6 dts

2016-10-14 Thread Jagan Teki
This is series add dts support for Engicam I.Core M6 qdl modules. just rebased on top of linux-next of previous set[1]. [1] http://www.spinics.net/lists/kernel/msg2358233.html Jagan Teki (5): ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support ARM: dts: imx6q: Add Engicam i.CoreM6

[PATCH v7 4/5] ARM: dts: imx6qdl-icore: Add usbotg support

2016-10-14 Thread Jagan Teki
From: Jagan Teki Add usbotg support for Engicam i.CoreM6 dql modules. Cc: Sascha Hauer Cc: Fabio Estevam Cc: Shawn Guo Cc: Matteo Lisi Cc: Michael Trimarchi Signed-off-by: Jagan Teki --- Changes for v7: - none Changes for v6: - none Changes for v5: - none Changes fo

[PATCH v7 5/5] ARM: dts: imx6qdl-icore: Add FEC support

2016-10-14 Thread Jagan Teki
From: Jagan Teki Add FEC support for Engicam i.CoreM6 dql modules. Observed similar 'eth0: link is not ready' issue which was discussed in [1] due rmii mode with external ref_clk, so added clock node along with the properties mentioned by Shawn in [2] FEC link log: $ ifconfig eth0

[PATCH v7 2/5] ARM: dts: imx6q: Add Engicam i.CoreM6 DualLite/Solo initial support

2016-10-14 Thread Jagan Teki
From: Jagan Teki i.CoreM6 DualLite/Solo modules are system on module solutions manufactured by Engicam with following characteristics: CPU NXP i.MX6 DL, 800MHz RAM 1GB, 32, 64 bit, DDR3-800/1066 NAND SLC,512MB Power supply Single 5V MAX LCD RES FULLHD and more inf

[PATCH v7 1/5] ARM: dts: imx6q: Add Engicam i.CoreM6 Quad/Dual initial support

2016-10-14 Thread Jagan Teki
From: Jagan Teki i.CoreM6 Quad/Dual modules are system on module solutions manufactured by Engicam with following characteristics: CPU NXP i.MX6 DQ, 800MHz RAM 1GB, 32, 64 bit, DDR3-800/1066 NAND SLC,512MB Power supply Single 5V MAX LCD RES FULLHD and more info at

[PATCH v7 3/5] ARM: dts: imx6qdl-icore: Add usbhost support

2016-10-14 Thread Jagan Teki
From: Jagan Teki Add usbhost support for Engicam i.CoreM6 dql modules. Cc: Sascha Hauer Cc: Fabio Estevam Cc: Shawn Guo Cc: Matteo Lisi Cc: Michael Trimarchi Signed-off-by: Jagan Teki --- Changes for v7: - none Changes for v6: - none Changes for v5: - none Changes f

Re: kernel BUG at arch/x86/kernel/traps.c:643! when run Redhat7(v3.10) in kvm guest

2016-10-14 Thread Rik van Riel
On Fri, 2016-10-14 at 14:14 +0800, Kefeng Wang wrote: > Hi all, > > We met BUG_ON in do_device_not_available(fpu exception handler) when > run redhat7 in kvm guest, > and there is no special test on this guest, only some network packet > receipt and transmission. > > I checked the new kernel vers

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Ard Biesheuvel
On 14 October 2016 at 10:25, Johannes Berg wrote: > On Fri, 2016-10-14 at 10:21 +0100, Ard Biesheuvel wrote: > >> It is annotated with a TODO, though :-) >> >> 38320c70d282b (Herbert Xu 2008-01-28 19:35:05 >> -0800 41) >> * TODO: Use spare space in skb for this where possible. > >

[PATCH v2 0/3] ARM: dts: imx6qdl cleanups

2016-10-14 Thread Jagan Teki
Patchset for imx6qdl devicetree files cleanups. Jagan Teki (3): arm: dts: imx6qdl: Fix "WARNING: please, no space before tabs" arm: dts: imx6qdl: Fix "ERROR: code indent should use tabs where possible" arm: dts: imx6qdl-wandboard-revb: Fix "ERROR: trailing whitespace" arch/arm/boot/dts

[PATCH v2 3/3] arm: dts: imx6qdl-wandboard-revb: Fix "ERROR: trailing whitespace"

2016-10-14 Thread Jagan Teki
Fixed error in trailing whitespace in wandboard-rev1 dtsi. Cc: Shawn Guo Signed-off-by: Jagan Teki --- Changes for v2: - none arch/arm/boot/dts/imx6qdl-wandboard-revb1.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/boot/dts/imx6qdl-wandboard-revb1.dts

[PATCH v2 1/3] arm: dts: imx6qdl: Fix "WARNING: please, no space before tabs"

2016-10-14 Thread Jagan Teki
Fixed no space before tabs warnings in respetcive imx6qdl dtsi files. Cc: Shawn Guo Signed-off-by: Jagan Teki --- Changes for v2: - none arch/arm/boot/dts/imx6qdl-apf6dev.dtsi | 14 +++--- arch/arm/boot/dts/imx6qdl-tx6.dtsi | 32 arch/ar

[PATCH v2 2/3] arm: dts: imx6qdl: Fix "ERROR: code indent should use tabs where possible"

2016-10-14 Thread Jagan Teki
Fixed code indent tabs in respetcive imx6qdl dtsi files. Cc: Shawn Guo Signed-off-by: Jagan Teki --- Changes for v2: - none arch/arm/boot/dts/imx6qdl-gw52xx.dtsi| 4 ++-- arch/arm/boot/dts/imx6qdl-gw53xx.dtsi| 4 ++-- arch/arm/boot/dts/imx6qdl-gw54xx.dtsi| 4 ++-

Re: [PATCH V3 05/10] acpi: apei: handle SEA notification type for ARMv8

2016-10-14 Thread Punit Agrawal
"Baicar, Tyler" writes: > Hello Punit, > > > On 10/12/2016 12:00 PM, Punit Agrawal wrote: >> Tyler Baicar writes: >> >>> ARM APEI extension proposal added SEA (Synchrounous External >>> Abort) notification type for ARMv8. >>> Add a new GHES error source handling function for SEA. If an error >>>

[GIT PULL (updated)] overlayfs update for 4.9

2016-10-14 Thread Miklos Szeredi
Hi Linus, Please pull updated branch from: git://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/vfs.git overlayfs-linus Left out the mnt_want_write_file() and clone_file_range() patches altogether, so this is now really overlayfs only. There's vfs_get_link() and that's technically in the VF

Re: [PATCH V3 10/10] arm64: KVM: add guest SEA support

2016-10-14 Thread Punit Agrawal
"Baicar, Tyler" writes: > Hello Punit, > > On 10/13/2016 7:14 AM, Punit Agrawal wrote: >> Hi Tyler, >> >> I know I've had my last comment already ;), but I thought I'd rather >> raise the question than stay confused... >> >> Tyler Baicar writes: >> >>> Currently external aborts are unsupported b

[PATCH] p54: memset(0) whole array

2016-10-14 Thread Jiri Slaby
gcc 7 complains: drivers/net/wireless/intersil/p54/fwio.c: In function 'p54_scan': drivers/net/wireless/intersil/p54/fwio.c:491:4: warning: 'memset' used with length equal to number of elements without multiplication by element size [-Wmemset-elt-size] Fix that by passing the correct size to mem

Re: [patch] irqchip/gicv3: fix a loop timeout

2016-10-14 Thread Sudeep Holla
On 14/10/16 08:26, Dan Carpenter wrote: We expect the loop to exit with "count" set to zero so let's switch it from a post-op to a pre-op count down. Otherwise we don't see the error message when this times out. Nice catch. Acked-by: Sudeep Holla -- Regards, Sudeep

Re: [PATCH v2] usb: dwc3: gadget: Wait for end transfer complete before free irq

2016-10-14 Thread Felipe Balbi
Hi, Baolin Wang writes: >> Baolin Wang writes: >>> I see what the problem is. Databook tells us we *must* set CMDIOC >>> when >>> issuing EndTransfer command and we should always wait for Command >>> Complete IRQ. However, we took a shortcut and just delayed for

Re: [PATCH] x86: remove duplicate rtit status MSR macro

2016-10-14 Thread Borislav Petkov
On Fri, Oct 14, 2016 at 08:42:20AM +0800, Longpeng(Mike) wrote: > The MSR_IA32_RTIT_STATUS has defined twice, so remove one. > > Signed-off-by: Longpeng(Mike) > --- > arch/x86/include/asm/msr-index.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/arch/x86/include/asm/msr-index.h > b/a

Re: [RFC PATCH v1 2/2] printk: external log buffer (CONFIG_LOGBUFFER)

2016-10-14 Thread Petr Mladek
On Wed 2016-10-12 03:12:56, Sean Hudson wrote: > On 10/5/2016 2:48 PM, Petr Mladek wrote: > > On Tue 2016-10-04 12:55:35, Sean Hudson wrote: > >> On 10/4/2016 6:27 AM, Petr Mladek wrote: > >>> IMHO, the only way is to revisit all locations when the log buffer > >>> is accessed and add all the neede

[RFC PATCH 2/2] hwmon: adt7411: add min, max and alarm attributes

2016-10-14 Thread Michael Walle
This patch adds support for the min, max and alarm attributes of the voltage and temperature channels. Additionally, the temp2_fault attribute is supported which indicates a fault of the external temperature diode. Signed-off-by: Michael Walle --- drivers/hwmon/adt7411.c | 306 ++

[RFC PATCH 1/2] hwmon: adt7411: update to new hwmon registration API

2016-10-14 Thread Michael Walle
This is also a preparation for to support more properties like min, max and alarm. Signed-off-by: Michael Walle --- drivers/hwmon/adt7411.c | 300 +--- 1 file changed, 179 insertions(+), 121 deletions(-) diff --git a/drivers/hwmon/adt7411.c b/drivers/

Re: [PATCH] fs: pipe, fix undefined behaviour

2016-10-14 Thread Vegard Nossum
On 14 October 2016 at 11:23, Jiri Slaby wrote: > echo | ./program > where ./program contains fcntl(0, F_SETPIPE_SZ, 0) this is triggered: > UBSAN: Undefined behaviour in ../include/linux/log2.h:63:13 > shift exponent 64 is too large for 64-bit type 'long unsigned int' > CPU: 3 PID: 4978 Comm: syz-

Re: [mac80211] BUG_ON with current -git (4.8.0-11417-g24532f7)

2016-10-14 Thread Johannes Berg
> So why is the performance hit acceptable for ESP but not for WPA? We > could easily implement the same thing, i.e., > kmalloc(GFP_ATOMIC)/kfree the aead_req struct rather than allocate it > on the stack Yeah, maybe we should. It's likely a much bigger allocation, but I don't actually know if th

  1   2   3   4   5   6   >