|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v6 for Xen 4.7 3/4] libxl: enable per-VCPU parameter settings for RTDS scheduler
On Tue, 2016-03-08 at 19:12 +0000, Wei Liu wrote: > Overall I think the patch is moving towards the right direction. > I also think so. Thanks a lot Wei for the review, BTW. > Just > that there are too many places where indentation need fixing. Please > fix them in the next iteration. I really don't like holding back > patches > just because of indentation issues. > Exactly. Trying to elaborate a bit more, this series does not include any complex algorithm or similar, so one may thing that it is not that hard to review. But it is indeed complex and hard to review, because the patches are rather big and because the API and the command line syntax we want to support is complex. If there are too many style issue, any review will likely end up focusing mostly, if not only, on them, for various reasons. E.g., when one finds a stile issue, avoiding commenting on it (e.g., because one wants to focus on "more important" aspects) means risking forgetting about it and, in the end, letting it hit the repository (if others also miss it or does the same). Also, we're all used to look at code that (well, mostly :-D) conforms to coding style, so it's harder to focus on code that does not. And more. Add to this that the most difficult part of the tools side of this series (like API and data structures) is actually ok. So, Chong, for us to be able to quickly and effectively help you forward, we need to ask you to do your best and get rid of (ideally) all coding style problems. Once that's done, we're not far from calling this all a done deal. :-) Thanks and Regards, Dario -- <<This happens because I choose it to happen!>> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) Attachment:
signature.asc _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
 |
Lists.xenproject.org is hosted with RackSpace, monitoring our |