Re: [PATCH net-next v3] netvsc: get rid of completion timeouts

2016-06-09 Thread David Miller
From: Vitaly Kuznetsov Date: Thu, 9 Jun 2016 12:44:03 +0200 > I'm hitting 5 second timeout in rndis_filter_set_rss_param() while setting > RSS parameters for the device. When this happens we end up returning > -ETIMEDOUT from the function and rndis_filter_device_add() falls back to > setting >

[PATCH 02/28] staging: unisys: visorbus: check parahotplug_request_complete_result

2016-06-10 Thread David Kershner
From: Erik Arfvidson This patch modifies the caller of parahotplug_request_complete() to check the return value and return appropriate result. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchipset.c | 5 - 1

[PATCH 03/28] staging: unisys: visorbus: remove unused module parameters

2016-06-10 Thread David Kershner
From: David Binder Removes unused module parameters from visorbus_main.c, in response to findings by SonarQube. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 9 + 1 file changed, 1 insertion

[PATCH 00/28] staging: unisys: fix visorbus & visorinput issues raised by tglx

2016-06-10 Thread David Kershner
functions staging: unisys: Remove reference to unused STANDALONE_CLIENT David Binder (13): staging: unisys: visorbus: remove unused module parameters staging: unisys: visorbus: remove unused struct staging: unisys: visorbus: modify format string to match argument staging: unisys: visornic

[PATCH 09/28] staging: unisys: visorinput: remove unnecessary locking

2016-06-10 Thread David Kershner
red state in devdata->interrupts_enabled. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 57 +- 1 file changed, 47 insertions(+), 10 deletions(-) diff --git a/drivers/staging/unisys/visorinput/visorinput.c b/d

[PATCH 06/28] staging: unisys: visornic: Correct comment spelling mistake

2016-06-10 Thread David Kershner
From: David Binder Fixes a comment spelling mistake in visornic. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visornic/visornic_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys

[PATCH 15/28] staging: unisys: Remove reference to unused STANDALONE_CLIENT

2016-06-10 Thread David Kershner
From: Bryan Thompson The STANDALONE_CLIENT define is no longer used by Unisys driver code. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging

[PATCH 08/28] staging: unisys: visorbus: removed unused periodic_test_workqueue

2016-06-10 Thread David Kershner
From: Tim Sell periodic_test_workqueue was an unused relic from the past, and was removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/unisys

[PATCH 17/28] staging: unisys: visorbus: fix commenting in vbusdevinfo.h

2016-06-10 Thread David Kershner
From: David Binder This patch ONLY touches comment lines, i.e., NO executable code is affected. * All functions worthy of documenting now use standard kerneldoc formatting. * Multi-line comments were tweaked so as to use appropriate conventions. * Minor typos were corrected. Signed-off-by

[PATCH 11/28] staging: unisys: visorbus: remove periodic_work.h/.c

2016-06-10 Thread David Kershner
From: Tim Sell These files were made no-longer-necessary by recent commits. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/include/periodic_work.h | 40 - drivers/staging/unisys/visorbus/Makefile| 1 - drivers/staging/unisys/visorbus

[PATCH 12/28] staging: unisys: visorbus: Make visordriver_callback_lock a mutex

2016-06-10 Thread David Kershner
From: Bryan Thompson visordriver_callback_lock is just a binary semaphore that logically makes more sense as a mutex. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 2 +- drivers/staging/unisys

[PATCH 01/28] staging: unisys: visorbus: remove return values for write_vbus functions

2016-06-10 Thread David Kershner
From: Erik Arfvidson This patch removes the return values from the write_vbus_* channel functions. Nobody was checking the return values and the value stored into the vbus info is not critical to the functioning of the device. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner

[PATCH 04/28] staging: unisys: visorbus: remove unused struct

2016-06-10 Thread David Kershner
From: David Binder Removes unused struct definition, channel_size_info, in response to findings by SonarQube. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 6 -- 1 file changed, 6 deletions(-) diff

[PATCH 05/28] staging: unisys: visorbus: modify format string to match argument

2016-06-10 Thread David Kershner
From: David Binder Modifies the format string of snprintf to expect an unsigned int instead of a signed one, per the supplied argument. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 2 +- 1 file changed

