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

Stefan Richter
2005-10-22T17:50:03Z

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
-=====-=-=-= =-=- =-==-