Re: [PATCH V3 1/4] ARM64 LPC: Indirect ISA port IO introduced

2016-09-17 Thread zhichang
Hi, Arnd, On 2016年09月14日 22:23, Arnd Bergmann wrote: > On Wednesday, September 14, 2016 10:16:28 PM CEST zhichang.yuan wrote: >>> >>> No need to guard includes with an #ifdef. >> If remove #ifdef here, extio.h should not contain any function external >> declarations whose definitions are in >> e

Re: [PATCH 2/8] drm/rockchip: Get rid of some unnecessary code

2016-09-17 Thread Tomasz Figa
Hi Mark, On Sun, Sep 18, 2016 at 10:50 AM, Mark yao wrote: > On 2016年09月14日 20:54, Tomasz Figa wrote: >> >> Current code implements prepare_fb and cleanup_fb callbacks only to >> grab/release fb references, which is already done by atomic framework >> when creating/destryoing plane state. Let's r

Re: Runtime failure running sh:qemu in -next due to 'sh: fix copy_from_user()'

2016-09-17 Thread Rob Landley
On 09/16/2016 09:23 PM, Guenter Roeck wrote: > On 09/16/2016 04:32 PM, Rich Felker wrote: >>> 4.6.3 from kernel.org. >> >> That is utterly ancient and probaby very buggy. I would recommend 5.x+ >> or at the very least 4.7 or 4.8. >> > Unfortunately that is the latest one available from kernel.org

Re: [PATCH v2 1/2] usb: dwc3: gadget: Add disconnect checking when changing function dynamically

2016-09-17 Thread Baolin Wang
Hi Felipe, Sorry for late reply due to my holiday. On 9 September 2016 at 18:47, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> When system has stpped the gadget, we should avoid queuing any requests >> which will cause tranfer failed. Thus adding some disconnect checking to >> avoid th

Re: Possible code defects: macros and precedence

