[lkml]   [2008]   [Aug]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [patch 1/3] crypto: Add a zlib crypto module
    On Fri, Aug 29, 2008 at 01:41:59PM +0000, Geert Uytterhoeven wrote:
    > From: Geert Uytterhoeven <>
    > Add a (de)compression module for the "zlib" format using the crypto API.

    I think we can safely conclude that our current compression
    interface sucks for what you're trying to achieve :)

    > While both the "zlib" and "deflate" crypto modules are implemented on top of
    > the zlib library, they differ in the following aspects:
    > - The "deflate" crypto module (used by IPSec and UBIFS) does not support
    > partial decompression, i.e. all compressed data has to be passed at once.
    > The "zlib" crypto module does support partial decompression;
    > zlib_decompress() will return -EAGAIN if not all compressed data has been
    > passed.
    > - The deflate crypto module uses the raw deflate data format (zlib is
    > initialized with a windowBits parameter of -DEFLATE_DEF_WINBITS = -11),
    > while e.g. squashfs and axfs use the zlib data format, with the default
    > windowBits parameter DEF_WBITS = 15.
    > Both parameters are incompatible with each other due to the different data
    > formats, as indicated by the sign of the windowbits parameter.
    > The absolute value of this parameter is the base two logarithm of the
    > maximum window size, and larger values are backwards compatible with
    > smaller values (as far as decompression is concerned).

    Therefore I suggest that we change the interface rather than
    mutilate the implementations.

    So here are a few things we should add:

    1) Separate alloc functions for comp/decomp to avoid the memory
    wastage you've identified.

    2) Provide parameters to these alloc functions through an opaque
    pointer. The format of the paramters will be determined by the
    name of the algorithm, i.e., if you want to change the parameters
    then you should change the name as well (e.g., deflate => deflate2).

    This removes the need to dupliate the implementation just because
    you want 15 instead of -11.

    3) Provide init/update/final (one set each for comp and decomp)
    functions similar to crypto_hash, in addition to the current
    comp/decomp functions.

    Visit Openswan at
    Email: Herbert Xu ~{PmV>HI~} <>
    Home Page:
    PGP Key:

     \ /
      Last update: 2008-08-30 08:25    [W:0.041 / U:1.720 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site