All of lore.kernel.org
 help / color / mirror / Atom feed
From: Fam Zheng <famz@redhat.com>
To: Colin Lord <clord@redhat.com>
Cc: qemu-devel@nongnu.org, kwolf@redhat.com, qemu-block@nongnu.org,
	mreitz@redhat.com
Subject: Re: [Qemu-devel] [PATCH v6 0/4] Dynamic module loading for block drivers
Date: Wed, 3 Aug 2016 14:28:37 +0800	[thread overview]
Message-ID: <20160803062837.GB11235@ad.usersys.redhat.com> (raw)
In-Reply-To: <1c794ef7-3c5a-a95c-7158-1cd1f0773f23@redhat.com>

On Tue, 08/02 10:17, Colin Lord wrote:
> It sounds like since this series removes dmg from the list of
> modularized drivers, there is a libbz2 dependency that may get pulled in
> from the dmg driver. I believe Fam was suggesting moving that part of
> the dmg driver into its own module (although not modularizing the entire
> dmg driver). Does that sound like a correct interpretation Fam? I do

Yes.

> like the idea, but I'm a little hesitant to put that into this series
> because I'd like to keep this series fairly simple so that hopefully
> there doesn't need to be too much more discussion about it. My
> internship will be ending soon so I'd prefer this series be finished
> sooner rather than later. Also I don't think anything in this series
> excludes this idea from being added later if we want. Of course there is
> always room for discussion, so if there are other thoughts I'm happy to
> listen.

I'll prepare a patch that can be applied before this series so that we don't
regress on dmg library.

Fam

      reply	other threads:[~2016-08-03  6:28 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-08-02 14:12 [Qemu-devel] [PATCH v6 0/4] Dynamic module loading for block drivers Colin Lord
2016-08-02 14:12 ` [Qemu-devel] [PATCH v6 1/4] blockdev: prepare iSCSI block driver for dynamic loading Colin Lord
2016-08-08 15:18   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2016-08-08 17:54     ` Colin Lord
2016-08-02 14:12 ` [Qemu-devel] [PATCH v6 2/4] blockdev: Add dynamic generation of module_block.h Colin Lord
2016-08-08 15:20   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2016-08-02 14:12 ` [Qemu-devel] [PATCH v6 3/4] blockdev: Add dynamic module loading for block drivers Colin Lord
2016-08-08 15:22   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2016-08-02 14:12 ` [Qemu-devel] [PATCH v6 4/4] blockdev: Modularize nfs block driver Colin Lord
2016-08-08 15:23   ` [Qemu-devel] [Qemu-block] " Stefan Hajnoczi
2016-08-02 14:17 ` [Qemu-devel] [PATCH v6 0/4] Dynamic module loading for block drivers Colin Lord
2016-08-03  6:28   ` Fam Zheng [this message]

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=20160803062837.GB11235@ad.usersys.redhat.com \
    --to=famz@redhat.com \
    --cc=clord@redhat.com \
    --cc=kwolf@redhat.com \
    --cc=mreitz@redhat.com \
    --cc=qemu-block@nongnu.org \
    --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.