[PATCH] staging: unisys: visorchannel: correct variable misspelling

2016-05-04 Thread David Kershner
From: David Binder Fixes the spelling of a global variable passed into a kernel macro. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchipset.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys

[PATCH v2 0/1] staging: unisys: Updates to the patch remove unused struct members

2016-05-05 Thread David Kershner
The patch staging: unisys: remove unused struct members did not remove a comment that referenced pnext. This series resends that. Changes since v1: - Remove additional comment line referencing pnext Erik Arfvidson (1): staging: unisys: remove unused struct members drivers/staging/uni

[PATCH v2 1/1] staging: unisys: remove unused struct members

2016-05-05 Thread David Kershner
From: Erik Arfvidson The following struct members were never used: putfile_active_buffer::pnext putfile_request::file_request_number putfile_request::data_sequence_number Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchipset.c | 6

[PATCH] staging: unisys: visornic: comment restructuring and removing bad diction

2016-05-05 Thread David Kershner
From: Erik Arfvidson The purpose of this patch is to clean up commenting and making the code with comments be pleasant to eyes. Also make commenting be consistent throughout the file. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visornic

[PATCH] staging: unisys: visorhba: fail gracefully for thread creation failures

2016-05-05 Thread David Kershner
(). Also, the other thread-related fields in visor_thread_info (has_stopped and id) were not being used, so visor_thead_info was just removed. We also now spit out an error message if thread creation fails. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys

[PATCH] staging: unisys: fix format string %Lx to %llx for u64

2016-05-05 Thread David Kershner
From: Erik Arfvidson this patch fixes the following sonarqube issue. %Lx in format string (no. 1) requires 'unsigned long long' but the argument type is 'unsigned long long' Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/

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

2016-05-06 Thread David Miller
From: Dexuan Cui Date: Wed, 4 May 2016 09:56:57 -0700 > +#define VMBUS_RINGBUFFER_SIZE_HVSOCK_RECV (5 * PAGE_SIZE) > +#define VMBUS_RINGBUFFER_SIZE_HVSOCK_SEND (5 * PAGE_SIZE) > + > +#define HVSOCK_RCV_BUF_SZVMBUS_RINGBUFFER_SIZE_HVSOCK_RECV ... > +struct hvsock_sock { ... > + /* The '

[PATCH 00/10] staging: unisys: driver updates

2016-05-06 Thread David Kershner
Patch series containing fixes for the s-Par unisys drivers. Each patch can be applied independently, if needed. Alexander Curtin (1): staging: unisys: visorhba: replace functionlike macro with function David Binder (2): staging: unisys: visorchannel: correct variable misspelling staging

[PATCH 03/10] staging: unisys: remove unused struct members

2016-05-06 Thread David Kershner
From: Erik Arfvidson The following struct members were never used: putfile_active_buffer::pnext putfile_request::file_request_number putfile_request::data_sequence_number Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by: Luis de Bethencourt --- drivers/staging

[PATCH 02/10] staging: unisys: visorchannel: correct variable misspelling

2016-05-06 Thread David Kershner
From: David Binder Fixes the spelling of a global variable passed into a kernel macro. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchipset.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys

[PATCH 01/10] staging: unisys: visorhba: replace functionlike macro with function

2016-05-06 Thread David Kershner
-by: Alexander Curtin Signed-off-by: Tim Sell Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/include/iochannel.h | 42 - drivers/staging/unisys/visorhba/visorhba_main.c | 26 --- 2 files changed, 22 insertions(+),

[PATCH 04/10] staging: unisys: fix format string %Lx to %llx for u64

2016-05-06 Thread David Kershner
From: Erik Arfvidson this patch fixes the following sonarqube issue. %Lx in format string (no. 1) requires 'unsigned long long' but the argument type is 'unsigned long long' Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/

[PATCH 10/10] staging: unisys: rename misleading var ii with frag

2016-05-06 Thread David Kershner
From: Erik Arfvidson This patch removes misleading variable name with a more appropriate name. Since ii is keeping track of fragments inside a for loop I renamed it frag. ii->frag Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visornic/visornic_mai

