Re: [PATCH v2 4/4] tpm/vtpm: fix kdoc warnings

2016-12-03 Thread Jarkko Sakkinen
On Wed, Nov 23, 2016 at 12:04:14PM +0200, Tomas Winkler wrote: > Use corret kdoc format for function description and eliminate warning > of type: > > tpm_ibmvtpm.c:66: warning: No description found for parameter 'count' > > Signed-off-by: Tomas Winkler Reviewed-by: Jarkko Sakkinen /Jarkkko >

Re: net: use-after-free in worker_thread

2016-12-03 Thread Andrey Konovalov
On Sat, Dec 3, 2016 at 2:49 PM, Eric Dumazet wrote: > On Sat, 2016-12-03 at 14:05 +0100, Andrey Konovalov wrote: >> On Sat, Dec 3, 2016 at 1:58 PM, Andrey Konovalov >> wrote: >> > +syzkal...@googlegroups.com >> > >> > On Sat, Dec 3, 2016 at 1:56 PM, Andrey Konovalov >> > wrote: >> >> Hi! >> >>

Re: [PATCH 1/1] net: ethernet: 3com: set error code on failures

2016-12-03 Thread Pan Bian
From: PanBian Hi, Lino, There is no special reason to map different errors to -EIO. In the original source code, err is set to -EIO on most error paths. I copied that. I guess it has no difference with the error value provided by the called functions in the view of callers of typhoon_init_one(

Re: [PATCH] IIO: Change msleep to usleep_range for small msecs

2016-12-03 Thread Aniroop Mathur
On Sat, Dec 3, 2016 at 2:30 PM, Jonathan Cameron wrote: > On 02/12/16 19:07, Aniroop Mathur wrote: >> On Wed, Nov 30, 2016 at 8:13 PM, Aniroop Mathur wrote: >>> On 30 Nov 2016 19:05, "Lars-Peter Clausen" wrote: >>> > >>> > On 11/27/2016 11:51 AM, Jonathan Cameron wrote: >>> > > On 26/11/16 03

Re: [PATCH v13 0/8] x86/arch_prctl Add ARCH_[GET|SET]_CPUID for controlling the CPUID instruction

2016-12-03 Thread Kyle Huey
On Fri, Dec 2, 2016 at 2:29 AM, Ingo Molnar wrote: > > * Kyle Huey wrote: > >> rr (http://rr-project.org/), a userspace record-and-replay reverse- >> execution debugger, would like to trap and emulate the CPUID instruction. >> This would allow us to a) mask away certain hardware features that rr

Re: Unkillable processes due to PTRACE_TRACEME again

2016-12-03 Thread Dmitry Vyukov
On Fri, Dec 2, 2016 at 10:02 PM, Pavel Machek wrote: > On Fri 2016-12-02 19:48:40, Dmitry Vyukov wrote: >> Hello, >> >> There was an issue discussed a year ago which leads to >> unkillalble/unwaitable zombie processes due to PTRACE_TRACEME: >> https://groups.google.com/d/msg/syzkaller/uGzwvhlCXAw/

Re: [PATCH v2 1/4] tpm: add kdoc for tpm_transmit and tpm_transmit_cmd

2016-12-03 Thread Jarkko Sakkinen
On Sat, Dec 03, 2016 at 05:32:36PM +0200, Jarkko Sakkinen wrote: > On Wed, Nov 23, 2016 at 12:04:11PM +0200, Tomas Winkler wrote: > > Functions tpm_transmit and transmit_cmd are referenced > > from other functions kdoc hence deserve documentation. > > > > Signed-off-by: Tomas Winkler > > Do you

Re: [PATCH v2 3/4] tmp: use pdev for parent device in tpm_chip_alloc

2016-12-03 Thread Jarkko Sakkinen
On Sat, Dec 03, 2016 at 05:35:30PM +0200, Jarkko Sakkinen wrote: > On Wed, Nov 23, 2016 at 12:04:13PM +0200, Tomas Winkler wrote: > > The tpm stack uses pdev name convention for the parent device. > > Fix that also in tpm_chip_alloc(). > > Usually I'm not too fond for minor style fixes but in this

Re: [PATCH v2 4/4] tpm/vtpm: fix kdoc warnings

2016-12-03 Thread Jarkko Sakkinen
On Sat, Dec 03, 2016 at 05:36:04PM +0200, Jarkko Sakkinen wrote: > On Wed, Nov 23, 2016 at 12:04:14PM +0200, Tomas Winkler wrote: > > Use corret kdoc format for function description and eliminate warning > > of type: > > > > tpm_ibmvtpm.c:66: warning: No description found for parameter 'count' > >

Re: [PATCH v2 2/4] tpm/tpm2-chip: fix kdoc errors

2016-12-03 Thread Jarkko Sakkinen
On Sat, Dec 03, 2016 at 05:34:17PM +0200, Jarkko Sakkinen wrote: > On Wed, Nov 23, 2016 at 12:04:12PM +0200, Tomas Winkler wrote: > > Use correct kdoc format, describe correct parameters and return values. > > > > Signed-off-by: Tomas Winkler > > Reviewed-by: Jarkko Sakkinen Tested-by: Jarkko

Re: [PATCH v2 0/4] tpm: kdoc cleanup

2016-12-03 Thread Jarkko Sakkinen
On Wed, Nov 23, 2016 at 12:04:10PM +0200, Tomas Winkler wrote: > Cleanup tpm in-code documentation. These are all applied to my master branch but please do answer to my question about TPM_DIGEST_SIZE so I can adjust that. /Jarkko

Re: [PATCH 1/1] net: caif: fix ineffective error check

2016-12-03 Thread Pan Bian
From: PanBian Hello Sergei, On Sat, Dec 03, 2016 at 04:17:51PM +0300, Sergei Shtylyov wrote: > Hello. > > On 12/3/2016 2:18 PM, Pan Bian wrote: > > >In function caif_sktinit_module(), the check of the return value of > >sock_register() seems ineffective. This patch fixes it. > > > >Bugzilla: h

usb/gadget: GPF in usb_gadget_unregister_driver

2016-12-03 Thread Andrey Konovalov
Hi! I've got the following error report while running the syzkaller fuzzer. On commit 3c49de52d5647cda8b42c4255cf8a29d1e22eff5 (Dec 2). general protection fault: [#1] SMP KASAN Dumping ftrace buffer: (ftrace buffer empty) Modules linked in: CPU: 0 PID: 10564 Comm: syz-executor0 Not taint

Re: [PATCH] fpga: fix sparse warnings in fpga-mgr and fpga-bridge

2016-12-03 Thread Moritz Fischer
On Fri, Dec 2, 2016 at 1:23 PM, Dinh Nguyen wrote: > Fix up these sparse warnings: > > drivers/fpga/fpga-mgr.c:189:21: warning: symbol '__fpga_mgr_get' was not > declared. Should it be static? > drivers/fpga/fpga-bridge.c:30:12: warning: symbol 'bridge_list_lock' was > not declared. Should it be s

[PATCH] uapi glibc compat: fix outer guard of net device flags enum

2016-12-03 Thread Jonas Gorski
Fix a wrong condition preventing the higher net device flags IFF_LOWER_UP etc to be defined if net/if.h is included before linux/if.h. The comment makes it clear the intention was to allow partial definition with either parts. This fixes compilation of userspace programs trying to use IFF_LOWER_U

Re: btrfs_destroy_inode warn (outstanding extents)

2016-12-03 Thread Dave Jones
On Thu, Dec 01, 2016 at 10:32:09AM -0500, Dave Jones wrote: > http://codemonkey.org.uk/junk/btrfs-destroy-inode-outstanding-extents.txt > > Also same bug, different run, but a different traceview > http://codemonkey.org.uk/junk/btrfs-destroy-inode-outstanding-extents-function-graph.txt > >

Re: [PATCH] ARM: pxa: ezx: fix a910 camera data

2016-12-03 Thread Antonio Ospite
On Sat, 26 Nov 2016 00:30:52 +0100 Stefan Schmidt wrote: > Hello. > Hi everyone, > On 25.11.2016 20:53, Robert Jarzmik wrote: > > Stefan Schmidt writes: > > > >> Hello. > >> > >> On 24.11.2016 17:29, Arnd Bergmann wrote: > >>> The camera_supply_dummy_device definition is shared between a780 a

[PATCH 0/2] ARM: mmp: Fine-tuning for sram_probe()

2016-12-03 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 3 Dec 2016 17:50:23 +0100 A few update suggestions were taken into account from static source code analysis. Markus Elfring (2): Check return values from ioremap() and kstrdup() Delete an unnecessary variable initialisation arch/arm/mach-mmp/sram.c | 19

[PATCH 1/2] ARM: mmp: Check return values from ioremap() and kstrdup() in sram_probe()

2016-12-03 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 3 Dec 2016 17:26:32 +0100 Two return values were not checked before their further use so far. This issue was detected by using the Coccinelle software. * Add a bit of exception handling. * Adjust tump targets. Fixes: 3c7241bd36e2a618fe20c91f6c69cc20f2d981f2 ("AR

[PATCH 2/2] ARM: mmp: Delete an unnecessary variable initialisation in sram_probe()

2016-12-03 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 3 Dec 2016 17:38:21 +0100 The local variable "ret" will be set to an appropriate value a bit later. Thus omit the explicit initialisation at the beginning. Signed-off-by: Markus Elfring --- arch/arm/mach-mmp/sram.c | 2 +- 1 file changed, 1 insertion(+), 1 delet

Re: [GIT PULL 1/3] ARM: exynos: Soc/mach for v4.10

2016-12-03 Thread Krzysztof Kozlowski
On Fri, Dec 02, 2016 at 10:52:57PM +0100, Arnd Bergmann wrote: > On Thursday, December 1, 2016 8:34:04 PM CET Krzysztof Kozlowski wrote: > > On Thu, Nov 24, 2016 at 08:08:27AM +0200, Krzysztof Kozlowski wrote: > > > Hi, > > > > > > This contains previous dts branch because SCU node in dts is neede

Re: [PATCH] x86/alternatives: Do not use sync_core() to serialize I$

2016-12-03 Thread Andy Lutomirski
On Sat, Dec 3, 2016 at 7:02 AM, Borislav Petkov wrote: > On Fri, Dec 02, 2016 at 08:28:44PM +0100, Borislav Petkov wrote: >> Ah, it is called only from apply_alternatives() but sure, it is safer >> this way. Lemme do that and run it through the boxes to see whether >> anything catches fire. > > Lo

Re: usb/gadget: GPF in usb_gadget_unregister_driver

2016-12-03 Thread Greg Kroah-Hartman
On Sat, Dec 03, 2016 at 05:36:35PM +0100, Andrey Konovalov wrote: > Hi! > > I've got the following error report while running the syzkaller fuzzer. > > On commit 3c49de52d5647cda8b42c4255cf8a29d1e22eff5 (Dec 2). > > general protection fault: [#1] SMP KASAN > Dumping ftrace buffer: >(ftr

Re: [PATCH] uio: uio_cif: Fix style issues

2016-12-03 Thread Greg KH
On Sat, Dec 03, 2016 at 07:39:44PM +0530, ChinmayVS wrote: > - Update header-file inclusion path. > - Fix indentation of switch and case. > - Remove extra space in function calls. > > Signed-off-by: ChinmayVS I need a "real" name here, one that you use on legal documents. Also, please break thi

Re: [SPCR] mmio32 iotype access requirements for X-Gene 8250(_dw) UART

2016-12-03 Thread Mark Salter
On Sat, 2016-12-03 at 05:06 -0500, Jon Masters wrote: > Hi Duc, all, > > (and changing the subject and trimming/adjusting the CC) > > On 12/02/2016 02:39 PM, Duc Dang wrote: > > > > On Fri, Dec 2, 2016 at 12:11 AM, Jon Masters wrote: > > > > > > You're welcome. > > > > > > (Unrelated) Note th

Re: usb/gadget: GPF in usb_gadget_unregister_driver

2016-12-03 Thread Felix Hädicke
Hi, > Hi! > > I've got the following error report while running the syzkaller fuzzer. > > On commit 3c49de52d5647cda8b42c4255cf8a29d1e22eff5 (Dec 2). > > general protection fault: [#1] SMP KASAN > Dumping ftrace buffer: >(ftrace buffer empty) > Modules linked in: > CPU: 0 PID: 10564 Comm:

Re: net: use-after-free in worker_thread

2016-12-03 Thread Cong Wang
On Sat, Dec 3, 2016 at 4:56 AM, Andrey Konovalov wrote: > Hi! > > I'm seeing lots of the following error reports while running the > syzkaller fuzzer. > > Reports appeared when I updated to 3c49de52 (Dec 2) from 2caceb32 (Dec 1). > >

Re: Re: [PATCH] Input: joystick: adi - change msleep to usleep_range for small msecs

2016-12-03 Thread Aniroop Mathur
On Tue, Nov 29, 2016 at 12:29 PM, vojt...@ucw.cz wrote: > On Mon, Nov 28, 2016 at 01:49:31PM +, Aniroop Mathur wrote: >> Hello Mr. Vojtech Pavlik, >> >> On 28 Nov 2016 17:23, "vojt...@ucw.cz" wrote: >> > >> > Hi. >> > >> > ADI_INIT_DELAY/ADI_DATA_DELAY doesn't have to be exact, and a long

[PATCH v2 0/3] Locality support for tpm_crb

2016-12-03 Thread Jarkko Sakkinen
The tpm_crb driver should follow the policy of reserving and relinquishing the locality it uses when multiple localities are used, like when TXT is another locality. Jarkko Sakkinen (3): tpm_crb: map locality registers tpm_crb: encapsulate crb_wait_for_reg_32 tpm_crb: request and relinquish

[PATCH v2 3/3] tpm_crb: request and relinquish locality 0

2016-12-03 Thread Jarkko Sakkinen
Request and relinquish locality for the driver use in order to be a better citizen in a multi locality environment like TXT. The locality is requested and relinquished as part of going into and waking up from idle. Signed-off-by: Jarkko Sakkinen --- drivers/char/tpm/tpm_crb.c | 36 ++

[PATCH v2 2/3] tpm_crb: encapsulate crb_wait_for_reg_32

2016-12-03 Thread Jarkko Sakkinen
Encapsulated crb_wait_for_reg32() so that state changes in other CRB registers than TPM_CRB_CTRL_REQ_x can be waited. Signed-off-by: Jarkko Sakkinen --- drivers/char/tpm/tpm_crb.c | 37 +++-- 1 file changed, 23 insertions(+), 14 deletions(-) diff --git a/drivers/

[PATCH v2 1/3] tpm_crb: map locality registers

2016-12-03 Thread Jarkko Sakkinen
In order to provide access to locality registers, this commits adds mapping of the head of the CRB registers, which are located right before the control area. Signed-off-by: Jarkko Sakkinen --- drivers/char/tpm/tpm_crb.c | 96 ++ 1 file changed, 64 ins

Re: [PATCH] Input: touchscreen: edt_ft5x06 - change msleep to usleep_range for small msecs

2016-12-03 Thread Simon Budig
Hello Mr, Mathur. On 29/11/16 21:54, Aniroop Mathur wrote: > I have submitted one patch as below for review to Linux Open Source. > The problem is that we do not have the hardware available with us to > test it and we would like to test it before actually applying it. > As you are the author of th

Re: net: use-after-free in worker_thread

2016-12-03 Thread Cong Wang
On Sat, Dec 3, 2016 at 9:41 AM, Cong Wang wrote: > On Sat, Dec 3, 2016 at 4:56 AM, Andrey Konovalov > wrote: >> Hi! >> >> I'm seeing lots of the following error reports while running the >> syzkaller fuzzer. >> >> Reports appeared when I updated to 3c49de52 (Dec 2) from 2caceb32 (Dec 1). >> >> =

Re: [PATCH] Input: touchscreen: edt_ft5x06 - change msleep to usleep_range for small msecs

2016-12-03 Thread Aniroop Mathur
Hello Mr. Simon, On Sat, Dec 3, 2016 at 10:58 PM, Simon Budig wrote: > Hello Mr, Mathur. > > On 29/11/16 21:54, Aniroop Mathur wrote: >> I have submitted one patch as below for review to Linux Open Source. >> The problem is that we do not have the hardware available with us to >> test it and we w

Re: scsi: use-after-free in bio_copy_from_iter

2016-12-03 Thread Johannes Thumshirn
On Sat, Dec 03, 2016 at 04:22:39PM +0100, Dmitry Vyukov wrote: > On Sat, Dec 3, 2016 at 11:38 AM, Johannes Thumshirn > wrote: > > On Fri, Dec 02, 2016 at 05:50:39PM +0100, Dmitry Vyukov wrote: > >> On Fri, Nov 25, 2016 at 8:08 PM, Dmitry Vyukov wrote: [...] Hi Dmitry, > > Thanks for looking

Re: [PATCH v2 1/7] Input: synaptics-rmi4 - add support for F03

2016-12-03 Thread Dmitry Torokhov
Hi Benjamin, On Fri, Dec 02, 2016 at 12:41:50PM +0100, Benjamin Tissoires wrote: > From: Lyude Paul > > This adds basic functionality for PS/2 passthrough on Synaptics > Touchpads using RMI4 through smbus. > > Reviewed-by: Andrew Duggan > Signed-off-by: Lyude Paul > Signed-off-by: Benjamin Ti

Re: [PATCH v2 4/7] Input: synaptics-rmi4 - store the attn data in the driver

2016-12-03 Thread Dmitry Torokhov
On Fri, Dec 02, 2016 at 04:34:01PM -0800, Andrew Duggan wrote: > On 12/02/2016 03:41 AM, Benjamin Tissoires wrote: > >Now that we have a proper API to set the attention data, there is > >no point in keeping it in the transport driver. > > > >Signed-off-by: Benjamin Tissoires > > Reviewed-by: Andr

Re: [PATCH v2 3/7] Input: synaptics-rmi4 - allow to add attention data

2016-12-03 Thread Dmitry Torokhov
On Fri, Dec 02, 2016 at 04:33:56PM -0800, Andrew Duggan wrote: > On 12/02/2016 03:41 AM, Benjamin Tissoires wrote: > >The HID implementation of RMI4 provides the data during > >the interrupt (in the input report). We need to provide > >a way for this transport driver to provide the attention > >dat

Re: [PATCH 1/1] btrfs: volumes: fix improper return value

2016-12-03 Thread Omar Sandoval
On Sat, Dec 03, 2016 at 07:01:45PM +0800, Pan Bian wrote: > Variable ret takes the errno on failures. However, it directly returns 0. > It may be better to return "ret". > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188741 > > Signed-off-by: Pan Bian > --- > fs/btrfs/volumes.c | 2 +

Re: [PATCH v2 0/7] RMI4 cleanups and switch hid-rmi to rmi_core

2016-12-03 Thread Dmitry Torokhov
On Fri, Dec 02, 2016 at 12:41:49PM +0100, Benjamin Tissoires wrote: > Hi, > > Well, the cleanup part of the series has already been applied, so > I guess this is just "switch hid-rmi to rmi_core". > > The series applies on top of Dmitri's synaptics-rmi4 branch > > However, as mentioned in v1, I'

[PATCH V2 00/15] Drivers: hv: CPU management fixes and a new uio driver

2016-12-03 Thread kys
From: K. Y. Srinivasan Fixes to handle CPU online/offline. Also included is a new uio driver for Hyper-V. V2: Updated commit logs (Greg KH) Re-implemented the API to detect if it is a vmbus device Alex Fluter (1): Tools: hv: kvp: configurable external scripts path Haiyang

[PATCH V2 01/15] Drivers: hv: vmbus: Raise retry/wait limits in vmbus_post_msg()

2016-12-03 Thread kys
From: Vitaly Kuznetsov DoS protection conditions were altered in WS2016 and now it's easy to get -EAGAIN returned from vmbus_post_msg() (e.g. when we try changing MTU on a netvsc device in a loop). All vmbus_post_msg() callers don't retry the operation and we usually end up with a non-functional

[PATCH V2 04/15] Drivers: hv: vmbus: Enhance the rescind callback functionality

2016-12-03 Thread kys
From: K. Y. Srinivasan Enhance the rescind callback functionality by permitting the passing of an opaque pointer. This functionality will be used by vmbus device drivers to implement rescind related cleanup more efficiently. Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel_mgmt.c | 11

[PATCH V2 05/15] hv: acquire vmbus_connection.channel_mutex in vmbus_free_channels()

2016-12-03 Thread kys
From: Vitaly Kuznetsov "kernel BUG at drivers/hv/channel_mgmt.c:350!" is observed when hv_vmbus module is unloaded. BUG_ON() was introduced in commit 85d9aa705184 ("Drivers: hv: vmbus: add an API vmbus_hvsock_device_unregister()") as vmbus_free_channels() codepath was apparently forgotten. Fixes

Re: [PATCH v2 0/3] Locality support for tpm_crb

2016-12-03 Thread Jarkko Sakkinen
CC linux-security-module On Sat, Dec 03, 2016 at 07:52:10PM +0200, Jarkko Sakkinen wrote: > The tpm_crb driver should follow the policy of reserving and > relinquishing the locality it uses when multiple localities are used, > like when TXT is another locality. > > Jarkko Sakkinen (3): > tpm_cr

[PATCH V2 15/15] hyperv: Add a function to detect if the device is a vmbus dev

2016-12-03 Thread kys
From: Haiyang Zhang On Hyper-V, every VF interface has a corresponding synthetic interface managed by netvsc that share the same MAC address. netvsc registers for netdev events to manage this association. Currently we use the MAC address to manage this association but going forward, we want to us

[PATCH V2 03/15] Drivers: hv: vmbus: Prevent sending data on a rescinded channel

2016-12-03 Thread kys
From: K. Y. Srinivasan After the channel is rescinded, the host does not read from the rescinded channel. Fail writes to a channel that has already been rescinded. Signed-off-by: K. Y. Srinivasan --- drivers/hv/ring_buffer.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff -

[PATCH V2 07/15] hv: init percpu_list in hv_synic_alloc()

2016-12-03 Thread kys
From: Vitaly Kuznetsov Initializing hv_context.percpu_list in hv_synic_alloc() helps to prevent a crash in percpu_channel_enq() when not all CPUs were online during initialization and it naturally belongs there. Signed-off-by: Vitaly Kuznetsov Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv

Re: [PATCH v2 1/3] tpm_crb: map locality registers

2016-12-03 Thread Jarkko Sakkinen
CC linux-security-module On Sat, Dec 03, 2016 at 07:52:11PM +0200, Jarkko Sakkinen wrote: > In order to provide access to locality registers, this commits adds > mapping of the head of the CRB registers, which are located right > before the control area. > > Signed-off-by: Jarkko Sakkinen > ---

[PATCH V2 02/15] hyperv: Fix spelling of HV_UNKOWN

2016-12-03 Thread kys
From: Haiyang Zhang Changed it to HV_UNKNOWN Signed-off-by: Haiyang Zhang Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel_mgmt.c |6 +++--- include/linux/hyperv.h|2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv/cha

[PATCH V2 09/15] hv: switch to cpuhp state machine for synic init/cleanup

2016-12-03 Thread kys
From: Vitaly Kuznetsov To make it possible to online/offline CPUs switch to cpuhp infrastructure for doing hv_synic_init()/hv_synic_cleanup(). Signed-off-by: Vitaly Kuznetsov Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv.c | 15 +++ drivers/hv/hyperv_vmbus.h |4

[PATCH V2 10/15] hv: make CPU offlining prevention fine-grained

2016-12-03 Thread kys
From: Vitaly Kuznetsov Since commit e513229b4c38 ("Drivers: hv: vmbus: prevent cpu offlining on newer hypervisors") cpu offlining was disabled. It is still true that we can't offline CPUs which have VMBus channels bound to them but we may have 'free' CPUs (e.v. we booted with maxcpus= parameter a

[PATCH V2 12/15] vmbus: add support for dynamic device id's

2016-12-03 Thread kys
From: Stephen Hemminger This patch adds sysfs interface to dynamically bind new UUID values to existing VMBus device. This is useful for generic UIO driver to act similar to uio_pci_generic. Signed-off-by: Stephen Hemminger Signed-off-by: K. Y. Srinivasan --- drivers/hv/vmbus_drv.c | 174 +++

[PATCH V2 11/15] hv: don't reset hv_context.tsc_page on crash

2016-12-03 Thread kys
From: Vitaly Kuznetsov It may happen that secondary CPUs are still alive and resetting hv_context.tsc_page will cause a consequent crash in read_hv_clock_tsc() as we don't check for it being not NULL there. It is safe as we're not freeing this page anyways. Signed-off-by: Vitaly Kuznetsov Signe

[PATCH V2 06/15] hv: allocate synic pages for all present CPUs

2016-12-03 Thread kys
From: Vitaly Kuznetsov It may happen that not all CPUs are online when we do hv_synic_alloc() and in case more CPUs come online later we may try accessing these allocated structures. Signed-off-by: Vitaly Kuznetsov Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv.c |4 ++-- 1 files chang

[PATCH V2 14/15] Tools: hv: kvp: configurable external scripts path

2016-12-03 Thread kys
From: Alex Fluter error when running hypervkvpd: $ sudo ./hv_kvp_daemon -n sh: hv_get_dns_info: command not found sh: hv_get_dhcp_info: command not found sh: hv_get_dns_info: command not found sh: hv_get_dhcp_info: command not found The external scripts are not installed in system path, adding

[PATCH V2 08/15] hv: change clockevents unbind tactics

2016-12-03 Thread kys
From: Vitaly Kuznetsov To get prepared to CPU offlining support we need co change the way how we unbind clockevent devices. As one CPU may go online/offline multiple times we need to bind it in hv_synic_init() and unbind it in hv_synic_cleanup(). There is an additional corner case: when we unload

Re: [PATCH v2 2/3] tpm_crb: encapsulate crb_wait_for_reg_32

2016-12-03 Thread Jarkko Sakkinen
CC linux-security-module On Sat, Dec 03, 2016 at 07:52:12PM +0200, Jarkko Sakkinen wrote: > Encapsulated crb_wait_for_reg32() so that state changes in other CRB > registers than TPM_CRB_CTRL_REQ_x can be waited. > > Signed-off-by: Jarkko Sakkinen > --- > drivers/char/tpm/tpm_crb.c | 37

[PATCH V2 13/15] uio-hv-generic: new userspace i/o driver for VMBus

2016-12-03 Thread kys
From: Stephen Hemminger This is a new driver to enable userspace networking on VMBus. It is based largely on the similar driver that already exists for PCI, and earlier work done by Brocade to support DPDK. Signed-off-by: Stephen Hemminger Signed-off-by: K. Y. Srinivasan --- MAINTAINERS

Re: [PATCH v2 3/3] tpm_crb: request and relinquish locality 0

2016-12-03 Thread Jarkko Sakkinen
CC linux-security-module On Sat, Dec 03, 2016 at 07:52:13PM +0200, Jarkko Sakkinen wrote: > Request and relinquish locality for the driver use in order to be > a better citizen in a multi locality environment like TXT. The > locality is requested and relinquished as part of going into and > waking

Re: [PATCH] Input: elantech - Add a special mode for a specific FW The touchapd which sample ver is 0x74 and hw_version is 0x03 have a fw bug which will cause crush sometimes, I add some work-around f

2016-12-03 Thread Dmitry Torokhov
On Fri, Dec 02, 2016 at 11:05:29PM +0100, ulrik.debie...@e2big.org wrote: > Hi, > > Thank you for the patch, see below my feedback on your patch. > Can you provide the contents of fw_verison, capabilities and samples ? > > It this fw bug present on multiple laptops ? > > > > On Fri, Dec 02, 2

Re: [PATCH] ARM: pxa: ezx: fix a910 camera data

2016-12-03 Thread Stefan Schmidt
Hello. On 03.12.2016 17:55, Antonio Ospite wrote: > On Sat, 26 Nov 2016 00:30:52 +0100 > Stefan Schmidt wrote: > >> Hello. >> > > Hi everyone, > >> On 25.11.2016 20:53, Robert Jarzmik wrote: >>> Stefan Schmidt writes: >>> Hello. On 24.11.2016 17:29, Arnd Bergmann wrote: > T

Re: [PATCH 1/1] netdev: broadcom: propagate error code

2016-12-03 Thread Michael Chan
On Sat, Dec 3, 2016 at 1:56 AM, Pan Bian wrote: > Function bnxt_hwrm_stat_ctx_alloc() always returns 0, even if the call > to _hwrm_send_message() fails. It may be better to propagate the errors > to the caller of bnxt_hwrm_stat_ctx_alloc(). > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?i

Re: [PATCH 11/39] Annotate hardware config module parameters in drivers/input/

2016-12-03 Thread Dmitry Torokhov
On Thu, Dec 01, 2016 at 12:31:03PM +, David Howells wrote: > When the kernel is running in secure boot mode, we lock down the kernel to > prevent userspace from modifying the running kernel image. Whilst this > includes prohibiting access to things like /dev/mem, it must also prevent > access

[PATCH V2 1/3 libnvdimm-pending] libnvdimm: remove else after return in nsio_rw_bytes()

2016-12-03 Thread Fabian Frederick
else after return is not needed. Signed-off-by: Fabian Frederick --- V2: -applied on top of libnvdimm-pending drivers/nvdimm/claim.c | 37 ++--- 1 file changed, 18 insertions(+), 19 deletions(-) diff --git a/drivers/nvdimm/claim.c b/drivers/nvdimm/claim.c index

Re: [PATCH 1/1] btrfs: fix improper return value

2016-12-03 Thread Omar Sandoval
On Sat, Dec 03, 2016 at 06:55:16PM +0800, Pan Bian wrote: > In function btrfs_uuid_tree_iterate(), errno is assigned to variable > ret on errors. However, it directly returns 0. It may be better to > return ret. > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188731 > > Signed-off-by: P

[PATCH V2 2/3 libnvdimm-pending] libnvdimm, namespace: avoid multiple sector calculations

2016-12-03 Thread Fabian Frederick
Use sector_t for cleared Suggested-by: Dan Williams Signed-off-by: Fabian Frederick --- V2: -applied on top of libnvdimm-pending drivers/nvdimm/pmem.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c index 0f47062..9e

[PATCH V2 3/3 libnvdimm-pending] libnvdimm, namespace: use octal for permissions

2016-12-03 Thread Fabian Frederick
According to commit f90774e1fd27 ("checkpatch: look for symbolic permissions and suggest octal instead") Signed-off-by: Fabian Frederick --- V2: applied on top of libnvdimm-pending Use calculated value instead of a | b (Suggested by Dan Williams) drivers/nvdimm/namespace_devs.c | 4 ++-- 1

Re: Unkillable processes due to PTRACE_TRACEME again

2016-12-03 Thread Pavel Machek
Hi! You cc-ed me on initial mail, I replied, and you removed me from cc. Ouch. On Sat 2016-12-03 16:55:49, Dmitry Vyukov wrote: > On Fri, Dec 2, 2016 at 10:02 PM, Pavel Machek wrote: > > On Fri 2016-12-02 19:48:40, Dmitry Vyukov wrote: > >> Hello, > >> > >> There was an issue discussed a year ag

Re: [PATCH v7] Input: synaptics-rmi4 - add support for F34 V7 bootloader

2016-12-03 Thread Dmitry Torokhov
Hi Nick, On Tue, Nov 29, 2016 at 10:49:55PM +, Nick Dyer wrote: > diff --git a/drivers/input/rmi4/rmi_f34.h b/drivers/input/rmi4/rmi_f34.h > index 6cee528..002677e 100644 > --- a/drivers/input/rmi4/rmi_f34.h > +++ b/drivers/input/rmi4/rmi_f34.h > @@ -33,6 +33,271 @@ > > #define F34_BOOTLOAD

[PATCH net-next] r8169: Add support for restarting auto-negotiation

2016-12-03 Thread Florian Fainelli
Implement ethtooll::nway_restart by utilizing mii_nway_restart. Signed-off-by: Florian Fainelli --- drivers/net/ethernet/realtek/r8169.c | 8 1 file changed, 8 insertions(+) diff --git a/drivers/net/ethernet/realtek/r8169.c b/drivers/net/ethernet/realtek/r8169.c index 2830190aaace..f9

Re: [PATCH V2 net-next] net: hns: Fix to conditionally convey RX checksum flag to stack

2016-12-03 Thread David Miller
From: Salil Mehta Date: Thu, 1 Dec 2016 12:09:22 + > But maybe now since we don't have any method to de-multiplex the kind of > checksum error (cannot depend upon register) we can have below code > re-arrangement: > > hns_nic_rx_checksum() { > /* check supported L3 protocol */ >

Re: [PATCH] stmmac: cleanup documenation, make it match reality

2016-12-03 Thread David Miller
From: Pavel Machek Date: Thu, 1 Dec 2016 11:32:18 +0100 > Fix english in documentation, make documentation match reality, remove > options that were removed from code. > > Signed-off-by: Pavel Machek Applied.

Re: [PATCH] irda: w83977af_ir: fix damaged whitespace

2016-12-03 Thread David Miller
From: Arnd Bergmann Date: Mon, 28 Nov 2016 15:19:43 +0100 > As David Miller pointed out for for the previous patch, the whitespace > in some functions looks rather odd. This was caused by commit 6329da5f258a > ("obsolete config in kernel source: USE_INTERNAL_TIMER"), which removed > some conditio

Re: [PATCH v5] PM/devfreq: add suspend frequency support

2016-12-03 Thread Pavel Machek
Hi! > diff --git a/include/linux/devfreq.h b/include/linux/devfreq.h > index 98c6993..8567153 100644 > --- a/include/linux/devfreq.h > +++ b/include/linux/devfreq.h > @@ -172,6 +172,7 @@ struct devfreq { > struct delayed_work work; > > unsigned long previous_freq; > + unsigned lo

Re: [PATCH] staging: greybus: Fix macro definition

2016-12-03 Thread kbuild test robot
Hi ChinmayVS, [auto build test ERROR on staging/staging-testing] [also build test ERROR on v4.9-rc7 next-20161202] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Chinmay-VS/staging-greybus-Fix-m

Re: [PATCH V2 net-next] net: hns: Fix to conditionally convey RX checksum flag to stack

2016-12-03 Thread David Miller
From: Salil Mehta Date: Thu, 1 Dec 2016 16:59:14 + > It looks to me the cumbersome check in the PATCH V2 should > be retained. I really want something simpler with small checks that are done in logical pieces in a straigtforward progression. The code in V2 is completely unreadable.

Re: [PATCH v7 0/4] Enabling Ring 3 MONITOR/MWAIT feature for Knights Landing

2016-12-03 Thread Pavel Machek
On Fri 2016-10-28 15:49:51, Grzegorz Andrejczuk wrote: > These patches enable Intel Xeon Phi x200 feature to use MONITOR/MWAIT > instruction in ring 3 (userspace) Patches set MSR 0x140 for all logical CPUs. > Then expose it as CPU feature and introduces elf HWCAP capability > for x86. Elf hardwa

Re: [SPCR] mmio32 iotype access requirements for X-Gene 8250(_dw) UART

2016-12-03 Thread Jon Masters
Hi Mark, On 12/03/2016 12:15 PM, Mark Salter wrote: > On Sat, 2016-12-03 at 05:06 -0500, Jon Masters wrote: >> There is a broader problem with X-Gene SPCR support. The problem is >> that the 16550 UART in X-Gene requires the 32-bit access quirk (the >> iotype is set to UPIO_MEM32 for the APMC0D08

Re: [PATCH 2/2] net: stmmac: unify mdio functions

2016-12-03 Thread David Miller
From: Corentin Labbe Date: Thu, 1 Dec 2016 16:19:41 +0100 > stmmac_mdio_{read|write} and stmmac_mdio_{read|write}_gmac4 are not > enought different for being split. > The only differences between thoses two functions are shift/mask for > addr/reg/clk_csr. > > This patch introduce a per platform

Re: [PATCH 1/2] net: stmmac: avoid Camelcase naming

2016-12-03 Thread David Miller
From: Corentin Labbe Date: Thu, 1 Dec 2016 16:19:40 +0100 > This patch simply rename regValue to value, like it was named in other > mdio functions. > > Signed-off-by: Corentin Labbe Applied.

Re: [PATCH 1/7] net: ethernet: ti: cpdma: am437x: allow descs to be plased in ddr

2016-12-03 Thread David Miller
From: Grygorii Strashko Date: Thu, 1 Dec 2016 17:34:26 -0600 > @@ -167,10 +167,10 @@ static struct cpdma_control_info controls[] = { > > /* various accessors */ > #define dma_reg_read(ctlr, ofs) __raw_readl((ctlr)->dmaregs + > (ofs)) > -#define chan_read(chan, fld) __raw

[git pull] drm fixes for v.4.9

2016-12-03 Thread Dave Airlie
Hi Linus, I awoke this morning to realise I hadn't sent my -fixes pull, I then discovered the office where my build and sign tags machine had a power cut, and since nobody will be in until tomorrow I can't restart my desktop. So this tag is unsigned due to that and the realisation I don't keep my

[PATCH] ARM: OMAP2+: PRM: Delete an error message for a failed memory allocation

2016-12-03 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 3 Dec 2016 21:46:02 +0100 Omit an extra message for a memory allocation failure in this function. Link: http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refactor_Strings-WSang_0.pdf Signed-off-by: Markus Elfring --- arch/arm/mach-omap2/prm_comm

Re: [PATCH 1/1] net: caif: fix ineffective error check

2016-12-03 Thread Sergei Shtylyov
On 12/03/2016 06:38 PM, Pan Bian wrote: In function caif_sktinit_module(), the check of the return value of sock_register() seems ineffective. This patch fixes it. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=188751 Signed-off-by: Pan Bian --- net/caif/caif_socket.c | 2 +- 1 file ch

Re: [PATCH 1/6] net: stmmac: return error if no DMA configuration is found

2016-12-03 Thread David Miller
When you post a series of related changes as a patch set, you must provide a proper "[PATCH 0/N] ..." posting which explains what the series is doing at a high level, how it is doing it, and why it is doing it that way. Please repost this entire series with a proper header posting included. Than

Re: [PATCH] dt-bindings: add MYIR Tech hardware vendor prefix

2016-12-03 Thread Rob Herring
On Mon, Nov 28, 2016 at 09:56:51AM +0200, Vladimir Zapolskiy wrote: > MYIR Tech Limited offers a range of ARM powered development boards and SoMs, > for details reference a list on http://elinux.org/Development_Platforms#ARM > or company's website http://myirtech.com > > Signed-off-by: Vladimir Za

Re: [RFC PATCH 2/2] Documentation: devictree: Add macb mdio bindings

2016-12-03 Thread Rob Herring
On Mon, Nov 28, 2016 at 03:19:27PM +0530, Harini Katakam wrote: > Add documentations for macb mdio driver. Bindings document h/w, not drivers. > > Signed-off-by: Harini Katakam > --- > .../devicetree/bindings/net/macb-mdio.txt | 31 > ++ > 1 file changed, 31 inser

Re: [PATCH 1/1 linux-next] ext4: fix block_validity definition

2016-12-03 Thread Theodore Ts'o
I've applied this patch with some further cleanups. The documentation for the mount options now reads: block_validity(*) These options enable or disable the in-kernel noblock_validityfacility for tracking filesystem metadata blocks within internal data struct

Re: [PATCH 2/2 v2] sched: use load_avg for selecting idlest group

2016-12-03 Thread Matt Fleming
On Fri, 02 Dec, at 07:31:04PM, Brendan Gregg wrote: > > For background, is this from the "A decade of wasted cores" paper's > patches? No, this patch fixes an issue I originally reported here, https://lkml.kernel.org/r/20160923115808.2330-1-m...@codeblueprint.co.uk Essentially, if you have an

Re: [PATCH 0/6] UBIFS related fscrypt updates

2016-12-03 Thread Theodore Ts'o
On Thu, Dec 01, 2016 at 11:14:52PM +0100, Richard Weinberger wrote: > This series applies on top of Ted's fscrypt tree[0] addresses the review > comments from Eric. > Ted, it would be awesome to have this patches in the v4.10 merge window. Your patch 4/6 won't apply on my branch because it referen

Re: [PATCH 39/39] mtd: nand: denali_dt: add compatible strings for UniPhier SoC variants

2016-12-03 Thread Dinh Nguyen
Hi, On Fri, Dec 2, 2016 at 8:49 PM, Marek Vasut wrote: > On 12/03/2016 03:41 AM, Masahiro Yamada wrote: >> Hi Rob, > > Hi! > >> 2016-12-03 1:26 GMT+09:00 Rob Herring : >> (Plan A) "denali,socfpga-nand" (for Altera SOCFPGA variant) "denali,uniphier-nand-v1"

[PATCH v2 1/3] ARM: dts: vf610-zii-dev-rev-b: Remove leftover PWM pingroup

2016-12-03 Thread Andrey Smirnov
Remove pwm0grp since it is: a) Not referenced anywhere in the DTS file (unlike Tower board it is based on, this board does not use/expose FTM0) b) Configures PTB2 and PTB3 in a way that contradicts pinctrl-mdio-mux Signed-off-by: Andrey Smirnov --- No changes si

[PATCH v2 2/3] ARM: dts: vf610-zii-dev: Add .dts file for rev. C

2016-12-03 Thread Andrey Smirnov
Add .dts file for rev. C of the board by factoring out commonalities into a shared include file (vf610-zii-dev.dtsi) and deriving revision specific file from it (vf610-zii-dev-rev-b.dts and vf610-zii-dev-reb-c.dts). Signed-off-by: Andrey Smirnov --- Changes since v1: - Removed fixed-link

[PATCH v2 3/3] ARM: dts: vf610-zii-dev-rev-b: Remove 'fixed-link' from DSA ports

2016-12-03 Thread Andrey Smirnov
Remove 'fixed-link' nodes from DSA ports since they are not needed (they are not limiting link's speed and the ports will be configured to their maximux speed as a default) Suggested-by: Andrew Lunn Signed-off-by: Andrey Smirnov --- Changes since v1: - Patch was not present in v1 arch

Re: [RFC PATCH 2/2] Documentation: devictree: Add macb mdio bindings

2016-12-03 Thread Florian Fainelli
Le 12/03/16 à 13:35, Rob Herring a écrit : > On Mon, Nov 28, 2016 at 03:19:27PM +0530, Harini Katakam wrote: >> Add documentations for macb mdio driver. > > Bindings document h/w, not drivers. > >> >> Signed-off-by: Harini Katakam >> --- >> .../devicetree/bindings/net/macb-mdio.txt | 3

Re: [PATCH] tty: serial: fsl_lpuart: fix del_timer_sync() vs timer routine deadlock

2016-12-03 Thread Bhuvanchandra DV
On 12/03/2016 02:58 AM, Nikita Yushchenko wrote: Problem found via lockdep: - lpuart_set_termios() calls del_timer_sync(&sport->lpuart_timer) while holding sport->port.lock - sport->lpuart_timer routine is lpuart_timer_func() that calls lpuart_copy_rx_to_tty() that acquires same lock. T

Re: [PATCH] ARM: dts: sunxi: Add num-cs for A20 spi nodes

2016-12-03 Thread kbuild test robot
Hi Emmanuel, [auto build test ERROR on robh/for-next] [also build test ERROR on v4.9-rc7 next-20161202] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Emmanuel-Vadot/ARM-dts-sunxi-Add-num-cs-for

Re: [PATCH 3/6] net: ethernet: ti: cpts: add support of cpts HW_TS_PUSH

2016-12-03 Thread Richard Cochran
On Mon, Nov 28, 2016 at 05:04:25PM -0600, Grygorii Strashko wrote: > This also change overflow polling period when HW_TS_PUSH feature is > enabled - overflow check work will be scheduled more often (every > 200ms) for proper HW_TS_PUSH events reporting. For proper reporting, you should make use of

<    1   2   3   >