Re: [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags

2016-12-14 Thread Marcel Holtmann
Hi Michael, > That's the default now, no need for makefiles to set it. > > Signed-off-by: Michael S. Tsirkin > --- > drivers/bluetooth/Makefile| 2 -- > drivers/net/can/Makefile | 1 - >

Re: [PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags

2016-12-14 Thread Kalle Valo
"Michael S. Tsirkin" writes: > That's the default now, no need for makefiles to set it. > > Signed-off-by: Michael S. Tsirkin > --- > drivers/bluetooth/Makefile| 2 -- > drivers/net/can/Makefile

[PATCH 8/8] Makefile: drop -D__CHECK_ENDIAN__ from cflags

2016-12-14 Thread Michael S. Tsirkin
That's the default now, no need for makefiles to set it. Signed-off-by: Michael S. Tsirkin --- drivers/bluetooth/Makefile| 2 -- drivers/net/can/Makefile | 1 - drivers/net/ethernet/altera/Makefile

Re: [PATCH] staging : osc : Remove braces from single-line body

2016-12-14 Thread Greg KH
On Thu, Dec 15, 2016 at 07:03:52AM +0530, Tabrez khan wrote: > Remove unnecessary braces {} for single if statement block. > This warning is found using checkpatch.pl. > > Signed-off-by: Tabrez khan > --- > drivers/staging/lustre/lustre/osc/osc_cache.c | 4 +--- > 1

[PATCH] staging : osc : Remove braces from single-line body

2016-12-14 Thread Tabrez khan
Remove unnecessary braces {} for single if statement block. This warning is found using checkpatch.pl. Signed-off-by: Tabrez khan --- drivers/staging/lustre/lustre/osc/osc_cache.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

[PATCH] staging: octeon: Call SET_NETDEV_DEV()

2016-12-14 Thread Florian Fainelli
The Octeon driver calls into PHYLIB which now checks for net_device->dev.parent, so make sure we do set it before calling into any MDIO/PHYLIB related function. Fixes: ec988ad78ed6 ("phy: Don't increment MDIO bus refcount unless it's a different owner") Reported-by: Aaro Koskinen

[PATCH 5/6] storvsc: properly handle SRB_ERROR when sense message is present

2016-12-14 Thread kys
From: Long Li When sense message is present on error, we should pass along to the upper layer to decide how to deal with the error. This patch fixes connectivity issues with Fiber Channel devices. Signed-off-by: Long Li Reviewed-by: K. Y. Srinivasan

[PATCH 2/6] storvsc: Remove the restriction on max segment size

2016-12-14 Thread kys
From: K. Y. Srinivasan Remove the artificially imposed restriction on max segment size. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- drivers/scsi/storvsc_drv.c |2 -- 1 files changed, 0 insertions(+), 2

[PATCH 1/6] storvsc: Enable tracking of queue depth

2016-12-14 Thread kys
From: K. Y. Srinivasan Enable tracking of queue depth. Signed-off-by: K. Y. Srinivasan Reviewed-by: Long Li --- drivers/scsi/storvsc_drv.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git

[PATCH 3/6] storvsc: Enable multi-queue support

2016-12-14 Thread kys
From: K. Y. Srinivasan Enable multi-q support. We will allocate the outgoing channel using the following policy: 1. We will make every effort to pick a channel that is in the same NUMA node that is initiating the I/O 2. The mapping between the

[PATCH 6/6] storvsc: properly set residual data length on errors

2016-12-14 Thread kys
From: Long Li On I/O errors, the Windows driver doesn't set data_transfer_length on error conditions other than SRB_STATUS_DATA_OVERRUN. In these cases we need to set data_transfer_length to 0, indicating there is no data transferred. On SRB_STATUS_DATA_OVERRUN,

[PATCH 4/6] storvsc: use tagged SRB requests if supported by the device

2016-12-14 Thread kys
From: Long Li Properly set SRB flags when hosting device supports tagged queuing. This patch improves the performance on Fiber Channel disks. Signed-off-by: Long Li Reviewed-by: K. Y. Srinivasan Signed-off-by: K. Y. Srinivasan

[PATCH 0/6] storvsc: Miscellaneous fixes and enhancements

2016-12-14 Thread kys
From: K. Y. Srinivasan Miscellaneous fixes and enhancements. K. Y. Srinivasan (3): storvsc: Enable tracking of queue depth storvsc: Remove the restriction on max segment size storvsc: Enable multi-queue support Long Li (3): storvsc: use tagged SRB requests if

[RFC PATCH 3/4] staging: android: ion: Remove page faulting support

2016-12-14 Thread Laura Abbott
Unclear if this is wanted or needed? Not-signed-off-by: Laura Abbott --- drivers/staging/android/ion/ion.c | 74 --- 1 file changed, 74 deletions(-) diff --git a/drivers/staging/android/ion/ion.c b/drivers/staging/android/ion/ion.c

[RFC PATCH 1/4] staging: android: ion: Some cleanup

2016-12-14 Thread Laura Abbott
- dmap_cnt isn't used. Remove it. - Ion has been using dma apis incorrectly to sync the caches. Remove bad usage in preparation for something better. - The alignment field doesn't actually change the alignment of an allocation, it only serves as an error check. This is basically pointless.

[RFC PATCH 4/4] staging: android: ion: Call dma_map_sg for syncing and mapping

2016-12-14 Thread Laura Abbott
Technically, calling dma_buf_map_attachment should return a buffer properly dma_mapped. Add calls to dma_map_sg to begin_cpu_access to ensure this happens. As a side effect, this lets Ion buffers take advantage of the dma_buf sync ioctls. Not-signed-off-by: Laura Abbott ---

[RFC PATCH 0/4] Ion caching (yet again) proof of concept

2016-12-14 Thread Laura Abbott
Hi, I've been (once again) looking at alternate caching models for Ion. Part of this work is also to make Ion fit better in to the dma_buf model. Ion is a bit unusual for dma_buf. Most drivers that support dma_buf have two parts: exporting buffers that a driver allocates and importing buffers

[RFC PATCH 2/4] staging: android: ion: Duplicate sg_table

2016-12-14 Thread Laura Abbott
Ion currently returns a single sg_table on each dma_map call. This is incorrect for later usage. Not-signed-off-by: Laura Abbott --- drivers/staging/android/ion/ion.c | 32 +++- 1 file changed, 31 insertions(+), 1 deletion(-) diff --git

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-14 Thread Stephen Hemminger
On Wed, 14 Dec 2016 15:27:58 -0800 Greg KH wrote: > On Wed, Dec 14, 2016 at 11:18:59PM +, Haiyang Zhang wrote: > > > > > > > -Original Message- > > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > > Sent: Saturday, December 10, 2016 7:21 AM > >

Re: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-14 Thread Greg KH
On Wed, Dec 14, 2016 at 11:18:59PM +, Haiyang Zhang wrote: > > > > -Original Message- > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Saturday, December 10, 2016 7:21 AM > > To: Stephen Hemminger > > Cc: Haiyang Zhang

RE: [PATCH 3/3] hv_netvsc: Implement VF matching based on serial numbers

2016-12-14 Thread Haiyang Zhang
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Saturday, December 10, 2016 7:21 AM > To: Stephen Hemminger > Cc: Haiyang Zhang ; o...@aepfle.de; > jasow...@redhat.com; linux-ker...@vger.kernel.org; >

Re: [PATCH] staging : i4l : Remove blank lines

2016-12-14 Thread Greg KH
On Thu, Dec 15, 2016 at 02:49:49AM +0530, Tabrez khan wrote: > Remove unnecessary blank lines. > This warning was found using checkpatch.pl. > > Signed-off-by: Tabrez khan > --- > drivers/staging/i4l/act2000/module.c | 1 - > 1 file changed, 1 deletion(-) > > diff

Re: [PATCH] staging : osc : coding style fix

2016-12-14 Thread Dilger, Andreas
On Dec 14, 2016, at 14:11, Tabrez khan wrote: > > Subject: [PATCH] staging : osc : coding style fix Thanks for sumbitting your patch. As a general rule, the patch summary line should try to describe (as best as possible in a single line) what the patch is actually

[PATCH] staging : osc : coding style fix

2016-12-14 Thread Tabrez khan
Remove unnecessary braces {} for single if statement block. This warning is found using checkpatch.pl. Signed-off-by: Tabrez khan --- drivers/staging/lustre/lustre/osc/osc_cache.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git

Re: [PATCH] staging : i4l : Remove blank lines

2016-12-14 Thread Greg KH
On Thu, Dec 15, 2016 at 01:38:18AM +0530, Tabrez Khan wrote: > From: Tabrez khan > > Remove blank lines. That says what you do, but you did not describe _why_ you did it, which is more important (hint, we can read the change and see what you did...) thanks, greg k-h

[PATCH] staging : i4l : Remove blank lines

2016-12-14 Thread Tabrez Khan
From: Tabrez khan Remove blank lines. Signed-off-by: Tabrez khan --- drivers/staging/i4l/act2000/module.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/i4l/act2000/module.c b/drivers/staging/i4l/act2000/module.c index

[driver-core:usermode_ro 4/4] fs/nfs/cache_lib.c:67:28: error: assignment of read-only location 'nfs_cache_getent_prog[0]'

2016-12-14 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git usermode_ro head: 00ed986b7a8fbb11b0c154e8dda5ee7c5ea48a43 commit: 00ed986b7a8fbb11b0c154e8dda5ee7c5ea48a43 [4/4] Introduce CONFIG_READONLY_USERMODEHELPER config: i386-allmodconfig (attached as .config) compiler:

[driver-core:usermode_ro 3/4] drivers/staging/rtl8192e/rtl8192e/rtl_dm.c:272:18: warning: initialization discards 'const' qualifier from pointer target type

2016-12-14 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git usermode_ro head: 00ed986b7a8fbb11b0c154e8dda5ee7c5ea48a43 commit: d57c93cde22bd5ec716de09bf84ae8576adf5e64 [3/4] Make static usermode helper binaries constant config: i386-randconfig-x011-201650 (attached as

[driver-core:usermode_ro 3/4] drivers/net/hamradio/baycom_epp.c:311:26: warning: initialization discards 'const' qualifier from pointer target type

2016-12-14 Thread kbuild test robot
tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git usermode_ro head: 00ed986b7a8fbb11b0c154e8dda5ee7c5ea48a43 commit: d57c93cde22bd5ec716de09bf84ae8576adf5e64 [3/4] Make static usermode helper binaries constant config: i386-randconfig-s1-201650 (attached as .config)

RE: [PATCH] drivers: staging: comedi: fix function prototypes

2016-12-14 Thread Hartley Sweeten
On December 14, 2016 6:42 AM, Piotr Gregor wrote: > Add names of parameters to function prototypes in comedi PCI. > Checkpatch reports now no errors. > > Signed-off-by: Piotr Gregor > --- > drivers/staging/comedi/comedi_pci.h | 18 ++ > 1 file changed, 10

[PATCH] drivers: staging: comedi: fix function prototypes

2016-12-14 Thread Piotr Gregor
Add names of parameters to function prototypes in comedi PCI. Checkpatch reports now no errors. Signed-off-by: Piotr Gregor --- drivers/staging/comedi/comedi_pci.h | 18 ++ 1 file changed, 10 insertions(+), 8 deletions(-) diff --git