[PATCH 06/10] staging: unisys: visorhba: fail gracefully for thread creation failures

2016-05-06 Thread David Kershner
(). Also, the other thread-related fields in visor_thread_info (has_stopped and id) were not being used, so visor_thead_info was just removed. We also now spit out an error message if thread creation fails. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys

[PATCH 05/10] staging: unisys: visornic: comment restructuring and removing bad diction

2016-05-06 Thread David Kershner
From: Erik Arfvidson The purpose of this patch is to clean up commenting and making the code with comments be pleasant to eyes. Also make commenting be consistent throughout the file. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by: Luis de Bethencourt --- drivers

[PATCH 07/10] staging: unisys: visorinput: handle_locking_key() simplifications

2016-05-06 Thread David Kershner
From: Tim Sell By directly returning for the error-case, a test was eliminated. Also, the old_state variable was removed as it was not necessary, and some redundant parens were removed. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c

[PATCH 08/10] staging: unisys: visorhba: main path needs to flow down the left margin

2016-05-06 Thread David Kershner
From: Tim Sell In del_scsipending_ent(), the error-path and main-path were switched, so the error-path is handled like 'if (err) return;', and the main-path flows down the left margin. This also allowed us to remove the initialization of "sent". Signed-off-by: Tim Sell

[PATCH 09/10] staging: unisys: visorhba: switch success handling to error handling

2016-05-06 Thread David Kershner
From: David Binder Addresses code audit feeback to switch from success handling to error handling in visorhba_main.c/process_disk_notify(). Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visorhba/visorhba_main.c | 17 + 1 file changed, 9

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

2016-05-07 Thread David Miller
From: Dexuan Cui Date: Sat, 7 May 2016 10:49:25 + > I should be able to make 'send', 'recv' here to pointers and use vmalloc() > to allocate the memory for them. I will do this. That's still unswappable kernel memory. People can open N sockets, where N is something on the order of the FD l

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

2016-05-08 Thread David Miller
From: Dexuan Cui Date: Sun, 8 May 2016 06:11:04 + > Thanks for pointing this out! > I understand, so I think I should add a module parameter, e.g., > "hv_sock.max_socket_number" with a default value, say, 1024? No, you should get rid of the huge multi-page buffers. __

[PATCH 0/9] staging: unisys: visorhba: fix scsi task mgmt completion handling and other cleanups

2016-05-12 Thread David Kershner
we can cleanly pass handles back-and-forth from/to the IO partition instead of raw pointers. This patchset simplifies and enhances the visorhba debugfs interface, and deletes a global array made no-longer-necessary by the cleanup. David Binder (4): staging: unisys: visornic: simplify visor

[PATCH 9/9] staging: unisys: visornic: check for error instead of success

2016-05-12 Thread David Kershner
From: David Binder Changes the conditional logic to check for an error code instead of a success code. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visornic/visornic_main.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a

[PATCH 1/9] staging: unisys: visorhba: delete processing of vdiskmgmt commands

2016-05-12 Thread David Kershner
From: Tim Sell We never issue SCSI commands of type CMD_VDISKMGMT_TYPE, so there is no need to have code that processes their completions. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorhba/visorhba_main.c | 22 -- 1 file changed, 22

[PATCH 5/9] staging: unisys: visorhba: visorhbas_open[] no longer used, so deleted

2016-05-12 Thread David Kershner
From: Tim Sell The prior patch which simplified the visorhba debugfs interface made it so visorhbas_open[] and VISORHBA_OPEN_MAX were no longer needed, so they have now been deleted. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorhba/visorhba_main.c

[PATCH 7/9] staging: unisys: visornic: cleanup error handling

2016-05-12 Thread David Kershner
From: David Binder Adjusts goto labels to prevent attempts to free unallocated resources. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visornic/visornic_main.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging

[PATCH 6/9] staging: unisys: visornic: simplify visornic if statements

2016-05-12 Thread David Kershner
From: David Binder Changes the conditional logic by looking for the absence of work to do, instead of the opposite. Signed-off-by: David Binder Reviewed-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visornic/visornic_main.c | 41 + 1 file

