[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace
From: |
Blue Swirl |
Subject: |
Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace |
Date: |
Sun, 28 Aug 2011 07:24:43 +0000 |
On Sat, Aug 27, 2011 at 5:56 PM, Lluís <address@hidden> wrote:
> Blue Swirl writes:
>
>> On Fri, Aug 26, 2011 at 7:12 PM, Lluís <address@hidden> wrote:
>>> Blue Swirl writes:
>>>
>>>> 957f1f99f263d57612807a9535f75ca4473f05f0 didn't consider
>>>> that qemu-timer-common.o is needed by simpletrace.
>>>
>>>> Fix by adding it to qga object list.
>>>
>>>> Signed-off-by: Blue Swirl <address@hidden>
>>>> ---
>>>> Makefile.objs | 3 +++
>>>> 1 files changed, 3 insertions(+), 0 deletions(-)
>>>
>>>> diff --git a/Makefile.objs b/Makefile.objs
>>>> index d1f3e5d..df11aec 100644
>>>> --- a/Makefile.objs
>>>> +++ b/Makefile.objs
>>>> @@ -404,6 +404,9 @@ qga-obj-y = $(addprefix qga/, $(qga-nested-y))
>>>> qga-obj-y += qemu-ga.o qemu-tool.o qemu-error.o qemu-sockets.o
>>>> module.o qemu-option.o cutils.o osdep.o
>>>> qga-obj-$(CONFIG_WIN32) += oslib-win32.o
>>>> qga-obj-$(CONFIG_POSIX) += oslib-posix.o
>>>> +ifeq ($(TRACE_BACKEND),simple)
>>>> +qga-obj-y += qemu-timer-common.o
>>>> +endif
>>>
>>>> vl.o: QEMU_CFLAGS+=$(GPROF_CFLAGS)
>>>
>>> I sent a patch that should fix it for everybody linking with the tracing
>>> objects:
>>>
>>> http://lists.gnu.org/archive/html/qemu-devel/2011-08/msg03150.html
>
>> With your patch, there are warnings from linker:
>> ../qemu-timer-common.o: warning: multiple common of `use_rt_clock'
>> ../qemu-timer-common.o: warning: previous common is here
>
> Ah, yes. These extra errors are fixed by the duplicate elimination patch
> :)
>
> http://lists.gnu.org/archive/html/qemu-devel/2011-08/msg02987.html
>
> So, you need both to keep it clean.
Using the sort function looks hackish to me. Maybe the linkage should
be controlled by configure instead?
- [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace, Blue Swirl, 2011/08/26
- Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace, Lluís, 2011/08/26
- Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace, Blue Swirl, 2011/08/27
- Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace, Lluís, 2011/08/27
- Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace,
Blue Swirl <=
- Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace, Lluís, 2011/08/28
- Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace, Blue Swirl, 2011/08/28
- Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace, Stefan Hajnoczi, 2011/08/29
- Re: [Qemu-devel] [PATCH 1/2] Fix guest agent build with simpletrace, Michael Roth, 2011/08/29
- [Qemu-devel] [PATCH] Add CONFIG_QEMU_TIMER to handle qemu-timer-common.o dep, Michael Roth, 2011/08/29
- Re: [Qemu-devel] [PATCH] Add CONFIG_QEMU_TIMER to handle qemu-timer-common.o dep, Stefan Hajnoczi, 2011/08/30
- Re: [Qemu-devel] [PATCH] Add CONFIG_QEMU_TIMER to handle qemu-timer-common.o dep, Lluís, 2011/08/30
- Re: [Qemu-devel] [PATCH] Add CONFIG_QEMU_TIMER to handle qemu-timer-common.o dep, Michael Roth, 2011/08/30
- Re: [Qemu-devel] [PATCH] Add CONFIG_QEMU_TIMER to handle qemu-timer-common.o dep, Lluís, 2011/08/30
- Re: [Qemu-devel] [PATCH] Add CONFIG_QEMU_TIMER to handle qemu-timer-common.o dep, Lluís, 2011/08/30