The patch titled
Subject: autofs4: change log print macros to not insert newline
has been added to the -mm tree. Its filename is
autofs4-change-log-print-macros-to-not-insert-newline.patch
This patch should soon appear at
http://ozlabs.org/~akpm/mmots/broken-out/autofs4-change-log-print-macros-to-not-insert-newline.patch
and later at
http://ozlabs.org/~akpm/mmotm/broken-out/autofs4-change-log-print-macros-to-not-insert-newline.patch
Before you just go and hit "reply", please:
a) Consider who else should be cc'ed
b) Prefer to cc a suitable mailing list as well
c) Ideally: find the original patch on the mailing list and do a
reply-to-all to that, adding suitable additional cc's
*** Remember to use Documentation/SubmitChecklist when testing your code ***
The -mm tree is included into linux-next and is updated
there every 3-4 working days
------------------------------------------------------
From: Ian Kent <ikent@xxxxxxxxxx>
Subject: autofs4: change log print macros to not insert newline
Common kernel coding practice is to include the newline of log prints
within the log text rather than hidden away in a macro.
To avoid introducing inconsistencies as changes are made change the log
macros to not include the newline.
Signed-off-by: Ian Kent <raven@xxxxxxxxxx>
Cc: Joe Perches <joe@xxxxxxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
---
fs/autofs4/autofs_i.h | 6 +++---
fs/autofs4/dev-ioctl.c | 14 +++++++-------
fs/autofs4/expire.c | 24 ++++++++++++------------
fs/autofs4/inode.c | 16 ++++++++--------
fs/autofs4/root.c | 26 +++++++++++++-------------
fs/autofs4/waitq.c | 12 ++++++------
6 files changed, 49 insertions(+), 49 deletions(-)
diff -puN fs/autofs4/autofs_i.h~autofs4-change-log-print-macros-to-not-insert-newline fs/autofs4/autofs_i.h
--- a/fs/autofs4/autofs_i.h~autofs4-change-log-print-macros-to-not-insert-newline
+++ a/fs/autofs4/autofs_i.h
@@ -36,15 +36,15 @@
/* #define DEBUG */
#define DPRINTK(fmt, ...) \
- pr_debug(KBUILD_MODNAME ":pid:%d:%s: " fmt "\n",\
+ pr_debug(KBUILD_MODNAME ":pid:%d:%s: " fmt, \
current->pid, __func__, ##__VA_ARGS__)
#define AUTOFS_WARN(fmt, ...) \
- pr_warn(KBUILD_MODNAME ":pid:%d:%s: " fmt "\n", \
+ pr_warn(KBUILD_MODNAME ":pid:%d:%s: " fmt, \
current->pid, __func__, ##__VA_ARGS__)
#define AUTOFS_ERROR(fmt, ...) \
- pr_err(KBUILD_MODNAME ":pid:%d:%s: " fmt "\n", \
+ pr_err(KBUILD_MODNAME ":pid:%d:%s: " fmt, \
current->pid, __func__, ##__VA_ARGS__)
/*
diff -puN fs/autofs4/dev-ioctl.c~autofs4-change-log-print-macros-to-not-insert-newline fs/autofs4/dev-ioctl.c
--- a/fs/autofs4/dev-ioctl.c~autofs4-change-log-print-macros-to-not-insert-newline
+++ a/fs/autofs4/dev-ioctl.c
@@ -75,7 +75,7 @@ static int check_dev_ioctl_version(int c
if ((param->ver_major != AUTOFS_DEV_IOCTL_VERSION_MAJOR) ||
(param->ver_minor > AUTOFS_DEV_IOCTL_VERSION_MINOR)) {
AUTOFS_WARN("ioctl control interface version mismatch: "
- "kernel(%u.%u), user(%u.%u), cmd(%d)",
+ "kernel(%u.%u), user(%u.%u), cmd(%d)\n",
AUTOFS_DEV_IOCTL_VERSION_MAJOR,
AUTOFS_DEV_IOCTL_VERSION_MINOR,
param->ver_major, param->ver_minor, cmd);
@@ -130,7 +130,7 @@ static int validate_dev_ioctl(int cmd, s
err = check_dev_ioctl_version(cmd, param);
if (err) {
AUTOFS_WARN("invalid device control module version "
- "supplied for cmd(0x%08x)", cmd);
+ "supplied for cmd(0x%08x)\n", cmd);
goto out;
}
@@ -138,14 +138,14 @@ static int validate_dev_ioctl(int cmd, s
err = invalid_str(param->path, param->size - sizeof(*param));
if (err) {
AUTOFS_WARN(
- "path string terminator missing for cmd(0x%08x)",
+ "path string terminator missing for cmd(0x%08x)\n",
cmd);
goto out;
}
err = check_name(param->path);
if (err) {
- AUTOFS_WARN("invalid path supplied for cmd(0x%08x)",
+ AUTOFS_WARN("invalid path supplied for cmd(0x%08x)\n",
cmd);
goto out;
}
@@ -373,7 +373,7 @@ static int autofs_dev_ioctl_setpipefd(st
new_pid = get_task_pid(current, PIDTYPE_PGID);
if (ns_of_pid(new_pid) != ns_of_pid(sbi->oz_pgrp)) {
- AUTOFS_WARN("not allowed to change PID namespace");
+ AUTOFS_WARN("not allowed to change PID namespace\n");
err = -EINVAL;
goto out;
}
@@ -661,7 +661,7 @@ static int _autofs_dev_ioctl(unsigned in
fn = lookup_dev_ioctl(cmd);
if (!fn) {
- AUTOFS_WARN("unknown command 0x%08x", command);
+ AUTOFS_WARN("unknown command 0x%08x\n", command);
return -ENOTTY;
}
@@ -754,7 +754,7 @@ int __init autofs_dev_ioctl_init(void)
r = misc_register(&_autofs_dev_ioctl_misc);
if (r) {
- AUTOFS_ERROR("misc_register failed for control device");
+ AUTOFS_ERROR("misc_register failed for control device\n");
return r;
}
diff -puN fs/autofs4/expire.c~autofs4-change-log-print-macros-to-not-insert-newline fs/autofs4/expire.c
--- a/fs/autofs4/expire.c~autofs4-change-log-print-macros-to-not-insert-newline
+++ a/fs/autofs4/expire.c
@@ -37,7 +37,7 @@ static int autofs4_mount_busy(struct vfs
struct path path = {.mnt = mnt, .dentry = dentry};
int status = 1;
- DPRINTK("dentry %p %pd", dentry, dentry);
+ DPRINTK("dentry %p %pd\n", dentry, dentry);
path_get(&path);
@@ -63,7 +63,7 @@ static int autofs4_mount_busy(struct vfs
status = 0;
done:
- DPRINTK("returning = %d", status);
+ DPRINTK("returning = %d\n", status);
path_put(&path);
return status;
}
@@ -189,7 +189,7 @@ static int autofs4_direct_busy(struct vf
unsigned long timeout,
int do_now)
{
- DPRINTK("top %p %pd", top, top);
+ DPRINTK("top %p %pd\n", top, top);
/* If it's busy update the expiry counters */
if (!may_umount_tree(mnt)) {
@@ -220,7 +220,7 @@ static int autofs4_tree_busy(struct vfsm
struct autofs_info *top_ino = autofs4_dentry_ino(top);
struct dentry *p;
- DPRINTK("top %p %pd", top, top);
+ DPRINTK("top %p %pd\n", top, top);
/* Negative dentry - give up */
if (!simple_positive(top))
@@ -228,7 +228,7 @@ static int autofs4_tree_busy(struct vfsm
p = NULL;
while ((p = get_next_positive_dentry(p, top))) {
- DPRINTK("dentry %p %pd", p, p);
+ DPRINTK("dentry %p %pd\n", p, p);
/*
* Is someone visiting anywhere in the subtree ?
@@ -274,11 +274,11 @@ static struct dentry *autofs4_check_leav
{
struct dentry *p;
- DPRINTK("parent %p %pd", parent, parent);
+ DPRINTK("parent %p %pd\n", parent, parent);
p = NULL;
while ((p = get_next_positive_dentry(p, parent))) {
- DPRINTK("dentry %p %pd", p, p);
+ DPRINTK("dentry %p %pd\n", p, p);
if (d_mountpoint(p)) {
/* Can we umount this guy */
@@ -363,7 +363,7 @@ static struct dentry *should_expire(stru
* offset (autofs-5.0+).
*/
if (d_mountpoint(dentry)) {
- DPRINTK("checking mountpoint %p %pd", dentry, dentry);
+ DPRINTK("checking mountpoint %p %pd\n", dentry, dentry);
/* Can we umount this guy */
if (autofs4_mount_busy(mnt, dentry))
@@ -376,7 +376,7 @@ static struct dentry *should_expire(stru
}
if (d_really_is_positive(dentry) && d_is_symlink(dentry)) {
- DPRINTK("checking symlink %p %pd", dentry, dentry);
+ DPRINTK("checking symlink %p %pd\n", dentry, dentry);
/*
* A symlink can't be "busy" in the usual sense so
* just check last used for expire timeout.
@@ -473,7 +473,7 @@ struct dentry *autofs4_expire_indirect(s
return NULL;
found:
- DPRINTK("returning %p %pd", expired, expired);
+ DPRINTK("returning %p %pd\n", expired, expired);
ino->flags |= AUTOFS_INF_EXPIRING;
smp_mb();
ino->flags &= ~AUTOFS_INF_NO_RCU;
@@ -505,12 +505,12 @@ int autofs4_expire_wait(struct dentry *d
if (ino->flags & AUTOFS_INF_EXPIRING) {
spin_unlock(&sbi->fs_lock);
- DPRINTK("waiting for expire %p name=%pd", dentry, dentry);
+ DPRINTK("waiting for expire %p name=%pd\n", dentry, dentry);
status = autofs4_wait(sbi, dentry, NFY_NONE);
wait_for_completion(&ino->expire_complete);
- DPRINTK("expire done status=%d", status);
+ DPRINTK("expire done status=%d\n", status);
if (d_unhashed(dentry))
return -EAGAIN;
diff -puN fs/autofs4/inode.c~autofs4-change-log-print-macros-to-not-insert-newline fs/autofs4/inode.c
--- a/fs/autofs4/inode.c~autofs4-change-log-print-macros-to-not-insert-newline
+++ a/fs/autofs4/inode.c
@@ -60,7 +60,7 @@ void autofs4_kill_sb(struct super_block
put_pid(sbi->oz_pgrp);
}
- DPRINTK("shutting down");
+ DPRINTK("shutting down\n");
kill_litter_super(sb);
if (sbi)
kfree_rcu(sbi, rcu);
@@ -221,7 +221,7 @@ int autofs4_fill_super(struct super_bloc
sbi = kzalloc(sizeof(*sbi), GFP_KERNEL);
if (!sbi)
return -ENOMEM;
- DPRINTK("starting up, sbi = %p", sbi);
+ DPRINTK("starting up, sbi = %p\n", sbi);
s->s_fs_info = sbi;
sbi->magic = AUTOFS_SBI_MAGIC;
@@ -270,14 +270,14 @@ int autofs4_fill_super(struct super_bloc
if (parse_options(data, &pipefd, &root_inode->i_uid, &root_inode->i_gid,
&pgrp, &pgrp_set, &sbi->type, &sbi->min_proto,
&sbi->max_proto)) {
- AUTOFS_ERROR("called with bogus options");
+ AUTOFS_ERROR("called with bogus options\n");
goto fail_dput;
}
if (pgrp_set) {
sbi->oz_pgrp = find_get_pid(pgrp);
if (!sbi->oz_pgrp) {
- AUTOFS_ERROR("could not find process group %d",
+ AUTOFS_ERROR("could not find process group %d\n",
pgrp);
goto fail_dput;
}
@@ -295,7 +295,7 @@ int autofs4_fill_super(struct super_bloc
if (sbi->max_proto < AUTOFS_MIN_PROTO_VERSION ||
sbi->min_proto > AUTOFS_MAX_PROTO_VERSION) {
AUTOFS_ERROR("kernel does not match daemon version "
- "daemon (%d, %d) kernel (%d, %d)",
+ "daemon (%d, %d) kernel (%d, %d)\n",
sbi->min_proto, sbi->max_proto,
AUTOFS_MIN_PROTO_VERSION, AUTOFS_MAX_PROTO_VERSION);
goto fail_dput;
@@ -308,11 +308,11 @@ int autofs4_fill_super(struct super_bloc
sbi->version = sbi->max_proto;
sbi->sub_version = AUTOFS_PROTO_SUBVERSION;
- DPRINTK("pipe fd = %d, pgrp = %u", pipefd, pid_nr(sbi->oz_pgrp));
+ DPRINTK("pipe fd = %d, pgrp = %u\n", pipefd, pid_nr(sbi->oz_pgrp));
pipe = fget(pipefd);
if (!pipe) {
- AUTOFS_ERROR("could not open pipe file descriptor");
+ AUTOFS_ERROR("could not open pipe file descriptor\n");
goto fail_dput;
}
ret = autofs_prepare_pipe(pipe);
@@ -332,7 +332,7 @@ int autofs4_fill_super(struct super_bloc
* Failure ... clean up.
*/
fail_fput:
- AUTOFS_ERROR("pipe file descriptor does not contain proper ops");
+ AUTOFS_ERROR("pipe file descriptor does not contain proper ops\n");
fput(pipe);
/* fall through */
fail_dput:
diff -puN fs/autofs4/root.c~autofs4-change-log-print-macros-to-not-insert-newline fs/autofs4/root.c
--- a/fs/autofs4/root.c~autofs4-change-log-print-macros-to-not-insert-newline
+++ a/fs/autofs4/root.c
@@ -108,7 +108,7 @@ static int autofs4_dir_open(struct inode
struct dentry *dentry = file->f_path.dentry;
struct autofs_sb_info *sbi = autofs4_sbi(dentry->d_sb);
- DPRINTK("file=%p dentry=%p %pd", file, dentry, dentry);
+ DPRINTK("file=%p dentry=%p %pd\n", file, dentry, dentry);
if (autofs4_oz_mode(sbi))
goto out;
@@ -138,7 +138,7 @@ static void autofs4_dentry_release(struc
struct autofs_info *ino = autofs4_dentry_ino(de);
struct autofs_sb_info *sbi = autofs4_sbi(de->d_sb);
- DPRINTK("releasing %p", de);
+ DPRINTK("releasing %p\n", de);
if (!ino)
return;
@@ -278,9 +278,9 @@ static int autofs4_mount_wait(struct den
if (ino->flags & AUTOFS_INF_PENDING) {
if (rcu_walk)
return -ECHILD;
- DPRINTK("waiting for mount name=%pd", dentry);
+ DPRINTK("waiting for mount name=%pd\n", dentry);
status = autofs4_wait(sbi, dentry, NFY_MOUNT);
- DPRINTK("mount wait done status=%d", status);
+ DPRINTK("mount wait done status=%d\n", status);
}
ino->last_used = jiffies;
return status;
@@ -340,7 +340,7 @@ static struct vfsmount *autofs4_d_automo
struct autofs_info *ino = autofs4_dentry_ino(dentry);
int status;
- DPRINTK("dentry=%p %pd", dentry, dentry);
+ DPRINTK("dentry=%p %pd\n", dentry, dentry);
/* The daemon never triggers a mount. */
if (autofs4_oz_mode(sbi))
@@ -427,7 +427,7 @@ static int autofs4_d_manage(struct dentr
struct autofs_info *ino = autofs4_dentry_ino(dentry);
int status;
- DPRINTK("dentry=%p %pd", dentry, dentry);
+ DPRINTK("dentry=%p %pd\n", dentry, dentry);
/* The daemon never waits. */
if (autofs4_oz_mode(sbi)) {
@@ -504,7 +504,7 @@ static struct dentry *autofs4_lookup(str
struct autofs_info *ino;
struct dentry *active;
- DPRINTK("name = %pd", dentry);
+ DPRINTK("name = %pd\n", dentry);
/* File name too long to exist */
if (dentry->d_name.len > NAME_MAX)
@@ -512,7 +512,7 @@ static struct dentry *autofs4_lookup(str
sbi = autofs4_sbi(dir->i_sb);
- DPRINTK("pid = %u, pgrp = %u, catatonic = %d, oz_mode = %d",
+ DPRINTK("pid = %u, pgrp = %u, catatonic = %d, oz_mode = %d\n",
current->pid, task_pgrp_nr(current), sbi->catatonic,
autofs4_oz_mode(sbi));
@@ -559,7 +559,7 @@ static int autofs4_dir_symlink(struct in
size_t size = strlen(symname);
char *cp;
- DPRINTK("%s <- %pd", symname, dentry);
+ DPRINTK("%s <- %pd\n", symname, dentry);
if (!autofs4_oz_mode(sbi))
return -EACCES;
@@ -699,7 +699,7 @@ static int autofs4_dir_rmdir(struct inod
struct autofs_info *ino = autofs4_dentry_ino(dentry);
struct autofs_info *p_ino;
- DPRINTK("dentry %p, removing %pd", dentry, dentry);
+ DPRINTK("dentry %p, removing %pd\n", dentry, dentry);
if (!autofs4_oz_mode(sbi))
return -EACCES;
@@ -742,7 +742,7 @@ static int autofs4_dir_mkdir(struct inod
if (!autofs4_oz_mode(sbi))
return -EACCES;
- DPRINTK("dentry %p, creating %pd", dentry, dentry);
+ DPRINTK("dentry %p, creating %pd\n", dentry, dentry);
BUG_ON(!ino);
@@ -844,7 +844,7 @@ static inline int autofs4_ask_umount(str
if (may_umount(mnt))
status = 1;
- DPRINTK("returning %d", status);
+ DPRINTK("returning %d\n", status);
status = put_user(status, p);
@@ -872,7 +872,7 @@ static int autofs4_root_ioctl_unlocked(s
struct autofs_sb_info *sbi = autofs4_sbi(inode->i_sb);
void __user *p = (void __user *)arg;
- DPRINTK("cmd = 0x%08x, arg = 0x%08lx, sbi = %p, pgrp = %u",
+ DPRINTK("cmd = 0x%08x, arg = 0x%08lx, sbi = %p, pgrp = %u\n",
cmd, arg, sbi, task_pgrp_nr(current));
if (_IOC_TYPE(cmd) != _IOC_TYPE(AUTOFS_IOC_FIRST) ||
diff -puN fs/autofs4/waitq.c~autofs4-change-log-print-macros-to-not-insert-newline fs/autofs4/waitq.c
--- a/fs/autofs4/waitq.c~autofs4-change-log-print-macros-to-not-insert-newline
+++ a/fs/autofs4/waitq.c
@@ -31,7 +31,7 @@ void autofs4_catatonic_mode(struct autof
return;
}
- DPRINTK("entering catatonic mode");
+ DPRINTK("entering catatonic mode\n");
sbi->catatonic = 1;
wq = sbi->queues;
@@ -101,7 +101,7 @@ static void autofs4_notify_daemon(struct
struct file *pipe = NULL;
size_t pktsz;
- DPRINTK("wait id = 0x%08lx, name = %.*s, type=%d",
+ DPRINTK("wait id = 0x%08lx, name = %.*s, type=%d\n",
(unsigned long) wq->wait_queue_token,
wq->name.len, wq->name.name, type);
@@ -164,7 +164,7 @@ static void autofs4_notify_daemon(struct
break;
}
default:
- AUTOFS_WARN("bad type %d!", type);
+ AUTOFS_WARN("bad type %d!\n", type);
mutex_unlock(&sbi->wq_mutex);
return;
}
@@ -453,7 +453,7 @@ int autofs4_wait(struct autofs_sb_info *
autofs_ptype_expire_indirect;
}
- DPRINTK("new wait id = 0x%08lx, name = %.*s, nfy=%d",
+ DPRINTK("new wait id = 0x%08lx, name = %.*s, nfy=%d\n",
(unsigned long) wq->wait_queue_token, wq->name.len,
wq->name.name, notify);
@@ -463,7 +463,7 @@ int autofs4_wait(struct autofs_sb_info *
autofs4_notify_daemon(sbi, wq, type);
} else {
wq->wait_ctr++;
- DPRINTK("existing wait id = 0x%08lx, name = %.*s, nfy=%d",
+ DPRINTK("existing wait id = 0x%08lx, name = %.*s, nfy=%d\n",
(unsigned long) wq->wait_queue_token, wq->name.len,
wq->name.name, notify);
mutex_unlock(&sbi->wq_mutex);
@@ -494,7 +494,7 @@ int autofs4_wait(struct autofs_sb_info *
recalc_sigpending();
spin_unlock_irqrestore(¤t->sighand->siglock, irqflags);
} else {
- DPRINTK("skipped sleeping");
+ DPRINTK("skipped sleeping\n");
}
status = wq->status;
_
Patches currently in -mm which might be from ikent@xxxxxxxxxx are
autofs4-make-autofs-log-prints-consistent.patch
autofs4-change-log-print-macros-to-not-insert-newline.patch
autofs4-use-pr_xxx-macros-directly-for-logging.patch
--
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
