Re: [PATCH] deprecate use of bdflush()

From: Bill Davidsen (davidsen@tmr.com)
Date: Tue Dec 03 2002 - 09:24:09 EST


On 2 Dec 2002, Robert Love wrote:

> On Mon, 2002-12-02 at 17:11, Andrew Morton wrote:
>
> > Ho-hum. I was going to do this months ago but general exhaustion
> > and sluggishness won out.
> >
> > We should tell the user which process called sys_bdflush() to aid
> > their expunging efforts.
>
> Good idea.
>
> I could do without the rate limiting, though - the print is after the
> CAP_SYS_ADMIN check. Root has plenty of other ways to print crap to the
> screen and it saves 32-bits from bss. But, uh, not a big deal at all
> either way.

My take on this is that it's premature. This would be fine in the 2.6.0-rc
series, but the truth is that the majority of 2.5 users boot 2.5 for
testing but run 2.4 for normal use. They aren't going to get rid of
bdflush and this just craps up the logs. At least with the occurrence
limit it will only happen a few times. I would like to see it once only,
myself, as a reminder rather than a nag.

While it's possible to do version dependent things in init scripts, in
this case there is no need, the call hurts nothing. Some users may post to
the list asking what it means, and others may actually do it, and break
their production systems.

-- 
bill davidsen <davidsen@tmr.com>
  CTO, TMR Associates, Inc
Doing interesting things with little computers since 1979.

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sat Dec 07 2002 - 22:00:16 EST