Date: Tue, 23 Dec 2008 13:18:47 -0800 From: "Li, Qing" <qing.li@bluecoat.com> To: "Hartmut Brandt" <hartmut.brandt@dlr.de>, "Qing Li" <qingli@freebsd.org> Cc: freebsd-net@freebsd.org, freebsd-current@freebsd.org Subject: RE: HEADSUP: arp-v2 has been committed Message-ID: <B583FBF374231F4A89607B4D08578A4302A8BCC4@bcs-mail03.internal.cacheflow.com> In-Reply-To: <4950F770.3090700@dlr.de> References: <55f001c9639d$875f14ec$7202020a@internal.cacheflow.com> <4950F770.3090700@dlr.de>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Hartmut, I appreciate your candid feedback. You raised many valid points.=20 I combined both of your emails in this reply, please see my=20 comments below ... <snip> >=20 > Also one thing that would be extremly helpful is a short description of > that interface arp(4). Currently one has to reverse engineer arp.c to > understand how to do things. > This project has been in the making for quite a long time. I had a lot of content prepared to write in the commit message in my head, but when the moment final arrived in the end, my mind went blank perhaps due to that much anticipation. Funny how that works.=20 I will provide more text on this subject after things have settled down. >=20 > - this is code maintained in another repository and imported to > FreeBSD. Luckily the cvs-times are over where this commit would have > taken the files of the vendor branch. But nevertheless it is never a > good idea to change code in contrib without pushing the changes > upstream. >=20 This was due to my lack of understanding about the structure of that section of the repository. Thank you for providing this=20 information. > > - you just removed a lot of code and left the ipNetToMedia table > entirely disfunctional. >=20 > - you obviously did not test the change. Otherwise you would have seen > that it did not work. > No, I did not test that piece of code. There were so much to do in=20 the end, and having modified arp and ndp myself, I estimated the fix would not be difficult even if I broke it. So I took a calculated gamble. Thank you for fixing my bugs. > > And another point: when changing external interfaces it might be > possible to ask for a full port build with the changes to look for the > fall-out on ports. I would say that this commit was a good candidate to > get the port maintainers into the boat earlier. > > not so happy, > You are absolutely right. This was a complete oversight on my part. I was telling myself "I think I am forgetting something".=20 ... then I remembered when the first port breakage report arrived ;-) To be fair though, I did send a message titled=20 "last call for L2/L3 rewrite code review" a week before the commit to net@, current@ and all of the developers. And I have sent many emails on this subject in the past few years. A couple of points I hope you could recognize: 1. The arp-v2 project replaces a major networking kernel design and=20 all of its dependencies that have been in operation for many years=20 (16+ ??). The networking kernel went through quite a surgery so=20 do expect things will continue to evolve. 2. This is the first time I am making such a major change in the kernel. Since I am still learning the process, I am bound to make mistakes but I will not repeat these mistakes in the future. My goal is to be diligent in monitoring the problem reports and provide timely responses and fixes. And finally I want to thank you and others for your hard work in helping me cleaning up the ports. Thanks again. Cheers, --Qing
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?B583FBF374231F4A89607B4D08578A4302A8BCC4>