Re: CVS commit: src/sys/dev

[yamt%mwd.biglobe.ne.jp@localhost (YAMAMOTO Takashi)]

> On Mar 5, 12:29pm, yamt%mwd.biglobe.ne.jp@localhost (YAMAMOTO Takashi) wrote:
> -- Subject: Re: CVS commit: src/sys/dev
> 
> | > Module Name:      src
> | > Committed By:     christos
> | > Date:             Mon Mar  5 02:55:32 UTC 2007
> | > 
> | > Modified Files:
> | >   src/sys/dev: vnd.c
> | > 
> | > Log Message:
> | > Fix compression problem from Cliff Wright:
> | >   Break one: because b_cylinder, and b_resid are one and the same.
> | > The work the routine was commented as being its primary function,
> | > end of patition adjustment, is wiped out, as at the end of
> | > the routine b_cylinder is set, splat, doing a wipe out of the adjustment
> | > to b_resid.
> | >   Break two: When doing the adjustment, a block count is created from a
> | > byte count, a block calculation is done, then the results are compared to
> | > the original byte count. i.e. apples to oranges, not blocks to blocks,
> | > but blocks to bytes.
> | >   Break three: since if all the other drivers that used this routine
> | > would have broken as vnd compress did, I must assume they always ignored
> | > the results of this routine. So if end of patition adjustment is
> | > really required then all these other drivers have been broken for a
> | > long time.
> | 
> | which "routine" are you talking about?
> 
> bounds_check_with_label()?
> 
> christos

are you going to fix it?

YAMAMOTO Takashi