[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH v3 7/7] xl: docs for xl config vnuma options



On 11/19/2013 05:23 PM, Dario Faggioli wrote:
Again, only one quick note (for now).

On lun, 2013-11-18 at 15:24 -0500, Elena Ufimtseva wrote:
xl: docs for xl config vnuma options

Changes to the documentation should happen in the same patch that
introduces the feature or modifies the behavior.

Although there is a certain logic to that, if there are major documentation changes, I find that having the documentation in the same exact patch makes the patch a lot harder to review. I much prefer to have the documentation happen at the end of the patch series, unless it's something really minor. (Although I generally won't complain either way.)

 -George

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.