diff options
author | Greg Kroah-Hartman <gregkh@suse.de> | 2009-09-01 23:12:58 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2009-09-15 15:02:02 -0400 |
commit | 689bf406829c8a5e44ebf8fc8918b77b3806ee48 (patch) | |
tree | 6afec7c8b003a8f581826f7ce400dd1ac26d69a6 /drivers | |
parent | f4266e342527b4309119edde777bb9fbb4e0ef33 (diff) |
Staging: hv: warn the world of a bug in the release function
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>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/staging/hv/vmbus_drv.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/staging/hv/vmbus_drv.c b/drivers/staging/hv/vmbus_drv.c index 4462d5bb87e..cc81ffd7e8a 100644 --- a/drivers/staging/hv/vmbus_drv.c +++ b/drivers/staging/hv/vmbus_drv.c | |||
@@ -965,6 +965,11 @@ Desc: Final callback release of the vmbus root device | |||
965 | static void vmbus_bus_release(struct device *device) | 965 | static void vmbus_bus_release(struct device *device) |
966 | { | 966 | { |
967 | DPRINT_ENTER(VMBUS_DRV); | 967 | DPRINT_ENTER(VMBUS_DRV); |
968 | /* FIXME */ | ||
969 | /* Empty release functions are a bug, or a major sign | ||
970 | * of a problem design, this MUST BE FIXED! */ | ||
971 | dev_err(device, "%s needs to be fixed!\n", __func__); | ||
972 | WARN_ON(1); | ||
968 | DPRINT_EXIT(VMBUS_DRV); | 973 | DPRINT_EXIT(VMBUS_DRV); |
969 | } | 974 | } |
970 | 975 | ||