From: James Carlson <carlsonj@workingcode.com>
To: Martin Jackson <mjackson220.list@gmail.com>
Cc: paulus@samba.org, linux-ppp@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: ppp_deflate + kmalloc
Date: Fri, 24 Jun 2011 14:02:40 +0000 [thread overview]
Message-ID: <4E049900.4080402@workingcode.com> (raw)
In-Reply-To: <BANLkTi=+2dqr1LPwG8MHpcYOyzJH57iavw@mail.gmail.com>
Martin Jackson wrote:
> In our android froyo-based system (omap3 hardware), we are getting the
> following problem where the ppp driver cannot kmalloc enough memory
> for the decomp buffer in the ppp driver.
>
> Trying to make a 4th-order kmalloc (I think that amounts to 64kB)
> seems ambitious. I do not understand why vmalloc is not being used
> here, like it is for the compression buffer. Is using vmalloc here an
> acceptable solution?
The code here shouldn't need contiguous pages, so vmalloc (even if
"slower") shouldn't be a problem.
But a higher-level question might be why you're bothering with RFC 1979
Deflate compression at all on this platform. I'd expect that you're
most likely going to end up talking to commercially-produced PPP servers
(possibly 3GPP or similar) at the other end, and very, very few of them
offer data compression with either RFC 1977 (BSD Compression) or RFC
1979. ("Very, very few" is probably being generous ...)
If it's always going to be negotiated away in practice, and if you're
having trouble with memory constraints, why not just ditch the baggage?
--
James Carlson 42.703N 71.076W <carlsonj@workingcode.com>
WARNING: multiple messages have this Message-ID (diff)
From: James Carlson <carlsonj@workingcode.com>
To: Martin Jackson <mjackson220.list@gmail.com>
Cc: paulus@samba.org, linux-ppp@vger.kernel.org, netdev@vger.kernel.org
Subject: Re: ppp_deflate + kmalloc
Date: Fri, 24 Jun 2011 10:02:40 -0400 [thread overview]
Message-ID: <4E049900.4080402@workingcode.com> (raw)
In-Reply-To: <BANLkTi=+2dqr1LPwG8MHpcYOyzJH57iavw@mail.gmail.com>
Martin Jackson wrote:
> In our android froyo-based system (omap3 hardware), we are getting the
> following problem where the ppp driver cannot kmalloc enough memory
> for the decomp buffer in the ppp driver.
>
> Trying to make a 4th-order kmalloc (I think that amounts to 64kB)
> seems ambitious. I do not understand why vmalloc is not being used
> here, like it is for the compression buffer. Is using vmalloc here an
> acceptable solution?
The code here shouldn't need contiguous pages, so vmalloc (even if
"slower") shouldn't be a problem.
But a higher-level question might be why you're bothering with RFC 1979
Deflate compression at all on this platform. I'd expect that you're
most likely going to end up talking to commercially-produced PPP servers
(possibly 3GPP or similar) at the other end, and very, very few of them
offer data compression with either RFC 1977 (BSD Compression) or RFC
1979. ("Very, very few" is probably being generous ...)
If it's always going to be negotiated away in practice, and if you're
having trouble with memory constraints, why not just ditch the baggage?
--
James Carlson 42.703N 71.076W <carlsonj@workingcode.com>
next prev parent reply other threads:[~2011-06-24 14:02 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-24 13:41 ppp_deflate + kmalloc Martin Jackson
2011-06-24 13:41 ` Martin Jackson
2011-06-24 14:02 ` James Carlson [this message]
2011-06-24 14:02 ` James Carlson
2011-06-26 21:17 ` Martin Jackson
2011-06-26 21:17 ` Martin Jackson
2011-06-28 4:15 ` David Miller
2011-06-28 4:15 ` David Miller
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=4E049900.4080402@workingcode.com \
--to=carlsonj@workingcode.com \
--cc=linux-ppp@vger.kernel.org \
--cc=mjackson220.list@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=paulus@samba.org \
/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.