From: Paolo Bonzini <pbonzini@redhat.com>
To: Dimitris Aragiorgis <dimara@arrikto.com>, qemu-devel@nongnu.org
Cc: kwolf@redhat.com, stefanha@redhat.com, qemu-block@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 4/4] block: Include qemu-log.o in block-obj-y
Date: Fri, 08 May 2015 13:41:52 +0200 [thread overview]
Message-ID: <554CA100.4070400@redhat.com> (raw)
In-Reply-To: <1431085109-19677-5-git-send-email-dimara@arrikto.com>
On 08/05/2015 13:38, Dimitris Aragiorgis wrote:
> Building the QEMU tools fails if we #define DEBUG_BLOCK inside
> block/raw-posix.c. This happens because qemu-log.o is missing from
> block-obj-y, which causes the link to fail. Fix this.
>
> Signed-off-by: Dimitris Aragiorgis <dimara@arrikto.com>
Let's just remove DEBUG_BLOCK messages. There are only two and they
only trigger for error cases.
Paolo
> ---
> Makefile.objs | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Makefile.objs b/Makefile.objs
> index 28999d3..98f6e02 100644
> --- a/Makefile.objs
> +++ b/Makefile.objs
> @@ -6,7 +6,7 @@ util-obj-y = util/ qobject/ qapi/ qapi-types.o qapi-visit.o qapi-event.o
> #######################################################################
> # block-obj-y is code used by both qemu system emulation and qemu-img
>
> -block-obj-y = async.o thread-pool.o
> +block-obj-y = async.o thread-pool.o qemu-log.o
> block-obj-y += nbd.o block.o blockjob.o
> block-obj-y += main-loop.o iohandler.o qemu-timer.o
> block-obj-$(CONFIG_POSIX) += aio-posix.o
>
prev parent reply other threads:[~2015-05-08 11:42 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-08 11:38 [Qemu-devel] [PATCH 0/4] Some fixes related to scsi-generic Dimitris Aragiorgis
2015-05-08 11:38 ` [Qemu-devel] [PATCH 1/4] Fix migration in case of scsi-generic Dimitris Aragiorgis
2015-05-08 11:44 ` Paolo Bonzini
2015-05-08 11:38 ` [Qemu-devel] [PATCH 2/4] block: Use bdrv_is_sg() everywhere Dimitris Aragiorgis
2015-05-08 11:43 ` Paolo Bonzini
2015-05-08 11:38 ` [Qemu-devel] [PATCH 3/4] raw-posix: Introduce hdev_is_sg() Dimitris Aragiorgis
2015-05-08 11:42 ` Paolo Bonzini
2015-05-08 11:38 ` [Qemu-devel] [PATCH 4/4] block: Include qemu-log.o in block-obj-y Dimitris Aragiorgis
2015-05-08 11:41 ` Paolo Bonzini [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=554CA100.4070400@redhat.com \
--to=pbonzini@redhat.com \
--cc=dimara@arrikto.com \
--cc=kwolf@redhat.com \
--cc=qemu-block@nongnu.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/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.