DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=gamma;
        h=subject:from:to:cc:in-reply-to:references:content-type:date
         :message-id:mime-version:x-mailer:content-transfer-encoding;
        b=wvnxTufCCukSPgWtjKzCeXKID/G8ZdsKS1SbGkLjGYbuXZAwVhA/5KRmZjHJkr01gp
         EQyiUhbsvyatfRreQIjSyNuOl4UBK1y3Ct5QnBroXkjdO4GriSHyNLXSGz8RgUDZChbY
         8wDQ0TntwnsBoX9b62wBUVZ8KHlV/BDrK9pJc=
Subject: Re: [PATCH 1/2] lib: add fast lzo decompressor
From: Andreas Robinson <andr345@gmail.com>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: John Stoffel <john@stoffel.org>, Alain Knaff <alain@knaff.lu>,
       linux-kernel@vger.kernel.org
In-Reply-To: <49D65700.5060202@zytor.com>
References: <1238593252-3435-1-git-send-email-andr345@gmail.com>
	 <1238593252-3435-2-git-send-email-andr345@gmail.com>
	 <49D3927A.2050406@zytor.com> <1238613730.10514.35.camel@andreas-desktop>
	 <18900.52255.798522.393088@stoffel.org>
	 <1238752196.8118.185.camel@andreas-desktop>  <49D65700.5060202@zytor.com>
Content-Type: text/plain
Date: Sat, 04 Apr 2009 16:34:36 +0200
Message-Id: <1238855676.18423.32.camel@andreas-desktop>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 903
Lines: 26

Hi,

after enabling checksumming (adler32) and adding an overrun check, based
on your suggestions and objections, the performance advantage of LZO on
the Eee 901 dropped to 67 ms with the fast, and 54 ms with the safe
decompressor. Apparently almost half of the speed boost came from
skipping the checksumming.

At this point I don't think this idea is worth pursuing further, when
LZO is so close to gzip in performance after all.

But I want thank you for discussing this topic and taking my patch
submission seriously.

I hope I did not waste too much of your time and that I will be able to
contribute something more useful next time.

Thanks,
Andreas


--
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/