TODO: Rewrite network stack.

TODO: Rewrite network stack.

[Antony Pavlov]

Hi!

I have inspected TODO file from barebox repo. Here is a quote:

  [ ] Rewrite network stack. The one-loop-for-all-protocols stack is just too
      ugly and big and uses thousands of global variables. Identify common
      functions and make a loop per network protocol

There is no much tiny TCP/IP stack candidates for porting to barebox.
The most notable is lwIP (http://savannah.nongnu.org/projects/lwip/).

See "project that use lwIP" list here: http://lwip.wikia.com/wiki/Projects_that_use_lwIP

lwip code is BSD-licensed. Is it possible to use lwip code in barebox?

-- 
Best regards,
  Antony Pavlov

P.S. Anyway TODO file from barebox repo is outdated (last change Jul 22 2010).

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: TODO: Rewrite network stack.

[Alessandro Rubini]

> The most notable is lwIP (http://savannah.nongnu.org/projects/lwip/).

I'm not happy at all about this code base, it looks quite ugly and
old-fashioned. It's not the barebox kind of quality. I'd stay with the
current code rather than lwip, despite the number of users.

If any, I'd evaluate picotcp (https://github.com/tass-belgium/picotcp).

/alessandro

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: TODO: Rewrite network stack.

[Antony Pavlov]

On Mon, 5 May 2014 14:29:44 +0200
Alessandro Rubini <rubini-list@gnudd.com> wrote:

> > The most notable is lwIP (http://savannah.nongnu.org/projects/lwip/).
> 
> I'm not happy at all about this code base, it looks quite ugly and
> old-fashioned. It's not the barebox kind of quality. I'd stay with the
> current code rather than lwip, despite the number of users.
> 
> If any, I'd evaluate picotcp (https://github.com/tass-belgium/picotcp).

Thanks for very interesting picotcp link!

At a glance picotcp looks better than lwip.

-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: TODO: Rewrite network stack.

[Sascha Hauer]

On Mon, May 05, 2014 at 02:55:12PM +0400, Antony Pavlov wrote:
> Hi!
> 
> I have inspected TODO file from barebox repo. Here is a quote:
> 
>   [ ] Rewrite network stack. The one-loop-for-all-protocols stack is just too
>       ugly and big and uses thousands of global variables. Identify common
>       functions and make a loop per network protocol
> 
> There is no much tiny TCP/IP stack candidates for porting to barebox.
> The most notable is lwIP (http://savannah.nongnu.org/projects/lwip/).
> 
> See "project that use lwIP" list here: http://lwip.wikia.com/wiki/Projects_that_use_lwIP
> 
> lwip code is BSD-licensed. Is it possible to use lwip code in barebox?
> 
> -- 
> Best regards,
>   Antony Pavlov
> 
> P.S. Anyway TODO file from barebox repo is outdated (last change Jul 22 2010).

Indeed. Maybe we should add the following item to it.

Sascha

diff --git a/TODO b/TODO
index 6350799..59fdade 100644
--- a/TODO
+++ b/TODO
@@ -76,6 +76,7 @@ TODO
     devices should be represented as a hierarchical tree in /dev
 [ ] Work out, how to mount driver on PCI devices. The mechanism should be
     transparent to the driver code.
+[ ] remove this file. It doesn't contain much useful information.
 
 DONE
 ----

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: TODO: Rewrite network stack.

[Uwe Kleine-König]

On Mon, May 05, 2014 at 03:22:40PM +0200, Sascha Hauer wrote:
> On Mon, May 05, 2014 at 02:55:12PM +0400, Antony Pavlov wrote:
> > Hi!
> > 
> > I have inspected TODO file from barebox repo. Here is a quote:
> > 
> >   [ ] Rewrite network stack. The one-loop-for-all-protocols stack is just too
> >       ugly and big and uses thousands of global variables. Identify common
> >       functions and make a loop per network protocol
> > 
> > There is no much tiny TCP/IP stack candidates for porting to barebox.
> > The most notable is lwIP (http://savannah.nongnu.org/projects/lwip/).
> > 
> > See "project that use lwIP" list here: http://lwip.wikia.com/wiki/Projects_that_use_lwIP
> > 
> > lwip code is BSD-licensed. Is it possible to use lwip code in barebox?
> > 
> > -- 
> > Best regards,
> >   Antony Pavlov
> > 
> > P.S. Anyway TODO file from barebox repo is outdated (last change Jul 22 2010).
> 
> Indeed. Maybe we should add the following item to it.
> 
> Sascha
> 
> diff --git a/TODO b/TODO
> index 6350799..59fdade 100644
> --- a/TODO
> +++ b/TODO
> @@ -76,6 +76,7 @@ TODO
>      devices should be represented as a hierarchical tree in /dev
>  [ ] Work out, how to mount driver on PCI devices. The mechanism should be
>      transparent to the driver code.
> +[ ] remove this file. It doesn't contain much useful information.
I took this patch, removed the file and wanted to move this entry below
DONE. This resulted in a git conflict. Please advise.

Uwe

>  
>  DONE
>  ----
> 
> -- 
> Pengutronix e.K.                           |                             |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
> Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: TODO: Rewrite network stack.

[Jean-Christophe PLAGNIOL-VILLARD]

On 14:29 Mon 05 May     , Alessandro Rubini wrote:
> 
> > The most notable is lwIP (http://savannah.nongnu.org/projects/lwip/).
> 
> I'm not happy at all about this code base, it looks quite ugly and
> old-fashioned. It's not the barebox kind of quality. I'd stay with the
> current code rather than lwip, despite the number of users.
> 
> If any, I'd evaluate picotcp (https://github.com/tass-belgium/picotcp).

uIP  too

Best Regards,
J.
> 
> /alessandro
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: TODO: Rewrite network stack.

[Antony Pavlov]

On Thu, 8 May 2014 14:43:15 +0200
Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com> wrote:

> On 14:29 Mon 05 May     , Alessandro Rubini wrote:
> > 
> > > The most notable is lwIP (http://savannah.nongnu.org/projects/lwip/).
> > 
> > I'm not happy at all about this code base, it looks quite ugly and
> > old-fashioned. It's not the barebox kind of quality. I'd stay with the
> > current code rather than lwip, despite the number of users.
> > 
> > If any, I'd evaluate picotcp (https://github.com/tass-belgium/picotcp).
> 
> uIP  too

Hmm, uIP looks like a light version of lwip or have I missed something?

> 
> Best Regards,
> J.
> > 
> > /alessandro
> > 
> > _______________________________________________
> > barebox mailing list
> > barebox@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/barebox


-- 
-- 
Best regards,
  Antony Pavlov

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox