From: Jeremy Fitzhardinge Subject: Re: [RFC PATCH 09/33] Add start-of-day setup hooks to subarch Date: Thu, 20 Jul 2006 09:27:16 -0400 Message-ID: <44BF84B4.2080700@goop.org> References: <20060718091807.467468000@sous-sol.org> <20060718091950.750213000@sous-sol.org> <20060720060736.GA25367@stusta.de> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Keir Fraser Cc: Adrian Bunk , Ian Pratt , xen-devel@lists.xensource.com, linux-kernel@vger.kernel.org, virtualization@lists.osdl.org, Chris Wright , Andrew Morton List-Id: xen-devel@lists.xenproject.org Keir Fraser wrote: >>> +struct shared_info *HYPERVISOR_shared_info = (struct shared_info >>> *)empty_zero_page; >>> +EXPORT_SYMBOL(HYPERVISOR_shared_info); >>> ... >> >> EXPORT_SYMBOL_GPL? > > Interrupt-control macros (local_irq_enable/disable and friends) used > this symbol, so GPLing it made non-GPL modules fail. We made a bunch > of the macros proper functions so this may no longer be the case. Given that this is just a page exported from the hypervisor, it doesn't make much sense to impose a GPL requirement on this symbol (to put it another way, this is more part of the Xen ABI than the Linux API, so Linux can't make much claim to it). J