Re: [PATCH 4/6] udf: Convert printks to pr_<level>

From: Jan Kara
Date: Mon Oct 10 2011 - 13:41:02 EST


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@xxxxxxx>
> -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@xxxxxxx>
SUSE Labs, CR
--
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/