Staging: hv: warn the world of a bug in the release function
authorGreg Kroah-Hartman <gregkh@suse.de>
Wed, 2 Sep 2009 03:12:58 +0000 (20:12 -0700)
committerGreg Kroah-Hartman <gregkh@suse.de>
Tue, 15 Sep 2009 19:02:02 +0000 (12:02 -0700)
All device release functions need to do something, if not, it's a bug.
By merely providing an "empty" release function, it gets the kernel to
shut up, but that's not solving the problem at all.  Stick a big fat
WARN_ON(1); in there to get people's attention.

Cc: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/hv/vmbus_drv.c

index 4462d5bb87e71fa87f44c854be7743196d2a32b0..cc81ffd7e8a826d1e5d3bf498141a9c009d69370 100644 (file)
@@ -965,6 +965,11 @@ Desc:      Final callback release of the vmbus root device
 static void vmbus_bus_release(struct device *device)
 {
        DPRINT_ENTER(VMBUS_DRV);
+       /* FIXME */
+       /* Empty release functions are a bug, or a major sign
+        * of a problem design, this MUST BE FIXED! */
+       dev_err(device, "%s needs to be fixed!\n", __func__);
+       WARN_ON(1);
        DPRINT_EXIT(VMBUS_DRV);
 }