Hi, On Tue, 2019-12-10 at 05:43 -0800, Waldek Kozaczuk wrote: > Hi, > > I think that Nadav had some code review comments he hoped to get > resolved before he could apply the entire series. I think he did > apply couple of the very first and trivial ones though. I wonder if > some of the individual patches like this one can be applied piece by > piece. All in all, I would love to finally merge this code. I think > that Nadav mentioned some of the bug fixes that are part of it as > well that would be nice to merge.
Well, the v6 support is required to be taken seriously I guess ;) At the moment we have to accept v6 connections on the host and bounce to a v4 port on OSv. Horrible. Certainly some parts of Charles' patch set look like they could be applied independently. Taking this one, Nadav didn't have any objections - but then it just languished. > As I understand Spirent (so you and Charles, right) maintain a I'm nothing to do with Spirent - not sure why you have that impression. > separate fork of OSv - https://github.com/SpirentOrion/osv - which > has many fixes in networking stack beyond the IPV6 patches that > Charles sent, right? I would love to bring them on to the original > repo. Now we have some sort of an automated test framework that could > let us test the networking stack against over 20 or more workloads - > https://github.com/cloudius-systems/osv/wiki/Automated-Testing-Framework I do maintain my own separate tree because we have a slew of stuff that we have to have fixed (multicast mac addresses on the interfaces, the long ongoing clock problem). After the clock issue, I just wanted to press on with our stuff and if you're not interested in fixing things, so be it. Certainly I can see why Charles didn't come back. > Lastly as far as cloud-init and program options fallout. Indeed > cloud-init was the only app I did not remove program option in lieu > of our custom lighter core/options.cc (boost and this specific > library were giving us a lot of grief). If you have a patch handy > with those fixes to cloud-init please send me one and I will happily > apply. Understood. I'd missed the implications when you did your initial post on the subject. My patch is intertwined with the network-module stuff, which is why I was asking the question. Its easier to submit one patch than unroll them. Rick -- You received this message because you are subscribed to the Google Groups "OSv Development" group. To unsubscribe from this group and stop receiving emails from it, send an email to osv-dev+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/osv-dev/9c70ae22f022c59baa1f508ee8deeeb715a4a9d2.camel%40rossfell.co.uk.
