All of lore.kernel.org
 help / color / mirror / Atom feed
From: devzero@web.de
To: akpm@linux-foundation.org
Cc: linux-kernel@vger.kernel.org, nitingupta.mail@gmail.com
Subject: Re: [PATCH] Add LZO1X compression support to the kernel
Date: Fri, 11 May 2007 22:48:15 +0200	[thread overview]
Message-ID: <1783772957@web.de> (raw)

>Why is this needed?  What code plans to use it?

it`s pretty useful because it`s and a damn fast and damn cpu friendly compression alorithm.

afaik, there is already a least one linux kernel-feature (under development) which is using lzo compression: 
see compressed caching project at http://linux-mm.org/CompressedCaching & http://linuxcompressed.sourceforge.net/

seems, they have also done porting it to the kernel, so there is probably choice between two implemetations to merge.

>How many buffer overruns are there in it?

i don`t know :) 
but, from a user-perspective,  lzo is really portable and seems to be a rock solid compression scheme.
i`m sucessfully using it for years (lzop utility) and i know projects which compress gigabytes of data every day with lzop.
furthermore, i know of at least 40 software projects using lzo compression, so this should have some level of maturity.

maybe i can add another software integrating lzo compression to the enumeration at http://www.lzop.de ? ;)

regards
roland


List:       linux-kernel
Subject:    Re: [PATCH] Add LZO1X compression support to the kernel
From:       Andrew Morton <akpm () linux-foundation ! org>
Date:       2007-05-10 6:21:29
Message-ID: 20070509232129.371f49d5.akpm () linux-foundation ! org
[Download message RAW]

On Wed, 02 May 2007 09:56:23 +0100 Richard Purdie <richard@openedhand.com> wrote:

> Current thinking is that lzo should get merged directly followed by the
> subsystem parts through their specific trees. It appears this should
> make it onto LKML despite the size so here goes.
> 
> Please keep in mind I haven't reformatted the LZO code itself as if I do
> so, it will make maintenance of it against any changes in LZO itself
> near impossible. In its current form, it should be possible to diff
> against upstream. All the bad formatting is confined to a handful of
> files in lib/lzo/ and the kernel interface should be clean.
> 
> I realise a maze of ifdefs still remain. I've already spent a lot of
> time removing a ton of them and going much further might start to affect
> diffability of the code - I hoping whats there is a good compromise.
> 
> I've asked the LZO author about the comments on lzo_copyright function
> but the code is GPLv2 licensed so is suitable for inclusion in the
> kernel.
> 
> 
> 
> Add LZO1X compression/decompression support to the kernel.
> 
> This is based on the standard userspace lzo library, particularly
> minilzo with the headers much trimmed down and simplified for kernel
> use. Its structured so that it should still diff with the userspace
> version for ease of future updating.

Well that's attractive-looking code.

Why is this needed?  What code plans to use it?

How many buffer overruns are there in it?
_____________________________________________________________________
Der WEB.DE SmartSurfer hilft bis zu 70% Ihrer Onlinekosten zu sparen!
http://smartsurfer.web.de/?mc=100071&distributionid=000000000066


             reply	other threads:[~2007-05-11 20:53 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-05-11 20:48 devzero [this message]
2007-05-13 13:06 ` [PATCH] Add LZO1X compression support to the kernel Jindrich Makovicka
  -- strict thread matches above, loose matches on Subject: below --
2007-05-12 10:41 devzero
2007-05-12 11:41 ` Jindrich Makovicka
2007-05-12 19:51   ` Satyam Sharma
2007-05-15 16:54 ` Pavel Machek
2007-05-10  8:49 Tomasz Chmielewski
2007-05-02  8:56 Richard Purdie
2007-05-02  9:35 ` Pekka Enberg
2007-05-04 18:28   ` Satyam Sharma
2007-05-10  6:21 ` Andrew Morton
2007-05-10  8:26   ` David Woodhouse
2007-05-10  8:53     ` Richard Purdie
2007-05-10 14:30   ` Jan Engelhardt
2007-05-12 11:17 ` Adrian Bunk
2007-05-12 15:17   ` Richard Purdie
2007-05-12 16:47     ` Andrew Morton
2007-05-12 20:56       ` Richard Purdie
2007-05-13 11:59         ` Markus F.X.J. Oberhumer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1783772957@web.de \
    --to=devzero@web.de \
    --cc=akpm@linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nitingupta.mail@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.