Re: ioctls, etc. (was Re: [PATCH 1/4] sas: add flag for locally attachedPHYs)

From: Stefan Richter
Date: Sat Oct 22 2005 - 12:50:25 EST


Christoph Hellwig wrote:
On Sat, Oct 22, 2005 at 12:42:27PM +0200, Stefan Richter wrote:
A. Post mock-ups and pseudo code about how to change the core, discuss.
B. Set up a scsi-cleanup tree. In this tree,
1. renovate the core (thereby break all command set drivers and
all transport subsystems),

No way. Doing things from scatch is a really bad idea. See how far we came
with Linux 2.6 scsi vs 2.4 scsi without throwing everything away and break the
world. Please submit changes to fix _one_ thing at a time and fix all users.
Repeat until done or you don't care anymore.

I agree with you. Alas my wording was misunderstandable und obviously carried a wrong tone.

I did not say "replace the core" in step 1. Also, the breakage which I refer to in step 1 would have to be immediately corrected in step 2 (although not for the whole subsystem at once, to allow for a fast cycle of validation of what happened in step 1). Furthermore I specifically said that most steps may (let me add: and should) overlap.
--
Stefan Richter
-=====-=-=-= =-=- =-==-
http://arcgraph.de/sr/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/