1. 15 5月, 2006 35 次提交
    • T
      [PATCH] libata-eh: add ATA and libata flags for new EH · 9be1e979
      Tejun Heo 提交于
      Add ATA and libata flags to be used by new EH.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      9be1e979
    • T
      [PATCH] libata-eh-fw: update SCSI command completion path for new EH · 246619da
      Tejun Heo 提交于
      SCSI command completion path used to do some part of EH including
      printing messages and obtaining sense data.  With new EH, all these
      are responsibilities of the EH, update SCSI command completion path to
      reflect this.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      246619da
    • T
      [PATCH] libata-eh-fw: update ata_exec_internal() for new EH · d95a717f
      Tejun Heo 提交于
      Update ata_exec_internal() such that it uses new EH framework.
      ->post_internal_cmd() is always invoked regardless of completion
      status.  Also, when ata_exec_internal() detects a timeout condition
      and new EH is in place, it freezes the port as timeout for normal
      commands would do.
      
      Note that ata_port_flush_task() is called regardless of
      wait_for_completion status.  This is necessary as exceptions unrelated
      to the qc can abort the qc, in which case PIO task could still be
      running after the wait for completion returns.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      d95a717f
    • T
      [PATCH] libata-eh-fw: update ata_scsi_error() for new EH · ad9e2762
      Tejun Heo 提交于
      Update ata_scsi_error() for new EH.  ata_scsi_error() is responsible
      for claiming timed out qcs and invoking ->error_handler in safe and
      synchronized manner.  As the state of the controller is unknown if a
      qc has timed out, the port is frozen in such cases.
      
      Note that ata_scsi_timed_out() isn't used for new EH.  This is because
      a timed out qc cannot be claimed by EH without freezing the port and
      freezing the port in ata_scsi_timed_out() results in unnecessary
      abortion of other active qcs.  ata_scsi_timed_out() can be removed
      once all drivers are converted to new EH.
      
      While at it, add 'TODO: kill' comments to old EH functions.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      ad9e2762
    • T
      [PATCH] libata-eh-fw: implement new EH scheduling from PIO · dafadcde
      Tejun Heo 提交于
      PIO executes without holding host_set lock, so it cannot be
      synchronized using the same mechanism as interrupt driven execution.
      port_task framework makes sure that EH is not entered until PIO task
      is flushed, so PIO task can be sure the qc in progress won't go away
      underneath it.  One thing it cannot be sure of is whether the qc has
      already been scheduled for EH by another exception condition while
      host_set lock was released.
      
      This patch makes ata_poll_qc-complete() handle such conditions
      properly and make it freeze the port if HSM violation is detected
      during PIO execution.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      dafadcde
    • T
      [PATCH] libata-eh-fw: implement freeze/thaw · e3180499
      Tejun Heo 提交于
      Freezing is performed atomic w.r.t. host_set->lock and once frozen
      LLDD is not allowed to access the port or any qc on it.  Also, libata
      makes sure that no new qc gets issued to a frozen port.
      
      A frozen port is thawed after a reset operation completes
      successfully, so reset methods must do its job while the port is
      frozen.  During initialization all ports get frozen before requesting
      IRQ, so reset methods are always invoked on a frozen port.
      
      Optional ->freeze and ->thaw operations notify LLDD that the port is
      being frozen and thawed, respectively.  LLDD can disable/enable
      hardware interrupt in these callbacks if the controller's IRQ mask can
      be changed dynamically.  If the controller doesn't allow such
      operation, LLDD can check for frozen state in the interrupt handler
      and ack/clear interrupts unconditionally while frozen.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      e3180499
    • T
      [PATCH] libata-eh-fw: implement ata_port_schedule_eh() and ata_port_abort() · 7b70fc03
      Tejun Heo 提交于
      ata_port_schedule_eh() directly schedules EH for @ap without
      associated qc.  Once EH scheduled, no further qc is allowed and EH
      kicks in as soon as all currently active qc's are drained.
      
      ata_port_abort() schedules all currently active commands for EH by
      qc_completing them with ATA_QCFLAG_FAILED set.  If ata_port_abort()
      doesn't find any qc to abort, it directly schedule EH using
      ata_port_schedule_eh().
      
      These two functions provide ways to invoke EH for conditions which
      aren't directly related to any specfic qc.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      7b70fc03
    • T
      [PATCH] libata-eh-fw: implement new EH scheduling via error completion · f686bcb8
      Tejun Heo 提交于
      There are several ways a qc can get schedule for EH in new EH.  This
      patch implements one of them - completing a qc with ATA_QCFLAG_FAILED
      set or with non-zero qc->err_mask.  ALL such qc's are examined by EH.
      
      New EH schedules a qc for EH from completion iff ->error_handler is
      implemented, qc is marked as failed or qc->err_mask is non-zero and
      the command is not an internal command (internal cmd is handled via
      ->post_internal_cmd).  The EH scheduling itself is performed by asking
      SCSI midlayer to schedule EH for the specified scmd.
      
      For drivers implementing old-EH, nothing changes.  As this change
      makes ata_qc_complete() rather large, it's not inlined anymore and
      __ata_qc_complete() is exported to other parts of libata for later
      use.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      f686bcb8
    • T
      [PATCH] libata-eh-fw: update ata_qc_from_tag() to enforce normal/EH qc ownership · f69499f4
      Tejun Heo 提交于
      New EH framework has clear distinction about who owns a qc.  Every qc
      starts owned by normal execution path - PIO, interrupt or whatever.
      When an exception condition occurs which affects the qc, the qc gets
      scheduled for EH.  Note that some events (say, link lost and regained,
      command timeout) may schedule qc's which are not directly related but
      could have been affected for EH too.  Scheduling for EH is atomic
      w.r.t. ap->host_set->lock and once schedule for EH, normal execution
      path is not allowed to access the qc in whatever way.  (PIO
      synchronization acts a bit different and will be dealt with later)
      
      This patch make ata_qc_from_tag() check whether a qc is active and
      owned by normal path before returning it.  If conditions don't match,
      NULL is returned and thus access to the qc is denied.
      __ata_qc_from_tag() is the original ata_qc_from_tag() and is used by
      libata core/EH layers to access inactive/failed qc's.
      
      This change is applied only if the associated LLDD implements new EH
      as indicated by non-NULL ->error_handler
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      f69499f4
    • T
      [PATCH] libata-eh-fw: use special reserved tag and qc for internal commands · 2ab7db1f
      Tejun Heo 提交于
      New EH may issue internal commands to recover from error while failed
      qc's are still hanging around.  To allow such usage, reserve tag
      ATA_MAX_QUEUE-1 for internal command.  This also makes it easy to tell
      whether a qc is for internal command or not.  ata_tag_internal() test
      implements this test.
      
      To avoid breaking existing drivers, ata_exec_internal() uses
      ATA_TAG_INTERNAL only for drivers which implement ->error_handler.
      For drivers using old EH, tag 0 is used.  Note that this makes
      ata_tag_internal() test valid only when ->error_handler is
      implemented.  This is okay as drivers on old EH should not and does
      not have any reason to use ata_tag_internal().
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      2ab7db1f
    • T
      [PATCH] libata-eh-fw: clear SError in ata_std_postreset() · dc2b3515
      Tejun Heo 提交于
      Clear SError in ata_std_postreset().  This is to clear SError bits
      which get set during reset.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      dc2b3515
    • T
      [PATCH] libata-eh-fw: add flags and operations for new EH · 9ec957f2
      Tejun Heo 提交于
      Add ATA_FLAG_EH_{PENDING|FROZEN}, ATA_ATA_QCFLAG_{FAILED|SENSE_VALID}
      and ops->freeze, thaw, error_handler, post_internal_cmd() for new EH.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      9ec957f2
    • T
      [PATCH] libata: use ATA printk helpers · f15a1daf
      Tejun Heo 提交于
      Use ATA printk helpers.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      f15a1daf
    • T
      [PATCH] libata: implement ATA printk helpers · 61440db6
      Tejun Heo 提交于
      Implement ata_{port|dev}_printk() which prefixes the message with
      proper identification string.  This change is necessary for later PM
      support because devices and links should be identified differently
      depending on how they are attached.
      
      This also helps unifying device id strings.  Currently, there are two
      forms in use (P is the port number D device number) - 'ataP(D):', and
      'ataP: dev D '.  These macros also make it harder to forget proper ID
      string (e.g. printing only port number when a device is in question).
      
      Debug message handling can be integrated into these printk macros by
      passing debug type and level via @lv.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      61440db6
    • T
      [PATCH] libata: use dev->ap · 3373efd8
      Tejun Heo 提交于
      Use dev->ap where possible and eliminate superflous @ap from functions
      and structures.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      3373efd8
    • T
      [PATCH] libata: add dev->ap · 38d87234
      Tejun Heo 提交于
      Add dev->ap which points back to the port the device belongs to.  This
      makes it unnecessary to pass @ap for silly reasons (e.g. printks).
      Also, this change is necessary to accomodate later PM support which
      will introduce ATA link inbetween port and device.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      38d87234
    • T
      [PATCH] libata: kill old SCR functions and sata_dev_present() · a0ab51ce
      Tejun Heo 提交于
      Kill now unused scr_{read|write|write_flush}() and sata_dev_present().
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      a0ab51ce
    • T
      [PATCH] libata: use new SCR and on/offline functions · 81952c54
      Tejun Heo 提交于
      Use new SCR and on/offline functions.  Note that for LLDD which know
      it implements SCR callbacks, SCR functions are guaranteed to succeed
      and ata_port_online() == !ata_port_offline().
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      81952c54
    • T
      [PATCH] libata: implement new SCR handling and port on/offline functions · 34bf2170
      Tejun Heo 提交于
      Implement ata_scr_{valid|read|write|write_flush}() and
      ata_port_{online|offline}().  These functions replace
      scr_{read|write}() and sata_dev_present().
      
      Major difference between between the new SCR functions and the old
      ones is that the new ones have a way to signal error to the caller.
      This makes handling SCR-available and SCR-unavailable cases in the
      same path easier.  Also, it eases later PM implementation where SCR
      access can fail due to various reasons.
      
      ata_port_{online|offline}() functions return 1 only when they are
      affirmitive of the condition.  e.g.  if SCR is unaccessible or
      presence cannot be determined for other reasons, these functions
      return 0.  So, ata_port_online() != !ata_port_offline().  This
      distinction is useful in many exception handling cases.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      34bf2170
    • T
      [PATCH] libata: init ap->cbl to ATA_CBL_SATA early · 838df628
      Tejun Heo 提交于
      Init ap->cbl to ATA_CBL_SATA in ata_host_init().  This is necessary
      for soon-to-follow SCR handling function changes.  LLDDs are free to
      change ap->cbl during probing.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      838df628
    • T
      [PATCH] sata_sil24: update TF image only when necessary · ce5f7f3d
      Tejun Heo 提交于
      Update TF image (pp->tf) only when necessary.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      ce5f7f3d
    • T
      [PATCH] libata: implement qc->result_tf · e61e0672
      Tejun Heo 提交于
      Add qc->result_tf and ATA_QCFLAG_RESULT_TF.  This moves the
      responsibility of loading result TF from post-compltion path to qc
      execution path.  qc->result_tf is loaded if explicitly requested or
      the qc failsa.  This allows more efficient completion implementation
      and correct handling of result TF for controllers which don't have
      global TF representation such as sil3124/32.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      e61e0672
    • T
      [PATCH] libata: remove postreset handling from ata_do_reset() · 96bd39ec
      Tejun Heo 提交于
      Make ata_do_reset() deal only with reset.  postreset is now the
      responsibility of the caller.  This is simpler and eases later
      prereset addition.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      96bd39ec
    • T
      [PATCH] libata: move ->set_mode() handling into ata_set_mode() · 3adcebb2
      Tejun Heo 提交于
      Move ->set_mode() handlng into ata_set_mode().
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      3adcebb2
    • T
      [PATCH] libata: use preallocated buffers · fe635c7e
      Tejun Heo 提交于
      It's not a very good idea to allocate memory during EH.  Use
      statically allocated buffer for dev->id[] and add 512byte buffer
      ap->sector_buf.  This buffer is owned by EH (or probing) and to be
      used as temporary buffer for various purposes (IDENTIFY, NCQ log page
      10h, PM GSCR block).
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      fe635c7e
    • T
      [PATCH] libata: hold host_set lock while finishing internal qc · 15869303
      Tejun Heo 提交于
      Hold host_set lock while finishing internal qc.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      15869303
    • T
      [PATCH] libata: clear ap->active_tag atomically w.r.t. command completion · 7401abf2
      Tejun Heo 提交于
      ap->active_tag was cleared in ata_qc_free().  This left ap->active_tag
      dangling after ata_qc_complete().  Spurious interrupts inbetween could
      incorrectly access the qc.  Clear active_tag in ata_qc_complete().
      This change is necessary for later EH changes.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      7401abf2
    • T
      [PATCH] libata: fix ->phy_reset class code handling in ata_bus_probe() · f8c2c420
      Tejun Heo 提交于
      ata_bus_probe() doesn't clear dev->class after ->phy_reset().  This
      can result in falsely enabled devices if probing fails.  Clear
      dev->class to ATA_DEV_UNKNOWN after fetching it.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      f8c2c420
    • T
      [PATCH] libata: kill duplicate prototypes · 6cd727b1
      Tejun Heo 提交于
      Kill duplicate prototypes for ata_eh_qc_complete/retry() in libata.h.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      6cd727b1
    • T
      [PATCH] libata: unexport ata_scsi_error() · e23befe9
      Tejun Heo 提交于
      While moving ata_scsi_error() from LLDD sht to libata transportt,
      EXPORT_SYMBOL_GPL() entry was left out.  Kill it.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      e23befe9
    • T
      [PATCH] ahci: hardreset classification fix · e4fac92a
      Tejun Heo 提交于
      AHCI calls ata_dev_classify() even when no device is attached which
      results in false class code.  Fix it.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      e4fac92a
    • T
      [PATCH] libata: rename ata_down_sata_spd_limit() and friends · 3c567b7d
      Tejun Heo 提交于
      Rename ata_down_sata_spd_limit() and friends to sata_down_spd_limit()
      and likewise for simplicity & consistency.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      3c567b7d
    • T
      [PATCH] libata: silly fix in ata_scsi_start_stop_xlat() · c44078c0
      Tejun Heo 提交于
      Don't directly access &qc->tf when tf == &qc->tf.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      c44078c0
    • T
      [PATCH] SCSI: implement shost->host_eh_scheduled · ee7863bc
      Tejun Heo 提交于
      libata needs to invoke EH without scmd.  This patch adds
      shost->host_eh_scheduled to implement such behavior.
      
      Currently the only user of this feature is libata and no general
      interface is defined.  This patch simply adds handling for
      host_eh_scheduled where needed and exports scsi_eh_wakeup() to
      modules.  The rest is upto libata.  This is the result of the
      following discussion.
      
      http://thread.gmane.org/gmane.linux.scsi/23853/focus=9760
      
      In short, SCSI host is not supposed to know about exceptions unrelated
      to specific device or command.  Such exceptions should be handled by
      transport layer proper.  However, the distinction is not essential to
      ATA and libata is planning to depart from SCSI, so, for the time
      being, libata will be using SCSI EH to handle such exceptions.
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      ee7863bc
    • L
      [PATCH] SCSI: Introduce scsi_req_abort_cmd (REPOST) · 89f48c4d
      Luben Tuikov 提交于
      Introduce scsi_req_abort_cmd(struct scsi_cmnd *).
      This function requests that SCSI Core start recovery for the
      command by deleting the timer and adding the command to the eh
      queue.  It can be called by either LLDDs or SCSI Core.  LLDDs who
      implement their own error recovery MAY ignore the timeout event if
      they generated scsi_req_abort_cmd.
      
      First post:
      http://marc.theaimsgroup.com/?l=linux-scsi&m=113833937421677&w=2Signed-off-by: NLuben Tuikov <ltuikov@yahoo.com>
      Signed-off-by: NTejun Heo <htejun@gmail.com>
      89f48c4d
  2. 27 4月, 2006 5 次提交