* [PATCH 1/5] nbd: use task_pid_nr() to get current pid
@ 2011-08-17 9:22 Amerigo Wang
2011-08-17 9:22 ` [PATCH 2/5] nbd: replace sysfs_create_file() with device_create_file() Amerigo Wang
` (3 more replies)
0 siblings, 4 replies; 5+ messages in thread
From: Amerigo Wang @ 2011-08-17 9:22 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, Greg Kroah-Hartman, WANG Cong, Paul Clements
From: WANG Cong <amwang@redhat.com>
Signed-off-by: WANG Cong <amwang@redhat.com>
---
drivers/block/nbd.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
index f533f33..a928287 100644
--- a/drivers/block/nbd.c
+++ b/drivers/block/nbd.c
@@ -405,7 +405,7 @@ static int nbd_do_it(struct nbd_device *lo)
BUG_ON(lo->magic != LO_MAGIC);
- lo->pid = current->pid;
+ lo->pid = task_pid_nr(current);
ret = sysfs_create_file(&disk_to_dev(lo->disk)->kobj, &pid_attr.attr);
if (ret) {
printk(KERN_ERR "nbd: sysfs_create_file failed!");
--
1.7.4.4
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 2/5] nbd: replace sysfs_create_file() with device_create_file()
2011-08-17 9:22 [PATCH 1/5] nbd: use task_pid_nr() to get current pid Amerigo Wang
@ 2011-08-17 9:22 ` Amerigo Wang
2011-08-17 9:22 ` [PATCH 3/5] nbd: replace printk KERN_ERR with dev_err() Amerigo Wang
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Amerigo Wang @ 2011-08-17 9:22 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, Greg Kroah-Hartman, WANG Cong, Paul Clements
From: WANG Cong <amwang@redhat.com>
Signed-off-by: WANG Cong <amwang@redhat.com>
---
drivers/block/nbd.c | 6 +++---
1 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
index a928287..da98360 100644
--- a/drivers/block/nbd.c
+++ b/drivers/block/nbd.c
@@ -406,9 +406,9 @@ static int nbd_do_it(struct nbd_device *lo)
BUG_ON(lo->magic != LO_MAGIC);
lo->pid = task_pid_nr(current);
- ret = sysfs_create_file(&disk_to_dev(lo->disk)->kobj, &pid_attr.attr);
+ ret = device_create_file(disk_to_dev(lo->disk), &pid_attr);
if (ret) {
- printk(KERN_ERR "nbd: sysfs_create_file failed!");
+ printk(KERN_ERR "nbd: device_create_file failed!");
lo->pid = 0;
return ret;
}
@@ -416,7 +416,7 @@ static int nbd_do_it(struct nbd_device *lo)
while ((req = nbd_read_stat(lo)) != NULL)
nbd_end_request(req);
- sysfs_remove_file(&disk_to_dev(lo->disk)->kobj, &pid_attr.attr);
+ device_remove_file(disk_to_dev(lo->disk), &pid_attr);
lo->pid = 0;
return 0;
}
--
1.7.4.4
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 3/5] nbd: replace printk KERN_ERR with dev_err()
2011-08-17 9:22 [PATCH 1/5] nbd: use task_pid_nr() to get current pid Amerigo Wang
2011-08-17 9:22 ` [PATCH 2/5] nbd: replace sysfs_create_file() with device_create_file() Amerigo Wang
@ 2011-08-17 9:22 ` Amerigo Wang
2011-08-17 9:22 ` [PATCH 4/5] nbd: lower the loglevel of an error message Amerigo Wang
2011-08-17 9:22 ` [PATCH 5/5] nbd: replace some printk with dev_warn() and dev_info() Amerigo Wang
3 siblings, 0 replies; 5+ messages in thread
From: Amerigo Wang @ 2011-08-17 9:22 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, Greg Kroah-Hartman, WANG Cong, Paul Clements
From: WANG Cong <amwang@redhat.com>
Signed-off-by: WANG Cong <amwang@redhat.com>
---
drivers/block/nbd.c | 50 +++++++++++++++++++++++++-------------------------
1 files changed, 25 insertions(+), 25 deletions(-)
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
index da98360..1b8e09f 100644
--- a/drivers/block/nbd.c
+++ b/drivers/block/nbd.c
@@ -158,8 +158,9 @@ static int sock_xmit(struct nbd_device *lo, int send, void *buf, int size,
sigset_t blocked, oldset;
if (unlikely(!sock)) {
- printk(KERN_ERR "%s: Attempted %s on closed socket in sock_xmit\n",
- lo->disk->disk_name, (send ? "send" : "recv"));
+ dev_err(disk_to_dev(lo->disk),
+ "Attempted %s on closed socket in sock_xmit\n",
+ (send ? "send" : "recv"));
return -EINVAL;
}
@@ -250,8 +251,8 @@ static int nbd_send_req(struct nbd_device *lo, struct request *req)
result = sock_xmit(lo, 1, &request, sizeof(request),
(nbd_cmd(req) == NBD_CMD_WRITE) ? MSG_MORE : 0);
if (result <= 0) {
- printk(KERN_ERR "%s: Send control failed (result %d)\n",
- lo->disk->disk_name, result);
+ dev_err(disk_to_dev(lo->disk),
+ "Send control failed (result %d)\n", result);
goto error_out;
}
@@ -270,8 +271,9 @@ static int nbd_send_req(struct nbd_device *lo, struct request *req)
lo->disk->disk_name, req, bvec->bv_len);
result = sock_send_bvec(lo, bvec, flags);
if (result <= 0) {
- printk(KERN_ERR "%s: Send data failed (result %d)\n",
- lo->disk->disk_name, result);
+ dev_err(disk_to_dev(lo->disk),
+ "Send data failed (result %d)\n",
+ result);
goto error_out;
}
}
@@ -328,14 +330,13 @@ static struct request *nbd_read_stat(struct nbd_device *lo)
reply.magic = 0;
result = sock_xmit(lo, 0, &reply, sizeof(reply), MSG_WAITALL);
if (result <= 0) {
- printk(KERN_ERR "%s: Receive control failed (result %d)\n",
- lo->disk->disk_name, result);
+ dev_err(disk_to_dev(lo->disk),
+ "Receive control failed (result %d)\n", result);
goto harderror;
}
if (ntohl(reply.magic) != NBD_REPLY_MAGIC) {
- printk(KERN_ERR "%s: Wrong magic (0x%lx)\n",
- lo->disk->disk_name,
+ dev_err(disk_to_dev(lo->disk), "Wrong magic (0x%lx)\n",
(unsigned long)ntohl(reply.magic));
result = -EPROTO;
goto harderror;
@@ -347,15 +348,15 @@ static struct request *nbd_read_stat(struct nbd_device *lo)
if (result != -ENOENT)
goto harderror;
- printk(KERN_ERR "%s: Unexpected reply (%p)\n",
- lo->disk->disk_name, reply.handle);
+ dev_err(disk_to_dev(lo->disk), "Unexpected reply (%p)\n",
+ reply.handle);
result = -EBADR;
goto harderror;
}
if (ntohl(reply.error)) {
- printk(KERN_ERR "%s: Other side returned error (%d)\n",
- lo->disk->disk_name, ntohl(reply.error));
+ dev_err(disk_to_dev(lo->disk), "Other side returned error (%d)\n",
+ ntohl(reply.error));
req->errors++;
return req;
}
@@ -369,8 +370,8 @@ static struct request *nbd_read_stat(struct nbd_device *lo)
rq_for_each_segment(bvec, req, iter) {
result = sock_recv_bvec(lo, bvec);
if (result <= 0) {
- printk(KERN_ERR "%s: Receive data failed (result %d)\n",
- lo->disk->disk_name, result);
+ dev_err(disk_to_dev(lo->disk), "Receive data failed (result %d)\n",
+ result);
req->errors++;
return req;
}
@@ -408,7 +409,7 @@ static int nbd_do_it(struct nbd_device *lo)
lo->pid = task_pid_nr(current);
ret = device_create_file(disk_to_dev(lo->disk), &pid_attr);
if (ret) {
- printk(KERN_ERR "nbd: device_create_file failed!");
+ dev_err(disk_to_dev(lo->disk), "device_create_file failed!\n");
lo->pid = 0;
return ret;
}
@@ -457,8 +458,8 @@ static void nbd_handle_req(struct nbd_device *lo, struct request *req)
if (rq_data_dir(req) == WRITE) {
nbd_cmd(req) = NBD_CMD_WRITE;
if (lo->flags & NBD_READ_ONLY) {
- printk(KERN_ERR "%s: Write on read-only\n",
- lo->disk->disk_name);
+ dev_err(disk_to_dev(lo->disk),
+ "Write on read-only\n");
goto error_out;
}
}
@@ -468,16 +469,15 @@ static void nbd_handle_req(struct nbd_device *lo, struct request *req)
mutex_lock(&lo->tx_lock);
if (unlikely(!lo->sock)) {
mutex_unlock(&lo->tx_lock);
- printk(KERN_ERR "%s: Attempted send on closed socket\n",
- lo->disk->disk_name);
+ dev_err(disk_to_dev(lo->disk),
+ "Attempted send on closed socket\n");
goto error_out;
}
lo->active_req = req;
if (nbd_send_req(lo, req) != 0) {
- printk(KERN_ERR "%s: Request send failed\n",
- lo->disk->disk_name);
+ dev_err(disk_to_dev(lo->disk), "Request send failed\n");
req->errors++;
nbd_end_request(req);
} else {
@@ -549,8 +549,8 @@ static void do_nbd_request(struct request_queue *q)
BUG_ON(lo->magic != LO_MAGIC);
if (unlikely(!lo->sock)) {
- printk(KERN_ERR "%s: Attempted send on closed socket\n",
- lo->disk->disk_name);
+ dev_err(disk_to_dev(lo->disk),
+ "Attempted send on closed socket\n");
req->errors++;
nbd_end_request(req);
spin_lock_irq(q->queue_lock);
--
1.7.4.4
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 4/5] nbd: lower the loglevel of an error message
2011-08-17 9:22 [PATCH 1/5] nbd: use task_pid_nr() to get current pid Amerigo Wang
2011-08-17 9:22 ` [PATCH 2/5] nbd: replace sysfs_create_file() with device_create_file() Amerigo Wang
2011-08-17 9:22 ` [PATCH 3/5] nbd: replace printk KERN_ERR with dev_err() Amerigo Wang
@ 2011-08-17 9:22 ` Amerigo Wang
2011-08-17 9:22 ` [PATCH 5/5] nbd: replace some printk with dev_warn() and dev_info() Amerigo Wang
3 siblings, 0 replies; 5+ messages in thread
From: Amerigo Wang @ 2011-08-17 9:22 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, Greg Kroah-Hartman, WANG Cong, Paul Clements
From: WANG Cong <amwang@redhat.com>
This is only an error, no need to use KERN_CRIT log level.
Signed-off-by: WANG Cong <amwang@redhat.com>
---
drivers/block/nbd.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
index 1b8e09f..2b5fc11 100644
--- a/drivers/block/nbd.c
+++ b/drivers/block/nbd.c
@@ -745,7 +745,7 @@ static int __init nbd_init(void)
BUILD_BUG_ON(sizeof(struct nbd_request) != 28);
if (max_part < 0) {
- printk(KERN_CRIT "nbd: max_part must be >= 0\n");
+ printk(KERN_ERR "nbd: max_part must be >= 0\n");
return -EINVAL;
}
--
1.7.4.4
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 5/5] nbd: replace some printk with dev_warn() and dev_info()
2011-08-17 9:22 [PATCH 1/5] nbd: use task_pid_nr() to get current pid Amerigo Wang
` (2 preceding siblings ...)
2011-08-17 9:22 ` [PATCH 4/5] nbd: lower the loglevel of an error message Amerigo Wang
@ 2011-08-17 9:22 ` Amerigo Wang
3 siblings, 0 replies; 5+ messages in thread
From: Amerigo Wang @ 2011-08-17 9:22 UTC (permalink / raw)
To: linux-kernel; +Cc: akpm, Greg Kroah-Hartman, WANG Cong, Paul Clements
From: WANG Cong <amwang@redhat.com>
Signed-off-by: WANG Cong <amwang@redhat.com>
---
drivers/block/nbd.c | 11 +++++------
1 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c
index 2b5fc11..3353d98 100644
--- a/drivers/block/nbd.c
+++ b/drivers/block/nbd.c
@@ -127,8 +127,7 @@ static void sock_shutdown(struct nbd_device *lo, int lock)
if (lock)
mutex_lock(&lo->tx_lock);
if (lo->sock) {
- printk(KERN_WARNING "%s: shutting down socket\n",
- lo->disk->disk_name);
+ dev_warn(disk_to_dev(lo->disk), "shutting down socket\n");
kernel_sock_shutdown(lo->sock, SHUT_RDWR);
lo->sock = NULL;
}
@@ -576,7 +575,7 @@ static int __nbd_ioctl(struct block_device *bdev, struct nbd_device *lo,
case NBD_DISCONNECT: {
struct request sreq;
- printk(KERN_INFO "%s: NBD_DISCONNECT\n", lo->disk->disk_name);
+ dev_info(disk_to_dev(lo->disk), "NBD_DISCONNECT\n");
blk_rq_init(NULL, &sreq);
sreq.cmd_type = REQ_TYPE_SPECIAL;
@@ -674,7 +673,7 @@ static int __nbd_ioctl(struct block_device *bdev, struct nbd_device *lo,
file = lo->file;
lo->file = NULL;
nbd_clear_que(lo);
- printk(KERN_WARNING "%s: queue cleared\n", lo->disk->disk_name);
+ dev_warn(disk_to_dev(lo->disk), "queue cleared\n");
if (file)
fput(file);
lo->bytesize = 0;
@@ -694,8 +693,8 @@ static int __nbd_ioctl(struct block_device *bdev, struct nbd_device *lo,
return 0;
case NBD_PRINT_DEBUG:
- printk(KERN_INFO "%s: next = %p, prev = %p, head = %p\n",
- bdev->bd_disk->disk_name,
+ dev_info(disk_to_dev(lo->disk),
+ "next = %p, prev = %p, head = %p\n",
lo->queue_head.next, lo->queue_head.prev,
&lo->queue_head);
return 0;
--
1.7.4.4
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2011-08-17 9:24 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-17 9:22 [PATCH 1/5] nbd: use task_pid_nr() to get current pid Amerigo Wang
2011-08-17 9:22 ` [PATCH 2/5] nbd: replace sysfs_create_file() with device_create_file() Amerigo Wang
2011-08-17 9:22 ` [PATCH 3/5] nbd: replace printk KERN_ERR with dev_err() Amerigo Wang
2011-08-17 9:22 ` [PATCH 4/5] nbd: lower the loglevel of an error message Amerigo Wang
2011-08-17 9:22 ` [PATCH 5/5] nbd: replace some printk with dev_warn() and dev_info() Amerigo Wang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Powered by JetHome