lkml.org 
[lkml]   [2001]   [Aug]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] make psaux reconnect adjustable
    Hi Linus,

    thanks for your comments.

    On Tue, Aug 14, 2001 at 09:58:55AM -0700, Linus Torvalds wrote:
    > I really have two comments, but I haven't followed the whole discussion,
    > so feel free to just say that it's been hashed out already:
    >
    > - sysconf entries are suspicious for stuff like this. If some code really
    > requires this to work correctly, that's exactly the kind of code that
    > would run automatically at bootup. A sysconf doesn't really help people
    > in that case - we'd be much better off with just a bootup switch.

    Maybe that's the difference of whether you define the kernel behaviour by
    deciding what goes in and the one that tries to avoid a breakage without
    changing the default behaviour ...
    Of course the sysctl is less intrusive (from a user's point of view).

    > - do we actually need the config switch AT ALL, whether at bootup or not?
    > What exactly breaks if we just always pass the AA 00 values through?
    > Apparently nothing ever breaks, which makes me suspect that people are
    > just being unnecessarily defensive.

    PS2 mouses need this ping thing to be operational after being plugged.
    But, there's no reason it needs to be done in the kernel.
    * It works for plain PS2 only (not: imps2, synps2, ...)
    * The userspace driver (gpm, X11) can do it as well, AFAICS

    I guess the thing has been introduced because it was more convenient than
    fixing userspace.
    Maybe the kernel does it a bit more efficient by throwing away the queue ...

    > In short, I'd prefer a patch that just unconditionally removes the code,
    > unless somebody KNOWS that it could break something. That failing, a
    > simple kernel command line option sounds better than more files in /proc.

    I'd be happy with removing it.
    Patch will follow!

    > Remember: the biggest mistake to do is to overdesign. The road to hell is
    > paved with good intentions.

    A few sysctls don't qualify as overdesign yet, I hope.

    Regards,
    --
    Kurt Garloff <garloff@suse.de> Eindhoven, NL
    GPG key: See mail header, key servers Linux kernel development
    SuSE GmbH, Nuernberg, DE SCSI, Security
    [unhandled content-type:application/pgp-signature]
    \
     
     \ /
      Last update: 2005-03-22 12:57    [W:2.525 / U:0.712 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site