From 4b08f4cf0f06b3345466f7d5d7ef4c04d45ecc56 Mon Sep 17 00:00:00 2001 From: Michael Brown Date: Thu, 28 Jun 2007 17:55:29 +0100 Subject: [PATCH] Quick hack to get image booting working again --- src/hci/commands/image_cmd.c | 63 ++++++++++++++++++++---------------- src/include/gpxe/image.h | 1 + src/include/usr/imgmgmt.h | 3 +- src/usr/autoboot.c | 10 +++++- src/usr/imgmgmt.c | 60 +++++++++++++--------------------- 5 files changed, 69 insertions(+), 68 deletions(-) diff --git a/src/hci/commands/image_cmd.c b/src/hci/commands/image_cmd.c index e1e22e3f6..4fe0ca9c2 100644 --- a/src/hci/commands/image_cmd.c +++ b/src/hci/commands/image_cmd.c @@ -72,19 +72,20 @@ static void imgfetch_core_syntax ( char **argv, int load ) { /** * The "imgfetch"/"module"/"kernel" command body * + * @v image_type Image type to assign (or NULL) + * @v load Image will be automatically loaded after fetching * @v argc Argument count * @v argv Argument list - * @v load Image will be automatically loaded after fetching - * @ret image Fetched image * @ret rc Return status code */ -static int imgfetch_core_exec ( int argc, char **argv, int load, - struct image **image ) { +static int imgfetch_core_exec ( struct image_type *image_type, int load, + int argc, char **argv ) { static struct option longopts[] = { { "help", 0, NULL, 'h' }, { "name", required_argument, NULL, 'n' }, { NULL, 0, NULL, 0 }, }; + struct image *image; const char *name = NULL; char *filename; int c; @@ -116,15 +117,33 @@ static int imgfetch_core_exec ( int argc, char **argv, int load, if ( ! name ) name = basename ( filename ); + /* Allocate image */ + image = alloc_image(); + if ( ! image ) { + printf ( "%s\n", strerror ( -ENOMEM ) ); + return -ENOMEM; + } + + /* Fill in image name */ + if ( name ) + strncpy ( image->name, name, ( sizeof ( image->name ) - 1 ) ); + + /* Set image type (if specified) */ + image->type = image_type; + + /* Fill in command line */ + imgfill_cmdline ( image, ( argc - optind ), &argv[optind] ); + + printf ( "name = %s, filename = %s\n", name, filename ); + /* Fetch the image */ - if ( ( rc = imgfetch ( filename, name, image ) ) != 0 ) { + if ( ( rc = imgfetch ( image, filename, load ) ) != 0 ) { printf ( "Could not fetch %s: %s\n", name, strerror ( rc ) ); + image_put ( image ); return rc; } - /* Fill in command line */ - imgfill_cmdline ( *image, ( argc - optind ), &argv[optind] ); - + image_put ( image ); return 0; } @@ -136,11 +155,10 @@ static int imgfetch_core_exec ( int argc, char **argv, int load, * @ret rc Exit code */ static int imgfetch_exec ( int argc, char **argv ) { - struct image *image; int rc; - if ( ( rc = imgfetch_core_exec ( argc, argv, 0, &image ) ) != 0 ) - return 1; + if ( ( rc = imgfetch_core_exec ( NULL, 0, argc, argv ) ) != 0 ) + return rc; return 0; } @@ -153,18 +171,10 @@ static int imgfetch_exec ( int argc, char **argv ) { * @ret rc Exit code */ static int kernel_exec ( int argc, char **argv ) { - struct image *image; int rc; - if ( ( rc = imgfetch_core_exec ( argc, argv, 1, &image ) ) != 0 ) - return 1; - - /* Load image */ - if ( ( rc = imgload ( image ) ) != 0 ) { - printf ( "Could not load %s: %s\n", image->name, - strerror ( rc ) ); - return 1; - } + if ( ( rc = imgfetch_core_exec ( NULL, 1, argc, argv ) ) != 0 ) + return rc; return 0; } @@ -177,14 +187,11 @@ static int kernel_exec ( int argc, char **argv ) { * @ret rc Exit code */ static int initrd_exec ( int argc, char **argv ) { - struct image *image; int rc; - if ( ( rc = imgfetch_core_exec ( argc, argv, 0, &image ) ) != 0 ) - return 1; - - /* Mark image as an intird */ - image->type = &initrd_image_type; + if ( ( rc = imgfetch_core_exec ( &initrd_image_type, 0, + argc, argv ) ) != 0 ) + return rc; return 0; } @@ -246,7 +253,7 @@ static int imgload_exec ( int argc, char **argv ) { } if ( ( rc = imgload ( image ) ) != 0 ) { printf ( "Could not load %s: %s\n", name, strerror ( rc ) ); - return 1; + return rc; } return 0; diff --git a/src/include/gpxe/image.h b/src/include/gpxe/image.h index 08a1dd905..7f09d9c6e 100644 --- a/src/include/gpxe/image.h +++ b/src/include/gpxe/image.h @@ -113,6 +113,7 @@ extern struct list_head images; #define for_each_image( image ) \ list_for_each_entry ( (image), &images, list ) +extern struct image * alloc_image ( void ); extern int register_image ( struct image *image ); extern void unregister_image ( struct image *image ); extern void promote_image ( struct image *image ); diff --git a/src/include/usr/imgmgmt.h b/src/include/usr/imgmgmt.h index 82a65f252..b3cbffbc4 100644 --- a/src/include/usr/imgmgmt.h +++ b/src/include/usr/imgmgmt.h @@ -9,8 +9,7 @@ struct image; -extern int imgfetch ( const char *filename, const char *name, - struct image **new_image ); +extern int imgfetch ( struct image *image, const char *filename, int load ); extern int imgload ( struct image *image ); extern int imgexec ( struct image *image ); extern struct image * imgautoselect ( void ); diff --git a/src/usr/autoboot.c b/src/usr/autoboot.c index 5d017229d..3936e4ca5 100644 --- a/src/usr/autoboot.c +++ b/src/usr/autoboot.c @@ -99,19 +99,27 @@ void netboot ( struct net_device *netdev ) { return; } printf ( "Booting \"%s\"\n", filename ); - if ( ( rc = imgfetch ( filename, NULL, &image ) ) != 0 ) { + image = alloc_image(); + if ( ! image ) { + printf ( "Out of memory\n" ); + return; + } + if ( ( rc = imgfetch ( image, filename, 0 ) ) != 0 ) { printf ( "Could not retrieve %s: %s\n", filename, strerror ( rc ) ); + image_put ( image ); return; } if ( ( rc = imgload ( image ) ) != 0 ) { printf ( "Could not load %s: %s\n", image->name, strerror ( rc ) ); + image_put ( image ); return; } if ( ( rc = imgexec ( image ) ) != 0 ) { printf ( "Could not execute %s: %s\n", image->name, strerror ( rc ) ); + image_put ( image ); return; } } diff --git a/src/usr/imgmgmt.c b/src/usr/imgmgmt.c index 9fe2e1493..df2519552 100644 --- a/src/usr/imgmgmt.c +++ b/src/usr/imgmgmt.c @@ -21,8 +21,9 @@ #include #include #include -#include -#include +#include +#include +#include #include /** @file @@ -31,6 +32,18 @@ * */ +static int imgfetch_autoload ( struct image *image ) { + int rc; + + if ( ( rc = register_image ( image ) ) != 0 ) + return rc; + + if ( ( rc = image_autoload ( image ) ) != 0 ) + return rc; + + return 0; +} + /** * Fetch an image * @@ -39,39 +52,18 @@ * @ret new_image Newly created image * @ret rc Return status code */ -int imgfetch ( const char *uri_string, const char *name, - struct image **new_image ) { - struct image *image; - struct async async; +int imgfetch ( struct image *image, const char *uri_string, int load ) { int rc; - /* Allocate new image */ - image = malloc ( sizeof ( *image ) ); - if ( ! image ) - return -ENOMEM; - memset ( image, 0, sizeof ( *image ) ); + printf ( "uri_string = %s\n", uri_string ); - /* Fill in image name */ - if ( name ) - strncpy ( image->name, name, ( sizeof ( image->name ) - 1 ) ); + if ( ( rc = create_downloader ( &monojob, image, + ( load ? imgfetch_autoload : + register_image ), + LOCATION_URI_STRING, + uri_string ) ) == 0 ) + rc = monojob_wait(); - /* Download the file */ - if ( ( rc = async_block_progress ( &async, - start_download ( uri_string, &async, - &image->data, - &image->len )))!=0) - goto err; - - /* Register the image */ - if ( ( rc = register_image ( image ) ) != 0 ) - goto err; - - *new_image = image; - return 0; - - err: - ufree ( image->data ); - free ( image ); return rc; } @@ -88,10 +80,6 @@ int imgload ( struct image *image ) { if ( ( rc = image_autoload ( image ) ) != 0 ) return rc; - /* If we succeed, move the image to the start of the list */ -#warning "No longer exists" - // promote_image ( image ); - return 0; } @@ -144,6 +132,4 @@ void imgstat ( struct image *image ) { */ void imgfree ( struct image *image ) { unregister_image ( image ); - ufree ( image->data ); - free ( image ); }