On Fri, Mar 24, 2017 at 12:29:39AM +0000, Xu, Anthony wrote:
> > Perhaps all trace.o files should be put into their own .a instead of
> > being added directly to the linker line:
> > 
> > COMMON_LDADDS = $(trace-obj-y) libqemuutil.a libqemustub.a
> > 
> > I think the linker would only pull in .o files containing symbols that are
> > actually referenced by the program.
> 
> Hi Stefan,
> 
> That's a good idea!
> 
> Below patch creates libqemutrace.a.  ./trace.o, ./qapi/trace.o and 
> ./util/trace.o are added into libqemuutil.a  to avoid recursive dependencies 
> between libqemuutil.a and libqemutrace.a.

Please see the guidelines for submitting patches:
http://wiki.qemu-project.org/Contribute/SubmitAPatch

Each patch revision should be sent as a separate email thread.

Each patch series must have your Signed-off-by and a commit
message/description.

Stefan

Attachment: signature.asc
Description: PGP signature

Reply via email to