Re: CVS commit: src/sys/fs/cd9660

[Jason Thorpe <thorpej%shagadelic.org@localhost>]

A good excuse to stop using link sets.

-- thorpej@iPhone

On May 3, 2008, at 2:43 PM, Chris Gilbert <chris%dokein.co.uk@localhost> wrote:

> Andrew Doran wrote:
> > Module Name:    src
> > Committed By:    ad
> > Date:        Sat May  3 15:57:41 UTC 2008
> >
> > Modified Files:
> >    src/sys/fs/cd9660: cd9660_vfsops.c
> >
> > Log Message:
> > Convert cd9660 to attach as a module.
> >
> >
> > To generate a diff of this commit:
> > cvs rdiff -r1.59 -r1.60 src/sys/fs/cd9660/cd9660_vfsops.c
> >
> > Please note that diffs are not public domain; they are subject to the
> > copyright notices on the relevant files.
>
> This appears to have broken rump, as rump needs the VFS_ATTACH that  
> was
> removed, otherwise there's no vfsops link_set:
>
> dependall ===> sys/rump/fs/bin/cd9660fs
>     create  cd9660fs/cd9660fs.d
>     create  cd9660fs/.depend
>    compile  cd9660fs/cd9660fs.o
>       link  cd9660fs/cd9660fs
> /extra/obj/current/i386/sys.clean/rump/librump/rumpkern/librump.a 
> (vfs_init.o):
> In function `vfsinit':
> /extra/cvs/src/sys/kern/vfs_init.c:340: undefined reference to
> `__start_link_set_vfsops'
> /extra/cvs/src/sys/kern/vfs_init.c:340: undefined reference to
> `__stop_link_set_vfsops'
>
> Thanks,
> Chris