Re: PR/49153 CVS commit: src/sys/rump/librump/rumpkern

[Andreas Gustafsson <gson%gson.org@localhost>]

The following reply was made to PR bin/49153; it has been noted by GNATS.

From: Andreas Gustafsson <gson%gson.org@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: pgoyette%NetBSD.org@localhost, John Nemeth <jnemeth%CornerstoneService.ca@localhost>, Antti Kantee <pooka%NetBSD.org@localhost>
Subject: Re: PR/49153 CVS commit: src/sys/rump/librump/rumpkern
Date: Fri, 22 Jan 2016 12:06:10 +0200

 There has been some further discussion about this issue in private
 email, which I'm summarizing here for the record.
 
 On the one hand, it seems to me that re-enabling autoloading by
 default in rump kernels would be a more correct fix than changing
 the tests to cope with it being disabled by default, since the
 commit of kern_module.c 1.98 disabled it unintentionally
 (this was confirmed by jnemeth).
 
 If we want this, it can be done by adding
 
     	CPPFLAGS+= -DMODULAR_DEFAULT_AUTOLOAD
 
 to src/sys/rump/librump/rumpkern/Makefile.rumpkern.
 
 On the other hand, pooka has come to question the utility of
 autoloading in rump kernels, and is considering removing it
 altogether.  If it is going to be removed, enabling it by default
 seems pointless.
 
 In any case, the test failures this PR was filed to report no longer
 occur, so I will close the PR.
 -- 
 Andreas Gustafsson, gson%gson.org@localhost