* [patch 0/5] hv cleanups
@ 2011-04-16 16:50 Olaf Hering
2011-04-16 16:50 ` [patch 1/5] hv: Reduce indention in vmbus_on_event Olaf Hering
` (4 more replies)
0 siblings, 5 replies; 7+ messages in thread
From: Olaf Hering @ 2011-04-16 16:50 UTC (permalink / raw)
To: Hank Janssen, Haiyang Zhang, Greg Kroah-Hartman; +Cc: linux-kernel
rebase patches to linux-next-20110415
drivers/staging/hv/blkvsc_drv.c | 70 +++++++++++++++++++-------------------
drivers/staging/hv/connection.c | 46 ++++++++++--------------
drivers/staging/hv/hv_mouse.c | 4 +-
drivers/staging/hv/netvsc.c | 2 +-
drivers/staging/hv/storvsc.c | 6 ++--
drivers/staging/hv/storvsc_drv.c | 4 +-
drivers/staging/hv/vmbus_drv.c | 8 ++---
7 files changed, 65 insertions(+), 75 deletions(-)
Olaf
^ permalink raw reply [flat|nested] 7+ messages in thread
* [patch 1/5] hv: Reduce indention in vmbus_on_event
2011-04-16 16:50 [patch 0/5] hv cleanups Olaf Hering
@ 2011-04-16 16:50 ` Olaf Hering
2011-04-16 16:50 ` [patch 2/5] hv: pass u32 to process_chn_event() Olaf Hering
` (3 subsequent siblings)
4 siblings, 0 replies; 7+ messages in thread
From: Olaf Hering @ 2011-04-16 16:50 UTC (permalink / raw)
To: Hank Janssen, Haiyang Zhang, Greg Kroah-Hartman; +Cc: linux-kernel
[-- Attachment #1: bug679942-vmbus_on_event.patch --]
[-- Type: text/plain, Size: 1741 bytes --]
Reduce indention in vmbus_on_event() by converting two
if (var) to if (!var)
Signed-off-by: Olaf Hering <olaf@aepfle.de>
---
compile tested.
drivers/staging/hv/connection.c | 33 +++++++++++++--------------------
1 file changed, 13 insertions(+), 20 deletions(-)
Index: linux-next/drivers/staging/hv/connection.c
===================================================================
--- linux-next.orig/drivers/staging/hv/connection.c
+++ linux-next/drivers/staging/hv/connection.c
@@ -287,30 +287,23 @@ void vmbus_on_event(unsigned long data)
u32 *recv_int_page = vmbus_connection.recv_int_page;
/* Check events */
- if (recv_int_page) {
- for (dword = 0; dword < maxdword; dword++) {
- if (recv_int_page[dword]) {
- for (bit = 0; bit < 32; bit++) {
- if (sync_test_and_clear_bit(bit,
- (unsigned long *)
- &recv_int_page[dword])) {
- relid = (dword << 5) + bit;
+ if (!recv_int_page)
+ return;
+ for (dword = 0; dword < maxdword; dword++) {
+ if (!recv_int_page[dword])
+ continue;
+ for (bit = 0; bit < 32; bit++) {
+ if (sync_test_and_clear_bit(bit, (unsigned long *)&recv_int_page[dword])) {
+ relid = (dword << 5) + bit;
- if (relid == 0) {
- /* special case - vmbus channel protocol msg */
- continue;
- } else {
- /* QueueWorkItem(VmbusProcessEvent, (void*)relid); */
- /* ret = WorkQueueQueueWorkItem(gVmbusConnection.workQueue, VmbusProcessChannelEvent, (void*)relid); */
- process_chn_event((void *)
- (unsigned long)relid);
- }
- }
+ if (relid == 0) {
+ /* special case - vmbus channel protocol msg */
+ continue;
}
+ process_chn_event((void *) (unsigned long)relid);
}
- }
+ }
}
- return;
}
/*
^ permalink raw reply [flat|nested] 7+ messages in thread
* [patch 2/5] hv: pass u32 to process_chn_event()
2011-04-16 16:50 [patch 0/5] hv cleanups Olaf Hering
2011-04-16 16:50 ` [patch 1/5] hv: Reduce indention in vmbus_on_event Olaf Hering
@ 2011-04-16 16:50 ` Olaf Hering
2011-04-16 16:50 ` [patch 3/5] hv: pass integer to tasklet_init() Olaf Hering
` (2 subsequent siblings)
4 siblings, 0 replies; 7+ messages in thread
From: Olaf Hering @ 2011-04-16 16:50 UTC (permalink / raw)
To: Hank Janssen, Haiyang Zhang, Greg Kroah-Hartman; +Cc: linux-kernel
[-- Attachment #1: bug679942-process_chn_event.patch --]
[-- Type: text/plain, Size: 1720 bytes --]
Change types in vmbus_on_event() to u32 since the input is u32 as well.
Pass u32 to process_chn_event() instead of casting arg to void* and back.
Update printk to reflect type change.
Signed-off-by: Olaf Hering <olaf@aepfle.de>
---
compile tested.
drivers/staging/hv/connection.c | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
Index: linux-next/drivers/staging/hv/connection.c
===================================================================
--- linux-next.orig/drivers/staging/hv/connection.c
+++ linux-next/drivers/staging/hv/connection.c
@@ -250,10 +250,9 @@ struct vmbus_channel *relid2channel(u32
/*
* process_chn_event - Process a channel event notification
*/
-static void process_chn_event(void *context)
+static void process_chn_event(u32 relid)
{
struct vmbus_channel *channel;
- u32 relid = (u32)(unsigned long)context;
/* ASSERT(relId > 0); */
@@ -271,7 +270,7 @@ static void process_chn_event(void *cont
* (void*)channel);
*/
} else {
- pr_err("channel not found for relid - %d\n", relid);
+ pr_err("channel not found for relid - %u\n", relid);
}
}
@@ -280,10 +279,10 @@ static void process_chn_event(void *cont
*/
void vmbus_on_event(unsigned long data)
{
- int dword;
- int maxdword = MAX_NUM_CHANNELS_SUPPORTED >> 5;
+ u32 dword;
+ u32 maxdword = MAX_NUM_CHANNELS_SUPPORTED >> 5;
int bit;
- int relid;
+ u32 relid;
u32 *recv_int_page = vmbus_connection.recv_int_page;
/* Check events */
@@ -300,7 +299,7 @@ void vmbus_on_event(unsigned long data)
/* special case - vmbus channel protocol msg */
continue;
}
- process_chn_event((void *) (unsigned long)relid);
+ process_chn_event(relid);
}
}
}
^ permalink raw reply [flat|nested] 7+ messages in thread
* [patch 3/5] hv: pass integer to tasklet_init()
2011-04-16 16:50 [patch 0/5] hv cleanups Olaf Hering
2011-04-16 16:50 ` [patch 1/5] hv: Reduce indention in vmbus_on_event Olaf Hering
2011-04-16 16:50 ` [patch 2/5] hv: pass u32 to process_chn_event() Olaf Hering
@ 2011-04-16 16:50 ` Olaf Hering
2011-04-16 16:50 ` [patch 4/5] hv: make vmbus_loglevel writeable Olaf Hering
2011-04-16 16:50 ` [patch 5/5] hv: remove double newline in DPRINT functions Olaf Hering
4 siblings, 0 replies; 7+ messages in thread
From: Olaf Hering @ 2011-04-16 16:50 UTC (permalink / raw)
To: Hank Janssen, Haiyang Zhang, Greg Kroah-Hartman; +Cc: linux-kernel
[-- Attachment #1: bug679942-tasklet_init.patch --]
[-- Type: text/plain, Size: 910 bytes --]
tasklet_init() takes an integer, so use one right away.
Signed-off-by: Olaf Hering <olaf@aepfle.de>
---
compile tested
drivers/staging/hv/vmbus_drv.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
Index: linux-next/drivers/staging/hv/vmbus_drv.c
===================================================================
--- linux-next.orig/drivers/staging/hv/vmbus_drv.c
+++ linux-next/drivers/staging/hv/vmbus_drv.c
@@ -578,10 +578,8 @@ static int vmbus_bus_init(struct pci_dev
hv_bus.bus.name = driver_name;
/* Initialize the bus context */
- tasklet_init(&hv_bus.msg_dpc, vmbus_on_msg_dpc,
- (unsigned long)NULL);
- tasklet_init(&hv_bus.event_dpc, vmbus_on_event,
- (unsigned long)NULL);
+ tasklet_init(&hv_bus.msg_dpc, vmbus_on_msg_dpc, 0);
+ tasklet_init(&hv_bus.event_dpc, vmbus_on_event, 0);
/* Now, register the bus with LDM */
ret = bus_register(&hv_bus.bus);
^ permalink raw reply [flat|nested] 7+ messages in thread
* [patch 4/5] hv: make vmbus_loglevel writeable
2011-04-16 16:50 [patch 0/5] hv cleanups Olaf Hering
` (2 preceding siblings ...)
2011-04-16 16:50 ` [patch 3/5] hv: pass integer to tasklet_init() Olaf Hering
@ 2011-04-16 16:50 ` Olaf Hering
2011-04-16 16:50 ` [patch 5/5] hv: remove double newline in DPRINT functions Olaf Hering
4 siblings, 0 replies; 7+ messages in thread
From: Olaf Hering @ 2011-04-16 16:50 UTC (permalink / raw)
To: Hank Janssen, Haiyang Zhang, Greg Kroah-Hartman; +Cc: linux-kernel
[-- Attachment #1: bug679942-vmbus_loglevel.patch --]
[-- Type: text/plain, Size: 713 bytes --]
make /sys/module/hv_vmbus/parameters/vmbus_loglevel writeable by root.
Signed-off-by: Olaf Hering <olaf@aepfle.de>
---
compile tested.
drivers/staging/hv/vmbus_drv.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Index: linux-next/drivers/staging/hv/vmbus_drv.c
===================================================================
--- linux-next.orig/drivers/staging/hv/vmbus_drv.c
+++ linux-next/drivers/staging/hv/vmbus_drv.c
@@ -827,7 +827,7 @@ static void __exit hv_pci_exit(void)
MODULE_LICENSE("GPL");
MODULE_VERSION(HV_DRV_VERSION);
-module_param(vmbus_loglevel, int, S_IRUGO);
+module_param(vmbus_loglevel, int, S_IRUGO|S_IWUSR);
module_init(hv_pci_init);
module_exit(hv_pci_exit);
^ permalink raw reply [flat|nested] 7+ messages in thread
* [patch 5/5] hv: remove double newline in DPRINT functions
2011-04-16 16:50 [patch 0/5] hv cleanups Olaf Hering
` (3 preceding siblings ...)
2011-04-16 16:50 ` [patch 4/5] hv: make vmbus_loglevel writeable Olaf Hering
@ 2011-04-16 16:50 ` Olaf Hering
2011-04-20 20:40 ` Greg KH
4 siblings, 1 reply; 7+ messages in thread
From: Olaf Hering @ 2011-04-16 16:50 UTC (permalink / raw)
To: Hank Janssen, Haiyang Zhang, Greg Kroah-Hartman; +Cc: linux-kernel
[-- Attachment #1: bug679942-dprintk_newline.patch --]
[-- Type: text/plain, Size: 14961 bytes --]
Remove double newline in DPRINT* functions, the macro already adds a newline.
Signed-off-by: Olaf Hering <olaf@aepfle.de>
---
compile tested
drivers/staging/hv/blkvsc_drv.c | 70 +++++++++++++++++++--------------------
drivers/staging/hv/hv_mouse.c | 4 +-
drivers/staging/hv/netvsc.c | 2 -
drivers/staging/hv/storvsc.c | 6 +--
drivers/staging/hv/storvsc_drv.c | 4 +-
5 files changed, 43 insertions(+), 43 deletions(-)
Index: linux-next/drivers/staging/hv/blkvsc_drv.c
===================================================================
--- linux-next.orig/drivers/staging/hv/blkvsc_drv.c
+++ linux-next/drivers/staging/hv/blkvsc_drv.c
@@ -474,7 +474,7 @@ static void blkvsc_shutdown(struct devic
if (!blkdev)
return;
- DPRINT_DBG(BLKVSC_DRV, "blkvsc_shutdown - users %d disk %s\n",
+ DPRINT_DBG(BLKVSC_DRV, "blkvsc_shutdown - users %d disk %s",
blkdev->users, blkdev->gd->disk_name);
spin_lock_irqsave(&blkdev->lock, flags);
@@ -504,7 +504,7 @@ static int blkvsc_do_flush(struct block_
{
struct blkvsc_request *blkvsc_req;
- DPRINT_DBG(BLKVSC_DRV, "blkvsc_do_flush()\n");
+ DPRINT_DBG(BLKVSC_DRV, "blkvsc_do_flush()");
if (blkdev->device_type != HARDDISK_TYPE)
return 0;
@@ -543,7 +543,7 @@ static int blkvsc_do_inquiry(struct bloc
unsigned char *buf;
unsigned char device_type;
- DPRINT_DBG(BLKVSC_DRV, "blkvsc_do_inquiry()\n");
+ DPRINT_DBG(BLKVSC_DRV, "blkvsc_do_inquiry()");
blkvsc_req = kmem_cache_zalloc(blkdev->request_pool, GFP_KERNEL);
if (!blkvsc_req)
@@ -574,7 +574,7 @@ static int blkvsc_do_inquiry(struct bloc
blkvsc_submit_request(blkvsc_req, blkvsc_cmd_completion);
- DPRINT_DBG(BLKVSC_DRV, "waiting %p to complete\n",
+ DPRINT_DBG(BLKVSC_DRV, "waiting %p to complete",
blkvsc_req);
wait_for_completion_interruptible(&blkvsc_req->request.wait_event);
@@ -594,7 +594,7 @@ static int blkvsc_do_inquiry(struct bloc
blkdev->device_type = UNKNOWN_DEV_TYPE;
}
- DPRINT_DBG(BLKVSC_DRV, "device type %d\n", device_type);
+ DPRINT_DBG(BLKVSC_DRV, "device type %d", device_type);
blkdev->device_id_len = buf[7];
if (blkdev->device_id_len > 64)
@@ -622,7 +622,7 @@ static int blkvsc_do_read_capacity(struc
struct scsi_sense_hdr sense_hdr;
struct vmscsi_request *vm_srb;
- DPRINT_DBG(BLKVSC_DRV, "blkvsc_do_read_capacity()\n");
+ DPRINT_DBG(BLKVSC_DRV, "blkvsc_do_read_capacity()");
blkdev->sector_size = 0;
blkdev->capacity = 0;
@@ -655,7 +655,7 @@ static int blkvsc_do_read_capacity(struc
blkvsc_submit_request(blkvsc_req, blkvsc_cmd_completion);
- DPRINT_DBG(BLKVSC_DRV, "waiting %p to complete\n",
+ DPRINT_DBG(BLKVSC_DRV, "waiting %p to complete",
blkvsc_req);
wait_for_completion_interruptible(&blkvsc_req->request.wait_event);
@@ -696,7 +696,7 @@ static int blkvsc_do_read_capacity16(str
struct scsi_sense_hdr sense_hdr;
struct vmscsi_request *vm_srb;
- DPRINT_DBG(BLKVSC_DRV, "blkvsc_do_read_capacity16()\n");
+ DPRINT_DBG(BLKVSC_DRV, "blkvsc_do_read_capacity16()");
blkdev->sector_size = 0;
blkdev->capacity = 0;
@@ -734,7 +734,7 @@ static int blkvsc_do_read_capacity16(str
blkvsc_submit_request(blkvsc_req, blkvsc_cmd_completion);
- DPRINT_DBG(BLKVSC_DRV, "waiting %p to complete\n",
+ DPRINT_DBG(BLKVSC_DRV, "waiting %p to complete",
blkvsc_req);
wait_for_completion_interruptible(&blkvsc_req->request.wait_event);
@@ -785,7 +785,7 @@ static int blkvsc_remove(struct device *
unsigned long flags;
int ret;
- DPRINT_DBG(BLKVSC_DRV, "blkvsc_remove()\n");
+ DPRINT_DBG(BLKVSC_DRV, "blkvsc_remove()");
if (!storvsc_drv_obj->base.dev_rm)
return -1;
@@ -906,7 +906,7 @@ static int blkvsc_submit_request(struct
DPRINT_DBG(BLKVSC_DRV, "blkvsc_submit_request() - "
"req %p type %s start_sector %lu count %ld offset %d "
- "len %d\n", blkvsc_req,
+ "len %d", blkvsc_req,
(blkvsc_req->write) ? "WRITE" : "READ",
(unsigned long) blkvsc_req->sector_start,
blkvsc_req->sector_count,
@@ -915,7 +915,7 @@ static int blkvsc_submit_request(struct
#if 0
for (i = 0; i < (blkvsc_req->request.data_buffer.len >> 12); i++) {
DPRINT_DBG(BLKVSC_DRV, "blkvsc_submit_request() - "
- "req %p pfn[%d] %llx\n",
+ "req %p pfn[%d] %llx",
blkvsc_req, i,
blkvsc_req->request.data_buffer.pfn_array[i]);
}
@@ -969,7 +969,7 @@ static int blkvsc_do_request(struct bloc
int pending = 0;
struct blkvsc_request_group *group = NULL;
- DPRINT_DBG(BLKVSC_DRV, "blkdev %p req %p sect %lu\n", blkdev, req,
+ DPRINT_DBG(BLKVSC_DRV, "blkdev %p req %p sect %lu", blkdev, req,
(unsigned long)blk_rq_pos(req));
/* Create a group to tie req to list of blkvsc_reqs */
@@ -991,7 +991,7 @@ static int blkvsc_do_request(struct bloc
bio_for_each_segment(bvec, bio, seg_idx) {
DPRINT_DBG(BLKVSC_DRV, "bio_for_each_segment() "
"- req %p bio %p bvec %p seg_idx %d "
- "databuf_idx %d\n", req, bio, bvec,
+ "databuf_idx %d", req, bio, bvec,
seg_idx, databuf_idx);
/* Get a new storvsc request */
@@ -1074,7 +1074,7 @@ static int blkvsc_do_request(struct bloc
/* Handle the last one */
if (blkvsc_req) {
- DPRINT_DBG(BLKVSC_DRV, "blkdev %p req %p group %p count %d\n",
+ DPRINT_DBG(BLKVSC_DRV, "blkdev %p req %p group %p count %d",
blkdev, req, blkvsc_req->group,
blkvsc_req->group->outstanding);
@@ -1092,7 +1092,7 @@ static int blkvsc_do_request(struct bloc
if (pending) {
DPRINT_DBG(BLKVSC_DRV, "adding blkvsc_req to "
"pending_list - blkvsc_req %p start_sect %lu"
- " sect_count %ld (%lu %ld)\n", blkvsc_req,
+ " sect_count %ld (%lu %ld)", blkvsc_req,
(unsigned long)blkvsc_req->sector_start,
blkvsc_req->sector_count,
(unsigned long)start_sector,
@@ -1111,7 +1111,7 @@ static int blkvsc_do_request(struct bloc
DPRINT_DBG(BLKVSC_DRV, "submitted blkvsc_req %p "
"start_sect %lu sect_count %ld (%lu %ld) "
- "ret %d\n", blkvsc_req,
+ "ret %d", blkvsc_req,
(unsigned long)blkvsc_req->sector_start,
blkvsc_req->sector_count,
(unsigned long)start_sector,
@@ -1131,7 +1131,7 @@ static void blkvsc_cmd_completion(struct
struct scsi_sense_hdr sense_hdr;
struct vmscsi_request *vm_srb;
- DPRINT_DBG(BLKVSC_DRV, "blkvsc_cmd_completion() - req %p\n",
+ DPRINT_DBG(BLKVSC_DRV, "blkvsc_cmd_completion() - req %p",
blkvsc_req);
vm_srb = &blkvsc_req->request.vstor_packet.vm_srb;
@@ -1159,7 +1159,7 @@ static void blkvsc_request_completion(st
DPRINT_DBG(BLKVSC_DRV, "blkdev %p blkvsc_req %p group %p type %s "
"sect_start %lu sect_count %ld len %d group outstd %d "
- "total outstd %d\n",
+ "total outstd %d",
blkdev, blkvsc_req, blkvsc_req->group,
(blkvsc_req->write) ? "WRITE" : "READ",
(unsigned long)blkvsc_req->sector_start,
@@ -1183,7 +1183,7 @@ static void blkvsc_request_completion(st
&blkvsc_req->group->blkvsc_req_list,
req_entry) {
DPRINT_DBG(BLKVSC_DRV, "completing blkvsc_req %p "
- "sect_start %lu sect_count %ld\n",
+ "sect_start %lu sect_count %ld",
comp_req,
(unsigned long)comp_req->sector_start,
comp_req->sector_count);
@@ -1199,7 +1199,7 @@ static void blkvsc_request_completion(st
* All the sectors have been xferred ie the
* request is done
*/
- DPRINT_DBG(BLKVSC_DRV, "req %p COMPLETED\n",
+ DPRINT_DBG(BLKVSC_DRV, "req %p COMPLETED",
comp_req->req);
kmem_cache_free(blkdev->request_pool,
comp_req->group);
@@ -1240,7 +1240,7 @@ static int blkvsc_cancel_pending_reqs(st
&pend_req->group->blkvsc_req_list,
req_entry) {
DPRINT_DBG(BLKVSC_DRV, "completing blkvsc_req %p "
- "sect_start %lu sect_count %ld\n",
+ "sect_start %lu sect_count %ld",
comp_req,
(unsigned long) comp_req->sector_start,
comp_req->sector_count);
@@ -1267,7 +1267,7 @@ static int blkvsc_cancel_pending_reqs(st
kmem_cache_free(blkdev->request_pool, comp_req);
}
- DPRINT_DBG(BLKVSC_DRV, "cancelling pending request - %p\n",
+ DPRINT_DBG(BLKVSC_DRV, "cancelling pending request - %p",
pend_req);
list_del(&pend_req->pend_entry);
@@ -1284,7 +1284,7 @@ static int blkvsc_cancel_pending_reqs(st
*/
DPRINT_DBG(BLKVSC_DRV,
"blkvsc_cancel_pending_reqs() - "
- "req %p COMPLETED\n", pend_req->req);
+ "req %p COMPLETED", pend_req->req);
kmem_cache_free(blkdev->request_pool,
pend_req->group);
}
@@ -1305,7 +1305,7 @@ static int blkvsc_do_pending_reqs(struct
/* Flush the pending list first */
list_for_each_entry_safe(pend_req, tmp, &blkdev->pending_list,
pend_entry) {
- DPRINT_DBG(BLKVSC_DRV, "working off pending_list - %p\n",
+ DPRINT_DBG(BLKVSC_DRV, "working off pending_list - %p",
pend_req);
ret = blkvsc_submit_request(pend_req,
@@ -1325,9 +1325,9 @@ static void blkvsc_request(struct reques
struct request *req;
int ret = 0;
- DPRINT_DBG(BLKVSC_DRV, "- enter\n");
+ DPRINT_DBG(BLKVSC_DRV, "- enter");
while ((req = blk_peek_request(queue)) != NULL) {
- DPRINT_DBG(BLKVSC_DRV, "- req %p\n", req);
+ DPRINT_DBG(BLKVSC_DRV, "- req %p", req);
blkdev = req->rq_disk->private_data;
if (blkdev->shutting_down || req->cmd_type != REQ_TYPE_FS ||
@@ -1340,7 +1340,7 @@ static void blkvsc_request(struct reques
if (ret != 0) {
DPRINT_DBG(BLKVSC_DRV,
- "- stop queue - pending_list not empty\n");
+ "- stop queue - pending_list not empty");
blk_stop_queue(queue);
break;
}
@@ -1349,11 +1349,11 @@ static void blkvsc_request(struct reques
ret = blkvsc_do_request(blkdev, req);
if (ret > 0) {
- DPRINT_DBG(BLKVSC_DRV, "- stop queue - no room\n");
+ DPRINT_DBG(BLKVSC_DRV, "- stop queue - no room");
blk_stop_queue(queue);
break;
} else if (ret < 0) {
- DPRINT_DBG(BLKVSC_DRV, "- stop queue - no mem\n");
+ DPRINT_DBG(BLKVSC_DRV, "- stop queue - no mem");
blk_requeue_request(queue, req);
blk_stop_queue(queue);
break;
@@ -1365,7 +1365,7 @@ static int blkvsc_open(struct block_devi
{
struct block_device_context *blkdev = bdev->bd_disk->private_data;
- DPRINT_DBG(BLKVSC_DRV, "- users %d disk %s\n", blkdev->users,
+ DPRINT_DBG(BLKVSC_DRV, "- users %d disk %s", blkdev->users,
blkdev->gd->disk_name);
mutex_lock(&blkvsc_mutex);
@@ -1388,7 +1388,7 @@ static int blkvsc_release(struct gendisk
{
struct block_device_context *blkdev = disk->private_data;
- DPRINT_DBG(BLKVSC_DRV, "- users %d disk %s\n", blkdev->users,
+ DPRINT_DBG(BLKVSC_DRV, "- users %d disk %s", blkdev->users,
blkdev->gd->disk_name);
mutex_lock(&blkvsc_mutex);
@@ -1409,7 +1409,7 @@ static int blkvsc_release(struct gendisk
static unsigned int blkvsc_check_events(struct gendisk *gd,
unsigned int clearing)
{
- DPRINT_DBG(BLKVSC_DRV, "- enter\n");
+ DPRINT_DBG(BLKVSC_DRV, "- enter");
return DISK_EVENT_MEDIA_CHANGE;
}
@@ -1417,7 +1417,7 @@ static int blkvsc_revalidate_disk(struct
{
struct block_device_context *blkdev = gd->private_data;
- DPRINT_DBG(BLKVSC_DRV, "- enter\n");
+ DPRINT_DBG(BLKVSC_DRV, "- enter");
if (blkdev->device_type == DVD_TYPE) {
blkvsc_do_read_capacity(blkdev);
@@ -1521,7 +1521,7 @@ static int blkvsc_ioctl(struct block_dev
*/
#if 0
case HDIO_GET_IDENTITY:
- DPRINT_INFO(BLKVSC_DRV, "HDIO_GET_IDENTITY\n");
+ DPRINT_INFO(BLKVSC_DRV, "HDIO_GET_IDENTITY");
if (copy_to_user((void __user *)arg, blkdev->device_id,
blkdev->device_id_len))
ret = -EFAULT;
Index: linux-next/drivers/staging/hv/hv_mouse.c
===================================================================
--- linux-next.orig/drivers/staging/hv/hv_mouse.c
+++ linux-next/drivers/staging/hv/hv_mouse.c
@@ -931,13 +931,13 @@ static void reportdesc_callback(struct h
input_device_ctx->connected = 1;
DPRINT_INFO(INPUTVSC_DRV,
- "HID device claimed by input\n");
+ "HID device claimed by input");
}
if (!hid_dev->claimed) {
DPRINT_ERR(INPUTVSC_DRV,
"HID device not claimed by "
- "input or hiddev\n");
+ "input or hiddev");
}
input_device_ctx->hid_device = hid_dev;
Index: linux-next/drivers/staging/hv/netvsc.c
===================================================================
--- linux-next.orig/drivers/staging/hv/netvsc.c
+++ linux-next/drivers/staging/hv/netvsc.c
@@ -1186,7 +1186,7 @@ static void netvsc_channel_cb(void *cont
default:
dev_err(&device->device,
"unhandled packet type %d, "
- "tid %llx len %d\n",
+ "tid %llx len %d",
desc->type, request_id,
bytes_recvd);
break;
Index: linux-next/drivers/staging/hv/storvsc.c
===================================================================
--- linux-next.orig/drivers/staging/hv/storvsc.c
+++ linux-next/drivers/staging/hv/storvsc.c
@@ -298,7 +298,7 @@ static void stor_vsc_on_io_completion(st
if (vstor_packet->vm_srb.scsi_status != 0 ||
vstor_packet->vm_srb.srb_status != 1) {
DPRINT_WARN(STORVSC,
- "cmd 0x%x scsi status 0x%x srb status 0x%x\n",
+ "cmd 0x%x scsi status 0x%x srb status 0x%x",
vstor_packet->vm_srb.cdb[0],
vstor_packet->vm_srb.scsi_status,
vstor_packet->vm_srb.srb_status);
@@ -309,7 +309,7 @@ static void stor_vsc_on_io_completion(st
if (vstor_packet->vm_srb.srb_status & 0x80) {
/* autosense data available */
DPRINT_WARN(STORVSC, "storvsc pkt %p autosense data "
- "valid - len %d\n", request,
+ "valid - len %d", request,
vstor_packet->vm_srb.sense_info_length);
memcpy(request->sense_buffer,
@@ -462,7 +462,7 @@ int stor_vsc_on_device_add(struct hv_dev
device_info->path_id = stor_device->path_id;
device_info->target_id = stor_device->target_id;
- DPRINT_DBG(STORVSC, "assigned port %u, path %u target %u\n",
+ DPRINT_DBG(STORVSC, "assigned port %u, path %u target %u",
stor_device->port_number, stor_device->path_id,
stor_device->target_id);
Index: linux-next/drivers/staging/hv/storvsc_drv.c
===================================================================
--- linux-next.orig/drivers/staging/hv/storvsc_drv.c
+++ linux-next/drivers/staging/hv/storvsc_drv.c
@@ -108,7 +108,7 @@ static int stor_vsc_initialize(struct hv
sizeof(struct vstor_packet) + sizeof(u64),
sizeof(u64)));
- DPRINT_INFO(STORVSC, "max io %u, currently %u\n",
+ DPRINT_INFO(STORVSC, "max io %u, currently %u",
stor_driver->max_outstanding_req_per_channel,
STORVSC_MAX_IO_REQUESTS);
@@ -833,7 +833,7 @@ static int storvsc_queuecommand_lck(stru
request->data_buffer.offset = sgl[0].offset;
for (i = 0; i < sg_count; i++) {
- DPRINT_DBG(STORVSC_DRV, "sgl[%d] len %d offset %d\n",
+ DPRINT_DBG(STORVSC_DRV, "sgl[%d] len %d offset %d",
i, sgl[i].length, sgl[i].offset);
request->data_buffer.pfn_array[i] =
page_to_pfn(sg_page((&sgl[i])));
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [patch 5/5] hv: remove double newline in DPRINT functions
2011-04-16 16:50 ` [patch 5/5] hv: remove double newline in DPRINT functions Olaf Hering
@ 2011-04-20 20:40 ` Greg KH
0 siblings, 0 replies; 7+ messages in thread
From: Greg KH @ 2011-04-20 20:40 UTC (permalink / raw)
To: Olaf Hering; +Cc: Hank Janssen, Haiyang Zhang, Greg Kroah-Hartman, linux-kernel
On Sat, Apr 16, 2011 at 06:50:43PM +0200, Olaf Hering wrote:
> Remove double newline in DPRINT* functions, the macro already adds a newline.
>
> Signed-off-by: Olaf Hering <olaf@aepfle.de>
This one no longer applies :(
thanks,
greg k-h
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2011-04-20 20:41 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-04-16 16:50 [patch 0/5] hv cleanups Olaf Hering
2011-04-16 16:50 ` [patch 1/5] hv: Reduce indention in vmbus_on_event Olaf Hering
2011-04-16 16:50 ` [patch 2/5] hv: pass u32 to process_chn_event() Olaf Hering
2011-04-16 16:50 ` [patch 3/5] hv: pass integer to tasklet_init() Olaf Hering
2011-04-16 16:50 ` [patch 4/5] hv: make vmbus_loglevel writeable Olaf Hering
2011-04-16 16:50 ` [patch 5/5] hv: remove double newline in DPRINT functions Olaf Hering
2011-04-20 20:40 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox