[lkml]   [2002]   [Apr]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] IDE TCQ #4
    Jens Axboe wrote:
    > On Tue, Apr 16 2002, Martin Dalecki wrote:
    >>Jens Axboe wrote:
    >>>yes this looks like a silly problem. the fix should be to have
    >>>ata_ar_get() set ATA_AR_RETURN in ar_flags:
    >>> if (!list_empty(&drive->free_req)) {
    >>> ar = list_ata_entry(drive->;
    >>> list_del(&ar->ar_queue);
    >>> ata_ar_init(drive, ar);
    >>> ar->ar_flags |= ATA_AR_RETURN;
    >>> }
    >>>and then only have ata_ar_put() readd it to the list when it is set:
    >>>static inline void ata_ar_put(ide_drive_t *drive, struct ata_request
    >>> if (ar->ar_flags & ATA_AR_RETURN)
    >>> list_add(&ar->ar_queue, &drive->free_req);
    >>> ...
    >>>Then you can also remove the ata_ar_put() conditional in
    >>>ide_end_drive_cmd(), just call ata_ar_put() unconditionally.
    >>Well something similar is already in IDE 37... I have just
    >>invented a flag ATA_AR_STATIC which get's set in ide_raw_taskfile
    >>ata_ar_put ich then checking for if (!(ar->ar_flags & ATA_AR_STATIC))...
    >>It has the desired effect in practice.
    > sure, just used ATA_AR_RETURN since it was there already. I'm not
    > particularly fond of that name though, and ATA_AR_STATIC isn't too good
    > either imo. how about ATA_AR_POOL? with the same semantics as
    > ATA_AR_RETURN, ie return to pool if flag is set.

    ATA_AR_POOL sounds good. It indicates where it's comming from and
    where it is going to remain. BTW.> Have you noticed the tactile steps
    forward I did in conversion of ide-cd.c to the new command submission in IDE 37?
    Any suggestions? It isn't really that abvious how to get finally rid
    of struct packt_command... but I think that I'm not far away now.
    The only thing that has to be done is to move the sense data and failed
    command used in ide-cd away from the packet command structure.
    I will not pass it as pointer around but just copy it directly to struct
    ata_devices driver_data field... I think.

    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:25    [W:0.024 / U:14.480 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site