Michel,
--- Michel D�nzer <[EMAIL PROTECTED]> wrote:
> On Wed, 2003-11-05 at 22:24, Bradley Chapman wrote:
> > Mr. Danzer,
>
> FYI, the correct transliteration of my last name would be 'Daenzer', but
> I feel more comfortable if people simply call me Michel.
Sorry about that. My default when speaking to people is to be polite; it's
usually a good idea ;-)
>
> > --- Michel Dnzer <[EMAIL PROTECTED]> wrote:
> > > On Wed, 2003-11-05 at 19:20, Ronny V. Vindenes wrote:
> > > >
> > > > I see the same thing using dri cvs (both with 1.9.0 radeon (kernel) and
> > > > 1.10.0 (dri cvs)) with 2.6.9-test9-mm2, plain -test9 works.
> > > >
> > > > [drm] Initialized radeon 1.10.0 20020828 on minor 0: ATI Radeon If R250
> > > > 9000
> > > > [drm:radeon_cp_init] *ERROR* radeon_cp_init called without lock held
> > > > [drm:radeon_unlock] *ERROR* Process 2073 using kernel context 0
> > > >
> > > > There's a patch in -mm2 that does something with drm/agp (haven't looked
> > > > at it yet):
> > > >
> > > > +drm-agp-module-dependency-fix.patch
> > > >
> > > > Maybe fix DRM<->AGP module dependencies so things autoload nicely.
>
> BTW, I believe this is Andrew's description of what the patch is
> supposed to do, not a suggestion of Ronny's.
>
> -#define DRM_AGP_GET (drm_agp_t *)inter_module_get("drm_agp")
> -#define DRM_AGP_PUT inter_module_put("drm_agp")
> +#define DRM_AGP_GET symbol_get(agp_drm)
> +#define DRM_AGP_PUT symbol_put(agp_drm)
>
> I wonder if 'drm_agp' was intentionally changed to 'agp_drm'... is there
> an accompanying agpgart patch? If not, can you try with
> symbol_{get,put}(drm_agp) ?
You're right, it does look like a thinko of some sort.
A guy named Ronny V. Vindenes sent me this patch:
diff -bBpur fix/drivers/char/drm/drm_agpsupport.h
linux-2.6.0-test9-mm2/drivers/char/drm/drm_agpsupport.h
--- fix/drivers/char/drm/drm_agpsupport.h 2003-11-06 00:49:53.299160210 +0100
+++ linux-2.6.0-test9-mm2/drivers/char/drm/drm_agpsupport.h 2003-11-06
00:27:43.000000000 +0100
@@ -37,8 +37,8 @@
#if __REALLY_HAVE_AGP
-#define DRM_AGP_GET symbol_get(agp_drm)
-#define DRM_AGP_PUT symbol_put(agp_drm)
+#define DRM_AGP_GET symbol_get(drm_agp)
+#define DRM_AGP_PUT symbol_put(drm_agp)
/**
* Pointer to the drm_agp_t structure made available by the agpgart module.
diff -bBpur fix/include/linux/agp_backend.h
linux-2.6.0-test9-mm2/include/linux/agp_backend.h
--- fix/include/linux/agp_backend.h 2003-11-06 00:50:03.247046605 +0100
+++ linux-2.6.0-test9-mm2/include/linux/agp_backend.h 2003-11-06
00:42:38.936727765 +0100
@@ -108,8 +108,7 @@ typedef struct {
int (*copy_info)(struct agp_kern_info *);
} drm_agp_t;
-/* Used by drm. */
-extern const drm_agp_t agp_drm;
+extern const drm_agp_t *drm_agp_p;
#endif /* __KERNEL__ */
#endif /* _AGP_BACKEND_H */
that he says fixes the problem. I'm patching the tree as we speak.
>
>
> > > Could either of you try loading the DRM with drm_opts=debug and see if
> > > there's anything interesting in the debugging output of the DRM?
> >
> > .....that the radeon DRM module is compiled into my kernel; it's not modular.
> >
> > Can drm_opts=debug be passed via the kernel command-line?
>
> I guess it can somehow, not sure whether it works as is or needs some
> additional magic though.
OK. If the patch fails I'll try it.
Brad
=====
Brad Chapman
Permanent e-mail: [EMAIL PROTECTED]
__________________________________
Do you Yahoo!?
Protect your identity with Yahoo! Mail AddressGuard
http://antispam.yahoo.com/whatsnewfree
-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive? Does it
help you create better code? SHARE THE LOVE, and help us help
YOU! Click Here: http://sourceforge.net/donate/
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel
