Re: [PATCH] news: Document new Xen hypervisor features
On 4/22/20 9:10 AM, Andrea Bolognani wrote:
On Wed, 2020-04-22 at 08:48 -0600, Jim Fehlig wrote:
> On 4/22/20 1:51 AM, Andrea Bolognani wrote:
>> On Tue, 2020-04-21 at 16:06 -0600, Jim Fehlig wrote:
>>> + <summary>
>>> + Xen: add support for e820_host hypervisor feature
>>
>> s/Xen: add/xen: Add/
>
> There is only one 'xen:' in news, all other instances are 'Xen:'. I
should stick
> with the majority.
If you look at most (all?) other entries, they look like
subsystem: Change
which is the same style we use for commit messages.
If you really want to split hairs, the subsytem is 'libxl'. But I don't think
that is good for news since in general folks know of the 'Xen' project, but not
'libxl'. However, now that the old xen driver is removed libxl could be renamed
to xen :-).
Xen seems to be the outlier when it comes to following this style,
both in release notes and to some extent in commit messages. It would
be great if it could fall in line with the rest of the code base.
Sorry for being out of line. I'll try to be more conscious of that in the future.
Regards,
Jim