[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU
From: |
Izik Eidus |
Subject: |
Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository |
Date: |
Thu, 31 Aug 2017 15:01:56 +0300 |
On Thu, Aug 31, 2017 at 2:26 PM, Paolo Bonzini <address@hidden> wrote:
>
>
> Il 31 ago 2017 9:43 AM, "Stefan Hajnoczi" <address@hidden> ha scritto:
>
> On Wed, Aug 30, 2017 at 03:07:38PM +0100, Daniel P. Berrange wrote:
> > On Wed, Aug 30, 2017 at 03:26:51AM -0500, Sergio Andres Gomez Del Real
> wrote:
> > > diff --git a/target/i386/hvf-utils/x86.c b/target/i386/hvf-utils/x86.c
> > > new file mode 100644
> > > index 0000000000..e3db2c9c8b
> > > --- /dev/null
> > > +++ b/target/i386/hvf-utils/x86.c
> > > @@ -0,0 +1,174 @@
> > > +/*
> > > + * Copyright (C) 2016 Veertu Inc,
> > > + * Copyright (C) 2017 Google Inc,
> > > + *
> > > + * This program is free software; you can redistribute it and/or
> > > + * modify it under the terms of the GNU General Public License as
> > > + * published by the Free Software Foundation; either version 2 or
> > > + * (at your option) version 3 of the License.
> > > + *
> > > + * This program is distributed in the hope that it will be useful,
> > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> > > + * GNU General Public License for more details.
> > > + *
> > > + * You should have received a copy of the GNU General Public License
> along
> > > + * with this program; if not, see <http://www.gnu.org/licenses/>.
> > > + */
> >
> > Again v2-or-v3-only.
> >
> > There's many more files with this same problem but I'll stop pointing
> > them out now.
> >
> > If this is to be included in QEMU, Veertu & Google (and any other
> > copyright holders) would have to agree to change these files to
> > v2-or-later
>
> Sergio: Have you or Paolo had any contact with the Veertu or Google
> authors about your Hypervisor.framework project? If you're already in
> contact with them you could raise the issue and ask them to join this
> email thread.
>
>
> Izik, Vincent (assuming you are the right person to contact at Google),
> can you reply to Daniel and Stefan?
>
Hi,
What I suggest is that we will send our patch' again as gpl2+ and clean the
entire stuff to make sure they are falling into the right copyright
category as required by QEMU.
I think it will be easier to me to just integrate it all to QEMU however I
need to know if to use the Google code or not (it is anyway not a big issue
from time perspective)
BTW, another thing that need to be integrated in order to make this stuff
useful is the vmnet patch's, it is apple NAT for vms that allow guests to
have networking... (altho that it come with a trick, without certificate it
will require root permission, while hypverisor framework itself can run
without root)
What do you guys think?
>
> Sergio worked on completing the QEMU port to Hypervisor.framework. The
> hvf-all.c file that Daniel pointed out as v2-only is derived from kvm-all.c
> and hax-all.c, and should be under v2-or-later license. The others seem to
> be either original or derived from Bochs, which is LGPL, so they could be
> LGPL or GPLv2+.
>
> Thanks,
>
> Paolo
>
>
> There are benefits to having this code upstream. If they ever want to
> rebase on qemu.git there will be less work for them.
>
> Stefan
>
>
>
- [Qemu-devel] [PATCH v2 13/13] hvf: inject General Protection Fault when vmexit through vmcall, (continued)
- [Qemu-devel] [PATCH v2 13/13] hvf: inject General Protection Fault when vmexit through vmcall, Sergio Andres Gomez Del Real, 2017/08/30
- [Qemu-devel] [PATCH v2 12/13] hvf: refactor event injection code for hvf, Sergio Andres Gomez Del Real, 2017/08/30
- [Qemu-devel] [PATCH v2 11/13] hvf: move fields from CPUState to CPUX86State, Sergio Andres Gomez Del Real, 2017/08/30
- [Qemu-devel] [PATCH v2 04/13] hvf: run hvf code through checkpatch.pl and fix style issues, Sergio Andres Gomez Del Real, 2017/08/30
- [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository, Sergio Andres Gomez Del Real, 2017/08/30
- Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository, Daniel P. Berrange, 2017/08/30
- Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository, Stefan Hajnoczi, 2017/08/31
- Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository, Paolo Bonzini, 2017/08/31
- Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository,
Izik Eidus <=
- Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository, Paolo Bonzini, 2017/08/31
- Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository, Izik Eidus, 2017/08/31
- Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository, Sergio Andrés Gómez del Real, 2017/08/31
- Message not available
- Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository, Izik Eidus, 2017/08/31
Re: [Qemu-devel] [PATCH v2 02/13] hvf: add code base from Google's QEMU repository, Stefan Hajnoczi, 2017/08/31
Re: [Qemu-devel] [PATCH v2 00/13] add support for Hypervisor.framework in QEMU, no-reply, 2017/08/30
Re: [Qemu-devel] [PATCH v2 00/13] add support for Hypervisor.framework in QEMU, no-reply, 2017/08/30
Re: [Qemu-devel] [PATCH v2 00/13] add support for Hypervisor.framework in QEMU, Stefan Hajnoczi, 2017/08/31