[PATCH 2/9] staging: unisys: visorhba: correct scsi task mgmt completion handling

2016-05-12 Thread David Kershner
: initiating type=1 taskmgmt command [ 364.029721] visorhba: notifying initiator with result=0x1 [ 364.029726] sd 0:0:1:1: visorhba: taskmgmt type=1 success; result=0x1 Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorhba/visorhba_main.c | 138

[PATCH 3/9] staging: unisys: visorhba: remove unused (and broken) logic

2016-05-12 Thread David Kershner
From: Tim Sell The handling of CMD_NOTIFYGUEST_TYPE messages from the IO partition appears to be only partially implemented, but fortunately it is never used in our current environment. This patch deletes the unused code. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers

[PATCH 4/9] staging: unisys: visorhba: simplify and enhance debugfs interface

2016-05-12 Thread David Kershner
seq_file interface, so that it could access the device context without resorting to a global array. This also simplified the function. Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorhba/visorhba_main.c | 136 +++- 1 file changed, 65

[PATCH 8/9] staging: unisys: visorhba: return 0 literal

2016-05-12 Thread David Kershner
From: David Binder Returns 0 instead of variable rc in visorhba_init(). Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visorhba/visorhba_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys/visorhba

[PATCH 2/8] staging: unisys: visorhba: "Prefer 'unsigned int'" checkpatch warnings

2016-05-13 Thread David Kershner
From: Tim Sell This patch fixes a few checkpatch warnings in visorhba: WARNING: Prefer 'unsigned int' to bare use of 'unsigned' Signed-off-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visorhba/visorhba_main.c | 6 +++--- 1 file changed

[PATCH 0/8] staging: unisys: Additional cleanups

2016-05-13 Thread David Kershner
This series cleans up some negative 1 returns and other minor cleanups. David Binder (2): staging: unisys: visornic: remove extraneous error check staging: unisys: visornic: change return statements Erik Arfvidson (5): staging: unisys: visorinput change -1 return value staging: unisys

[PATCH 4/8] staging: unisys: visorhba change -1 return value

2016-05-13 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return value to -EBUSY Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorhba/visorhba_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/unisys/visorhba

[PATCH 8/8] staging: unisys: visornic: change return statements

2016-05-13 Thread David Kershner
From: David Binder Changes return statements in visornic_rx() to use literals instead of a variable. Also changes function description to reflect the correct return type. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/visornic/visornic_main.c | 16

[PATCH 6/8] staging: unisys: visorchipset change -1 return value

2016-05-13 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return value to -EINVAL Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/visorchipset.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys/visorbus

[PATCH 7/8] staging: unisys: iovmcall_gnuc.h change -1 return values

2016-05-13 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return values to -EPERM. This operation is not supported is a good alternative to -1 because the return is basically telling the caller that the processor doesn't support vmcall operations. Signed-off-by: Erik Arfvidson Signed-off-by:

[PATCH 1/8] staging: unisys: visornic: remove extraneous error check

2016-05-13 Thread David Kershner
From: David Binder Removes an extraneous error check in devdata_initialize(), and updates the function comment accordingly. Signed-off-by: David Binder Reviewed-by: Tim Sell Signed-off-by: David Kershner --- drivers/staging/unisys/visornic/visornic_main.c | 4 +--- 1 file changed, 1

[PATCH 3/8] staging: unisys: visorinput change -1 return value

2016-05-13 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return value to -EINVAL Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers/staging/unisys/visorinput/visorinput.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/unisys/visorinput

[PATCH 5/8] staging: unisys: visorbus change -1 return values

2016-05-13 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return values to -EFAULT since it would be the most appropriate, given that this error would only occur in an unexpected bad offset field. Resulting in a bad address. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner --- drivers

Re: [PATCH v11 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

2016-05-15 Thread David Miller
From: Dexuan Cui Date: Sun, 15 May 2016 09:52:42 -0700 > Changes since v10 > > 1) add module params: send_ring_page, recv_ring_page. They can be used to > enlarge the ringbuffer size to get better performance, e.g., > # modprobe hv_sock recv_ring_page=16 send_ring_page=16 > By default, recv_rin

