From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Fitzhardinge Subject: Re: [PATCH 09/12] x86: Add static initiazlier for descriptors Date: Wed, 29 Oct 2008 23:58:11 +1100 Message-ID: <49085DE3.4010508@goop.org> References: <> <1224904532-9586-1-git-send-email-ice799@gmail.com> <52f8491fb5d55c0171840b01638ea6db0a98f3be.1224903712.git.ice799@gmail.com> <03269a2e8f455b431612a32e8cce048e98c75b27.1224903712.git.ice799@gmail.com> <4810cc08c12cd78612b53f9b52d7ba93a9b03fd0.1224903712.git.ice799@gmail.com> <8834ad1ab5a5ab9b4902ab708160b23f078f3758.1224903712.git.ice799@gmail.com> <4d0ab6d642a4edc3ecc7367df7043ecc8f0a7f98.1224903712.git.ice799@gmail.com> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-newbie-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Joe Damato Cc: linux-x86_64@vger.kernel.org, linux-newbie@vger.kernel.org, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org Joe Damato wrote: > Add static initializer for LDT/GDT/TSS descriptors. > > Signed-off-by: Joe Damato > --- > include/asm-x86/desc_defs.h | 14 ++++++++++++++ > 1 files changed, 14 insertions(+), 0 deletions(-) > > diff --git a/include/asm-x86/desc_defs.h b/include/asm-x86/desc_defs.h > index a1c6516..5119051 100644 > --- a/include/asm-x86/desc_defs.h > +++ b/include/asm-x86/desc_defs.h > @@ -91,6 +91,20 @@ typedef struct desc_struct tss_desc; > , .dpl = (hi >> 13) & 3 \ > , .p = (hi >> 15) & 1 \ > , .base1 = (hi >> 16) & 0xffff } > +#define __DESC_INITIALIZER(lo,hi) \ > + { .limit0 = lo & 0xffff \ > + , .base0 = (lo >> 16) & 0xffff \ > + , .base1 = hi & 0xff \ > + , .type = (hi >> 8) & 0xf \ > + , .s = (hi >> 12) & 1 \ > + , .dpl = (hi >> 13) & 3 \ > + , .p = (hi >> 15) & 1 \ > + , .limit = (hi >> 16) & 0xf \ > + , .avl = (hi >> 20) & 1 \ > + , .l = (hi >> 21) & 1 \ > + , .d = (hi >> 22) & 1 \ > + , .g = (hi >> 23) & 1 \ > + ,.base2 = (hi >> 24) & 0xf } > Reasonable as a transition, I guess, but it would be better to just make the initializers pass in proper values. But I think it would be better to implement this with a union rather than repacking the packed values. J -- To unsubscribe from this list: send the line "unsubscribe linux-newbie" 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.linux-learn.org/faqs