Re: [2.4] scsi per-host lock was Re: smp dead lock of io_request_lock/queue_lock patch

From: Doug Ledford (dledford_at_redhat.com)
Date: 01/20/04

  • Next message: Andrew Morton: "Re: page allocation failure"
    To: Marcelo Tosatti <marcelo.tosatti@cyclades.com>
    Date:	Mon, 19 Jan 2004 22:21:02 -0500
    
    

    On Mon, 2004-01-19 at 21:19, Marcelo Tosatti wrote:
    > Doug,
    >
    > Where can I find a copy of uptodate iorl and mlqueue patchsets ?

    Red Hat's 2.4.21-9.EL kernel? ;-)

    Seriously, I've already set up 3 new bk trees on linux-scsi.bkbits.net.
    They are 2.4-for-marcelo, 2.4-drivers, and 2.4-everything. Right now
    the trees are still just clones of your current tree. The plan,
    however, is to sort through all of the scsi patches we put into our
    kernels and put them into the various trees. Obviously, the everything
    tree will have both the iorl and mlqueue patches plus the other stuff.
    The for-marcelo tree is stuff that is agreed to be appropriate for
    mainline. The drivers tree is where I'll start sticking driver updates
    that come across linux-scsi for the 2.4 kernel.

    > I cant enjoy the feeling of splitting a global lock at this point in 2.4's
    > lifetime, but it has been in RedHat's and SuSE's trees for long.
    >
    > A reason to have it its the dependacy of mlqueue patch. You said most
    > drivers are handling the BUSY/QUEUE_FULL hangs by themselves, you mention
    > "IBM zfcp" / Emulex which dont. What other drivers suffer from the same
    > problem ?

    Well, that's a hard to answer question. First, the problem only shows
    up on drivers that actually enabled tagged queuing and use a reasonably
    deep queue depth (normally). So, that automatically strikes out most of
    the older ISA and crap hardware drivers since they only do a command at
    a time. The better drivers; aic7xxx (new and old), ncr53c8xx, sym1 and
    sym2, qla2x00, etc.; work around the problem. It's mainly newer drivers
    that vendors are starting to write and who don't know the history of the
    broken mid layer handling of QUEUE_FULL and BUSY situations. So, I
    cited the two I know of off the top of my head, and there may be others
    but I would actually have to look at the source code to identify them
    (when 98% of the scsi market is QLogic and NCR/SYM and aic7xxx, those
    other drivers fall through the cracks). Of course, one of the other
    issues is that each driver that tries to handle this situation itself
    does it in a different manner, and the correctness of those varying
    implementations is also an issue to consider.

    -- 
      Doug Ledford <dledford@redhat.com>     919-754-3700 x44233
             Red Hat, Inc.
             1801 Varsity Dr.
             Raleigh, NC 27606
    -
    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/
    

  • Next message: Andrew Morton: "Re: page allocation failure"

    Relevant Pages

    • Re: Bad design contributes to bizarre death.
      ... If a driver going 60 mph in a 20 mph zone hits a concrete ... barricade on the side of the road and is killed do we blame the ... so these same drivers can "safely" exceed the speed limit by 20-40 mph. ... trees on the rights of way to make sure no one is injured by those trees! ...
      (rec.bicycles.misc)
    • Re: linux-next: the purge :-)
      ... empty) of the ldp tree for drivers that are readier for integration. ... I've now moved my trees around a bit, and would like it if you could ... next major kernel is out. ... There had been need of rain for many days. ...
      (Linux-Kernel)
    • Re: [patch 2.6.21-rc5-git] make /proc/acpi/wakeup more useful
      ... The tools will likely need to understand how ACPI ... Though it may be that "parallel trees" should go away. ... would be the case with a more normal driver structure. ... Converting such legacy drivers is ...
      (Linux-Kernel)
    • Re: linux-next: the purge :-)
      ... empty) of the ldp tree for drivers that are readier for integration. ... I've now moved my trees around a bit, and would like it if you could ...
      (Linux-Kernel)