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

David Kershner david.kershner at unisys.com
Fri Jun 10 23:35:29 UTC 2016


This patchset comprises the first 26 patches of the previously-submitted
patchset (but retracted): 

	[PATCH v4 00/29] Fixed issues raised by tglx, then move visorbus to
			 drivers/virt

then adds 2 patches to visorinput that:
	* fixes a device initialization race condition
	* converts a semaphore to a mutex

As described in the email NAKing the previously-submitted patchset,
the reason we are re-submitting this now is to make things a bit
cleaner by separating the fixes we need to make to the code in
staging from the patchset that actually moves the code out of staging.

The intent of this patchset is to fix all known outstanding
issues with code in drivers/staging/unisys/, so that subsequent
patchsets can move these drivers out of staging.

tglx: The following patchset fixes issues you raised during your
code review of visorbus on 5/18, and visorinput on 6/1. 

Converts visorbus to use a kernel timer for periodic device-specific
callbacks instead of a workqueue, making the implementation in
periodic_work.c and periodic_work.h no longer necessary.  These files
are then deleted.

The visordriver_callback_lock has been switched to a mutex.

Several module parameters and structures were removed that were no
longer being used.

Bryan Thompson (4):
  staging: unisys: visorbus: 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 (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: Correct comment spelling mistake
  staging: unisys: include: Remove thread-related enum members
  staging: unisys: visorbus: fix commenting in vbusdevinfo.h
  staging: unisys: visorbus: fix commenting in visorbus_main.c
  staging: unisys: visorbus: fix visorchannel.c comments
  staging: unisys: visorbus: Rectify commenting in visorchipset.c
  staging: unisys: visorbus: Move visorbus-unique functions to private
    header
  staging: unisys: visorbus: rectify kerneldoc comment for struct
  staging: unisys: visorbus: Remove notifier-related code from visorbus
  staging: unisys: visorbus: Rename function to follow existing
    convention

David Kershner (1):
  staging: unisys: Move vbushelper.h to visorbus directory

Erik Arfvidson (2):
  staging: unisys: visorbus: remove return values for write_vbus
    functions
  staging: unisys: visorbus: check parahotplug_request_complete_result

Tim Sell (8):
  staging: unisys: visorbus: removed unused periodic_test_workqueue
  staging: unisys: visorinput: remove unnecessary locking
  staging: unisys: visorbus: use kernel timer instead of workqueue
  staging: unisys: visorbus: remove periodic_work.h/.c
  staging: unisys: visorbus: remove unused parameter from function
  staging: unisys: visorbus: fix visorbus_private.h comments
  staging: unisys: visorinput: ensure proper locking wrt creation & ints
  staging: unisys: visorinput: make lock_visor_dev a mutex

 drivers/staging/unisys/include/guestlinuxdebug.h   |   4 +-
 drivers/staging/unisys/include/periodic_work.h     |  40 --
 drivers/staging/unisys/include/visorbus.h          |  80 +--
 drivers/staging/unisys/visorbus/Makefile           |   1 -
 drivers/staging/unisys/visorbus/periodic_work.c    | 204 -------
 drivers/staging/unisys/visorbus/vbusdeviceinfo.h   |  65 ++-
 .../unisys/{include => visorbus}/vbushelper.h      |   0
 drivers/staging/unisys/visorbus/visorbus_main.c    | 615 +++++++++++----------
 drivers/staging/unisys/visorbus/visorbus_private.h |  73 ++-
 drivers/staging/unisys/visorbus/visorchannel.c     | 269 +++------
 drivers/staging/unisys/visorbus/visorchipset.c     | 434 ++++++++-------
 drivers/staging/unisys/visorinput/visorinput.c     | 141 +++--
 drivers/staging/unisys/visornic/visornic_main.c    |   2 +-
 13 files changed, 833 insertions(+), 1095 deletions(-)
 delete mode 100644 drivers/staging/unisys/include/periodic_work.h
 delete mode 100644 drivers/staging/unisys/visorbus/periodic_work.c
 rename drivers/staging/unisys/{include => visorbus}/vbushelper.h (100%)

-- 
1.9.1



More information about the devel mailing list