[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] Xen 4.5 development update (September update). Feature freeze slip by two weeks.
- To: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
- From: Dave Scott <Dave.Scott@xxxxxxxxxx>
- Date: Tue, 7 Oct 2014 12:48:27 +0000
- Accept-language: en-GB, en-US
- Cc: "artem.mygaiev@xxxxxxxxxxxxxxx" <artem.mygaiev@xxxxxxxxxxxxxxx>, "ufimtseva@xxxxxxxxx" <ufimtseva@xxxxxxxxx>, "gross@xxxxxxxx" <gross@xxxxxxxx>, "christoffer.dall@xxxxxxxxxx" <christoffer.dall@xxxxxxxxxx>, "dongxiao.xu@xxxxxxxxx" <dongxiao.xu@xxxxxxxxx>, Stefano Stabellini <Stefano.Stabellini@xxxxxxxxxx>, "mengxu@xxxxxxxxxxxxx" <mengxu@xxxxxxxxxxxxx>, "chao.p.peng@xxxxxxxxxxxxxxx" <chao.p.peng@xxxxxxxxxxxxxxx>, "zhigang.x.wang@xxxxxxxxxx" <zhigang.x.wang@xxxxxxxxxx>, "parth.dixit@xxxxxxxxxx" <parth.dixit@xxxxxxxxxx>, "Paul.Skentzos@xxxxxxxxxxxxxxx" <Paul.Skentzos@xxxxxxxxxxxxxxx>, "vijay.kilari@xxxxxxxxx" <vijay.kilari@xxxxxxxxx>, "rcojocaru@xxxxxxxxxxxxxxx" <rcojocaru@xxxxxxxxxxxxxxx>, Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>, "guijianfeng@xxxxxxxxxxxxxx" <guijianfeng@xxxxxxxxxxxxxx>, "daniel.kiper@xxxxxxxxxx" <daniel.kiper@xxxxxxxxxx>, "josh.whitehead@xxxxxxxxxxxxxxx" <josh.whitehead@xxxxxxxxxxxxxxx>, Zoltan Kiss <zoltan.kiss@xxxxxxxxxx>, "avanzini.arianna@xxxxxxxxx" <avanzini.arianna@xxxxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>, "serge.broslavsky@xxxxxxxxxx" <serge.broslavsky@xxxxxxxxxx>, "yjhyun.yoo@xxxxxxxxxxx" <yjhyun.yoo@xxxxxxxxxxx>, "olaf@xxxxxxxxx" <olaf@xxxxxxxxx>, "Steve.Vande.rLeest@xxxxxxxxxxxxxxx" <Steve.Vande.rLeest@xxxxxxxxxxxxxxx>, Ian Campbell <Ian.Campbell@xxxxxxxxxx>, "wency@xxxxxxxxxxxxxx" <wency@xxxxxxxxxxxxxx>, "mcgrof@xxxxxxxx" <mcgrof@xxxxxxxx>, "julien.grall@xxxxxxxxxx" <julien.grall@xxxxxxxxxx>, George Dunlap <George.Dunlap@xxxxxxxxxx>, "robert.vanvossen@xxxxxxxxxxxxxxx" <robert.vanvossen@xxxxxxxxxxxxxxx>, "shantong.kang@xxxxxxxxx" <shantong.kang@xxxxxxxxx>, "roy.franz@xxxxxxxxxx" <roy.franz@xxxxxxxxxx>, "yang.z.zhang@xxxxxxxxx" <yang.z.zhang@xxxxxxxxx>, Paul Durrant <Paul.Durrant@xxxxxxxxxx>, "msw@xxxxxxxxxx" <msw@xxxxxxxxxx>, Ian Jackson <Ian.Jackson@xxxxxxxxxx>, Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>, "andrii.tseglytskyi@xxxxxxxxxxxxxxx" <andrii.tseglytskyi@xxxxxxxxxxxxxxx>, "jgross@xxxxxxxx" <jgross@xxxxxxxx>, Thomas Leonard <talex5@xxxxxxxxx>, Wei Liu <wei.liu2@xxxxxxxxxx>, Jonathan Ludlam <Jonathan.Ludlam@xxxxxxxxxx>, "Vijaya.Kumar@xxxxxxxxxxxxxxxxxx" <Vijaya.Kumar@xxxxxxxxxxxxxxxxxx>, "suriyan.r@xxxxxxxxx" <suriyan.r@xxxxxxxxx>, Dario Faggioli <dario.faggioli@xxxxxxxxxx>, Eddie Dong <eddie.dong@xxxxxxxxx>, "Kelly.Zytaruk@xxxxxxx" <Kelly.Zytaruk@xxxxxxx>, "dslutz@xxxxxxxxxxx" <dslutz@xxxxxxxxxxx>, Ross Lagerwall <ross.lagerwall@xxxxxxxxxx>, "Aravind.Gopalakrishnan@xxxxxxx" <Aravind.Gopalakrishnan@xxxxxxx>, David Vrabel <david.vrabel@xxxxxxxxxx>, "Suravee.Suthikulpanit@xxxxxxx" <Suravee.Suthikulpanit@xxxxxxx>, "aravindp@xxxxxxxxx" <aravindp@xxxxxxxxx>, "tiejun.chen@xxxxxxxxx" <tiejun.chen@xxxxxxxxx>, Malcolm Crossley <malcolm.crossley@xxxxxxxxxx>, "yanghy@xxxxxxxxxxxxxx" <yanghy@xxxxxxxxxxxxxx>, "tklengyel@xxxxxxxxxxxxx" <tklengyel@xxxxxxxxxxxxx>, "feng.wu@xxxxxxxxx" <feng.wu@xxxxxxxxx>, Roger Pau Monne <roger.pau@xxxxxxxxxx>
- Delivery-date: Tue, 07 Oct 2014 12:48:32 +0000
- List-id: Xen developer discussion <xen-devel.lists.xen.org>
- Thread-index: AQHP2KLAnpA9QPeGpUmP2KAJaN7u7JwSO/KAgBJLOAA=
- Thread-topic: [Xen-devel] Xen 4.5 development update (September update). Feature freeze slip by two weeks.
Hi Konrad,
On 25 Sep 2014, at 22:26, Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> wrote:
> On Thu, Sep 25, 2014 at 10:25:42AM +0100, George Dunlap wrote:
>> On 09/25/2014 10:22 AM, Dave Scott wrote:
>>> Hi Konrad,
>>>
>>> On 10 Sep 2014, at 18:05, <konrad.wilk@xxxxxxxxxx> <konrad.wilk@xxxxxxxxxx>
>>> wrote:
>>>
>>>> * extend the xenstore ring with a 'closing' signal (fair)
>>>> RFC patch posted
>>>> - David Scott
>>> I’d like to propose an exception for this one. The title is perhaps
>>> underselling it: although it is a protocol extension (and hence a new
>>> feature) the main motivation is to fix a bug in hvmloader which causes HVM
>>> guests to hang during booting on a busy machine. The bug is fairly easy to
>>> reproduce with ~200 VMs: you’ll probably find at least one failed to boot.
>>>
>>> The patch set has got to a v4 but needs a little bit of work to clarify the
>>> relationship between the closing signal and the existing RESET_WATCHES
>>> xenstore protocol request.
>
> It looked (from a brief look) as it has OCaml and I have no experience
> with that. Is there somebody who can review it?
>
> I hadn't dug in it to give it yet an opinion (sorry).
I’ve managed to gather an Acked-by from IanJ (NB without reviewing the OCaml
parts):
http://lists.xenproject.org/archives/html/xen-devel/2014-09/msg04255.html
On 26 Sep 2014, at 15:00, Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx> wrote:
> Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
>
> I'm happy to take this from you without really reviewing the ocaml
> code.
and I’ve got a positive review from Jon Ludlam (an OCaml expert)[added to cc:]
just now on the OCaml parts:
http://lists.xenproject.org/archives/html/xen-devel/2014-10/msg00748.html
On 7 Oct 2014, at 12:26, Jon Ludlam <jonathan.ludlam@xxxxxxxxxxxxx> wrote:
> Handling the exception higher up looks a fair bit nicer, and my previous
> comment has been addressed, so it looks fine to me.
>
> Reviewed-by: Jon Ludlam <jonathan.ludlam@xxxxxxxxxx>
FWIW I think this is still worth taking for 4.5 because
1. it fixes a real bug (HVM VMs occasionally failing to boot because of bad
hvmloader/xenstored interaction)
2. the oxenstored changes are quite unobtrusive: the flag is sampled at the
bottom, an exception bubbles all the way (almost) to the top and is handled
there. If something were to go wrong, that exception would be caught at the
main loop allowing the process to continue.
What do you think?
Cheers,
Dave
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel
|
