From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Guthro Subject: Re: Xen-changelog Digest, Vol 38, Issue 55 Date: Tue, 22 Apr 2008 13:19:43 -0400 Message-ID: <480E1E2F.2080800@virtualiron.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: xen-devel Cc: Jan Beulich List-Id: xen-devel@lists.xenproject.org I pulled in the latest changes after seeing the Changelog Digests go by - but seem to be having problems compiling against the latest 2.6.18-xen kernel changeset (523:f128849f9e78) http://xenbits.xensource.com/linux-2.6.18-xen.hg?rev/f128849f9e78 xen-changelog-request@lists.xensource.com wrote: > 6. [linux-2.6.18-xen] linux/x86: set up machine_e820 in > copy_e820_map() for it to be usable (Xen patchbot-linux-2.6.18-xen) > > ---------------------------------------------------------------------- > This is the chunk of the patch that is giving me problems: (errors below) > + > +#ifdef CONFIG_XEN > + if (is_initial_xendomain()) { > + struct xen_memory_map memmap; > + > + memmap.nr_entries = E820MAX; > + set_xen_guest_handle(memmap.buffer, machine_e820.map); > + > + if (HYPERVISOR_memory_op(XENMEM_machine_memory_map, &memmap)) > + BUG(); > + machine_e820.nr_map = memmap.nr_entries; > + } else > + machine_e820 = e820; > +#endif > + > return 0; > } > CC arch/x86_64/kernel/e820.o /home/bguthro/dev/unstable/linux-2.6.18-xen.hg/arch/x86_64/kernel/e820-xen.c: In function ‘copy_e820_map’: /home/bguthro/dev/unstable/linux-2.6.18-xen.hg/arch/x86_64/kernel/e820-xen.c:613: error: ‘machine_e820’ undeclared (first use in this function) /home/bguthro/dev/unstable/linux-2.6.18-xen.hg/arch/x86_64/kernel/e820-xen.c:613: error: (Each undeclared identifier is reported only once /home/bguthro/dev/unstable/linux-2.6.18-xen.hg/arch/x86_64/kernel/e820-xen.c:613: error: for each function it appears in.) Is this changeset complete - or did an extern declaration get left out somehow? Ben Guthro