2016-09-17 Thread Julia Lawall
On Sat, 17 Sep 2016, Joe Perches wrote: > On Sat, 2016-09-17 at 22:24 +0200, Julia Lawall wrote: > > (A 2.2MB message that (perhaps thankfully) didn't get through to lkml) > > > Below is the Coccinelle output for the case where the definition of the > > macro is a single expression.  There is al

Re: [PATCH v2 2/2] usb: dwc3: Wait for control tranfer completed when stopping gadget

2016-09-17 Thread Baolin Wang
Hi Felipe, On 9 September 2016 at 19:03, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> diff --git a/drivers/usb/dwc3/core.c b/drivers/usb/dwc3/core.c >> index 057739d..22787b6 100644 >> --- a/drivers/usb/dwc3/core.c >> +++ b/drivers/usb/dwc3/core.c >> @@ -999,6 +999,7 @@ static int dwc3

Re: Possible code defects: macros and precedence

2016-09-17 Thread Julia Lawall
On Sat, 17 Sep 2016, Joe Perches wrote: > On Sat, 2016-09-17 at 22:24 +0200, Julia Lawall wrote: > > diff -u -p a/lib/sha1.c b/lib/sha1.c > [] > > @@ -49,18 +49,18 @@ > >   * the input data, the next mix it from the 512-bit array. > >   */ > >  #define SHA_SRC(t) get_unaligned_be32((__u32 *)data

Re: [PATCH 4/5] ipc/msg: Lockless security checks for msgsnd

2016-09-17 Thread Manfred Spraul
Hi Davidlohr, Just as with msgrcv (along with the rest of sysvipc since a few years ago), perform the security checks without holding the ipc object lock. Thinking about it: isn't this wrong? CPU1: * msgrcv() * ipcperms() CPU2: * msgctl(), change permissions ** msgctl() returns, new perm

drivers/gpio/gpiolib.c:3215: undefined reference to `of_get_named_gpiod_flags'

2016-09-17 Thread kbuild test robot
Hi Linus, 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: d4690f1e1cdabb4d61207b6787b1605a0dc0aeab commit: 2527ecc9195e9c66252af24c4689e8a67cd4ccb9 gpio: Fix OF build problem on UM date: 4 weeks ag

[PATCH 1/2] rtc: Add some dummy static inline functions

2016-09-17 Thread Baolin Wang
The patch 2 in this patchset adding tracepoints for alarmtimers will build failed on S390 platform, due to S390 defconfig did not define CONFIG_RTC_LIB macro to define the rtc_ktime_to_tm() function which is used in this patch. Thus we should add some ummy static inline functions in case CONFIG_RTC

[PATCH 2/2] time: alarmtimer: Add the trcepoints for alarmtimer

2016-09-17 Thread Baolin Wang
For system debugging, we sometimes want to know who sets one alarm timer, the time of the timer, when the timer started and fired and so on. Thus adding tracepoints can help us trace the alarmtimer information. For example, when we debug the system supend/resume, if the system is always resumed by

Re: [tip regression] efi: Allow drivers to reserve boot services forever == toxic

2016-09-17 Thread Mike Galbraith
On Sat, 2016-09-17 at 20:58 +0100, Matt Fleming wrote: > These addresses are pretty low. Can you try the hacky patch plus > Ricardo's change in commit 3dad6f7f6975 ("x86/efi: Defer efi_esrt_init > until after memblock_x86_fill"). He fixed a bug where it's possible to > run out of memblock regions.

[PATCH 00/10] firewire-net: Fine-tuning for several function implementations

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 18 Sep 2016 08:04:02 +0200 Some update suggestions were taken into account from static source code analysis. Markus Elfring (10): Use kmalloc_array() in fwnet_broadcast_start() Rename a jump label in fwnet_broadcast_start() Rename jump labels in fwnet_init()

[PATCH 01/] firewire-net: Use kmalloc_array() in fwnet_broadcast_start()

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 17 Sep 2016 21:55:42 +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 02/10] firewire-net: Rename a jump label in fwnet_broadcast_start()

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 17 Sep 2016 22:02:44 +0200 Adjust jump labels according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/firewire/net.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/firewire/net.

[PATCH 03/10] firewire-net: Rename jump labels in fwnet_init()

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 17 Sep 2016 22:11:25 +0200 Adjust jump labels according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/firewire/net.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/firewire/net.c b/dr

[PATCH 04/10] firewire-net: Rename jump labels in fwnet_probe()

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 17 Sep 2016 22:17:12 +0200 Adjust jump labels according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/firewire/net.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/firewire/net.c b

[PATCH 05/10] firewire-net: Rename a jump label in fwnet_tx()

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 17 Sep 2016 22:22:58 +0200 Adjust jump labels according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/firewire/net.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/firewire/net.c

[PATCH 06/10] firewire-net: Rename a jump label in fwnet_send_packet()

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 17 Sep 2016 22:27:52 +0200 Adjust a jump label according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/firewire/net.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/firewire/net.c b/driv

[PATCH 07/10] firewire-net: Rename a jump label in fwnet_incoming_packet()

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 17 Sep 2016 22:32:14 +0200 Adjust jump labels according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/firewire/net.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/firewire/net.c b

[PATCH 08/10] firewire-net: Rename a jump label in fwnet_finish_incoming_packet()

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 17 Sep 2016 22:40:05 +0200 Adjust a jump label according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/firewire/net.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/firewire/net.c b/driv

[PATCH 09/10] firewire-net: Rename jump labels in fwnet_pd_new()

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 17 Sep 2016 22:48:46 +0200 Adjust jump labels according to the current Linux coding style convention. Signed-off-by: Markus Elfring --- drivers/firewire/net.c | 13 ++--- 1 file changed, 6 insertions(+), 7 deletions(-) diff --git a/drivers/firewire/net.

[PATCH 10/10] firewire-net: Adjust checks for null pointers in five functions

2016-09-17 Thread SF Markus Elfring
From: Markus Elfring Date: Sun, 18 Sep 2016 07:48:56 +0200 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The script "checkpatch.pl" can point information out like the following. Comparison to NULL could be written !… Thus fix the affected source code

Re: [PATCH linux-firmware 06/12] WHENCE: Remove references to two nvidia firmware files that were never added

2016-09-17 Thread Alexandre Courbot
On 09/18/2016 11:02 AM, Ben Hutchings wrote: > * PGP Signed by an unknown key > > Signed-off-by: Ben Hutchings Stupid copy-paste. Thanks for catching this! Reviewed-by: Alexandre Courbot > --- > WHENCE | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/WHENCE b/WHENCE > index 6002834

Re: [PATCH] KVM: VMX: Enable MSR-BASED TPR shadow even if w/o APICv

2016-09-17 Thread Wanpeng Li
2016-09-15 23:58 GMT+08:00 Radim Krčmář : > 2016-09-15 15:05+0800, Wanpeng Li: >> 2016-09-14 20:03 GMT+08:00 Radim Krčmář : >>> 2016-09-14 11:40+0200, Paolo Bonzini: On 14/09/2016 09:58, Wanpeng Li wrote: > From: Wanpeng Li > > I observed that kvmvapic(to optimize flexpriority=N o

Re: [PATCH] KVM: VMX: Enable MSR-BASED TPR shadow even if w/o APICv

2016-09-17 Thread Wanpeng Li
2016-09-18 14:53 GMT+08:00 Wanpeng Li : > 2016-09-15 23:58 GMT+08:00 Radim Krčmář : >> 2016-09-15 15:05+0800, Wanpeng Li: >>> 2016-09-14 20:03 GMT+08:00 Radim Krčmář : 2016-09-14 11:40+0200, Paolo Bonzini: > On 14/09/2016 09:58, Wanpeng Li wrote: >> From: Wanpeng Li >> >> I ob

<    1   2   3