[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility f
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions |
Date: |
Mon, 23 Jul 2012 15:59:58 -0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, Jul 23, 2012 at 04:49:07PM +0000, Blue Swirl wrote:
> On Mon, Jul 16, 2012 at 5:42 PM, Eduardo Habkost <address@hidden> wrote:
> > On Sat, Jul 14, 2012 at 09:14:30AM +0000, Blue Swirl wrote:
> > [...]
> >> >> > diff --git a/tests/Makefile b/tests/Makefile
> >> >> > index b605e14..89bd890 100644
> >> >> > --- a/tests/Makefile
> >> >> > +++ b/tests/Makefile
> >> >> > @@ -15,6 +15,7 @@ check-unit-y +=
> >> >> > tests/test-string-output-visitor$(EXESUF)
> >> >> > check-unit-y += tests/test-coroutine$(EXESUF)
> >> >> > check-unit-y += tests/test-visitor-serialization$(EXESUF)
> >> >> > check-unit-y += tests/test-iov$(EXESUF)
> >> >> > +check-unit-y += tests/test-x86-cpuid$(EXESUF)
> >> >>
> >> >> This probably tries to build the cpuid test also for non-x86 targets
> >> >> and break them all.
> >> >
> >> > I don't think there's any concept of "targets" for the check-unit tests.
> >>
> >> How about:
> >> check-qtest-i386-y = tests/test-x86-cpuid$(EXESUF)
> >
> > test-x86-cpuid is not a qtest test case.
>
> Why not? I don't think it is a unit test either, judging from what the
> other unit tests do.
It is absolutely a unit test. I don't know why you don't think so. It
simply checks the results of the APIC ID calculation functions.
>
> >
> >>
> >> > I had to do the following, to be able to make a test that uses the
> >> > target-i386 code:
> >> >
> >> >> > +tests/test-x86-cpuid.o: QEMU_INCLUDES += -Itarget-i386
> >> >
> >> > Any suggestions to avoid this hack would be welcome.
> >>
> >> Maybe it would be simpler to adjust #include path in the file.
> >
> > Using the full path on the #include line would break in case
> > target-i386/topology.h include other files from the target-i386
> > directory.
>
> That's fragile. Maybe the target-xyz files should use the full path.
Yes, it would be fragile. That's why I used -Itarget-i386 (not a perfect
solution, but more likely to stay working and not break easily).
I don't know if I understand what you propose. You mean to change the
256 target-specific #include lines inside qemu?
$ git grep -f <(find target-* -name '*.h' | sed -e
'address@hidden/]*/@#include "@' | sort -u) target-* | wc -l
256
--
Eduardo
- Re: [Qemu-devel] [QEMU PATCH 2/7] hw/apic.c: rename bit functions to not conflict with bitops.h, (continued)
- [Qemu-devel] [Seabios RFC PATCH 1/1] get lapic IDs from fw_cfg, Eduardo Habkost, 2012/07/10
- [Qemu-devel] [QEMU RFC PATCH 4/7] i386: create apic_id_for_cpu() function, Eduardo Habkost, 2012/07/10
- [Qemu-devel] [QEMU RFC PATCH 7/7] generate APIC IDs according to CPU topology, Eduardo Habkost, 2012/07/10
- [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Eduardo Habkost, 2012/07/10
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Blue Swirl, 2012/07/12
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Eduardo Habkost, 2012/07/13
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Blue Swirl, 2012/07/14
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Eduardo Habkost, 2012/07/16
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Blue Swirl, 2012/07/23
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions,
Eduardo Habkost <=
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Blue Swirl, 2012/07/23
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Eduardo Habkost, 2012/07/23
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Blue Swirl, 2012/07/23
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Eduardo Habkost, 2012/07/23
- Re: [Qemu-devel] [QEMU RFC PATCH 6/7] i386: topology & APIC ID utility functions, Blue Swirl, 2012/07/24
[Qemu-devel] [QEMU PATCH 3/7] kvm: set vcpu_id to APIC ID instead of CPU index, Eduardo Habkost, 2012/07/10
[Qemu-devel] [QEMU RFC PATCH 5/7] pc: write lapic info (apic IDs) to fw_cfg so seabios can use it, Eduardo Habkost, 2012/07/10
Re: [Qemu-devel] [RFC 0/7+1] QEMU APIC ID + topology bug + CPU hotplug, Igor Mammedov, 2012/07/12