From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jeremy Fitzhardinge Subject: Re: pvops DomU kernels [was Re: "Hotplug Scripts not working ..." error on jaunty] Date: Fri, 23 Oct 2009 13:37:08 -0700 Message-ID: <4AE213F4.7020204@goop.org> References: <940bcfd20910212256w3ba99c50v4fedf4dad034c5c8@mail.gmail.com> <4AE0C8E1.3030608@goop.org> <940bcfd20910230121r174c7876m230c0b2728495b56@mail.gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <940bcfd20910230121r174c7876m230c0b2728495b56@mail.gmail.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Dulloor Cc: Boris Derzhavets , Xen-devel , "Marc - A. Dahlhaus" , Keir Fraser List-Id: xen-devel@lists.xenproject.org On 10/23/09 01:21, Dulloor wrote: > * jeremy pvops git (with SMP) - Your patch works. I had commented out > detect_extended_topology as a work-around. > OK, good. > * jeremy pvops git (without SMP) - Build fails at link time, since > "register_task_migration_notifier" definition is guarded by CONFIG_SMP > (rightly so). This function is used in pvclock_init_vsyscall. Should > the call in pvclock_init_vsyscall be guarded by CONFIG_SMP or should > another "dummy" version of register_task_migration_notifier be defined > for !CONFIG_SMP. > Yes, the second sounds like the right answer. > * Linus' git - Nothing on Linux console. Crash rip [BUG()] - > (gdb) x/i 0xffffffff818da4ee > 0xffffffff818da4ee : ud2a > > Crash is because HYPERVISOR_update_va_mapping fails. > > I have attached the "xm dmesg", which also shows the error(in mm.c) > leading to the crash. If you can't think of any existing problem, I > will spend time debugging tomorrow. > Its not at all clear to me what Xen is objecting to here. There are a few paths where it can end up at could_not_pin in get_page_from_l1e(), but I don't know which one is happening here. I think it's the "if ( unlikely(real_pg_owner != pg_owner) )" conditional, but I don't know what real_pg_owner means here. Keir? J