Re: [PATCH net-next v2 0/6] hv_netvsc: avoid races on mtu change/set channels

2016-05-16 Thread David Miller
From: Vitaly Kuznetsov Date: Fri, 13 May 2016 13:55:19 +0200 > Changes since v1: > - Rebased to net-next [Haiyang Zhang] > > Original description: > > MTU change and set channels operations are implemented as netvsc device > re-creation destroying internal structures (struct net_device stays).

[PATCH 0/5] add bus driver for Unisys s-Par paravirtualized devices to arch/x86

2016-05-17 Thread David Kershner
. David Kershner (5): staging: unisys: Move vbushelper.h to visorbus directory include: linux: visorbus: Add visorbus to include/linux directory Documentation: Move visorbus documentation from staging to Documentation/ staging: unisys: visorbus: Have visorbus use include/linux/visorbus

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

2016-05-17 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 --- Documentation/ABI/stable/sysfs-bus

[PATCH 1/5] staging: unisys: Move vbushelper.h to visorbus directory

2016-05-17 Thread David Kershner
Only visorbus needs this header file so move it to visorbus directory. Signed-off-by: David Kershner --- drivers/staging/unisys/include/vbushelper.h | 46 drivers/staging/unisys/visorbus/vbushelper.h | 46 2 files changed, 46 insertions

[PATCH 4/5] staging: unisys: visorbus: Have visorbus use include/linux/visorbus

2016-05-17 Thread David Kershner
Since we have now moved the include directories over to include/linux/visorbus this patch makes all of visorbus use the new include folders. Signed-off-by: David Kershner --- drivers/staging/unisys/visorbus/controlvmchannel.h | 2 +- drivers/staging/unisys/visorbus/periodic_work.c| 3

[PATCH 2/5] include: linux: visorbus: Add visorbus to include/linux directory

2016-05-17 Thread David Kershner
Update include/linux to include the s-Par associated common include header files needed for the s-Par visorbus. Signed-off-by: David Kershner --- include/linux/visorbus/channel.h | 572 +++ include/linux/visorbus/channel_guid.h| 55 +++ include/linux

Re: [PATCH v11 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

2016-05-18 Thread David Miller
I'm travelling and very busy with the merge window. So sorry I won't be able to think about this for some time. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH net-next] tools: hv: Add a script for bonding synthetic and VF NICs

2016-05-23 Thread David Miller
From: Haiyang Zhang Date: Thu, 19 May 2016 10:23:38 -0700 > This example script creates 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

Re: [PATCH net-next 0/5] hv_netvsc: cleanup after untangling the pointer mess

2016-05-23 Thread David Miller
From: Vitaly Kuznetsov Date: Mon, 23 May 2016 17:50:33 +0200 > After we made traveling through our internal structures explicit it became > obvious that some functions take arguments they don't need just to do > redundant pointer travel and get to what they really need while their > callers alrea

[PATCH 07/24] staging: unisys: visorinput: remove unnecessary locking

2016-05-24 Thread David Kershner
red state in devdata->interrupts_enabled. 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/drivers/staging/

[PATCH 00/24] Fixes comments made by tglx, then move visorbus to drivers/virt

2016-05-24 Thread David Kershner
nisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements staging: unisys: visorbus: Remove unused functions staging: unisys: Remove reference to unused STANDALONE_CLIENT David Binder (12): staging: unisys: visorbus: remove unused module parameters staging: unisys: visorbus: remove unused s

[PATCH 01/24] staging: unisys: visorbus: remove unused module parameters

2016-05-24 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 --- drivers/staging/unisys/visorbus/visorbus_main.c | 9 + 1 file changed, 1 insertion(+), 8 deletions(-) diff

[PATCH 18/24] staging: unisys: visorbus: make visorchipset function descriptions more kerneldoc-like

2016-05-24 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorchipset.c now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging

[PATCH 20/24] staging: unisys: visorbus: Add kerneldoc-style comments for visorbus API

