On Fri, Jul 22, 2016 at 03:59:58PM +0800, Chen Hanxiao wrote:
 From: Chen Hanxiao <chenhanxiao(a)gmail.com>
 
   If we failed to add something to initrd, just error out.
 
 Signed-off-by: Chen Hanxiao <chenhanxiao(a)gmail.com>
 ---
  src/ext2_initrd.ml | 6 +++++-
  1 file changed, 5 insertions(+), 1 deletion(-)
 
 diff --git a/src/ext2_initrd.ml b/src/ext2_initrd.ml
 index d4a4e2f..d9a3a99 100644
 --- a/src/ext2_initrd.ml
 +++ b/src/ext2_initrd.ml
 @@ -151,8 +151,12 @@ let rec build_initrd debug tmpdir modpath initrd =
    visit topset;
    close_out chan;
  
 +  let num_visted = StringSet.cardinal !visited in
    if debug >= 1 then
 -    printf "supermin: ext2: wrote %d modules to minimal initrd\n%!"
(StringSet.cardinal !visited);
 +    printf "supermin: ext2: wrote %d modules to minimal initrd\n%!"
num_visted;
 +
 +  if num_visited == 0 then
 +    error "failed to write any modules into minimal initrd, try 'depmod
-a'" 
NACK.  It's not a bug if no modules are added, since all features
needed might be compiled into the kernel.
Rich.
-- 
Richard Jones, Virtualization Group, Red Hat 
http://people.redhat.com/~rjones
Read my programming and virtualization blog: 
http://rwmj.wordpress.com
virt-df lists disk usage of guests without needing to install any
software inside the virtual machine.  Supports Linux and Windows.
http://people.redhat.com/~rjones/virt-df/