[lkml]   [2002]   [Sep]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] Experimental IDE oops dumper v0.1
    On Mon, Sep 16, 2002 at 08:18:07AM -0400, Alan Cox wrote:
    > > + /* Pause (at least 400ns) in case we just issued a command */
    > > + oopser_usec(1);
    > > + for (i = 0; i < 1000; i++) {
    > > + b = inb(HD_STATUS);
    > > + if (!(b & BUSY_STAT)) {
    > > + if (b & ERR_STAT) return 0;
    > > + if (b & flag) return 1;
    > > + }
    > > + oopser_usec(1000);
    > This will stop working on SATA when VDMA goes into newer controllers btw.

    I think VDMA is an extra feature, the old PIO will still remain - it has
    to, for other register accesses, and the data port is not much

    > Be careful here - one or two drives get nIEN backwards, you might just
    > want to turn off interrupts and be done with it
    > > + oopser_usec(1); /* 400ns according to spec */
    > > + outb(0x0a, HD_CMD);
    > You really need to reset and reprogram/retune the controller as well.

    No. The controller is always tuned for both PIO and DMA. This is a must
    - not all commands are DMA (eg IDENTIFY) and for for example ATAPI some
    data is sent over PIO and some over DMA even for a single command. So
    using PIO when the drive is switched to DMA is perfectly OK.

    > I like the infrastructure but the IDE dumper code is wishful thinking
    > in one or two spots. You don't know if the controller is in DMA modes,
    > tuned for different things to the drives or legacy free.

    Legacy free? That is wishful thinking. Not going to happen any soon.
    Unfortunately. As for DMA vs PIO, see above.

    > Im not sure what
    > to do for legacy free cases but the other bits like LBA48 and retuning
    > probably can be handled with some small chipset specific hooks

    Retuning not needed, LBA48 might be needed. Not sure about LBA48 in PIO
    mode, it might work even without chipset support - it's usually the
    LBA48+DMA combination that confuses the chips.

    Vojtech Pavlik
    SuSE Labs
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2005-03-22 13:28    [W:0.023 / U:6.216 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site