[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v3 04/15] argo: init, destroy and soft-reset, with enable command line opt
On Mon, Jan 14, 2019 at 6:47 AM Wei Liu <wei.liu2@xxxxxxxxxx> wrote: > > Hi all > > The locking scheme seems to be remaining sticking point. The rest are > mostly cosmetic issues (FAOD, they still need to be addressed). Frankly > I don't think there is enough time to address all the technical details, > but let me sum up each side's position and see if we can reach an > amicable solution. > > From maintainers and reviewers' point of view: > > 1. Maintainers / reviewers don't like complexity unless absolutely > necessary. > 2. Maintainers / reviewers feel they have a responsibility to understand > the code and algorithm. > > Yet being the gatekeepers doesn't necessarily mean we understand every > technical details and every usecase. We would like to, but most of the > time it is unrealistic. > > Down to this specific patch series: > > Roger thinks the locking scheme is too complex. Christopher argues > that's necessary for short-live channels to be performant. > > Both have their point. > > I think having a complex locking scheme is inevitable, just like we did > for performant grant table several years ago. Regardless of the timing > issue we have at hand, asking Christopher to implement a stripped down > version creates more work for him. > > Yet ignoring Roger's concerns is unfair to him as well, since he put in > so much time and effort to understand the algorithm and provide > suggestions. It is in fact unreasonable to ask anyone to fully > understand the locking mechanism and check the implementation is correct > in a few days (given the series was posted in Dec and there were major > holidays in between, plus everyone had other commitments). > > To unblock this, how about we make Christopher maintainer of Argo? He > and OpenXT will be on the hook for further improvement. And I believe it > would be in their best interest to keep Argo bug-free and eventually > make it become supported. > > So: > > 1. Make sure Argo is self-contained -- this requires careful review for > interaction between Argo and other parts of the hypervisor. > 2. Argo is going to be experimental and off-by-default -- this is the > default status for new feature anyway. > 3. Make Christopher maintainer of Argo -- this would be a natural thing > to do anyway. > > Does this work for everyone? Yes, this will work for me. Thanks for the summary and proposal. Christopher _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxxxxxxxxx https://lists.xenproject.org/mailman/listinfo/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |