From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Stefan Weil <sw@weilnetz.de>
Cc: "Fam Zheng" <fam@euphon.net>,
"Lucian Petrut" <lpetrut@cloudbasesolutions.com>,
"Philippe Mathieu-Daudé" <philmd@redhat.com>,
qemu-devel@nongnu.org, "Justin Terry" <juterry@microsoft.com>,
"Alex Bennée" <alex.bennee@linaro.org>,
"Ilias Maratos" <i.maratos@gmail.com>
Subject: Re: [Qemu-devel] [PATCH 0/2] testing: Build WHPX enabled binaries
Date: Thu, 19 Sep 2019 12:41:52 +0100 [thread overview]
Message-ID: <20190919114152.GK20217@redhat.com> (raw)
In-Reply-To: <04454c37-ad8b-b69b-3f40-bb78f0c20ef2@weilnetz.de>
On Thu, Sep 19, 2019 at 01:18:57PM +0200, Stefan Weil wrote:
> Am 19.09.2019 um 12:59 schrieb Philippe Mathieu-Daudé:
> > Add a job to cross-build QEMU with WHPX enabled.
> >
> > Use the Win10SDK headers from the Android Project, as commented
> > in https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg03842.html
> >
> > Based-on: <20190918121101.30690-1-philmd@redhat.com>
> > https://lists.gnu.org/archive/html/qemu-devel/2019-09/msg03844.html
> >
> > Philippe Mathieu-Daudé (2):
> > tests/docker: Add fedora-win10sdk-cross image
> > .shippable.yml: Build WHPX enabled binaries
> >
> > .shippable.yml | 2 ++
> > tests/docker/Makefile.include | 1 +
> > .../dockerfiles/fedora-win10sdk-cross.docker | 21 +++++++++++++++++++
> > 3 files changed, 24 insertions(+)
> > create mode 100644 tests/docker/dockerfiles/fedora-win10sdk-cross.docker
> >
>
> Please note that the required header files are part of the Win10SDK
> which is not published under a free license, so I am afraid that they
> cannot be used with QEMU code to produce free binaries.
Can you elaborate on why you think that is the case ?
Looking at the 3 header files used, all they contain are struct
definitions and integer constants, fortunately no functional
code in macros or inline functions. As such, it is questionable
whether the headers alone can be considered copyrightable material.
Any compatible implementation that follows the WHPX API/ABI will by
neccessity be the same, modulo code style.
Further, from a GPL POV, these headers should fall under the system
libraries exception, as WHPX is core low level operating system
functionality.
There's a valid legal question as to how to we got access to the
files in the first place. Assuming they are freely distributable
though, given their hosting on Google Android downloads.
So the fact that the headers are not under an open source / free
license doesn't look like it should impact the ability to distribute
the resulting QEMU binaries.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
next prev parent reply other threads:[~2019-09-19 11:59 UTC|newest]
Thread overview: 75+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-19 10:59 [Qemu-devel] [PATCH 0/2] testing: Build WHPX enabled binaries Philippe Mathieu-Daudé
2019-09-19 10:59 ` [Qemu-devel] [PATCH 1/2] tests/docker: Add fedora-win10sdk-cross image Philippe Mathieu-Daudé
2019-09-19 11:28 ` Alex Bennée
2019-09-19 11:35 ` Philippe Mathieu-Daudé
2019-09-19 10:59 ` [Qemu-devel] [PATCH 2/2] .shippable.yml: Build WHPX enabled binaries Philippe Mathieu-Daudé
2019-09-19 11:02 ` Philippe Mathieu-Daudé
2019-09-19 11:09 ` [Qemu-devel] [PATCH 0/2] testing: " Thomas Huth
2019-09-19 11:21 ` Philippe Mathieu-Daudé
2019-09-19 11:18 ` Stefan Weil
2019-09-19 11:26 ` Philippe Mathieu-Daudé
2020-05-20 10:25 ` Philippe Mathieu-Daudé
2020-05-20 10:25 ` [Bug 1879672] " Philippe Mathieu-Daudé
2019-09-19 11:41 ` Daniel P. Berrangé [this message]
2019-09-19 11:54 ` [Qemu-devel] " Philippe Mathieu-Daudé
2019-09-19 13:25 ` Daniel P. Berrangé
-- strict thread matches above, loose matches on Subject: below --
2019-09-20 11:33 [PATCH v2 0/3] " Philippe Mathieu-Daudé
2019-09-20 11:33 ` [PATCH v2 1/3] target/i386: Fix broken build with WHPX enabled Philippe Mathieu-Daudé
2019-09-20 11:37 ` Paolo Bonzini
2019-09-20 11:33 ` [PATCH v2 2/3] tests/docker: Add fedora-win10sdk-cross image Philippe Mathieu-Daudé
2019-09-20 11:33 ` [PATCH v2 3/3] .shippable.yml: Build WHPX enabled binaries Philippe Mathieu-Daudé
2019-09-20 15:17 ` [PATCH v2 0/3] testing: " Philippe Mathieu-Daudé
2019-09-20 16:53 ` Justin Terry (VM)
2020-05-20 10:26 ` Philippe Mathieu-Daudé
2020-05-20 10:26 ` [Bug 1879672] " Philippe Mathieu-Daudé
2020-07-31 8:33 ` Philippe Mathieu-Daudé
2020-07-31 8:33 ` [Bug 1879672] " Philippe Mathieu-Daudé
2020-07-31 23:31 ` [EXTERNAL] " Sunil Muthuswamy
2020-08-03 10:51 ` Philippe Mathieu-Daudé
2020-08-03 10:51 ` [Bug 1879672] " Philippe Mathieu-Daudé
2020-08-03 11:28 ` Stefan Weil
2020-08-03 20:25 ` Stefan Weil
2020-08-04 6:43 ` Thomas Huth
2020-08-04 6:55 ` Stefan Weil
2020-08-04 7:23 ` Philippe Mathieu-Daudé
2020-08-04 7:23 ` [Bug 1879672] " Philippe Mathieu-Daudé
2020-08-04 7:42 ` Stefan Weil
2020-08-04 7:52 ` Philippe Mathieu-Daudé
2020-08-04 7:52 ` [Bug 1879672] " Philippe Mathieu-Daudé
2020-08-04 8:10 ` Thomas Huth
2020-08-04 9:23 ` Daniel P. Berrangé
2020-08-04 9:23 ` [Bug 1879672] " Daniel Berrange
2020-08-18 21:20 ` [EXTERNAL] " Sunil Muthuswamy
2020-08-19 3:36 ` Philippe Mathieu-Daudé
2020-08-19 3:36 ` [Bug 1879672] " Philippe Mathieu-Daudé
2019-09-21 6:34 ` no-reply
2019-11-07 19:48 [PATCH v2] WHPX: support for xcr0 Sunil Muthuswamy
2019-11-07 20:05 ` Stefan Weil
2019-11-07 22:52 ` Sunil Muthuswamy
2020-05-20 10:27 ` Philippe Mathieu-Daudé
2020-05-20 10:27 ` [Bug 1879672] " Philippe Mathieu-Daudé
2019-11-12 18:52 ` Sunil Muthuswamy
2019-11-13 14:56 ` Paolo Bonzini
2022-04-28 6:31 ` Paolo Bonzini
2020-02-18 20:38 [PATCH] WHPX: Assigning maintainer for Windows Hypervisor Platform Sunil Muthuswamy
2020-02-18 20:51 ` Justin Terry (SF)
2020-02-19 8:31 ` Philippe Mathieu-Daudé
2020-02-19 15:50 ` [EXTERNAL] " Justin Terry (SF)
2020-02-21 7:54 ` Stefan Weil
2020-02-24 19:43 ` Sunil Muthuswamy
2020-05-20 10:29 ` Philippe Mathieu-Daudé
2020-05-20 10:29 ` [Bug 1879672] " Philippe Mathieu-Daudé
2020-05-19 21:59 ` Sunil Muthuswamy
2020-05-20 10:29 ` Philippe Mathieu-Daudé
2020-05-20 10:29 ` [Bug 1879672] " Philippe Mathieu-Daudé
2020-02-21 17:15 ` Paolo Bonzini
2020-05-20 10:14 [Bug 1879672] [NEW] QEMU installer with WHPX support Philippe Mathieu-Daudé
2020-05-20 10:25 ` [Bug 1879672] " Daniel Berrange
2020-05-20 12:30 ` Stefan Weil
2020-06-08 7:18 ` Philippe Mathieu-Daudé
2020-11-04 20:01 ` John Snow
2020-11-04 23:12 ` Philippe Mathieu-Daudé
2020-11-04 23:12 ` Philippe Mathieu-Daudé
2021-05-09 15:38 ` Philippe Mathieu-Daudé
2021-05-09 18:33 ` Stefan Weil
2021-05-27 7:45 ` Philippe Mathieu-Daudé
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190919114152.GK20217@redhat.com \
--to=berrange@redhat.com \
--cc=alex.bennee@linaro.org \
--cc=fam@euphon.net \
--cc=i.maratos@gmail.com \
--cc=juterry@microsoft.com \
--cc=lpetrut@cloudbasesolutions.com \
--cc=philmd@redhat.com \
--cc=qemu-devel@nongnu.org \
--cc=sw@weilnetz.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).