Re: [PATCH] size-in-bytes

From: Andries.Brouwer@cwi.nl
Date: Sat Feb 16 2002 - 17:45:26 EST


    From dalecki@evision-ventures.com Sat Feb 16 23:05:50 2002

    Well I see that you have killed all places where it get's used - that's
    fine!
    But why don't just kill (fix) the initializations as well. You have already:

    g->part[minor(dev)].nr_sects;

    and friends right there at your hands ;-).

Yes, of course there is a reason this patch is called 02*.
I planned a road to happiness in 92 steps, where each of the steps
does something clear and simple, simplifies the tree, beautifies
the code, restructures in a clearly necessary way.
Indeed, your suggested steps are also there.
Five or six of these steps found their way into the kernel,
(some thanks to Christoph Hellwig) but there is still a long way to go.
The present patch is just a rediff of step 02.

I do not know what the best strategy is, these times.
I see you and Vojtech do good things to the IDE code,
but would myself prefer to do such things in a series
of really small steps. That way it is also very clear
for Andre what happens.

Andries
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sat Feb 23 2002 - 21:00:13 EST