linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCHSET #upstream] libata: separate out SFF and BMDMA
@ 2008-10-21  9:17 Tejun Heo
  2008-10-21  9:17 ` [PATCH 01/22] pata_sch: use ata_pci_sff_init_one() Tejun Heo
                   ` (27 more replies)
  0 siblings, 28 replies; 44+ messages in thread
From: Tejun Heo @ 2008-10-21  9:17 UTC (permalink / raw)
  To: jeff, linux-ide, linux-scsi


Hello,

This is the first take of separate-out-SFF-and-BMDMA patchset.

Historically, ATA meant SFF (or TASKFILE interface) plus BMDMA and
that holds true for many modern SATA controllers.  This was reflected
in libata implementation and libata core layer and SFF/BMDMA support
used to be all tangled in one piece.

For SFF/BMDMA drivers, this assumption is fine but for drivers which
have native interfaces (e.g. ahci, sata_sil24), this meant that those
drivers had to emulate SFF/BMDMA to ceratin not-so-well defined level
libata core layer expects.  A lot of this vague dependency on
SFF/BMDMA was removed during recent SFF/BMDMA separation but there
still are SFF/BMDMA specific assumptions in libata core layer.

Another issue which surpassed with the addition of all the legacy,
platform and somewhat peculiar PATA drivers is that now there are
enough drivers which use SFF interface but use its own DMA interface
or don't support DMA at all.  This, in similar fashion, led those
drivers to fake certain level of BMDMA functionality to please the
libata SFF/BMDMA layer.  This led to the following interesting
solutions in low level drivers.

1. For BMDMA drivers which don't use traditional BMDMA register,
   ata_bmdma_mode_filter() incorrectly inhibits DMA modes.  Those
   drivers either have to inherit from ata_sff_port_ops or clear
   ->mode_filter explicitly.

2. non-BMDMA drivers call into BMDMA PRD table allocation.  It doesn't
   actually allocate PRD table if bmdma_addr is not initialized but is
   still confusing.

3. For BMDMA drivers which don't use traditional BMDMA register, some
   methods might not be invoked as expected (e.g. bmdma_stop from
   ata_sff_post_internal_cmd()).

4. SFF drivers w/ custom DMA interface implement noop BMDMA ops
   worrying libata core might call into one of them.

This patchset further separates SFF/BMDMA support from the rest of the
libata and separate out BMDMA from SFF.  This patchset contains the
following 22 patches.

 0001-pata_sch-use-ata_pci_sff_init_one.patch
 0002-sata_inic162x-inic162x-is-not-dependent-on-CONFIG_A.patch
 0003-sata_mv-remove-unnecessary-initialization.patch
 0004-libata-sff-clear-IRQ-from-ata_sff_error_handler-o.patch
 0005-libata-sff-kill-unused-prototype-and-make-ata_dev_s.patch
 0006-libata-kill-ATA_FLAG_DISABLED.patch
 0007-sata_inic162x-kill-PORT_PRD_ADDR-initialization.patch
 0008-libata-sff-reorder-SFF-BMDMA-functions.patch
 0009-pata_cmd640-bf54x-icside-fix-inheritance.patch
 0010-libata-sff-clean-up-BMDMA-initialization.patch
 0011-libata-sff-introduce-ata_sff_init-exit-and-ata_sf.patch
 0012-libata-sff-ap-last_-ctl-are-SFF-specific.patch
 0013-libata-sff-port_task-is-SFF-specific.patch
 0014-libata-sff-separate-out-BMDMA-EHs.patch
 0015-libata-sff-ata_sff_-dumb_-qc_prep-are-BMDMA-specifi.patch
 0016-libata-sff-prd-is-BMDMA-specific.patch
 0017-libata-sff-separate-out-BMDMA-qc_issue.patch
 0018-libata-sff-ata_sff_irq_clear-is-BMDMA-specific.patch
 0019-libata-sff-separate-out-BMDMA-irq-handler.patch
 0020-libata-sff-separate-out-BMDMA-init.patch
 0021-sata_qstor-kill-dummy-BMDMA-ops.patch
 0022-libata-sff-make-BMDMA-optional.patch

0001-0010 make things aligned for the future changes.

 0004 changes the default SFF error handler behavior such that
 unnecessary register accesses are removed.  I'm fairly sure this
 change is safe (tested on various SFF/BMDMA controllers) but we'll
 need to look out for problem reports.

 0006 kills ATA_FLAG_DISABLED (finally).  This patch touches SCSI SAS
 drivers.

 0010 cleans up BMDMA initialization and bmdma_addr condition checks.

0011-0013 further separate out SFF assumptions from libata core layer.

0014-0021 separate out BMDMA from SFF.

