[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] xen and agpgart

Somehow you are ending up in the following code in the following code
in __change_page_attr (arch/i386/mm/pageattr.c):

        if (cpu_has_pse && (page_count(kpte_page) == 1)) {
                list_add(&kpte_page->lru, &df_list);
                revert_page(kpte_page, address);

This ought to be impossible, as cpu_has_pse should be 0.
Add some debug printing around that code -- e.g., print the value of
cpu_has_pse, and also the contents of e.g.,
boot_cpu_data.x86_capability[0], boot_cpu_data.x86_capability[1].

Perhaps the capability flags are getting corrupted - I think the value
must be correct at boot time or you would crash while booting!

Meanwhile I've checked in a fix for wbinvd. I've fixed the wbinvd()
macro in system.h, but agp.h uses the instruction directly. You'll
have to patch that file yourself -- really a patch ought to be sent to
the maintainer, but I'll wait until I've fixed the broken uses of
ioremap_nocache in various GART drivers.

 -- Keir

> On Sat, Sep 04, 2004 at 04:26:01AM +0100, Keir Fraser wrote:
> > Can you make your vmlinux file available that goes with that oops?
> > If so I'll take a look.
> Sure.  I've placed the vmlinux and related files here:
> http://sekrit.org/dist/xen/
> The source changes I've made to get to this point were turning the cache
> flush operation into a noop and s/virt_to_phys/virt_to_bus/ in the agp
> drivers, and in arch/xen/i386/mm/ioremap.c.
>                               -- Gerald

This SF.Net email is sponsored by BEA Weblogic Workshop
FREE Java Enterprise J2EE developer tools!
Get your free copy of BEA WebLogic Workshop 8.1 today.
Xen-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.