23 Aug 2016

48334

0

Google SoC lwIP Conclusion

My Google Summer of Code project proposal stated that I would add TCP support to the network branch of ReactOS, which sought to integrate lwIP 1.4.1 as the protocol level network driver for the operating system, to ultimately be tested by replacing the network driver in an installation of Windows Server 2003 with my driver. The full proposal can be found here. At the time of my proposal, I underestimated the amount of effort a fully working network driver would take. I spent a large chunk of time familiarizing myself with Windows' AFD/TDI driver framework as wll as lwIP before I could write maintainable, robust driver code. I documented my various struggles with AFD/TDI semantics and NT Interrupt Request Packets in the following blog posts.

Week 1

Week 2

Week 3

Week 4

Week 5

Week 6

Week 7

After an initial attempt at supporting TCP, I spent most of the rest of the summer grappling with lwIP's inherent lack of thread-safety in a totally asynchronous NT kernel environment. Not only did I struggle with lwIP's threading problems, I had to do some code restructuring to make my own data structures thread safe. The following blog posts document my discovery and resolution of these problems. 

Wekk 8

Week 9

Week 10

Week 11

Week 12

Getting a good grasp of both the NT kernel driver framework and lwIP took up the first seven weeks. Resolving multi-threading issues took the next five weeks. With only a single week left before the end of Summer of Code, I could not possibly finish full browser support, let alone a fully featured and robust driver that could replace the Windows Server 2003 driver. In the last week, I did as much as I could to flesh out UDP (which I inadvertantly broke) and TCP to the point of usability. 

Week 13

I definitely did not manage to finish my goals for this Summer of Code project that I laid out in the beginning. However, I did end up with a fairly robust TCP/IP driver capable of supporting C console applications that use TCP and UDP sockets. I managed to only extensively test it on loopback, but my tests proved that the TCP driver could handle an excessive number of short-lived user sockets on multiple threads without errors. 

The ReactOS project uses SVN for version control. As with other Summer of Code participants, ReactOS gave me my own working branch. I used two different branches, linked below. 

Initial branch, used until partway through the project

Final branch, used for the remainder of the project

All changes made after each branch's creation represent my work. Instructions for building and running ReactOS are available on the ReactOS wiki. The final branch, linked above, is a functional fork of ReactOS trunk where I replaced the majority of the contents of the drivers/network/tcpip/ directory. Most of the code in that directory was written before I started my project, and most of my modifications reside in address.c and address.h. All changes in other files were made to support changes made in address.c and address.h. 

This branch also includes the C console applications - "server.exe" and "client.exe" - that I used to test TCP functionality. The source code for those programs reside in the base/applications/network/ directory, in tcpclient/ and tcpserver/. They can be run by opening up two command consoles in ReactOS and typing "server" in one and "client" in the other. 

This driver is still incomplete and not able to handle full fledged browsers, since some TDI messages remain unimplemented or partially implemented. 

I would like to thank Amine Khaldi, Art Yerkes, Ziliang Guo, and Thomas Faber from ReactOS for supporting my development effort. 

This blog post represents the personal opinion of the author and is not representative of the position of the ReactOS Project.

The Blog Posts

Opinions, technical details, side projects or lovely kittens created directly by the ReactOS Devs.

Their opinions are theirs, so...well...any injuries, wounds, or dead-kittens due them, are..well..their own responsibility.

Visit the "Project News" for official statements.