On 16-06-21, 10:31, Linus Walleij wrote: > Hi Enrico, > > On Tue, Jun 15, 2021 at 7:49 PM Enrico Weigelt, metux IT consult > <info@xxxxxxxxx> wrote: > > > Introduce new GPIO driver for virtual GPIO devices via virtio. > > > > The driver implements the virtio-gpio protocol (ID 41), which can be > > used by either VM guests (e.g. bridging virtual gpios from the guest > > to real gpios in the host or attaching simulators for automatic > > application testing), as well as virtio-gpio hardware devices. > > > > Signed-off-by: Enrico Weigelt, metux IT consult <info@xxxxxxxxx> > > So now there are two contesting patches for this and that creates a > social problem for us as maintainers. I am not too happy about that. > > This situation activates the kernel management style document so > I advise involved parties to familiarize themselves with it: > https://www.kernel.org/doc/html/latest/process/management-style.html > > Can we get the discussion down to actual technical points? +1 I can not agree more to this. > We really need a virtio GPIO driver, no doubt, so if everyone could > just work toward that goal and compromise with their specific pet > peeves that would be great. Enrico, I am not looking to get any credits for the code or spec here. I don't really care about that. For the very same reason I kept you as the author of the 1st patch in the kernel series, so git keeps showing you as the original author. All I wanted to work on was the backend (in rust). This is what happened for I2C for example, Jie Deng (Intel) worked on the spec and Linux driver and I helped review it, make him fix a thing or two and that's all. I worked on the rust implementation for the backend then. You only ever sent 1 real versions of the Linux driver, that too "6-months-back", there were no real blockers anywhere and you never attempted to upstream anything. Similarly, you "never" sent the specification properly to the virtio lists for review. You sent it once as an attachment to an email, which no one ever received. When I tried to move this forward, invested a lot of time into making it better from specification to code, reviews started happening, you decided to start blocking it again. You should be rather happy that something that you worked on is making progress, specially when you didn't get time to do the same. You wrote this in your patch: > > Status: > > * this driver is now field tested for about 6 month > > (against KVM+Qemu as well as some HW/FPGA implementation) Linux upstream doesn't really care about this, you can ask any Linux Maintainer about this. If your code and specification isn't doing the right thing, and isn't good enough, you will be asked to update it upon reviews. YOU JUST CAN'T SAY I WON'T because I have products based on this version. This is not how any open source project works. The code and specification here doesn't belong to a single person or company. It is for everyone's use. > > * virtio device ID officially allocated Correct. > > * virtio spec has been submitted to virtio TC Which specification are you talking about here ? The only specification I can see on virtio lists is the one I sent. And the driver you tried to send isn't aligned to that for sure, and it takes us back from all the improvements I tried to do. I am not saying that my version of the specification is the best and there is no flaw in it. There surely is, but that can't be fixed by sending another version of it. You need to make a technical point about it and convince people that what you are saying is correct and it is required for your use-case (not existing downstream solution). There is no point going backwards now after making so much of progress. Even if you try to send your version, it will slowly and slowly reach very close to my latest version of code and spec. Because your version of the code and spec weren't good enough for everyone. It doesn't matter if you have real products on your earlier version, you can keep using that in your downstream solution, but Linux kernel and specification are going to get an improved version (from yours or mine, but that doesn't matter here). You need to accept that changes to that are inevitable since there are many users of gpio-virtio, not just you and me, but many more (Like Bjorn expressed his interest in this today for Qcom stuff). Again, it would be better if you can discuss further on technical merits or demerits in the currently circulated specification and give your invaluable suggestions on the same. Else we will end up spending few more months with just this and it won't get us anywhere. Thanks. -- viresh _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linuxfoundation.org/mailman/listinfo/virtualization
