Re: [PATCH 2/2] mm: Changed pr_warning() to pr_warn()

From: Joe Perches
Date: Sun Mar 09 2014 - 18:16:07 EST


On Sun, 2014-03-09 at 17:15 +0900, Choi Gi-yong wrote:
> Signed-off-by: Choi Gi-yong <yong@xxxxxxxx>

[]

> diff --git a/mm/percpu.c b/mm/percpu.c

> @@ -812,8 +812,8 @@ fail_unlock:
> fail_unlock_mutex:
> mutex_unlock(&pcpu_alloc_mutex);
> if (warn_limit) {
> - pr_warn("PERCPU: allocation failed, size=%zu align=%zu, "
> - "%s\n", size, align, err);
> + pr_warn("PERCPU: allocation failed, size=%zu align=%zu, %s\n",
> + size, align, err);

The second line should use 3 tabs for indentation.

pr_warn("PERCPU: allocation failed, size=%zu align=%zu, %s\n",
size, align, err);

If you want to become familiar with kernel style
and patching something, please practice on some
files in drivers/staging.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/