lkml.org 
[lkml]   [2006]   [Dec]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [GIT PATCH] more Driver core patches for 2.6.19
    On 12/13/06, Greg KH <gregkh@suse.de> wrote:
    > On Wed, Dec 13, 2006 at 12:38:05PM -0800, Michael K. Edwards wrote:
    > > Seriously, though, please please pretty please do not allow a facility
    > > for "going through a simple interface to get accesses to irqs and
    > > memory regions" into the mainline kernel, with or without toy ISA
    > > examples.
    >
    > Why? X does it today :)

    Er, I rest my case? Anyway, the history around X11 is completely
    different from the present situation, and there were good arguments at
    the time for structuring the problem so that the X server was
    relatively kernel-neutral even if it was banging directly on a
    framebuffer, and later on 2-D acceleration hardware. As graphics
    chips have become more sophisticated, pure-userspace X11 has become
    less tenable.

    > > Embedded systems integrators have enough trouble with chip vendors who
    > > think that exposing the device registers to userspace constitutes a
    > > "driver".
    >
    > Yes, and because of this, they create binary only drivers today. Lots
    > of them. All over the place. Doing crazy stupid crap in kernelspace.

    It does not get less crazy and stupid because we open a big hole from
    kernelspace to userspace and let them pretend that they have a GPL
    "driver" when all of the chip init logic is peeked and poked from the
    same closed code in userspace. Most low-level drivers (the kind that
    involve IRQs and registers local to the CPU; USB is different) cannot
    be done right from userspace and we shouldn't encourage people to try.

    > Then there are people who do irq stuff in userspace but get it wrong.
    > I've seen that happen many times in lots of different research papers
    > and presentations.

    Their problems need not become our problems.

    > This interface does it correctly, and it allows those people who for
    > some reason feel they do want to keep their logic in non-gpl code, to do
    > it.

    People who want to keep their logic in non-GPL code do so by providing
    binary-only drivers. That's a sane compromise in certain sectors, and
    occasionally results in the eventual opening of the driver (when the
    illusion of competitive advantage in closed-ness wears off) in
    integrable shape. A customer with some leverage and technical skill
    can negotiate for access to the source code so that he can fix the
    bugs that are biting him, rebuild with a toolchain that isn't from the
    Dark Ages, and so forth: a real-life demonstration to the vendor that
    letting outsiders work on the code will sell more chips. But if you
    actively encourage a brain-damaged userspace driver strategy, that
    opportunity is lost.

    > It also allows code that needs floating point to not be in the kernel
    > and in one instance using this interface actually sped up the device
    > because of the lack of the need to go between kernel and userspace a
    > bunch of times.

    What instance is that? Are you sure it wasn't a case of things being
    done in the driver that should have been done in a library all along?

    > > The correct description is more like "porting shim for MMU-less RTOS
    > > tasks"; and if the BSP vendors of the world can make a nickel
    > > supplying them, more power to them. Just not in mainline, please.
    >
    > Again, X does this today, and does does lots of other applications.
    > This is a way to do it in a sane manner, to keep people who want to do
    > floating point out of the kernel, and to make some embedded people much
    > happier to use Linux, gets them from being so mad at Linux because we
    > keep changing the internal apis, and makes me happier as they stop
    > violating my copyright by creating closed drivers in the kernel.

    Today's problem is that too many chip suppliers' in-house software
    people have neither the skills nor the schedule room nor the influence
    to insist that drivers be written competently and maintainably,
    whether or not they are maintained in the public view. So chipmakers
    turn to the BSP vendors, whose business model is built around being
    the only people who can do incremental development on the code they
    write.

    I'm not at all hostile to BSP vendors or to chipmakers who wind up in
    this position; that's the way most of the industry works nowadays.
    But I do want to know when I'm dealing with that situation, because
    the appropriate strategies for getting a product to market are
    different when it costs the chip vendor real cash money each time they
    commission a recompile to meet some customer's expectations.
    Pretending that driver code doesn't need reviewing every few months as
    the kernel's locking strategies, timer handling, internal APIs, etc.
    evolve makes things worse.

    Just like the only thing worse that a salesperson who's on commission
    is a salesperson who isn't on commission, the only thing worse than a
    closed device driver in kernelspace is a closed device driver in
    userspace.

    Cheers,
    - Michael
    -
    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/

    \
     
     \ /
      Last update: 2006-12-13 22:51    [W:4.022 / U:0.136 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site