- 15 Sep, 2009 40 commits
-
-
Bill Pemberton authored
This variable generated an unused variable warning due to other code in the fuction being commented out. This comments out the variable defination so that the code compiles without warnings. Signed-off-by:
Bill Pemberton <wfp5p@virginia.edu> Cc: Hank Janssen <hjanssen@microsoft.com> Signed-off-by:
Greg Kroah-Hartman <gregkh@suse.de>
-
Bill Pemberton authored
The return value of driver_for_each_device() is now checked. A non-zero value simply generates a warning message, but it's better than not checking at all. Signed-off-by:
-
Nicolas Palix authored
Remove typedef DEVICE_OBJECT and use a struct named hv_device instead. Remove typedef PDEVICE_OBJECT which aliases a pointer and use struct hv_device * instead. Here is the semantic patch to perform this transformation: (http://coccinelle.lip6.fr/) //<smpl> @rm_PDEVICE_OBJECT@ @@ -typedef struct _DEVICE_OBJECT *PDEVICE_OBJECT; @rm_DEVICE_OBJECT@ @@ -typedef struct _DEVICE_OBJECT +struct hv_device {...} -DEVICE_OBJECT ; @fixtypedef_PDEVICE_OBJECT@ typedef PDEVICE_OBJECT; @@ -PDEVICE_OBJECT +struct hv_device* @fixtypedef_DEVICE_OBJECT@ typedef DEVICE_OBJECT; @@ -DEVICE_OBJECT +struct hv_device @fixstruct__DEVICE_OBJECT@ @@ struct -_DEVICE_OBJECT +hv_device //</smpl> Signed-off-by:
Nicolas Palix <npalix@diku.dk> Cc: Hank Janssen <hjanssen@microsoft.com> Signed-off-by:
-
Bill Pemberton authored
WORKQUEUE was a wrapper around struct workqueue_struct so just use that instead. Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Note that this struct doesn't appear to be used anywhere and can probably be removed. Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
physAddr was declared but never used in HvInit() Signed-off-by:
-
Bill Pemberton authored
vmbus_ctl_table_hdr, vmbus_dev_ctl_table, vmbus_ctl_table, and vmus_root_ctl_table were never used. This removes them. Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
Remove preprocessor check for X2V_LINUX in osd.c Signed-off-by:
-
Bill Pemberton authored
blkvsc_ioctl() had the wrong parameter list for struct block_device_operations Signed-off-by:
-
Bill Pemberton authored
blkvsc_release() had the wrong parameter list for struct block_device_operations Signed-off-by:
-
Bill Pemberton authored
blkvsc_open() had the wrong parameter list for struct block_device_operations Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Bill Pemberton authored
There were a few places that used %lx when they should have used %x and a few places that used %d when they should have used %ld Signed-off-by:
-
Bill Pemberton authored
Remove C99 // comments with traditional /* */ comments Signed-off-by:
-
Bill Pemberton authored
Signed-off-by:
-
Greg Kroah-Hartman authored
Right now they can not be built into the kernel due to global symbol name conflicts that this code is causing. Thanks to Nicolas Palix <npalix@diku.dk> for pointing this out. Reported-by:
-
Nicolas Palix authored
Here is a short excerpt of the semantic patch performing this transformation: (http://www.emn.fr/x-info/coccinelle/) //<smpl> @@ expression E1,E2,E3; @@ - kzalloc(E1*sizeof(E2),E3) + kcalloc(E1,sizeof(E2),E3) @@ expression E1,E3; type T; @@ - kzalloc(E1*sizeof(T),E3) + kcalloc(E1,sizeof(T),E3) //</smpl> Signed-off-by:
-
Moritz Muehlenhoff authored
Signed-off-by:
Moritz Muehlenhoff <jmm@debian.org> Cc: Hank Janssen <hjanssen@microsoft.com> Signed-off-by:
-
Greg Kroah-Hartman authored
No one was even using them. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
After LogMsg was converted to printk, lots of build warnings showed up as no one was checking the arguments to LogMsg. This patch fixes them all. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
Use the "real" print_hex_dump_bytes call instead of a wrapper function. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
Use the "real" printk call instead of a wrapper function. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
Use the "real" mb call instead of a wrapper function. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
Use the "real" udelay call instead of a wrapper function. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
Now that there are no users of the wrapper functions for spinlocks, remove them. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
Don't use the wrapper functions for this lock, make it a real lock so that we know what is going on. I don't think we really want to be doing a irqsave for this code, but I left it alone to preserve the original codepath. It should be reviewed later. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
Don't use the wrapper functions for this lock, make it a real lock so that we know what is going on. I don't think we really want to be doing a irqsave for this code, but I left it alone to preserve the original codepath. It should be reviewed later. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
Don't use the wrapper functions for this lock, make it a real lock so that we know what is going on. I don't think we really want to be doing a irqsave for this code, but I left it alone to preserve the original codepath. It should be reviewed later. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by: -
Greg Kroah-Hartman authored
Don't use the wrapper functions for this lock, make it a real lock so that we know what is going on. I don't think we really want to be doing a irqsave for this code, but I left it alone to preserve the original codepath. It should be reviewed later. Cc: Hank Janssen <hjanssen@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Signed-off-by:
-
