Re: PR/47974 CVS commit: src/tests/fs/common

[Martin Husemann <martin%duskware.de@localhost>]

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

From: Martin Husemann <martin%duskware.de@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: reinoud%NetBSD.org@localhost
Subject: Re: PR/47974 CVS commit: src/tests/fs/common
Date: Sun, 30 Jun 2013 17:37:44 +0200

 With some more tuning it now end up with:
 
 udf_mountfs (args=0x7f7ff7b66be0, l=0x7f7ff7b62000, mp=0x7f7ff6fc0000, 
     devvp=0x7f7ff6f01250)
     at /usr/src/sys/rump/fs/lib/libudf/../../../../fs/udf/udf_vfsops.c:612
 612             sector_size = ump->discinfo.sector_size;
 (gdb) 
 614             while ((1 << bshift) < sector_size)
 (gdb) print sector_size
 $7 = 512
 
 since the discinfo could not be retrieved from the image via MMCGETDISCINFO,
 the default values have been filled in:
 
 (gdb) print ump->discinfo 
 $9 = {mmc_profile = 1, mmc_class = 1, disc_state = 3 '\003', 
   last_session_state = 3 '\003', bg_format_state = 3 '\003', 
   link_block_penalty = 0 '\000', mmc_cur = 3106, mmc_cap = 3106, 
   disc_flags = 8, disc_id = 0, disc_barcode = 0, application_code = 0 '\000', 
   unused1 = "\000\000", last_possible_lba = 10000, sector_size = 512, 
   num_sessions = 1, num_tracks = 1, first_track = 1, 
   first_track_last_session = 1, last_track_last_session = 1, unused2 = 0, 
   reserved1 = {0, 0, 0, 0}, reserved2 = {0, 0, 0}, reserved3 = {0, 0, 0, 0, 0, 
     0, 0, 0}}
 
 
 but with the simplistic aproach to fake a all-0-image instead of newfs_udf
 I used, the mount fails because num_anchors is 0:
 
 661             /* read all anchors to get volume descriptor sequence */
 662             num_anchors = udf_read_anchors(ump);
 663             if (num_anchors == 0)
 664                     return EINVAL;
 
 Reinoud, can you add -F support to newfs_udf? Untill then I see no use in
 including the udf tests in the official runs.
 
 Martin