Skip to content

Commit

Permalink
switch /dev/kmsg to ->write_iter()
Browse files Browse the repository at this point in the history
Signed-off-by: Al Viro <[email protected]>
  • Loading branch information
Al Viro committed Oct 9, 2014
1 parent cd678fc commit 849f312
Showing 1 changed file with 7 additions and 13 deletions.
20 changes: 7 additions & 13 deletions kernel/printk/printk.c
Original file line number Diff line number Diff line change
Expand Up @@ -519,14 +519,13 @@ struct devkmsg_user {
char buf[8192];
};

static ssize_t devkmsg_writev(struct kiocb *iocb, const struct iovec *iv,
unsigned long count, loff_t pos)
static ssize_t devkmsg_write(struct kiocb *iocb, struct iov_iter *from)
{
char *buf, *line;
int i;
int level = default_message_loglevel;
int facility = 1; /* LOG_USER */
size_t len = iov_length(iv, count);
size_t len = iocb->ki_nbytes;
ssize_t ret = len;

if (len > LOG_LINE_MAX)
Expand All @@ -535,13 +534,10 @@ static ssize_t devkmsg_writev(struct kiocb *iocb, const struct iovec *iv,
if (buf == NULL)
return -ENOMEM;

line = buf;
for (i = 0; i < count; i++) {
if (copy_from_user(line, iv[i].iov_base, iv[i].iov_len)) {
ret = -EFAULT;
goto out;
}
line += iv[i].iov_len;
buf[len] = '\0';
if (copy_from_iter(buf, len, from) != len) {
kfree(buf);
return -EFAULT;
}

/*
Expand All @@ -567,10 +563,8 @@ static ssize_t devkmsg_writev(struct kiocb *iocb, const struct iovec *iv,
line = endp;
}
}
line[len] = '\0';

printk_emit(facility, level, NULL, 0, "%s", line);
out:
kfree(buf);
return ret;
}
Expand Down Expand Up @@ -802,7 +796,7 @@ static int devkmsg_release(struct inode *inode, struct file *file)
const struct file_operations kmsg_fops = {
.open = devkmsg_open,
.read = devkmsg_read,
.aio_write = devkmsg_writev,
.write_iter = devkmsg_write,
.llseek = devkmsg_llseek,
.poll = devkmsg_poll,
.release = devkmsg_release,
Expand Down

0 comments on commit 849f312

Please sign in to comment.