On 2/15/06, Ian McDonald <[EMAIL PROTECTED]> wrote: > On 2/16/06, Arnaldo Carvalho de Melo <[EMAIL PROTECTED]> wrote: > > Hi Ian, Andrea, > > > > Can somebody please poke holes in the attached patch? It moves > > more code out of the > > CCIDs into the CCID infrastructure, simplifying some stuff. > > > > With this rework ccid_operations becomes the struct proto > > counterpart in ccid land (struct sock has a "operations" pointer to > > struct proto). > > > > I saw the need for this when working on Andrea's ccid feature > > negotiation patch, that I'm > > now reworking to take advantage of the attached patch. > > > As a baseline I downloaded acme-net-2.6.17.git and did some testing. > When I test with netem in the middle between two boxes the transmitter > locks solid.... > > Test > ./ttcp_acme -c -l500 -n100 -t jandi2 > ./ttcp_acme -c -l500 -r > > Netem configured as follows in middle box: > sudo tc qdisc add dev lan1 root netem delay 50ms loss 10% > sudo tc qdisc add dev lan0 root netem delay 50ms loss 10% > > I don't believe this happens with older version of DCCP (just pre > CCID2) so I will start trying to track down.... > > I don't have so much time so may take a little while. Feel free to > investigate earlier if you wish. > > As such I haven't tested the patch as baseline not stable... >
http://master.kernel.org/git/?p=linux/kernel/git/acme/net-2.6.17.git;a=shortlog;h=ccid_operations I tested with the code that is in the "ccid_operations" of my net-2.6.17 git tree (can you give it a try? Perhaps I did some last minute fix over the one you tested), but without a middle box, tomorrow I'll revive this parisc and use another machine to test on real machines, not just qemu. - Arnaldo - To unsubscribe from this list: send the line "unsubscribe dccp" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