[PATCH 07/28] staging: unisys: include: Remove thread-related enum members

2016-06-10 Thread David Kershner
From: David Binder Code relating to ktheads was previously removed from s-Par driver code. This patch cleans up lingering remnants of kthreads by removing thread- related enum types. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys

[PATCH 10/28] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-10 Thread David Kershner
, because none of them ever perform any operations that would be invalid in atomic context. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/include/visorbus.h | 10 +++-- drivers/staging/unisys/visorbus/visorbus_main.c | 54 +++-- 2 files

[PATCH 25/28] staging: unisys: visorbus: fix visorbus_private.h comments

2016-06-10 Thread David Kershner
From: Tim Sell This patch ONLY touches comment lines, i.e., NO executable code is affected. Comments were fixed in visorbus_private.h: * Minor typos were corrected. * Useless comments were removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus

[PATCH 19/28] staging: unisys: visorbus: fix visorchannel.c comments

2016-06-10 Thread David Kershner
From: David Binder This patch ONLY touches comment lines, i.e., NO executable code is affected. Comments were fixed in visorchannel.c: * All functions worthy of documenting now use standard kerneldoc formatting. * Multi-line comments were tweaked so as to use appropriate conventions. * Minor

[PATCH 18/28] staging: unisys: visorbus: fix commenting in visorbus_main.c

2016-06-10 Thread David Kershner
From: David Binder This patch ONLY touches comment lines, i.e., NO executable code is affected. * All functions worthy of documenting now use standard kerneldoc formatting. * Improper uses of kerneldoc formatting were converted to standard multi-line comments. * Multi-line comments were

[PATCH 22/28] staging: unisys: visorbus: rectify kerneldoc comment for struct

2016-06-10 Thread David Kershner
From: David Binder Fixes the kerneldoc comment for struct visor_device - the struct members were not listed with the appropriate @ prefix. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/include/visorbus.h | 34 +++ 1 file

[PATCH 13/28] staging: unisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements

2016-06-10 Thread David Kershner
Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 1 - drivers/staging/unisys/visorbus/visorchannel.c | 17 - drivers/staging/unisys/visorbus/visorchipset.c | 2 -- 3 files changed, 20 deletions(-) diff --git a/drivers

[PATCH 23/28] staging: unisys: visorbus: Remove notifier-related code from visorbus

2016-06-10 Thread David Kershner
From: David Binder When this functionality was first implemented, visorchipset and visorbus were separate drivers, which necessitated a registration mechanism for them to communicate. More-recently, visorchipset and visorbus were combined into a single driver, and now exist as separate source

[PATCH 14/28] staging: unisys: visorbus: Remove unused functions

2016-06-10 Thread David Kershner
From: Bryan Thompson Remove visorbus_clear_channel, visorchannel_signalqueue_slots_avail, visorchannel_signalqueue_max_slots, visorchannel_clear, and visorchannel_debug which are no longer called by any driver. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell

[PATCH 26/28] staging: unisys: Move vbushelper.h to visorbus directory

