2009-11-17 11:59:03

by Franko Fang

[permalink] [raw]
Subject: [PATCH 1/5]net: PPP buffer too small for higher speed connections

From: fangxiaozhi <[email protected]>
1. This patch is based on the kernel of 2.6.32-rc7
2. In this patch, we enlarge the out buffer size to optimize the upload speed for the ppp connection. Then it can support the upload of HSUPA data cards.
Signed-off-by: fangxiaozhi <[email protected]>
-----------------------------------------------------------------------------------------
--- a/drivers/net/ppp_async.c 2009-10-12 05:43:56.000000000 +0800
+++ b/drivers/net/ppp_async.c 2009-10-15 16:29:56.000000000 +0800
@@ -36,7 +36,7 @@

#define PPP_VERSION "2.4.2"

-#define OBUFSIZE 256
+#define OBUFSIZE 4096

/* Structure for storing local state. */
struct asyncppp {


******************************************************************************************
This email and its attachments contain confidential information from HUAWEI, which is intended only for the person or entity whose address is listed above. Any use of the information contained here in any way (including, but not limited to, total or partial disclosure, reproduction, or dissemination) by persons other than the intended recipient(s) is prohibited. If you receive this email in error, please notify the sender by phone or email
immediately and delete it!
*****************************************************************************************


2009-11-17 12:02:19

by David Miller

[permalink] [raw]
Subject: Re: [PATCH 1/5]net: PPP buffer too small for higher speed connections

From: fangxiaozhi 00110321 <[email protected]>
Date: Tue, 17 Nov 2009 19:59:05 +0800

> From: fangxiaozhi <[email protected]>
> 1. This patch is based on the kernel of 2.6.32-rc7
> 2. In this patch, we enlarge the out buffer size to optimize the upload speed for the ppp connection. Then it can support the upload of HSUPA data cards.
> Signed-off-by: fangxiaozhi <[email protected]>

Applied to net-next-2.6, thanks.

2009-11-17 12:04:08

by Franko Fang

[permalink] [raw]
Subject: Re: [PATCH 1/5]net: PPP buffer too small for higher speed connections

Well, that's great.

Thanks very much for your help.
----- Original Message -----
From: "David Miller" <[email protected]>
To: <[email protected]>
Cc: <[email protected]>; <[email protected]>; <[email protected]>; <[email protected]>
Sent: Tuesday, November 17, 2009 8:02 PM
Subject: Re: [PATCH 1/5]net: PPP buffer too small for higher speed connections


> From: fangxiaozhi 00110321 <[email protected]>
> Date: Tue, 17 Nov 2009 19:59:05 +0800
>
>> From: fangxiaozhi <[email protected]>
>> 1. This patch is based on the kernel of 2.6.32-rc7
>> 2. In this patch, we enlarge the out buffer size to optimize the upload speed for the ppp connection. Then it can support the upload of HSUPA data cards.
>> Signed-off-by: fangxiaozhi <[email protected]>
>
> Applied to net-next-2.6, thanks.

2009-11-17 17:20:57

by Lennart Sorensen

[permalink] [raw]
Subject: Re: [PATCH 1/5]net: PPP buffer too small for higher speed connections

On Tue, Nov 17, 2009 at 07:59:05PM +0800, fangxiaozhi 00110321 wrote:
> From: fangxiaozhi <[email protected]>
> 1. This patch is based on the kernel of 2.6.32-rc7
> 2. In this patch, we enlarge the out buffer size to optimize the upload speed for the ppp connection. Then it can support the upload of HSUPA data cards.
> Signed-off-by: fangxiaozhi <[email protected]>
> -----------------------------------------------------------------------------------------
> --- a/drivers/net/ppp_async.c 2009-10-12 05:43:56.000000000 +0800
> +++ b/drivers/net/ppp_async.c 2009-10-15 16:29:56.000000000 +0800
> @@ -36,7 +36,7 @@
>
> #define PPP_VERSION "2.4.2"
>
> -#define OBUFSIZE 256
> +#define OBUFSIZE 4096
>
> /* Structure for storing local state. */
> struct asyncppp {

I don't know what an HSUPA connection is, so what kind of speed is that?
I am just wondering if this would affect ppp on other connections that
are async (in a positive manner that is).

--
Len Sorensen

2009-11-17 20:35:50

by Dan Williams

[permalink] [raw]
Subject: Re: [PATCH 1/5]net: PPP buffer too small for higher speed connections

On Tue, 2009-11-17 at 12:21 -0500, Lennart Sorensen wrote:
> On Tue, Nov 17, 2009 at 07:59:05PM +0800, fangxiaozhi 00110321 wrote:
> > From: fangxiaozhi <[email protected]>
> > 1. This patch is based on the kernel of 2.6.32-rc7
> > 2. In this patch, we enlarge the out buffer size to optimize the upload speed for the ppp connection. Then it can support the upload of HSUPA data cards.
> > Signed-off-by: fangxiaozhi <[email protected]>
> > -----------------------------------------------------------------------------------------
> > --- a/drivers/net/ppp_async.c 2009-10-12 05:43:56.000000000 +0800
> > +++ b/drivers/net/ppp_async.c 2009-10-15 16:29:56.000000000 +0800
> > @@ -36,7 +36,7 @@
> >
> > #define PPP_VERSION "2.4.2"
> >
> > -#define OBUFSIZE 256
> > +#define OBUFSIZE 4096
> >
> > /* Structure for storing local state. */
> > struct asyncppp {
>
> I don't know what an HSUPA connection is, so what kind of speed is that?

(Google and Wikipedia know everything and are only a click away...)

High-Speed Uplink Packet Access is a enhancement for around 5.7Mbps
uplinks on mobile broadband networks based on GSM/UMTS standards.

In the end, what all the mobile manufacturers like Huawei should already
be doing is converting their devices to use an AT-based control port and
a netdev-based data port like Option (hso) and Ericsson (cdc-acm and
cdc-ether) already have. Then we don't need to do the useless PPP
session between the host and the card. With GSM/UMTS/HSPA, PPP never
goes over the air, it's purely between the card and the host. And thus
is pointless and a netdev-type architecture would work a lot better.

Dan

> I am just wondering if this would affect ppp on other connections that
> are async (in a positive manner that is).
>