All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <aliguori@us.ibm.com>
To: Laszlo Ersek <lersek@redhat.com>,
	pbonzini@redhat.com, hch@lst.de, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 0/7] introduce BSD-licensed block driver for "raw"
Date: Fri, 16 Aug 2013 09:59:07 -0500	[thread overview]
Message-ID: <874napfzr8.fsf@codemonkey.ws> (raw)
In-Reply-To: <1376662507-5212-1-git-send-email-lersek@redhat.com>

Laszlo Ersek <lersek@redhat.com> writes:

> Paolo asked me to write such a driver based on his textual specification
> alone. The first patch captures his email in full, the rest re-quotes
> parts that are being implemented.
>
> The tree compiles at each patch. The series passes "make check-block".
>
> "block/raw.c" is not removed because I wanted to keep it out of my
> series and out of my brain.
>
> Disclaimer: I couldn't care less if the raw block driver was public
> domain or AGPLv3+, as long as it qualifies as free software. I'm only
> trying to do what Paolo asked of me.

Generally speaking, rewriting parts of QEMU to be !GPL is something I
would strongly, strongly oppose.

I believe that Paolo had a good reason for this though.  I suppose the
logic is that we want to expose a "libqemublock" that libvirt can use such
that it can stop parsing qcow2 files.

Now libvirt just needs LGPLv2+, right?

Is the JSON mode of qemu-img info not sufficient for libvirt's purposes?

Is there additional logic behind having a libqemublock?

Regards,

Anthony Liguori

>
> Laszlo Ersek (7):
>   add skeleton for BSD licensed "raw" BlockDriver
>   raw_bsd: emit debug events in bdrv_co_readv() and bdrv_co_writev()
>   raw_bsd: add raw_create()
>   raw_bsd: introduce "special members"
>   raw_bsd: add raw_create_options
>   raw_bsd: register bdrv_raw
>   switch raw block driver from "raw.o" to "raw_bsd.o"
>
>  block/Makefile.objs |    2 +-
>  block/raw_bsd.c     |  186 +++++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 187 insertions(+), 1 deletions(-)
>  create mode 100644 block/raw_bsd.c

  parent reply	other threads:[~2013-08-16 14:59 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-08-16 14:15 [Qemu-devel] [PATCH 0/7] introduce BSD-licensed block driver for "raw" Laszlo Ersek
2013-08-16 14:15 ` [Qemu-devel] [PATCH 1/7] add skeleton for BSD licensed "raw" BlockDriver Laszlo Ersek
2013-08-16 14:15 ` [Qemu-devel] [PATCH 2/7] raw_bsd: emit debug events in bdrv_co_readv() and bdrv_co_writev() Laszlo Ersek
2013-08-16 14:15 ` [Qemu-devel] [PATCH 3/7] raw_bsd: add raw_create() Laszlo Ersek
2013-08-16 14:15 ` [Qemu-devel] [PATCH 4/7] raw_bsd: introduce "special members" Laszlo Ersek
2013-08-20  8:11   ` Kevin Wolf
2013-08-16 14:15 ` [Qemu-devel] [PATCH 5/7] raw_bsd: add raw_create_options Laszlo Ersek
2013-08-16 14:15 ` [Qemu-devel] [PATCH 6/7] raw_bsd: register bdrv_raw Laszlo Ersek
2013-08-16 14:15 ` [Qemu-devel] [PATCH 7/7] switch raw block driver from "raw.o" to "raw_bsd.o" Laszlo Ersek
2013-08-18 14:29   ` Paolo Bonzini
2013-08-20  7:51     ` Kevin Wolf
2013-08-16 14:59 ` Anthony Liguori [this message]
2013-08-16 15:06   ` [Qemu-devel] [PATCH 0/7] introduce BSD-licensed block driver for "raw" Daniel P. Berrange
2013-08-18 14:25   ` Paolo Bonzini
2013-08-19 16:24     ` Laszlo Ersek
2013-08-20  8:21 ` Kevin Wolf
2013-08-21  8:20   ` Laszlo Ersek
2013-08-21  9:03     ` Paolo Bonzini

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=874napfzr8.fsf@codemonkey.ws \
    --to=aliguori@us.ibm.com \
    --cc=hch@lst.de \
    --cc=lersek@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.