All of lore.kernel.org
 help / color / mirror / Atom feed
* [0/16 ver2] gdth combined patchset & call for testers
@ 2007-10-02 20:05 Boaz Harrosh
  2007-10-02 20:40 ` [PATCH 01/16] gdth: Make one abuse of scsi_cmnd less obvious Boaz Harrosh
                   ` (16 more replies)
  0 siblings, 17 replies; 19+ 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] 19+ messages in thread
* [RFC 0/16] gdth combined patchset & call for testers
@ 2007-09-30 19:44 Boaz Harrosh
  2007-09-30 20:14 ` [PATCH 14/16] gdth: Setup proper per-command private data Boaz Harrosh
  0 siblings, 1 reply; 19+ 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] 19+ messages in thread

end of thread, other threads:[~2007-10-02 21:22 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-10-02 20:05 [0/16 ver2] gdth combined patchset & call for testers Boaz Harrosh
2007-10-02 20:40 ` [PATCH 01/16] gdth: Make one abuse of scsi_cmnd less obvious Boaz Harrosh
2007-10-02 20:46 ` [PATCH 02/16] gdth: Stop abusing ->done for internal commands Boaz Harrosh
2007-10-02 20:48 ` [PATCH 03/16] gdth: split out isa probing Boaz Harrosh
2007-10-02 20:49 ` [PATCH 04/16] gdth: split out eisa probing Boaz Harrosh
2007-10-02 20:51 ` [PATCH 05/16] gdth: split out pci probing Boaz Harrosh
2007-10-02 20:54 ` [PATCH 06/16] gdth: Remove 2.4.x support, in-kernel changelog Boaz Harrosh
2007-10-02 20:55 ` [PATCH 07/16] gdth: kill gdth_{read,write}[bwl] wrappers Boaz Harrosh
2007-10-02 20:57 ` [PATCH 08/16] gdth: Reorder scsi_host_template intitializers Boaz Harrosh
2007-10-02 20:59 ` [PATCH 09/16] gdth: Remove virt hosts Boaz Harrosh
2007-10-02 21:05 ` [PATCH 10/16] gdth: clean up host private data Boaz Harrosh
2007-10-02 21:08 ` [PATCH 11/16] gdth: gdth_interrupt() gdth_get_status() & gdth_wait() fixes Boaz Harrosh
2007-10-02 21:09 ` [PATCH 12/16] gdth: switch to modern scsi host registration Boaz Harrosh
2007-10-02 21:11 ` [PATCH 13/16] gdth: Remove gdth_ctr_tab[] Boaz Harrosh
2007-10-02 21:14 ` [PATCH 14/16] gdth: Setup proper per-command private data Boaz Harrosh
2007-10-02 21:16 ` [PATCH 15/16] gdth: Move members from SCp to gdth_cmndinfo, stage 2 Boaz Harrosh
2007-10-02 21:18 ` [PATCH 16/16] gdth: !use_sg cleanup and use of scsi accessors Boaz Harrosh
2007-10-02 21:22 ` [0/16 ver2] gdth combined patchset & call for testers Boaz Harrosh
  -- strict thread matches above, loose matches on Subject: below --
2007-09-30 19:44 [RFC 0/16] " Boaz Harrosh
2007-09-30 20:14 ` [PATCH 14/16] gdth: Setup proper per-command private data Boaz Harrosh

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.