X-Git-Url: http://git.ozlabs.org/?p=petitboot;a=blobdiff_plain;f=devices%2Fyaboot-parser.c;h=8d65ce27e1cc7bd6f29268785b0df31d6394b0fc;hp=f457abd7dca06682741a19dc7829f305023bb8e7;hb=84a9a8660a90551f2f934ac77ac0c9b2c2e8c079;hpb=678dbd282f917c68c4fd6badfc14fcf464796f5d

diff --git a/devices/yaboot-parser.c b/devices/yaboot-parser.c
index f457abd..8d65ce2 100644
--- a/devices/yaboot-parser.c
+++ b/devices/yaboot-parser.c
@@ -15,7 +15,6 @@
 
 static struct device *dev;
 static const char *mountpoint;
-static char partition_mntpoint[PATH_MAX];
 static char *defimage;
 
 char *
@@ -103,13 +102,13 @@ void process_image(char *label)
 
 	opt.name = label;
 	cfgopt = cfg_get_strg(label, "image");
-	opt.boot_image_file = join_paths(mountpoint, cfgopt);
+	opt.boot_image_file = resolve_path(cfgopt, mountpoint);
 	if (cfgopt == defimage)
-		printf("This one is default. What do we do about it?\n");
+		pb_log("This one is default. What do we do about it?\n");
 
 	cfgopt = cfg_get_strg(label, "initrd");
 	if (cfgopt)
-		opt.initrd_file = join_paths(mountpoint, cfgopt);
+		opt.initrd_file = resolve_path(cfgopt, mountpoint);
 
 	opt.boot_args = make_params(label, NULL);
 
@@ -164,7 +163,7 @@ static int yaboot_parse(const char *devicepath, const char *_mountpoint)
 	close(fd);
 	
 	if (cfg_parse(filepath, conf_file, conf_len)) {
-		printf("Error parsing yaboot.conf\n");
+		pb_log("Error parsing yaboot.conf\n");
 		return 0;
 	}
 
@@ -182,8 +181,8 @@ static int yaboot_parse(const char *devicepath, const char *_mountpoint)
 	}
 	dev->icon_file = strdup(generic_icon_file(guess_device_type()));
 
-	/* Mount the 'partition' which is what all the image filenames
-	   are relative to */
+	/* If we have a 'partiton=' directive, update the default mountpoint
+	 * to use that instead of the current mountpoint */
 	tmpstr = cfg_get_strg(0, "partition");
 	if (tmpstr) {
 		char *endp;
@@ -203,16 +202,9 @@ static int yaboot_parse(const char *devicepath, const char *_mountpoint)
 			/* and add our own... */
 			sprintf(endp+1, "%d", partnr);
 
-			/* FIXME: udev may not have created the device node
-			   yet. And on removal, unmount_device() only unmounts
-			   it once, while in fact it may be mounted twice. */
-			if (mount_device(new_dev, partition_mntpoint)) {
-				printf("Error mounting image partition\n");
-				return 0;
-			}
-			mountpoint = partition_mntpoint;
-			dev->id = new_dev;
-		}				
+			mountpoint = mountpoint_for_device(new_dev);
+			free(new_dev);
+		}
 	}
 
 	defimage = cfg_get_default();