Re: [Xen-devel] [PATCH v3 15/15] tools/tests: Enable xen-access on ARM

[Tamas K Lengyel <tamas.lengyel@xxxxxxxxxxxx>]

On Thu, Sep 4, 2014 at 12:06 AM, Tamas K Lengyel <tamas.lengyel@xxxxxxxxxxxx> wrote:

On Wed, Sep 3, 2014 at 10:27 PM, Julien Grall <julien.grall@xxxxxxxxxx> wrote:

Hello Tamas,

On 02/09/14 05:15, Tamas K Lengyel wrote:

On Mon, Sep 1, 2014 at 11:26 PM, Julien Grall <julien.grall@xxxxxxxxxx

<mailto:julien.grall@linaro.org>> wrote:
    On 01/09/14 10:22, Tamas K Lengyel wrote:

        diff --git a/tools/tests/xen-access/__Makefile
        b/tools/tests/xen-access/__Makefile
        index 65eef99..698355c 100644
        --- a/tools/tests/xen-access/__Makefile
        +++ b/tools/tests/xen-access/__Makefile

        @@ -7,9 +7,7 @@ CFLAGS += $(CFLAGS_libxenctrl)
           CFLAGS += $(CFLAGS_libxenguest)
           CFLAGS += $(CFLAGS_xeninclude)

        -TARGETS-y :=
        -TARGETS-$(CONFIG_X86) += xen-access
        -TARGETS := $(TARGETS-y)
        +TARGETS := xen-access


    I would move the definition of HAS_MEM_ACCESS from arch/*/Rules.mk
    to config/*.mk and use the defition here to build or not xen-access.


I'm not a fan of that approach. How about this:

What is the problem with this solution?

Honestly, I tried to wrap my head around what is going on in config/*.mk but it didn't really click yet. It does look more complicated to have the HAS_MEM_ACCESS in config/*.mk then in arch/*/Rules.mk..
 

Never mind, I got it. I'm only going to move HAS_MEM_ACCESS in this patch, I'll leave the definition of it in arch/*/Rules.mk for the rest of the series as it will only become required here.

Tamas

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel