Messages in this thread | | | Date | Fri, 30 Mar 2012 14:17:44 -0700 | From | Greg KH <> | Subject | Re: [PATCH] kmsg: Use vmalloc instead of kmalloc when writing |
| |
On Fri, Mar 30, 2012 at 09:05:52PM +0000, Arnd Bergmann wrote: > On Friday 30 March 2012, Thomas Gleixner wrote: > > I think so. This is an interface to inject stuff into dmesg. Limiting > > that to a reasonable size makes sense. We can probably limit it to > > something small like 1024, but I don't know about the "ideas" of those > > folks who think that it's a great idea to do it at all. > > I guess a page would be a reasonable size, similar to what we do for > sysfs.
Ok. Sasha, as you seem to have noticed this, care to dig in syslog and systemd to get an idea of the buffer sizes they are expecting to pass into kmsg, and if they can handle a short write properly? If so, restricting it to a page is fine with me, otherwise we might want to make it a bit bigger.
thanks,
greg k-h
| |