On Wed, 4 Sep 2002, Suparna Bhattacharya wrote:
>
> Oh yes, even I had this fixed this in the bio traversal patches
> I had posted (had this in the core patch, and mentioned it
> in the description in the note :) ), guess it went unnoticed.
Well, I've never seen a "this should go in" about it.
Also, it was apparently mixed up with the "bio splitup" stuff, which was
discussed at least with Jens, and I feel strongly that we shouldn't split,
we should build up. Jens was working on exactly that.
In other words, I absolutely hate the fact that a major bug-fix was
(a) not marked as such and sent to me
and
(b) mixed up with experimental work for other drivers
Even now (assuming I hadn't fixed it on my own), I would have preferred to
get that fix separately, as it would have impacted the floppy driver, for
example (the fix broke the floppy driver even more than it was before,
because the floppy driver was stupidly trying to work around the original
bug by hand).
Imagine what a horror it is to figure out why a large experimental patch
breaks an existing driver? My first reaction would have been to just throw
the large new patch away, since it obviously broke the floppy even more.
Instead, if I had been passed the bug-fix only, and the floppy had broken
worse that it was originally, it would have been absolutely _obvious_
where the problem was.
In short: please please PLEASE keep fixes to existing code separate from
new stuff. It makes everything easier, and I have absolutely no problems
with applying "obvious fixes" even if they might break something else.
In contrast, the new stuff I really don't know if it should go in at all,
considering that it's trivial (and CPU-efficient) to build up legal bio
request on the fly and _not_ depend on splitting them later (or at least
making splitting a special thing only used by things like MD and other
such indirection layers).
Linus
-
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 Sep 07 2002 - 22:00:22 EST