Re: [PATCH] news: Document new Xen hypervisor features

2020-04-23 Thread Andrea Bolognani
On Wed, 2020-04-22 at 11:55 -0600, Jim Fehlig wrote: > On 4/22/20 11:06 AM, Andrea Bolognani wrote: > > [1] Although it will never be a full rename, because it still need to > > handle libxl:// connection URLs. > > libxl:// was never supported. DV rightly pointed out that introducing >

Re: [PATCH] news: Document new Xen hypervisor features

2020-04-22 Thread Jim Fehlig
On 4/22/20 11:06 AM, Andrea Bolognani wrote: On Wed, 2020-04-22 at 10:43 -0600, Jim Fehlig wrote: On 4/22/20 9:10 AM, Andrea Bolognani wrote: 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

Re: [PATCH] news: Document new Xen hypervisor features

2020-04-22 Thread Andrea Bolognani
On Wed, 2020-04-22 at 10:43 -0600, Jim Fehlig wrote: > On 4/22/20 9:10 AM, Andrea Bolognani wrote: > > 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

Re: [PATCH] news: Document new Xen hypervisor features

2020-04-22 Thread Jim Fehlig
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: + + Xen: add support for e820_host hypervisor feature s/Xen: add/xen: Add/

Re: [PATCH] news: Document new Xen hypervisor features

2020-04-22 Thread Andrea Bolognani
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: > > > + > > > + Xen: add support for e820_host hypervisor feature > > > > s/Xen: add/xen: Add/ > > There is only one 'xen:'

Re: [PATCH] news: Document new Xen hypervisor features

2020-04-22 Thread Jim Fehlig
On 4/22/20 1:51 AM, Andrea Bolognani wrote: On Tue, 2020-04-21 at 16:06 -0600, Jim Fehlig wrote: + + 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.

Re: [PATCH] news: Document new Xen hypervisor features

2020-04-22 Thread Andrea Bolognani
On Tue, 2020-04-21 at 16:06 -0600, Jim Fehlig wrote: > + > + Xen: add support for e820_host hypervisor feature s/Xen: add/xen: Add/ I suggest also putting either single or double quotes around the name of the feature for clarity. > + > + > + e820_host

[PATCH] news: Document new Xen hypervisor features

2020-04-21 Thread Jim Fehlig
Signed-off-by: Jim Fehlig --- docs/news.xml | 22 ++ 1 file changed, 22 insertions(+) diff --git a/docs/news.xml b/docs/news.xml index 99c65447b0..074dc22e97 100644 --- a/docs/news.xml +++ b/docs/news.xml @@ -66,6 +66,28 @@ feedback from users on how well it works