MIME-Version: 1.0
In-Reply-To: <1450217797-19295-6-git-send-email-mcgrof@do-not-panic.com>
References: <1450217797-19295-1-git-send-email-mcgrof@do-not-panic.com> <1450217797-19295-6-git-send-email-mcgrof@do-not-panic.com>
From: Andy Lutomirski <luto@amacapital.net>
Date: Thu, 17 Dec 2015 12:48:44 -0800
Message-ID: <CALCETrW_rdxW9h0JDCG_a7PCbW88QOv2KMVdWrDwd+Zr5MfG0w@mail.gmail.com>
Subject: Re: [RFC v1 5/8] x86/init: move ebda reservations into linker table
To: "Luis R. Rodriguez" <mcgrof@do-not-panic.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>, Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
        Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
        Rusty Russell <rusty@rustcorp.com.au>,
        Boris Ostrovsky <boris.ostrovsky@oracle.com>, mcb30@ipxe.org,
        Juergen Gross <jgross@suse.com>, Jan Beulich <JBeulich@suse.com>,
        Joerg Roedel <joro@8bytes.org>,
        Andrey Ryabinin <ryabinin.a.a@gmail.com>,
        Andrey Konovalov <andreyknvl@google.com>, long.wanglong@huawei.com,
        qiuxishi@huawei.com, Andrey Ryabinin <aryabinin@virtuozzo.com>,
        Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
        valentinrothberg@gmail.com, peter.senna@gmail.com,
        Luis Rodriguez <mcgrof@suse.com>, X86 ML <x86@kernel.org>,
        Xen Devel <xen-devel@lists.xensource.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1132
Lines: 27

On Tue, Dec 15, 2015 at 2:16 PM, Luis R. Rodriguez
<mcgrof@do-not-panic.com> wrote:
> From: "Luis R. Rodriguez" <mcgrof@suse.com>
>
> This lets us annotate its requirements specifically for
> PC and lguest subarchitectures. While at it since head.c
> just has ebda data rename it. Since we're using linker tables
> and both x86 32-bit and 64-bit have them we don't need
> to declare a call for this separately. This lets us
> also keep this declared static now.
>
> Since we're using linker tables and have support to annotate
> subarchitecture do that instead. pv_legacy() is incorrect as
> its really only for legacy PV guests. There's no need for
> pv_legacy() check anymore now.

I'm entirely ignorant of anything going on in gPXE/iPXE.

Can you explain what a linker table *does*?  It looks like all you've
done in this patch is to move code around.  What actually happens?

--Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/