On Mon, Mar 09, 2015 at 12:20:58PM +0000, Justin Cormack wrote: > On Mar 9, 2015 11:27 AM, "Patrick Welche" <[email protected]> wrote: > > > > On Sat, Mar 07, 2015 at 09:55:08PM +0100, J. Hannken-Illjes wrote: > > > On 07 Mar 2015, at 18:03, Patrick Welche <[email protected]> wrote: > > > > > > > - if (modclass != MODULE_CLASS_EXEC || error != ENOENT) > > > > + if ((modclass != MODULE_CLASS_EXEC || error != > ENOENT) && > > > > + root_device != NULL) > > > > > > > > but why? Compiler bug (gcc)? > > > > > > Given this fragment is "#ifdef DEBUG" it looks like rump_server has to > > > be linked with librumpvfs in the DEBUG case? > > > > Indeed! The attached seems to fix the build - comments? > > > > -lrumpvfs needs to be inside the whole-archive linker flags.
Worse than that: the additional reference to root_device appeared in kern_module.c. That source file is used by librump. Does librump really want to depend on librumpvfs if DEBUG? (probably not) Cheers, Patrick
