Commit 3798ff31 authored by Tim Sell's avatar Tim Sell Committed by Greg Kroah-Hartman
Browse files

staging: unisys: visornic: don't destroy global workqueues until devs destroyed 



visornic_cleanup() was previously incorrectly destroying its global
workqueues prior to cleaning up the devices which used them.  This patch
corrects the order of these operations.

Signed-off-by: default avatarTim Sell <Timothy.Sell@unisys.com>
Signed-off-by: default avatarBenjamin Romer <benjamin.romer@unisys.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 46df8226

drivers/staging/unisys/visornic/visornic_main.c

+2 −1
Original line number Diff line number Diff line
@@ -2241,6 +2241,8 @@ static int visornic_init(void) 
 */

static void visornic_cleanup(void)

{

	visorbus_unregister_visor_driver(&visornic_driver);



	if (visornic_serverdown_workqueue) {

		flush_workqueue(visornic_serverdown_workqueue);

		destroy_workqueue(visornic_serverdown_workqueue);
@@ -2251,7 +2253,6 @@ static void visornic_cleanup(void) 
	}

	debugfs_remove_recursive(visornic_debugfs_dir);



	visorbus_unregister_visor_driver(&visornic_driver);

	kfree(dev_num_pool);

	dev_num_pool = NULL;

}