[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date: Mon, 10 Oct 2011 19:40:58 +0200
From: Jan Kara <jack@...e.cz>
To: Joe Perches <joe@...ches.com>
Cc: Jan Kara <jack@...e.cz>, linux-kernel@...r.kernel.org,
NamJae Jeon <linkinjeon@...il.com>
Subject: Re: [PATCH 4/6] udf: Convert printks to pr_<level>
On Mon 10-10-11 10:25:10, Joe Perches wrote:
> On Mon, 2011-10-10 at 18:41 +0200, Jan Kara wrote:
> > On Mon 10-10-11 08:23:34, Joe Perches wrote:
> > > I forgot to mention that udf_err emits at KERN_CRIT.
> > > I think that's not good.
> > I've changed udf_err() to call pr_err() because
> > KERN_CRIT is really too much.
>
> If this means you applied it or applied it with
> modifications, I think it'd be good to either
> reply to the 0/n message with something like
> "applied to my tree", or reply to each message
> as appropriate.
Sure, that's what I wanted to do but I first wanted to test what I have,
stumbled across an issue with patch 1 which delayed me a bit...
> That reply lets other people like NamJae proceed
> appropriately. Otherwise, it's left to some
> pull request or patch series posted by you
> potentially months later to figure out what's
> current.
>
> Is there a udf git tree?
On kernel.org which is down... I didn't bother to setup some other place
yet.
> The stuff on sourceforge is not current
> and the MAINTAINERS W: entry doesn't work.
Actually, the link is ancient and I don't use SF for UDF development so
I've added a patch to my tree which removes the link.
>
> Maybe:
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 644894a..58eb5f1 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -6624,7 +6624,7 @@ F: drivers/net/ethernet/8390/ne-h8300.c
>
> UDF FILESYSTEM
> M: Jan Kara <jack@...e.cz>
> -W: http://linux-udf.sourceforge.net
> +W: http://sourceforge.net/projects/linux-udf/develop
> S: Maintained
> F: Documentation/filesystems/udf.txt
> F: fs/udf/
>
> cheers, Joe
>
>
--
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists