public inbox for linux-scsi@vger.kernel.org
 help / color / mirror / Atom feed
* [RFC 0/16] gdth combined patchset & call for testers
@ 2007-09-30 19:44 Boaz Harrosh
  2007-09-30 19:50 ` [PATCH 1/16] gdth: split out isa probing Boaz Harrosh
                   ` (20 more replies)
  0 siblings, 21 replies; 48+ messages in thread
From: Boaz Harrosh @ 2007-09-30 19:44 UTC (permalink / raw)
  To: Christoph Hellwig, Jeff Garzik, James Bottomley, Matthew Wilcox,
	achim_leubner
  Cc: linux-scsi

Hi!

following is an attempted at unified patchset for the gdth driver.

They try to incorporate floating patches to gdth from:
Christoph Hellwig
Jeff Garzik
Matthew Wilcox
and Me Boaz Harrosh

They are done in the mindset of "likelihood of inducing breakage",
hence the need for testers. Christoph & Jeff please review for any
obvious easy-to-catch bugs.

Some short explanations:
[1/16] gdth: split out isa probing - Christoph Hellwig
[2/16] gdth: split out eisa probing - Christoph Hellwig
[3/16] gdth: split out pci probing - Christoph Hellwig
  These three are from Christoph and where ACKed by
  Jeff at the time.

[4/16] gdth: Remove 2.4.x support, in-kernel changelog - Jeff Garzik
  Same but partial work was done both by Christoph and Matthew.

[5/16] gdth: kill gdth_{read,write}[bwl] wrappers - Jeff Garzik
[6/16] Reorder scsi_host_template intitializers
[7/16] gdth: make some virt ctrlr code common
  These 3 are from Jeff's patchset 6 & 7 where the same patch

[8/16] gdth: Remove virt hosts - Christoph && Boaz
  Here we need an executive decision! The issue is as stated by Christoph:

    "The virt_ctr option allows to register a new scsi_host for each bus
    on the raid controller.  This non-default option makes no sense with
    the current scsi code and prevents cleaning up the host registration,
    so remove it."

  I agree. This is just exactly the same as done buy scsi-ml scans but only
  more resource consuming. Unless I'm totally missing something, perhaps it is
  just a leftover from old kernels.

  But if it is decided that this "virt_ctr" fixture is absolutely needed than
  I have a patch for re-enabling it at: "after the patchset", done in a different
  way. Because for now it prevents the cleanups I need.

  Also this patch can Just be merged with [7/16] but I wanted it separate in the 
  case we decide for "virt_ctr" fixture return.

[9/16] gdth: clean up host private data - Christoph && Boaz
  This is based on the same patch from Christoph, but taken one step
  farther, by just passing the ha pointer everywhere instead of hanum.
  Christoph please acknowledge your signed-of-by on this patch.

[10/16] gdth: gdth_get_status() return pointer to host not its index - Boaz
  This logically belongs to [9/16] but is separated for reviewing and bisect-ability
  As it is a sensitive matter.

[11/16] gdth: switch to modern scsi host registration - Christoph
  Christoph what is missing from here is the remove of the deprecated 
  pci_find_device() call. Can I Just use pci_get_device() of the same signature
  or do I need to call some other pci_ members after that?

[12/16] gdth: Remove gdth_ctr_tab[] - Boaz
  I took Christoph's cleanup one step farther and got read of the statically
  allocated gdth_ctr_tab[]. In it's place I use the new link-list introduced
  by the [11/16] patch.

[13/16] gdth: Make one abuse of scsi_cmnd less obvious - Matthew Wilcox
  This is the first patch sent by Matthew Wilcox, rebased to all above
  patches. Matthew, thanks, it saved me from a much uglier hack I had
  with regard to per-command-private-data.

[14/16] gdth: Setup proper per-command private data - Boaz
[15/16] gdth: Move members from SCp to gdth_cmndinfo, stage 2 - Boaz
  These two move me much closer to the agenda I had in all this,
  which is: "gdth diss-abuse of of scsi_cmnd IO members".
  With these patches I also conform to Matthew's second patch:
  "gdth: Stop abusing ->done for internal commands"

[16/16] gdth: !use_sg cleanup and use of scsi accessors
  And finally this one.


Boaz


