|To:||Julius Volz <juliusv@xxxxxxxxxx>|
|Subject:||Re: [PATCH 00/26] IPVS: Add first IPv6 support to IPVS.|
|From:||Patrick McHardy <kaber@xxxxxxxxx>|
|Date:||Wed, 11 Jun 2008 20:42:09 +0200|
Julius Volz wrote:
On Wed, Jun 11, 2008 at 7:23 PM, Patrick McHardy <kaber@xxxxxxxxx> wrote:I briefly looked over the patches I didn't comment on. I think there's too much duplication everywhere, a lot of them look like they could avoid almost all duplication by handling differences at a higher layer or simply sharing the code (like hashing).Yes, the duplication is high unfortunately. I must admit that I didn't feel secure enough to restructure all the existing code without breaking it, so I copied lots of functions and modified them for IPv6. My main goal was to keep all the old v4 stuff working first and then remove the duplication later (or hope for smarter people). So I obviously don't expect this to be ready for inclusion, but I will have a lot of time to work on it (I'm doing it as an intern project) and learn as long as I get good feedback like yours on what to improve.
Another question I was unsure about: is the breaking of the userspace-to-kernel interface even acceptable at all? I think the code would get ugly (and have even more duplication) if you wanted to keep the backwards compatibility. And you have to compile ipvsadm for your kernel version anyways.
Usually its not acceptable. Why do you have compile ipvsadm for specific kernel versions? -- To unsubscribe from this list: send the line "unsubscribe lvs-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
|<Prev in Thread]||Current Thread||[Next in Thread>|
|Previous by Date:||Re: [PATCH 00/26] IPVS: Add first IPv6 support to IPVS., Julius Volz|
|Next by Date:||Re: [PATCH 00/26] IPVS: Add first IPv6 support to IPVS., Julius Volz|
|Previous by Thread:||Re: [PATCH 00/26] IPVS: Add first IPv6 support to IPVS., Julius Volz|
|Next by Thread:||Re: [PATCH 00/26] IPVS: Add first IPv6 support to IPVS., Julius Volz|
|Indexes:||[Date] [Thread] [Top] [All Lists]|