0022 makes BMDMA optional and reorganizes Kconfig options according to
the groups of drivers they belong to.

This patchset is on top of

  linus#master (2515ddc6db8eb49a79f0fe5e67ff09ac7c81eab4)
+ [1] libata-sff-fix-ata_sff_post_internal_cmd
+ [2] libata-initialize-port_task-when-!CONFIG_ATA_SFF
+ [3] libata-eh-fix-slave-link-EH-action-mask-handling
+ [4] libata-transfer-EHI-control-flags-to-slave-ehc_i

and makes the followings changes.

 drivers/ata/Kconfig                 |  360 +++----
 drivers/ata/Makefile                |   47 
 drivers/ata/ata_piix.c              |    4 
 drivers/ata/libata-core.c           |  172 ---
 drivers/ata/libata-eh.c             |   11 
 drivers/ata/libata-scsi.c           |    3 
 drivers/ata/libata-sff.c            | 1757 ++++++++++++++++++++----------------
 drivers/ata/libata.h                |   37 
 drivers/ata/pata_acpi.c             |   10 
 drivers/ata/pata_ali.c              |    7 
 drivers/ata/pata_amd.c              |    2 
 drivers/ata/pata_artop.c            |    2 
 drivers/ata/pata_atiixp.c           |    4 
 drivers/ata/pata_bf54x.c            |   20 
 drivers/ata/pata_cmd640.c           |   10 
 drivers/ata/pata_cmd64x.c           |    2 
 drivers/ata/pata_cs5520.c           |    4 
 drivers/ata/pata_cs5530.c           |    6 
 drivers/ata/pata_cs5535.c           |    2 
 drivers/ata/pata_cs5536.c           |    2 
 drivers/ata/pata_cypress.c          |    2 
 drivers/ata/pata_efar.c             |    2 
 drivers/ata/pata_hpt366.c           |    4 
 drivers/ata/pata_hpt37x.c           |    6 
 drivers/ata/pata_hpt3x2n.c          |    4 
 drivers/ata/pata_hpt3x3.c           |    2 
 drivers/ata/pata_icside.c           |    7 
 drivers/ata/pata_it8213.c           |    2 
 drivers/ata/pata_it821x.c           |    8 
 drivers/ata/pata_jmicron.c          |    2 
 drivers/ata/pata_marvell.c          |    2 
 drivers/ata/pata_mpc52xx.c          |    1 
 drivers/ata/pata_netcell.c          |    2 
 drivers/ata/pata_ninja32.c          |    2 
 drivers/ata/pata_ns87415.c          |    4 
 drivers/ata/pata_oldpiix.c          |    4 
 drivers/ata/pata_optidma.c          |    2 
 drivers/ata/pata_pdc2027x.c         |    6 
 drivers/ata/pata_pdc202xx_old.c     |    4 
 drivers/ata/pata_platform.c         |    1 
 drivers/ata/pata_radisys.c          |    4 
 drivers/ata/pata_sc1200.c           |    6 
 drivers/ata/pata_scc.c              |   15 
 drivers/ata/pata_sch.c              |   12 
 drivers/ata/pata_serverworks.c      |    8 
 drivers/ata/pata_sil680.c           |    4 
 drivers/ata/pata_sis.c              |    2 
 drivers/ata/pata_sl82c105.c         |    2 
 drivers/ata/pata_triflex.c          |    2 
 drivers/ata/pata_via.c              |    2 
 drivers/ata/pdc_adma.c              |   74 -
 drivers/ata/sata_inic162x.c         |   25 
 drivers/ata/sata_mv.c               |   15 
 drivers/ata/sata_nv.c               |  264 ++---
 drivers/ata/sata_promise.c          |   30 
 drivers/ata/sata_qstor.c            |  102 --
 drivers/ata/sata_sil.c              |    7 
 drivers/ata/sata_sil24.c            |    9 
 drivers/ata/sata_sis.c              |    4 
 drivers/ata/sata_svw.c              |    4 
 drivers/ata/sata_sx4.c              |    9 
 drivers/ata/sata_uli.c              |    6 
 drivers/ata/sata_via.c              |    6 
 drivers/ata/sata_vsc.c              |   12 
 drivers/scsi/ipr.c                  |    7 
 drivers/scsi/libsas/sas_scsi_host.c |    2 
 include/linux/ata.h                 |    2 
 include/linux/libata.h              |   91 +
 68 files changed, 1653 insertions(+), 1602 deletions(-)

Thanks.

--
tejun

[1] http://article.gmane.org/gmane.linux.ide/35364
[2] http://article.gmane.org/gmane.linux.ide/35363
[3] http://article.gmane.org/gmane.linux.ide/35390
[4] http://article.gmane.org/gmane.linux.ide/35393

