Re: [PATCH] Add source address to sunrpc svc errors

From: Valdis . Kletnieks
Date: Tue Aug 28 2007 - 15:28:08 EST


On Mon, 27 Aug 2007 17:43:33 EDT, "J. Bruce Fields" said:

> Looks like a reasonable idea to me, thanks! Any objection to just
> calling it "svc_printk" instead of "svc_printkerr"?
>
> I also wonder whether these shouldn't all be dprintk's instead of
> printk's. One misbehaving client could create a lot of noise in the
> logs.

I shouldn't have to rebuild my kernel with debugging enabled just to see
who is throwing trash at my machine. printk(KERN_INFO maybe and/or using
a printk_ratelimit.

Attachment: pgp00000.pgp
Description: PGP signature