[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/3] qemu_chr_new_from_opts: do not overwrite ex
From: |
Stefano Stabellini |
Subject: |
Re: [Qemu-devel] [PATCH 1/3] qemu_chr_new_from_opts: do not overwrite existing chr->filename |
Date: |
Tue, 28 May 2013 11:43:24 +0100 |
User-agent: |
Alpine 2.02 (DEB 1266 2009-07-14) |
On Tue, 28 May 2013, Gerd Hoffmann wrote:
> On 05/27/13 20:13, Stefano Stabellini wrote:
> > Set the new filename only if it hasn't been already set (for example by
> > qemu_chr_open_pty).
> >
> > This fixes a regression that prevents Xen HVM domains from booting.
> > It was introduced by:
> >
> > commit 2b220025993e76d4116781ca91a4fabc5ad9c722
> > Author: Lei Li <address@hidden>
> > Date: Tue May 21 18:27:59 2013 +0800
>
> FYI: just posted a pull req which reverts this one (you are cc'ed).
Right, your two patches also solve the problem I was seeing.
I'll drop the first patch of my series.