2016-06-10 Thread David Kershner
Only visorbus needs this header file so move it to visorbus directory. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/{include => visorbus}/vbushelper.h | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename drivers/staging/unisys/{include => vi

[PATCH 21/28] staging: unisys: visorbus: Move visorbus-unique functions to private header

2016-06-10 Thread David Kershner
From: David Binder Moves function prototypes that are unique to visorbus from include/visorbus.h to visorbus/visorbus_private.h. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 25

[PATCH 28/28] staging: unisys: visorinput: make lock_visor_dev a mutex

2016-06-10 Thread David Kershner
From: Tim Sell Since lock_visor_dev is a binary semaphore it makes more sense to use a mutex lock. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff

[PATCH 20/28] staging: unisys: visorbus: Rectify commenting in visorchipset.c

2016-06-10 Thread David Kershner
From: David Binder Adds kerneldoc formatting to appropriate functions. Other multi-line comments now use proper formatting. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchipset.c | 292 - 1

[PATCH 27/28] staging: unisys: visorinput: ensure proper locking wrt creation & ints

2016-06-10 Thread David Kershner
ned-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 74 +- 1 file changed, 49 insertions(+), 25 deletions(-) diff --git a/drivers/staging/unisys/visorinput/visorinput.c b/drivers/staging/unisys/visorinput/visorinput.c index f633985..c13e698 10

[PATCH 24/28] staging: unisys: visorbus: Rename function to follow existing convention

2016-06-10 Thread David Kershner
From: David Binder Renames visorchipset_device_pause_response to device_pause_response, thereby following the convention that other responder functions follow. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c| 4 ++-- drivers

[PATCH 16/28] staging: unisys: visorbus: remove unused parameter from function

2016-06-10 Thread David Kershner
From: Tim Sell The off parameter to visorchannel_create_guts() was never used, so it was removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchannel.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers

Re: [PATCH net-next,v2] tools: hv: Add a script to help bonding synthetic and VF NICs

2016-06-10 Thread David Miller
From: Haiyang Zhang Date: Fri, 10 Jun 2016 14:23:12 -0700 > This script helps to create bonding network devices based on synthetic NIC > (the virtual network adapter usually provided by Hyper-V) and the matching > VF NIC (SRIOV virtual function). So the synthetic NIC and VF NIC can > function as

[PATCH RESEND 00/28] staging: unisys: fix visorbus & visorinput issues raised by tglx

2016-06-10 Thread David Kershner
staging: unisys: visorbus: Remove unused functions staging: unisys: Remove reference to unused STANDALONE_CLIENT David Binder (13): staging: unisys: visorbus: remove unused module parameters staging: unisys: visorbus: remove unused struct staging: unisys: visorbus: modify format string to

[PATCH RESEND 02/28] staging: unisys: visorbus: check parahotplug_request_complete_result

2016-06-10 Thread David Kershner
From: Erik Arfvidson This patch modifies the caller of parahotplug_request_complete() to check the return value and return appropriate result. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchipset.c | 5 - 1

[PATCH RESEND 04/28] staging: unisys: visorbus: remove unused struct

2016-06-10 Thread David Kershner
From: David Binder Removes unused struct definition, channel_size_info, in response to findings by SonarQube. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 6 -- 1 file changed, 6 deletions(-) diff

[PATCH RESEND 03/28] staging: unisys: visorbus: remove unused module parameters

2016-06-10 Thread David Kershner
From: David Binder Removes unused module parameters from visorbus_main.c, in response to findings by SonarQube. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 9 + 1 file changed, 1 insertion

[PATCH RESEND 01/28] staging: unisys: visorbus: remove return values for write_vbus functions

2016-06-10 Thread David Kershner
From: Erik Arfvidson This patch removes the return values from the write_vbus_* channel functions. Nobody was checking the return values and the value stored into the vbus info is not critical to the functioning of the device. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner

[PATCH RESEND 24/28] staging: unisys: visorbus: Rename function to follow existing convention

2016-06-10 Thread David Kershner
From: David Binder Renames visorchipset_device_pause_response to device_pause_response, thereby following the convention that other responder functions follow. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c| 4 ++-- drivers

[PATCH RESEND 17/28] staging: unisys: visorbus: fix commenting in vbusdevinfo.h

2016-06-10 Thread David Kershner
From: David Binder This patch ONLY touches comment lines, i.e., NO executable code is affected. * All functions worthy of documenting now use standard kerneldoc formatting. * Multi-line comments were tweaked so as to use appropriate conventions. * Minor typos were corrected. Signed-off-by

[PATCH RESEND 21/28] staging: unisys: visorbus: Move visorbus-unique functions to private header

2016-06-10 Thread David Kershner
From: David Binder Moves function prototypes that are unique to visorbus from include/visorbus.h to visorbus/visorbus_private.h. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 25

[PATCH RESEND 05/28] staging: unisys: visorbus: modify format string to match argument

2016-06-10 Thread David Kershner
From: David Binder Modifies the format string of snprintf to expect an unsigned int instead of a signed one, per the supplied argument. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 2 +- 1 file changed

[PATCH RESEND 27/28] staging: unisys: visorinput: ensure proper locking wrt creation & ints

2016-06-10 Thread David Kershner
ned-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 74 +- 1 file changed, 49 insertions(+), 25 deletions(-) diff --git a/drivers/staging/unisys/visorinput/visorinput.c b/drivers/staging/unisys/visorinput/visorinput.c index f633985..c13e698 10

[PATCH RESEND 14/28] staging: unisys: visorbus: Remove unused functions

2016-06-10 Thread David Kershner
From: Bryan Thompson Remove visorbus_clear_channel, visorchannel_signalqueue_slots_avail, visorchannel_signalqueue_max_slots, visorchannel_clear, and visorchannel_debug which are no longer called by any driver. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell

[PATCH RESEND 12/28] staging: unisys: visorbus: Make visordriver_callback_lock a mutex

2016-06-10 Thread David Kershner
From: Bryan Thompson visordriver_callback_lock is just a binary semaphore that logically makes more sense as a mutex. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 2 +- drivers/staging/unisys

[PATCH RESEND 18/28] staging: unisys: visorbus: fix commenting in visorbus_main.c

2016-06-10 Thread David Kershner
From: David Binder This patch ONLY touches comment lines, i.e., NO executable code is affected. * All functions worthy of documenting now use standard kerneldoc formatting. * Improper uses of kerneldoc formatting were converted to standard multi-line comments. * Multi-line comments were

[PATCH RESEND 07/28] staging: unisys: include: Remove thread-related enum members

2016-06-10 Thread David Kershner
From: David Binder Code relating to ktheads was previously removed from s-Par driver code. This patch cleans up lingering remnants of kthreads by removing thread- related enum types. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys

[PATCH RESEND 13/28] staging: unisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements

2016-06-10 Thread David Kershner
Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 1 - drivers/staging/unisys/visorbus/visorchannel.c | 17 - drivers/staging/unisys/visorbus/visorchipset.c | 2 -- 3 files changed, 20 deletions(-) diff --git a/drivers

[PATCH RESEND 16/28] staging: unisys: visorbus: remove unused parameter from function

2016-06-10 Thread David Kershner
From: Tim Sell The off parameter to visorchannel_create_guts() was never used, so it was removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchannel.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/drivers

[PATCH RESEND 19/28] staging: unisys: visorbus: fix visorchannel.c comments

2016-06-10 Thread David Kershner
From: David Binder This patch ONLY touches comment lines, i.e., NO executable code is affected. Comments were fixed in visorchannel.c: * All functions worthy of documenting now use standard kerneldoc formatting. * Multi-line comments were tweaked so as to use appropriate conventions. * Minor

[PATCH RESEND 10/28] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-06-10 Thread David Kershner
, because none of them ever perform any operations that would be invalid in atomic context. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/include/visorbus.h | 10 +++-- drivers/staging/unisys/visorbus/visorbus_main.c | 54 +++-- 2 files

[PATCH RESEND 06/28] staging: unisys: visornic: Correct comment spelling mistake

2016-06-10 Thread David Kershner
From: David Binder Fixes a comment spelling mistake in visornic. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visornic/visornic_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys

[PATCH RESEND 08/28] staging: unisys: visorbus: removed unused periodic_test_workqueue

2016-06-10 Thread David Kershner
From: Tim Sell periodic_test_workqueue was an unused relic from the past, and was removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorbus_main.c | 15 --- 1 file changed, 15 deletions(-) diff --git a/drivers/staging/unisys

[PATCH RESEND 25/28] staging: unisys: visorbus: fix visorbus_private.h comments

2016-06-10 Thread David Kershner
From: Tim Sell This patch ONLY touches comment lines, i.e., NO executable code is affected. Comments were fixed in visorbus_private.h: * Minor typos were corrected. * Useless comments were removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus

[PATCH RESEND 23/28] staging: unisys: visorbus: Remove notifier-related code from visorbus

2016-06-10 Thread David Kershner
From: David Binder When this functionality was first implemented, visorchipset and visorbus were separate drivers, which necessitated a registration mechanism for them to communicate. More-recently, visorchipset and visorbus were combined into a single driver, and now exist as separate source

[PATCH RESEND 11/28] staging: unisys: visorbus: remove periodic_work.h/.c

2016-06-10 Thread David Kershner
From: Tim Sell These files were made no-longer-necessary by recent commits. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/include/periodic_work.h | 40 - drivers/staging/unisys/visorbus/Makefile| 1 - drivers/staging/unisys/visorbus

[PATCH RESEND 15/28] staging: unisys: Remove reference to unused STANDALONE_CLIENT

2016-06-10 Thread David Kershner
From: Bryan Thompson The STANDALONE_CLIENT define is no longer used by Unisys driver code. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging

[PATCH RESEND 28/28] staging: unisys: visorinput: make lock_visor_dev a mutex

2016-06-10 Thread David Kershner
From: Tim Sell Since lock_visor_dev is a binary semaphore it makes more sense to use a mutex lock. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff

[PATCH RESEND 09/28] staging: unisys: visorinput: remove unnecessary locking

2016-06-10 Thread David Kershner
red state in devdata->interrupts_enabled. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 57 +- 1 file changed, 47 insertions(+), 10 deletions(-) diff --git a/drivers/staging/unisys/visorinput/visorinput.c b/d

[PATCH RESEND 20/28] staging: unisys: visorbus: Rectify commenting in visorchipset.c

2016-06-10 Thread David Kershner
From: David Binder Adds kerneldoc formatting to appropriate functions. Other multi-line comments now use proper formatting. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchipset.c | 292 - 1

[PATCH RESEND 26/28] staging: unisys: Move vbushelper.h to visorbus directory

2016-06-10 Thread David Kershner
Only visorbus needs this header file so move it to visorbus directory. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/vbushelper.h | 46 drivers/staging/unisys/visorbus/vbushelper.h | 46 2 files

[PATCH RESEND 22/28] staging: unisys: visorbus: rectify kerneldoc comment for struct

2016-06-10 Thread David Kershner
From: David Binder Fixes the kerneldoc comment for struct visor_device - the struct members were not listed with the appropriate @ prefix. Signed-off-by: David Binder --- drivers/staging/unisys/include/visorbus.h | 34 +++ 1 file changed, 17 insertions(+), 17

[PATCH 1/3] include: linux: visorbus: Add visorbus to include/linux directory

2016-06-10 Thread David Kershner
folders. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/MAINTAINERS| 2 +- drivers/staging/unisys/visorbus/Makefile | 2 -- drivers/staging/unisys/visorbus/controlvmchannel.h| 2 +- drivers

[PATCH 2/3] Documentation: Move visorbus documentation from staging to Documentation/

2016-06-10 Thread David Kershner
This patch simple does a git mv of the drivers/staging/unisys/Documentation directory to Documentation. Renames overview.txt to visorbus.txt and renames sysfs-platform-visorchipset to the correct name sysfs-bus-visorbus. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- .../ABI/stable

[PATCH 3/3] drivers: Add visorbus to the drivers/virt directory

2016-06-10 Thread David Kershner
visorbus is currently located at drivers/staging/visorbus, this patch moves it to drivers/virt. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/Kconfig| 3 +-- drivers/staging/unisys/Makefile

[PATCH 0/3] move visorbus out of staging to drivers/virt/visorbus

2016-06-10 Thread David Kershner
This patchset comprises the last 3 patches of the previously-submitted patchset (but retracted): [PATCH v4 00/29] Fixed issues raised by tglx, then move visorbus to drivers/virt David Kershner (3): include: linux: visorbus: Add visorbus to include/linux direc

Re: [PATCH 3/3] staging: lowmemorykiller: select the task with maximum rss to kill

2016-06-21 Thread David Rientjes
On Tue, 21 Jun 2016, Ganesh Mahendran wrote: > Current task selecting logic in LMK does not fully aware of the memory > pressure. It may select the task with maximum score adj, but with > least tasksize. > > For example, if min_score_adj is 200, and there are 2 tasks in system: >task a: score

Re: [PATCH v12 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-06-28 Thread David Miller
From: Dexuan Cui Date: Fri, 24 Jun 2016 07:45:24 + > + while ((ret = vmalloc(size)) == NULL) > + ssleep(1); This is completely, and entirely, unacceptable. If the allocation fails, you return an error and release your resources. You don't just loop forever waiting for it to

Re: [PATCH v12 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-06-28 Thread David Miller
From: Dexuan Cui Date: Tue, 28 Jun 2016 09:59:21 + > The idea here is: IMO the syscalls sys_read()/write() shoudn't return > -ENOMEM, so I have to make sure the buffer allocation succeeds? You have to fail if resources cannot be allocated. ___ deve

Re: [PATCH v13 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-06-30 Thread David Miller
From: Dexuan Cui Date: Wed, 29 Jun 2016 11:30:40 + > @@ -1509,4 +1509,18 @@ static inline void commit_rd_index(struct > vmbus_channel *channel) > } > > > +struct vmpipe_proto_header { > + u32 pkt_type; It is wasteful to have two empty lines before this structure definition, one is

Re: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-07-04 Thread David Miller
From: Dexuan Cui Date: Tue, 5 Jul 2016 01:58:31 + > Not sure if you had a chance to review this version. Why me? Other people have to review this too. > Now I have a question: may I split the include/linux/socket.h change > and ask you to pre-allocate the number for AF_HYPERV to allow > ba

Re: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-07-04 Thread David Miller
From: Dexuan Cui Date: Tue, 5 Jul 2016 06:46:24 + >> From: David Miller [mailto:da...@davemloft.net] >> Sent: Tuesday, July 5, 2016 14:27 >> To: Dexuan Cui >> Subject: Re: [PATCH v14 net-next 1/1] hv_sock: introduce Hyper-V Sockets >> >> From: Dexuan Cu

Re: [PATCH RESEND net-next] netvsc: Use the new in-place consumption APIs in the rx path

2016-07-08 Thread David Miller
From: k...@exchange.microsoft.com Date: Tue, 5 Jul 2016 16:52:46 -0700 > From: K. Y. Srinivasan > > Use the new APIs for eliminating a copy on the receive path. These new APIs > also > help in minimizing the number of memory barriers we end up issuing (in the > ringbuffer code) since we can be

Re: [PATCH net-next,v3] tools: hv: Add a script to help bonding synthetic and VF NICs

2016-07-12 Thread David Miller
From: Haiyang Zhang Date: Mon, 11 Jul 2016 17:06:42 -0700 > From: Haiyang Zhang > > This script helps to create bonding network devices based on synthetic NIC > (the virtual network adapter usually provided by Hyper-V) and the matching > VF NIC (SRIOV virtual function). So the synthetic NIC and

Re: [PATCH] staging: octeon: Use IS_ENABLED() instead of checking for built-in or module

2016-07-14 Thread David Daney
On 07/14/2016 10:06 AM, Javier Martinez Canillas wrote: The IS_ENABLED() macro checks if a Kconfig symbol has been enabled either built-in or as a module, use that macro instead of open coding the same. Signed-off-by: Javier Martinez Canillas Acked-by: David Daney --- drivers/staging

Re: [PATCH v17 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-07-19 Thread David Miller
From: Dexuan Cui Date: Wed, 20 Jul 2016 01:48:18 + >> From: kbuild test robot [mailto:l...@intel.com] >> Sent: Wednesday, July 20, 2016 1:10 >> >> Hi, >> >> [auto build test WARNING on net-next/master] >> >> url:https://github.com/0day-ci/linux/commits/Dexuan-Cui/introduce- >> Hyper-V-

Re: [PATCH net-next] hv_netvsc: Fix VF register on bonding devices

2016-07-25 Thread David Miller
From: Haiyang Zhang Date: Fri, 22 Jul 2016 18:14:50 -0700 > From: Haiyang Zhang > > Added a condition to avoid bonding devices with same MAC registering > as VF. > > Signed-off-by: Haiyang Zhang > Reviewed-by: K. Y. Srinivasan Applied, thanks. ___

Re: [PATCH v18 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-07-25 Thread David Miller
From: Dexuan Cui Date: Sat, 23 Jul 2016 01:35:51 + > +static struct sock *hvsock_create(struct net *net, struct socket *sock, > + gfp_t priority, unsigned short type) > +{ > + struct hvsock_sock *hvsk; > + struct sock *sk; > + > + sk = sk_alloc(net, A

Re: [PATCH v18 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-07-25 Thread David Miller
From: Dexuan Cui Date: Tue, 26 Jul 2016 03:09:16 + > BTW, during the past month, at least 7 other people also reviewed > the patch and gave me quite a few good comments, which have > been addressed. Correction: Several people gave coding style and simple corrections to your patch. Very few

Re: [PATCH v18 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-07-26 Thread David Miller
From: Dexuan Cui Date: Tue, 26 Jul 2016 07:09:41 + > I googled "S390 hypervisor socket" but didn't find anything related (I think). That would be net/iucv/ There's also VMWare's stuff under net/vmw_vsock It's just absolutely rediculous to make a new hypervisor socket interface over and ove

drivers/staging/ks7010/ks_wlan_net.c:1287: suspicious if ?

2016-08-01 Thread David Binderman
else return -EINVAL; } else if (enabled) { /* 2 */ I am not sure of a suitable fix for this. Regards David Binderman ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH net 2/4] hv_netvsc: reset vf_inject on VF removal

2016-08-12 Thread David Miller
From: Vitaly Kuznetsov Date: Thu, 11 Aug 2016 12:58:55 +0200 > We reset vf_inject on VF going down (netvsc_vf_down()) but we don't on > VF removal (netvsc_unregister_vf()) so vf_inject stays 'true' while > vf_netdev is already NULL and we're trying to inject packets into NULL > net device in netv

Re: [RFC 1/2] netvsc: reference counting fix

2016-08-14 Thread David Miller
From: Stephen Hemminger Date: Sat, 13 Aug 2016 11:35:59 -0700 > This is how I think it should be fixed, but not tested yet. > > Subjec: netvsc: use device not module reference counts > > Fix how the cross-device reference counting is handled. When VF is > associated with the synthetic interfac

Re: [PATCH net v2 0/5] hv_netvsc: fixes for VF removal path

2016-08-15 Thread David Miller
t; series. PATCH1 is required to support the change. > > Changes since v1: > - Re-arrange patches in the series to not introduce new issues [David Miller] > - Add PATCH5 which fixes a new issue I discovered while testing. > - Add Haiyang' A-b tags to PATCH1-4 > > With rega

Re: [PATCH net-next] hv_netvsc: Implement batching of receive completions

2016-08-19 Thread David Miller
From: Haiyang Zhang Date: Fri, 19 Aug 2016 14:47:09 -0700 > From: Haiyang Zhang > > The existing code uses busy retry when unable to send out receive > completions due to full ring buffer. It also gives up retrying after limit > is reached, and causes receive buffer slots not being recycled. >

linux-4.8-rc3/drivers/staging/rtl8723au/hal/rtl8723a_bt-coexist.c:3533: pointless test ?

2016-08-22 Thread David Binderman
Config.HCIExtensionVer >= 1) { Suggest remove second test. Regards David Binderman ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [RESEND PATCH] staging: octeon: Use IS_ENABLED() instead of checking for built-in or module

2016-08-25 Thread David Daney
built-in and module enable details. Signed-off-by: Javier Martinez Canillas Acked-by: David Daney --- Changes since first post: - Added David Daney Acked-by tag. - Rebased on top of latest master. You changed something about the patch. In this case the Subject is not "RESEND", but

Re: Improving OCTEON II 10G Ethernet performance

2016-08-25 Thread David Daney
On 08/25/2016 11:22 AM, Aaro Koskinen wrote: Hi, On Thu, Aug 25, 2016 at 09:50:15AM -0700, David Daney wrote: Ideally we would configure the packet classifiers on the RX side to create multiple RX queues based on a hash of the TCP 5-tuple, and handle each queue with a single NAPI instance

Re: Improving OCTEON II 10G Ethernet performance

2016-08-25 Thread David Daney
On 08/24/2016 06:29 PM, Ed Swierk wrote: I'm trying to migrate from the Octeon SDK to a vanilla Linux 4.4 kernel for a Cavium OCTEON II (CN6880) board running in 64-bit little-endian mode. So far I've gotten most of the hardware features I need working, including XAUI/RXAUI, USB, boot bus and I2C

Re: Improving OCTEON II 10G Ethernet performance

2016-08-25 Thread David Daney
On 08/25/2016 02:18 PM, Aaro Koskinen wrote: Hi, On Thu, Aug 25, 2016 at 01:11:45PM -0700, David Daney wrote: On 08/25/2016 11:22 AM, Aaro Koskinen wrote: On Thu, Aug 25, 2016 at 09:50:15AM -0700, David Daney wrote: Ideally we would configure the packet classifiers on the RX side to create

Re: [PATCH 0/6] constify ethtool_ops structures

2016-08-31 Thread David Miller
From: Julia Lawall Date: Wed, 31 Aug 2016 09:30:42 +0200 > Constify ethtool_ops structures. Patches 2-6 applied, thanks. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 0/9] staging: octeon: multi rx group (queue) support

2016-08-31 Thread David Daney
ke it back to ping. This happens only on CN68XX, and I found the root cause. I will send a new series later today. Thanks, cn68xx does have a slightly different SSO and interrupt implementation, so it will probably need some special case code to handle both styles o

[PATCH 08/21] staging: unisys: visorbus: Convert visorchipset.h refs

2016-09-01 Thread David Kershner
From: David Binder visorbus_private.h was previously named visorchipset.h. This patch removes references to the old filename, and updates them with the new filename. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus

[PATCH 13/21] staging: unisys: visorbus: Merge controlvm completion status to controlvmchannel

2016-09-01 Thread David Kershner
The completion status are valid only for the controlvm channel so they should be moved into its header file. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/controlvmchannel.h | 76 .../unisys/visorbus/controlvmcompletionstatus.h

[PATCH 18/21] staging: unisys: visorbus: Move non global guestlinuxdebug to visorbus

2016-09-01 Thread David Kershner
Guestlinuxdebug.h is only used by visorbus, move the include file into its directory instead of the global one. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/{include => visorbus}/guestlinuxdebug.h | 0 1 file changed, 0 insertions(+), 0 deletions(-) ren

[PATCH 03/21] staging: unisys: visornic_main.c remove unused pound defines

2016-09-01 Thread David Kershner
From: Erik Arfvidson This removes two unused pound defines in the visornic_main.c file. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visornic/visornic_main.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/unisys/visornic

[PATCH 21/21] staging: unisys: visorbus: cleanup include listings

2016-09-01 Thread David Kershner
Cleaned up include section of files so that they only include the files needed by them. If an include file includes the file already don't repeat it. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorbus_main.c | 2 +- drivers/staging/unisys/vis

[PATCH 07/21] staging: unisys: Move vbushelper.h contents to visorbus_private.h

2016-09-01 Thread David Kershner
From: Bryan Thompson The contents of vbushelper.h are now only used by visorbus, so it no longer needs to be a general include file and it can be incorporated in the visorbus private header. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers

[PATCH 09/21] staging: unisys: visorbus: Merge channel_guid.h with channel.h

2016-09-01 Thread David Kershner
From: David Binder Merges the contents of channel_guid.h with channel.h, as the former was not a proper header file. Therefore, the channel_guid.h is no longer needed. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include

[PATCH 04/21] staging: unisys: visorbus: remove g_devicechangestate_packet

2016-09-01 Thread David Kershner
From: Jon Frisch This patch removes the unused struct controlvm_message_packet g_devicechangestate_packet. The struct is set but never called. Signed-off-by: Jon Frisch Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchipset.c | 2 -- 1 file changed, 2 deletions

[PATCH 19/21] staging: unisys: remove check for GNUC

2016-09-01 Thread David Kershner
The check for GNUC is redundant and is not needed for vmcallinterface.h. Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/vmcallinterface.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/unisys/visorbus/vmcallinterface.h b/drivers

[PATCH 11/21] staging: unisys: visorinput: Remove ultrainputreport.h include

2016-09-01 Thread David Kershner
From: Bryan Thompson Stop ultrainputreport.h from including itself. Signed-off-by: Bryan Thompson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorinput/ultrainputreport.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/unisys

<    7   8   9   10   11   12   13   14   15   16   >