2016-05-24 Thread David Kershner
From: David Binder Adds kerneldoc-style comments for those functions which may be used outside of the visorbus driver. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging/unisys/include/visorbus.h | 127 drivers/staging/unisys/visorbus

[PATCH 19/24] staging: unisys: visorbus: Move visorbus-unique functions to private header

2016-05-24 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 --- drivers/staging/unisys/include/visorbus.h | 25 drivers/staging/unisys

[PATCH 22/24] include: linux: visorbus: Add visorbus to include/linux directory

2016-05-24 Thread David Kershner
folders. Signed-off-by: David Kershner --- drivers/staging/unisys/MAINTAINERS| 2 +- drivers/staging/unisys/visorbus/Makefile | 2 -- drivers/staging/unisys/visorbus/controlvmchannel.h| 2 +- drivers/staging/unisys/visorbus

[PATCH 05/24] staging: unisys: include: Remove thread-related enum members

2016-05-24 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 --- drivers/staging/unisys/include

[PATCH 16/24] staging: unisys: visorbus: make visorbus_private.h function descriptions more kerneldoc-like

2016-05-24 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorbus_private.h now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers

[PATCH 24/24] drivers: Add visorbus to the drivers directory

2016-05-24 Thread David Kershner
visorbus is currently located at drivers/staging/visorbus, this patch moves it to drivers/virt. Signed-off-by: David Kershner --- drivers/staging/unisys/Kconfig| 3 +-- drivers/staging/unisys/Makefile | 1 - drivers

[PATCH 04/24] staging: unisys: visornic: Correct comment spelling mistake

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

[PATCH 12/24] staging: unisys: visorbus: Remove unused functions

2016-05-24 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 --- drivers/staging

[PATCH 08/24] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-05-24 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 21/24] staging: unisys: Move vbushelper.h to visorbus directory

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

[PATCH 17/24] staging: unisys: visorbus: make visorchannel function descriptions more kerneldoc-like

2016-05-24 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorchannel.c now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers/staging

[PATCH 09/24] staging: unisys: visorbus: remove periodic_work.h/.c

2016-05-24 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 11/24] staging: unisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements

2016-05-24 Thread David Kershner
Signed-off-by: David Kershner --- 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/staging/unisys/visorbus

[PATCH 23/24] Documentation: Move visorbus documentation from staging to Documentation/

2016-05-25 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 --- .../ABI/stable/sysfs-bus-visorbus

[PATCH 10/24] staging: unisys: visorbus: Make visordriver_callback_lock a mutex

2016-05-25 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 --- drivers/staging/unisys/include/visorbus.h | 3 ++- drivers/staging/unisys/visorbus/visorbus_main.c | 10

[PATCH 06/24] staging: unisys: visorbus: removed unused periodic_test_workqueue

2016-05-25 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 03/24] staging: unisys: visorbus: modify format string to match argument

2016-05-25 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 --- drivers/staging/unisys/visorbus/visorbus_main.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH 15/24] staging: unisys: visorbus: make function descriptions more kerneldoc-like

2016-05-25 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorbus_main.c now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers

[PATCH 13/24] staging: unisys: Remove reference to unused STANDALONE_CLIENT

2016-05-25 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 --- drivers/staging/unisys/include/visorbus.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/staging/unisys/include/visorbus.h

[PATCH 02/24] staging: unisys: visorbus: remove unused struct

2016-05-25 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 --- drivers/staging/unisys/visorbus/visorbus_main.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/drivers/staging

[PATCH 14/24] staging: unisys: visorbus: vbusdeviceinfo function descriptions more kerneldoc-like

2016-05-25 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in vbusdeviceinfo.h now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner --- drivers

[PATCH v2 13/27] staging: unisys: visorbus: Make visordriver_callback_lock a mutex

2016-05-31 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 | 3 ++- drivers/staging/unisys

[PATCH v2 09/27] staging: unisys: visorbus: removed unused periodic_test_workqueue

2016-05-31 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 v2 04/27] staging: unisys: visorbus: remove unused module parameters

