Messages in this thread | | | Date | Sat, 21 Jun 2003 18:45:43 +0200 | Subject | Re: [PATCH - RFC] loop.c | From | Fruhwirth Clemens <> |
| |
On Sat, Jun 21, 2003 at 05:07:47PM +0200, Andries.Brouwer@cwi.nl wrote: > For a long time we have had a somewhat unfortunate situation > where people wanting to use cryptoloop had to collect some > kernel patches and util-linux patches elsewhere. > Now that we have crypto in the kernel this can be rectified. > > As far as I can see this requires two things: > - crypto transfer functions must be registered with loop.c, > that is, loop_register_transfer() must be called > - loop.c must be fixed > > The first point is handled by cryptoloop-0.2-2.5.58.diff. > Concerning the second point, several patches are floating around. > Below a patch that Jari Ruusu sent me. > > This is a RFC. > Clemens - any comments on the crypto side? > Jens - any comments on the block I/O side?
I haven't looked at it in detail but Jari's patches have done the right things at all times. If they do things right is another issue. I prefer Adam Richter's patches better but if the decision is in favour for Jari's I have no technical objections.
But you have to make yourself clear:
This patch will change the IV metric to 512 byte and the loop transfer function has to be allowed to increase the IV for itself after any 512byte chunk. (Otherwise the transfer function would have to return after a single chunk, let loop.c compute the next IV and be called by loop.c again.)
I've always wanted it this way. Nothing which has ever been merged officially has used loop.c's IV calculation and the unofficial stuff fixed the calculation years ago (one and a half actually). So I can't see a reason to maintain backward compatiblity, because I hardly suspect that there is even a user base. I come to this conclusion because loop.c's old IV calculation isn't even done properly. This patch breaks the old IV calculation: http://lwn.net/Articles/2677/ . It has been merged a year ago and nobody ever recognized the breakage. So let's face it: There is no user base. We are talking about backward compatiblity for nobody here.
If you, Andries, are really volunteering to split the patch up, you're my hero :)
Regards, Clemens [unhandled content-type:application/pgp-signature] | |