^ permalink raw reply	[flat|nested] 44+ messages in thread

end of thread, other threads:[~2010-05-08 15:33 UTC | newest]

Thread overview: 44+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-10-21  9:17 [PATCHSET #upstream] libata: separate out SFF and BMDMA Tejun Heo
2008-10-21  9:17 ` [PATCH 01/22] pata_sch: use ata_pci_sff_init_one() Tejun Heo
2008-10-21  9:17 ` [PATCH 02/22] sata_inic162x: inic162x is not dependent on CONFIG_ATA_SFF Tejun Heo
2008-10-21  9:17 ` [PATCH 03/22] sata_mv: remove unnecessary initialization Tejun Heo
2008-10-21  9:17 ` [PATCH 04/22] libata-sff: clear IRQ from ata_sff_error_handler() only when necessary Tejun Heo
2008-10-21  9:53   ` Sergei Shtylyov
2008-10-21 11:45     ` [PATCH 04/22, UPDATED] " Tejun Heo
2008-10-28  4:10   ` [PATCH 04/22] " Jeff Garzik
2008-10-28  4:28     ` Tejun Heo
2008-10-21  9:17 ` [PATCH 05/22] libata-sff: kill unused prototype and make ata_dev_select() static Tejun Heo
2010-05-07 13:47   ` Sergei Shtylyov
2008-10-21  9:17 ` [PATCH 06/22] libata: kill ATA_FLAG_DISABLED Tejun Heo
2008-10-21  9:17 ` [PATCH 07/22] sata_inic162x: kill PORT_PRD_ADDR initialization Tejun Heo
2008-10-21  9:17 ` [PATCH 08/22] libata-sff: reorder SFF/BMDMA functions Tejun Heo
2008-10-21  9:17 ` [PATCH 09/22] pata_cmd640/bf54x/icside: fix inheritance Tejun Heo
2008-10-21  9:17 ` [PATCH 10/22] libata-sff: clean up BMDMA initialization Tejun Heo
2008-10-28  4:16   ` Jeff Garzik
2008-10-21  9:17 ` [PATCH 11/22] libata-sff: introduce ata_sff_init/exit() and ata_sff_port_init() Tejun Heo
2008-10-21  9:17 ` [PATCH 12/22] libata-sff: ap->[last_]ctl are SFF specific Tejun Heo
2008-10-21  9:17 ` [PATCH 13/22] libata-sff: port_task is " Tejun Heo
2008-10-21  9:17 ` [PATCH 14/22] libata-sff: separate out BMDMA EHs Tejun Heo
2008-10-21  9:17 ` [PATCH 15/22] libata-sff: ata_sff_[dumb_]qc_prep are BMDMA specific Tejun Heo
2008-10-21  9:17 ` [PATCH 16/22] libata-sff: prd is " Tejun Heo
2008-10-21  9:17 ` [PATCH 17/22] libata-sff: separate out BMDMA qc_issue Tejun Heo
2008-10-21  9:17 ` [PATCH 18/22] libata-sff: ata_sff_irq_clear() is BMDMA specific Tejun Heo
2008-10-21  9:17 ` [PATCH 19/22] libata-sff: separate out BMDMA irq handler Tejun Heo
2008-10-21  9:17 ` [PATCH 20/22] libata-sff: separate out BMDMA init Tejun Heo
2008-10-21 15:49   ` [PATCH 20/22 UPDATED] " Tejun Heo
2008-10-21  9:17 ` [PATCH 21/22] sata_qstor: kill dummy BMDMA ops Tejun Heo
2008-10-21 10:10   ` Sergei Shtylyov
2008-10-21 11:27     ` Tejun Heo
2008-10-21  9:18 ` [PATCH 22/22] libata-sff: make BMDMA optional Tejun Heo
2008-10-21  9:26 ` [PATCHSET #upstream] libata: separate out SFF and BMDMA Tejun Heo
2008-10-21 11:36 ` Sergei Shtylyov
2008-10-21 11:43   ` Tejun Heo
2008-10-27 19:43 ` Mark Lord
2008-10-28  1:06   ` Tejun Heo
2008-10-28  4:20 ` Jeff Garzik
2008-11-03 14:26   ` Tejun Heo
2008-11-04  5:35     ` Jeff Garzik
2008-11-04  5:38       ` Tejun Heo
2008-10-28  4:31 ` [PATCH 04/22 DESC UPDATED] libata-sff: clear IRQ from ata_sff_error_handler() only when necessary Tejun Heo
2010-05-07 13:59 ` [PATCHSET #upstream] libata: separate out SFF and BMDMA Sergei Shtylyov
2010-05-08 15:33   ` Tejun Heo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).