2016-05-31 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 v2 11/27] staging: unisys: visorbus: use kernel timer instead of workqueue

2016-05-31 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 v2 10/27] staging: unisys: visorinput: remove unnecessary locking

2016-05-31 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 v2 24/27] staging: unisys: Move vbushelper.h to visorbus directory

2016-05-31 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 v2 03/27] staging: unisys: iovmcall_gnuc.h change -1 return values

2016-05-31 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return values to -EPERM. This operation is not supported is a good alternative to -1 because the return is basically telling the caller that the processor doesn't support vmcall operations. Signed-off-by: Erik Arfvidson Signed-off-by:

[PATCH v2 02/27] staging: unisys: visorchipset change -1 return value

2016-05-31 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return value to -EINVAL Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/visorbus/visorchipset.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers

[PATCH v2 05/27] staging: unisys: visorbus: remove unused struct

2016-05-31 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 v2 06/27] staging: unisys: visorbus: modify format string to match argument

2016-05-31 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 v2 07/27] staging: unisys: visornic: Correct comment spelling mistake

2016-05-31 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 v2 25/27] include: linux: visorbus: Add visorbus to include/linux directory

2016-05-31 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 v2 23/27] staging: unisys: visorbus: Add kerneldoc-style comments for visorbus API

2016-05-31 Thread David Kershner
From: David Binder Adds kerneldoc-style comments for those functions which may be used outside of the visorbus driver. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim Sell --- drivers/staging/unisys/include/visorbus.h | 127 drivers

[PATCH v2 12/27] staging: unisys: visorbus: remove periodic_work.h/.c

2016-05-31 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 v2 27/27] drivers: Add visorbus to the drivers directory

2016-05-31 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 v2 16/27] staging: unisys: Remove reference to unused STANDALONE_CLIENT

2016-05-31 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 v2 21/27] staging: unisys: visorbus: make visorchipset function descriptions more kerneldoc-like

2016-05-31 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorchipset.c now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim

[PATCH v2 22/27] staging: unisys: visorbus: Move visorbus-unique functions to private header

2016-05-31 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 v2 15/27] staging: unisys: visorbus: Remove unused functions

2016-05-31 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 v2 00/27] Fixed issues raised by tglx, then move visorbus to drivers/virt

2016-05-31 Thread David Kershner
: Make visordriver_callback_lock a mutex staging: unisys: visorbus: Remove unnecessary EXPORT_SYMBOL statements staging: unisys: visorbus: Remove unused functions staging: unisys: Remove reference to unused STANDALONE_CLIENT David Binder (12): staging: unisys: visorbus: remove unused module

[PATCH v2 01/27] staging: unisys: visorbus change -1 return values

2016-05-31 Thread David Kershner
From: Erik Arfvidson This patch changes the vague -1 return values to -EFAULT since it would be the most appropriate, given that this error would only occur in an unexpected bad offset field. Resulting in a bad address. Signed-off-by: Erik Arfvidson Signed-off-by: David Kershner Reviewed-by

[PATCH v2 18/27] staging: unisys: visorbus: make function descriptions more kerneldoc-like

2016-05-31 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorbus_main.c now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim

[PATCH v2 08/27] staging: unisys: include: Remove thread-related enum members

2016-05-31 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 v2 20/27] staging: unisys: visorbus: make visorchannel function descriptions more kerneldoc-like

2016-05-31 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in visorchannel.c now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim

[PATCH v2 26/27] Documentation: Move visorbus documentation from staging to Documentation/

2016-05-31 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 v2 17/27] staging: unisys: visorbus: vbusdeviceinfo function descriptions more kerneldoc-like

2016-05-31 Thread David Kershner
From: David Binder Per audit feedback from Thomas Gleixner, function descriptions in vbusdeviceinfo.h now utilize a more kerneldoc-like formatting. The affected comments do not implement other kerneldoc requirements. Signed-off-by: David Binder Signed-off-by: David Kershner Reviewed-by: Tim

<    4   5   6   7   8   9   10   11   12   13   >