^ permalink raw reply	[flat|nested] 48+ messages in thread
* [0/16 ver2] gdth combined patchset & call for testers
@ 2007-10-02 20:05 Boaz Harrosh
  2007-10-02 21:18 ` [PATCH 16/16] gdth: !use_sg cleanup and use of scsi accessors Boaz Harrosh
  0 siblings, 1 reply; 48+ messages in thread
From: Boaz Harrosh @ 2007-10-02 20:05 UTC (permalink / raw)
  To: James Bottomley, Mike Christie, Jeff Garzik, Matthew Wilcox,
	linux-scsi

Hi!

following is an attempted at unified patchset for the gdth driver.

They try to incorporate floating patches to gdth from:
Christoph Hellwig
Jeff Garzik
Matthew Wilcox
and Me Boaz Harrosh

They are done in the mindset of "likelihood of inducing breakage",
hence the need for testers. Christoph & Jeff please review for any
obvious easy-to-catch bugs. Also there was a massive code shifts
and rebasing I hope nothing was lost.

The differences from last set are:
- Matthews patches are first so they can go in. They
  are a bit modified so please take these and not the
  original. Mainly they restrain from cleanups

- Fix Christoph bug that was spotted by jeff of the misplaced
  #endif. Also when compiling with out CONFIG_{ISA,EISA,PCI}
  compiler would complain about unused params, functions and
  so on so fixed that.

- More Aggressive fix for gdth_interrupt() and friends
  Christoph pleeeeeeease review.

- Rebasing to all above. I hope I'll get the From:
  right this time

Some short explanations:
[01/16] gdth: Make one abuse of scsi_cmnd less obvious - Matthew Wilcox
[02/16] gdth: Stop abusing ->done for internal commands - Matthew Wilcox
  Patches from Matthew, but cleaned for future patchability.

[03/16] gdth: split out isa probing - Christoph Hellwig
[04/16] gdth: split out eisa probing - Christoph Hellwig
[05/16] gdth: split out pci probing - Christoph Hellwig
  These three are from Christoph and where ACKed by
  Jeff at the time. Revised by me for warning fixes when
  one of the buses ISA/EISA/PCI is not defined.

[06/16] gdth: Remove 2.4.x support, in-kernel changelog - Jeff Garzik
  Same but partial work was done both by Christoph and Matthew.

[07/16] gdth: kill gdth_{read,write}[bwl] wrappers - Jeff Garzik
[08/16] Reorder scsi_host_template intitializers
[09/16] gdth: Remove virt hosts - Christoph && Boaz
  Here we need an executive decision! The issue is as stated by Christoph:

    "The virt_ctr option allows to register a new scsi_host for each bus
    on the raid controller.  This non-default option makes no sense with
    the current scsi code and prevents cleaning up the host registration,
    so remove it."

  I agree. This is just exactly the same as done buy scsi-ml scans but only
  more resource consuming. Unless I'm totally missing something, perhaps it is
  just a leftover from old kernels.

[10/16] gdth: clean up host private data - Christoph && Boaz
  This is based on the same patch from Christoph, but taken one step
  farther, by just passing the ha pointer everywhere instead of hanum.

[11/16] gdth_interrupt gdth_get_status & gdth_wait fixes - Boaz
  This logically belongs to [9/16] but is separated for reviewing and 
  bisect-ability As it is a sensitive matter. 
  It is done aggressively this time around Please review!

[12/16] gdth: switch to modern scsi host registration - Christoph
  Christoph what is missing from here is the remove of the deprecated 
  pci_find_device() call. Can I Just use pci_get_device() of the same signature
  or do I need to call some other pci_ members after that?

[13/16] gdth: Remove gdth_ctr_tab[] - Boaz
  I took Christoph's cleanup one step farther and got read of the statically
  allocated gdth_ctr_tab[]. In it's place I use the new link-list introduced
  by the [11/16] patch.

[14/16] gdth: Setup proper per-command private data - Boaz
[15/16] gdth: Move members from SCp to gdth_cmndinfo, stage 2 - Boaz
  These two move me much closer to the agenda I had in all this,
  which is: "gdth diss-abuse of of scsi_cmnd IO members".
  With these patches I also conform to Matthew's second patch:
  "gdth: Stop abusing ->done for internal commands"

[16/16] gdth: !use_sg cleanup and use of scsi accessors
  And finally this one.

Boaz


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

end of thread, other threads:[~2007-10-03 18:16 UTC | newest]

Thread overview: 48+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-09-30 19:44 [RFC 0/16] gdth combined patchset & call for testers Boaz Harrosh
2007-09-30 19:50 ` [PATCH 1/16] gdth: split out isa probing Boaz Harrosh
2007-10-02 17:17   ` Rolf Eike Beer
2007-10-03 16:00     ` Jeff Garzik
2007-09-30 19:50 ` [PATCH 2/16] gdth: split out eisa probing Boaz Harrosh
2007-10-02 17:20   ` Rolf Eike Beer
2007-10-03 17:27     ` Christoph Hellwig
2007-10-03 17:32       ` Rolf Eike Beer
2007-10-03 17:38         ` Christoph Hellwig
2007-10-03 17:59           ` Jeff Garzik
2007-10-03 18:05             ` Christoph Hellwig
2007-10-03 18:07               ` Jeff Garzik
2007-09-30 19:55 ` [PATCH 3/16] gdth: split out pci probing Boaz Harrosh
2007-09-30 19:57 ` [PATCH 4/16] gdth: Remove 2.4.x support, in-kernel changelog Boaz Harrosh
2007-09-30 19:58 ` [PATCH 5/16] gdth: kill gdth_{read,write}[bwl] wrappers Boaz Harrosh
2007-09-30 19:59 ` [PATCH 6/16] Reorder scsi_host_template intitializers Boaz Harrosh
2007-09-30 20:01 ` [PATCH 7/16] gdth: make some virt ctrlr code common Boaz Harrosh
2007-09-30 21:22   ` Christoph Hellwig
2007-09-30 20:03 ` [PATCH 8/16] gdth: Remove virt hosts Boaz Harrosh
2007-09-30 20:06 ` [PATCH 9/16] gdth: clean up host private data Boaz Harrosh
2007-09-30 21:23   ` Christoph Hellwig
2007-09-30 20:09 ` [PATCH 10/16] gdth: gdth_get_status() return pointer to host not its index Boaz Harrosh
2007-09-30 21:26   ` Christoph Hellwig
2007-10-02 11:04     ` Boaz Harrosh
2007-10-02 11:10       ` Boaz Harrosh
2007-09-30 20:10 ` [PATCH 11/16] gdth: switch to modern scsi host registration Boaz Harrosh
2007-09-30 20:12 ` [PATCH 12/16] gdth: Remove gdth_ctr_tab[] Boaz Harrosh
2007-09-30 20:13 ` [PATCH 13/16] gdth: Make one abuse of scsi_cmnd less obvious Boaz Harrosh
2007-09-30 21:28   ` Christoph Hellwig
2007-09-30 23:21     ` Matthew Wilcox
2007-10-01 13:56       ` Boaz Harrosh
2007-10-01 14:23         ` Jeff Garzik
2007-09-30 20:14 ` [PATCH 14/16] gdth: Setup proper per-command private data Boaz Harrosh
2007-09-30 20:16 ` [PATCH 15/16] gdth: Move members from SCp to gdth_cmndinfo, stage 2 Boaz Harrosh
2007-10-02 18:02   ` Rolf Eike Beer
2007-10-03 18:15     ` Christoph Hellwig
2007-09-30 20:17 ` [PATCH 16/16] gdth: !use_sg cleanup and use of scsi accessors Boaz Harrosh
2007-10-01 14:06   ` Boaz Harrosh
2007-10-01 14:19   ` [PATCH 16/16 ver2] " Boaz Harrosh
2007-09-30 21:00 ` [RFC 0/16] gdth combined patchset & call for testers Jeff Garzik
2007-09-30 21:07 ` Jeff Garzik
2007-09-30 21:36   ` Christoph Hellwig
2007-09-30 22:53     ` Jeff Garzik
2007-09-30 21:27 ` Jeff Garzik
2007-10-01 14:29   ` Boaz Harrosh
2007-09-30 21:33 ` Christoph Hellwig
2007-09-30 22:53 ` Jeff Garzik
  -- strict thread matches above, loose matches on Subject: below --
2007-10-02 20:05 [0/16 ver2] " Boaz Harrosh
2007-10-02 21:18 ` [PATCH 16/16] gdth: !use_sg cleanup and use of scsi accessors Boaz Harrosh

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox