2014-11-04 01:05:06

by Anton Blanchard

Subject: [PATCH] quota: Add log level to printk

Signed-off-by: Anton Blanchard <[email protected]>
---

Index: b/fs/quota/dquot.c
===================================================================
--- a/fs/quota/dquot.c
+++ b/fs/quota/dquot.c
@@ -2743,7 +2743,7 @@ static int __init dquot_init(void)
for (i = 0; i < nr_hash; i++)
INIT_HLIST_HEAD(dquot_hash + i);

- printk("Dquot-cache hash table entries: %ld (order %ld, %ld bytes)\n",
+ pr_info("Dquot-cache hash table entries: %ld (order %ld, %ld bytes)\n",
nr_hash, order, (PAGE_SIZE << order));

register_shrinker(&dqcache_shrinker);

2014-11-04 03:06:43

by Joe Perches

Subject: Re: [PATCH] quota: Add log level to printk

On Tue, 2014-11-04 at 12:05 +1100, Anton Blanchard wrote:
> Signed-off-by: Anton Blanchard <[email protected]>
> ---
>
> Index: b/fs/quota/dquot.c
> ===================================================================
> --- a/fs/quota/dquot.c
> +++ b/fs/quota/dquot.c
> @@ -2743,7 +2743,7 @@ static int __init dquot_init(void)
> for (i = 0; i < nr_hash; i++)
> INIT_HLIST_HEAD(dquot_hash + i);
>
> - printk("Dquot-cache hash table entries: %ld (order %ld, %ld bytes)\n",
> + pr_info("Dquot-cache hash table entries: %ld (order %ld, %ld bytes)\n",
> nr_hash, order, (PAGE_SIZE << order));

The other printks in the file have a "VFS: " prefix.
Maybe this one too?

2014-11-04 12:09:06

by Jan Kara

Subject: Re: [PATCH] quota: Add log level to printk

On Mon 03-11-14 19:06:39, Joe Perches wrote:
> On Tue, 2014-11-04 at 12:05 +1100, Anton Blanchard wrote:
> > Signed-off-by: Anton Blanchard <[email protected]>
> > ---
> >
> > Index: b/fs/quota/dquot.c
> > ===================================================================
> > --- a/fs/quota/dquot.c
> > +++ b/fs/quota/dquot.c
> > @@ -2743,7 +2743,7 @@ static int __init dquot_init(void)
> > for (i = 0; i < nr_hash; i++)
> > INIT_HLIST_HEAD(dquot_hash + i);
> >
> > - printk("Dquot-cache hash table entries: %ld (order %ld, %ld bytes)\n",
> > + pr_info("Dquot-cache hash table entries: %ld (order %ld, %ld bytes)\n",
> > nr_hash, order, (PAGE_SIZE << order));
>
> The other printks in the file have a "VFS: " prefix.
> Maybe this one too?
I'll merge Anton's patch and include additional VFS:.

Honza
--
Jan Kara <[email protected]>
SUSE Labs, CR