lkml.org 
[lkml]   [2002]   [Dec]   [18]   [last100]   RSS Feed
Views: [more markup]   [less markup]   [headers]   [forward]  
 
Messages in this thread
/
FromEd Tomlinson <>
SubjectRe: [drm:drm_init] *ERROR* Cannot initialize the agpgart module.
DateWed, 18 Dec 2002 07:57:53 -0500
On December 18, 2002 04:46 am, Rusty Russell wrote:
> In message <20021218102004.A4947@suse.de> you write:
> > On Wed, Dec 18, 2002 at 06:07:48PM +1100, Rusty Russell wrote:
> >  > Dave, it's true that it's my fault, but I'm afraid it looks like your
> >  > bug 8).  This is most likely karma for not reporting bugs in the
> >  > module code when you had problems 8)
> >
> > Bah, my bad. 8)
>
> That's OK, you're in good company as the first I heard from Alan was
> that he gave up kernel work because "modules was so broken". 8(
>
> >  > static int __init agp_backend_initialize(struct pci_dev *dev)
> >  >
> >  > ....
> >  > int agp_register_driver (struct pci_dev *dev)
> >  > {
> >  > ....
> >  > 	ret_val = agp_backend_initialize(dev);
> >
> > Whoops. Thanks, fixed up here.
> > I'll bet I duped that bug in other places too, as I've rejigged
> > stuff around. I'll double check those paths in a mo.
> >
> >  > Ed, does it work if you take all the __init out of the agp code?
> >
> > My moneys on it working. The oops looked like it was jumping to oblivion
> > when it called agp_backend_initialize. So the new modutils discards
> > __init sections ? That's a new feature isn't it ?
>
> Yeah, kind of a bonus.  It's actually arch-dependent.
>
> I've added an item in my TODO list to check for relocations on the
> non-init section which point into the init sections.  It'd be cute.
> It'll probably never go into the main kernel, and it's no actually an
> error, you can imagine code which does:
>
> 	if (initializing)
> 		some_init_func();

Dave when you have this in a bk tree let me know and I will pull and 
verify it working here.

Thanks to both of you

Ed Tomlinson
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 11:31    [W:0.225 / U:0.340 seconds]
©2003-2008 Jasper Spaans