Bernd Petrovitsch wrote:
> The "register" keyword is and was always from start *at most* a hint to
> the C compiler to use a register for that variable (similar to "inline"
> BTW).
> So every C compiler is allowed to simply ignore the "register" for any
> reason - be it "not implemented" or "the co
On Tue, 2007-05-22 at 14:38 +0530, Nitin Gupta wrote:
[...]
> On 5/18/07, Andrey Panin <[EMAIL PROTECTED]> wrote:
> > On 138, 05 18, 2007 at 03:28:31PM +0530, Nitin Gupta wrote:
> > > + register const unsigned char *ip;
> >
> > register keyword is meaningless for today's compiler.
>
> But can
On May 22 2007 14:38, Nitin Gupta wrote:
> On 5/18/07, Andrey Panin <[EMAIL PROTECTED]> wrote:
>> On 138, 05 18, 2007 at 03:28:31PM +0530, Nitin Gupta wrote:
>> > + register const unsigned char *ip;
>>
>> register keyword is meaningless for today's compiler.
>
> But can we assume that gcc is
On May 18 2007 15:28, Nitin Gupta wrote:
> +/* lzo1x.h -- public interface of the LZO1X compression algorithm
> +
> + This file is part of the LZO real-time data compression library.
> +
> + Copyright (C) 2005 Markus Franz Xaver Johannes Oberhumer
> + Copyright (C) 2004 Markus Franz Xaver Jo
Nitin Gupta wrote:
Ok. I will make them inline functions now.
Btw, the only reason why any sane compiler would not inline
them is because it has reason to believe the end-result will
be more efficient. AFAIK you only need to use
__always_inline where it matters for correctness (like, for
exampl
Hi,
On 5/18/07, Richard Purdie <[EMAIL PROTECTED]> wrote:
> This patch, as of yet, only gives 'non-safe' version of decompressor.
> The 'safe' version will be included soon.
How are you planning to add that back?
Please see newer patch posted.
The LZO author had some concerns about this c
On 5/22/07, Pekka Enberg <[EMAIL PROTECTED]> wrote:
Nitin Gupta wrote:
> What if compiler decides not to actully inline them? In that case
> there will be significant perf. hit.
Then you use __always_inline.
Pekka
Ok. I will make them inline functions now.
Thanks,
Ni
Hi,
On 5/18/07, Andrey Panin <[EMAIL PROTECTED]> wrote:
On 138, 05 18, 2007 at 03:28:31PM +0530, Nitin Gupta wrote:
> + register const unsigned char *ip;
register keyword is meaningless for today's compiler.
But can we assume that gcc is being used? What if we use compiler for
which it do
Nitin Gupta wrote:
What if compiler decides not to actully inline them? In that case
there will be significant perf. hit.
Then you use __always_inline.
Pekka
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PRO
Hi,
On 5/18/07, Pekka Enberg <[EMAIL PROTECTED]> wrote:
On 5/18/07, Nitin Gupta <[EMAIL PROTECTED]> wrote:
> +#define DX2(p,s1,s2) \
> + (size_t)((p)[2]) << (s2)) ^ (p)[1]) << (s1)) ^ (p)[0])
> +#define DX3(p,s1,s2,s3) ((DX2((p)+1,s2,s3) << (s1)) ^ (p)[0])
> +#define DMUL(a,b) ((s
Bill Rugolsky Jr. wrote:
I'm certainly missing something but what are the advantages of this
code (over current gzip etc.), and what will be using it?
Richard's patchset added it to the crypto library and wired it into
the JFFS2 file system. We recently started using LZO in a userland UDP
pro
On Sat, 2007-05-19 at 14:55 -0400, Bill Rugolsky Jr. wrote:
> On Fri, May 18, 2007 at 11:14:57PM +0200, Krzysztof Halasa wrote:
> > I'm certainly missing something but what are the advantages of this
> > code (over current gzip etc.), and what will be using it?
>
> Richard's patchset added it to t
On Fri, May 18, 2007 at 11:14:57PM +0200, Krzysztof Halasa wrote:
> I'm certainly missing something but what are the advantages of this
> code (over current gzip etc.), and what will be using it?
Richard's patchset added it to the crypto library and wired it into
the JFFS2 file system. We recentl
>I'm certainly missing something but what are the advantages of this
>code (over current gzip etc.), and what will be using it?
lzo compresses/decompresses much faster and using less cpu
this is how it compares:
bzip2: best compression, but damn slow performance
gzip: good compression with good
"Nitin Gupta" <[EMAIL PROTECTED]> writes:
> Facts for LZO (at least for original code. Should hold true for this
> port also - hence the RFC!):
> - The compressor can never overrun buffer.
> - The "non-safe" version of decompressor can never overrun buffer if
> compressed data is unmodified. I am
On Fri, May 18, 2007 at 03:28:31PM +0530, Nitin Gupta wrote:
> +/* lzo1x.h -- public interface of the LZO1X compression algorithm
> +
> + This file is part of the LZO real-time data compression library.
> +
> + Copyright (C) 2005 Markus Franz Xaver Johannes Oberhumer
> + Copyright (C) 2004 Ma
On 138, 05 18, 2007 at 03:28:31PM +0530, Nitin Gupta wrote:
> Hi,
>
> This is kernel port of LZO1X de/compression algo stripped down to just ~500
> LOC!
> It is derived from original LZO 2.02 code found at:
> http://www.oberhumer.com/opensource/lzo/download/
> The code has also been reformatted t
On Fri, 2007-05-18 at 16:57 +0530, Nitin Gupta wrote:
> On 5/18/07, Heikki Orsila <[EMAIL PROTECTED]> wrote:
> > Good work..
> >
> > On Fri, May 18, 2007 at 03:28:31PM +0530, Nitin Gupta wrote:
> > > Facts for LZO (at least for original code. Should hold true for this
> > > port also - hence the RF
Hi,
On Fri, 2007-05-18 at 15:28 +0530, Nitin Gupta wrote:
> This is kernel port of LZO1X de/compression algo stripped down to just ~500
> LOC!
> It is derived from original LZO 2.02 code found at:
> http://www.oberhumer.com/opensource/lzo/download/
> The code has also been reformatted to match ge
Hi,
Thanks for review. My comments inline.
On 5/18/07, Heikki Orsila <[EMAIL PROTECTED]> wrote:
Good work..
On Fri, May 18, 2007 at 03:28:31PM +0530, Nitin Gupta wrote:
> Facts for LZO (at least for original code. Should hold true for this
> port also - hence the RFC!):
> - The compressor can
On 5/18/07, Pekka Enberg <[EMAIL PROTECTED]> wrote:
So how about making that a little less verbose. Say like:
Copyright (c) 1996-2005 Markus Franz Xaver and Johannes Oberhumer
Oh, the author's name really is "Markus Franz Xaver Johannes
Oberhumer." But please make the copyright statement one l
On 5/18/07, Nitin Gupta <[EMAIL PROTECTED]> wrote:
+ Copyright (C) 2005 Markus Franz Xaver Johannes Oberhumer
+ Copyright (C) 2004 Markus Franz Xaver Johannes Oberhumer
[snip]
So how about making that a little less verbose. Say like:
Copyright (c) 1996-2005 Markus Franz Xaver and Johannes
Good work..
On Fri, May 18, 2007 at 03:28:31PM +0530, Nitin Gupta wrote:
> Facts for LZO (at least for original code. Should hold true for this
> port also - hence the RFC!):
> - The compressor can never overrun buffer.
> - The "non-safe" version of decompressor can never overrun buffer if
> compr
Hi,
This is kernel port of LZO1X de/compression algo stripped down to just ~500 LOC!
It is derived from original LZO 2.02 code found at:
http://www.oberhumer.com/opensource/lzo/download/
The code has also been reformatted to match general kernel style.
Facts for LZO (at least for original code.
24 matches
Mail list logo
