From patchwork Tue Jan 20 20:13:00 2009 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jonthan Brassow X-Patchwork-Id: 3333 X-Patchwork-Delegate: agk@redhat.com Received: from hormel.redhat.com (hormel1.redhat.com [209.132.177.33]) by demeter.kernel.org (8.14.2/8.14.2) with ESMTP id n0KK8VZ6018611 for ; Tue, 20 Jan 2009 12:08:32 -0800 Received: from listman.util.phx.redhat.com (listman.util.phx.redhat.com [10.8.4.110]) by hormel.redhat.com (Postfix) with ESMTP id 0CB5D619883; Tue, 20 Jan 2009 15:13:03 -0500 (EST) Received: from int-mx2.corp.redhat.com (nat-pool.util.phx.redhat.com [10.8.5.200]) by listman.util.phx.redhat.com (8.13.1/8.13.1) with ESMTP id n0KKD1bR030569 for ; Tue, 20 Jan 2009 15:13:01 -0500 Received: from ns3.rdu.redhat.com (ns3.rdu.redhat.com [10.11.255.199]) by int-mx2.corp.redhat.com (8.13.1/8.13.1) with ESMTP id n0KKD1Xc030349 for ; Tue, 20 Jan 2009 15:13:02 -0500 Received: from [10.15.80.1] (hydrogen.msp.redhat.com [10.15.80.1]) by ns3.rdu.redhat.com (8.13.8/8.13.8) with ESMTP id n0KKD161006430 for ; Tue, 20 Jan 2009 15:13:01 -0500 From: Jonathan Brassow To: dm-devel@redhat.com Date: Tue, 20 Jan 2009 14:13:00 -0600 Message-Id: <1232482380.19993.46.camel@hydrogen.msp.redhat.com> Mime-Version: 1.0 X-Scanned-By: MIMEDefang 2.58 on 172.16.27.26 X-loop: dm-devel@redhat.com Subject: [dm-devel] [PATCH 11 of 12]: dm-exception-store-generalize-table-args.patch X-BeenThere: dm-devel@redhat.com X-Mailman-Version: 2.1.5 Precedence: junk Reply-To: device-mapper development List-Id: device-mapper development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dm-devel-bounces@redhat.com Errors-To: dm-devel-bounces@redhat.com brassow Rework 'dm_exception_store_create' arguments to make way for cleaner interface when new constructor table format is introduced. Signed-off-by: Jonthan Brassow --- dm-devel mailing list dm-devel@redhat.com https://www.redhat.com/mailman/listinfo/dm-devel Index: linux-2.6/drivers/md/dm-exception-store.c =================================================================== --- linux-2.6.orig/drivers/md/dm-exception-store.c +++ linux-2.6/drivers/md/dm-exception-store.c @@ -195,16 +195,15 @@ static int set_chunk_size(struct dm_exce return 0; } -int dm_exception_store_create(struct dm_target *ti, int argc, char **argv, - unsigned *args_used, +int dm_exception_store_create(const char *type_name, struct dm_target *ti, + int argc, char **argv, struct dm_exception_store **store) { int r = 0; struct dm_exception_store_type *type; struct dm_exception_store *tmp_store; - char persistent; - if (argc < 3) { + if (argc < 2) { ti->error = "Insufficient exception store arguments"; return -EINVAL; } @@ -215,13 +214,7 @@ int dm_exception_store_create(struct dm_ return -ENOMEM; } - persistent = toupper(*argv[1]); - if (persistent != 'P' && persistent != 'N') { - ti->error = "Persistent flag is not P or N"; - return -EINVAL; - } - - type = get_type(argv[1]); + type = get_type(type_name); if (!type) { ti->error = "Exception store type not recognised"; r = -EINVAL; @@ -231,6 +224,12 @@ int dm_exception_store_create(struct dm_ tmp_store->type = type; tmp_store->ti = ti; + /* + * COW-dev and chunk_size are common to all types of + * exception stores and are stored directly in the + * dm_exception_store and not passed on to the + * constructor for the dm_exception_store_type + */ r = dm_get_device(ti, argv[0], 0, 0, FMODE_READ | FMODE_WRITE, &tmp_store->cow); if (r) { @@ -238,17 +237,21 @@ int dm_exception_store_create(struct dm_ goto bad_cow; } - r = set_chunk_size(tmp_store, argv[2], &ti->error); - if (r) + r = set_chunk_size(tmp_store, argv[1], &ti->error); + if (r) { + ti->error = "Unable to set chunk size"; goto bad_cow; + } + + argc -= 2; + argv += 2; - r = type->ctr(tmp_store, 0, NULL); + r = type->ctr(tmp_store, argc, argv); if (r) { ti->error = "Exception store type constructor failed"; goto bad_ctr; } - *args_used = 3; *store = tmp_store; return 0; Index: linux-2.6/drivers/md/dm-exception-store.h =================================================================== --- linux-2.6.orig/drivers/md/dm-exception-store.h +++ linux-2.6/drivers/md/dm-exception-store.h @@ -167,8 +167,8 @@ static inline chunk_t sector_to_chunk(st int dm_exception_store_type_register(struct dm_exception_store_type *type); int dm_exception_store_type_unregister(struct dm_exception_store_type *type); -int dm_exception_store_create(struct dm_target *ti, int argc, char **argv, - unsigned *args_used, +int dm_exception_store_create(const char *type_name, struct dm_target *ti, + int argc, char **argv, struct dm_exception_store **store); void dm_exception_store_destroy(struct dm_exception_store *store); Index: linux-2.6/drivers/md/dm-snap.c =================================================================== --- linux-2.6.orig/drivers/md/dm-snap.c +++ linux-2.6/drivers/md/dm-snap.c @@ -571,6 +571,43 @@ static int init_hash_tables(struct dm_sn } /* + * create_exception_store + * @ti + * @argc + * @argv + * @args_used + * @store: contains newly allocated dm_exception_store + * + * Possible formats for argv:: + * p/n + * + * Returns: 0 on success, -Exxx on error + */ +static int create_exception_store(struct dm_target *ti, unsigned argc, + char **argv, unsigned *args_used, + struct dm_exception_store **store) +{ + char *tmp_argv[2]; + + *store = NULL; + + if (1 /* less change patch to patch */) { + if (argc < 3) { + ti->error = "Insufficient exception store arguments"; + return -EINVAL; + } + + tmp_argv[0] = argv[0]; /* COW dev */ + tmp_argv[1] = argv[2]; /* chunk size */ + + *args_used = 3; + + return dm_exception_store_create(toupper(*argv[1]), ti, 2, + tmp_argv, store); + } +} + +/* * Construct a snapshot mapping:

*/ static int snapshot_ctr(struct dm_target *ti, unsigned int argc, char **argv) @@ -592,12 +629,9 @@ static int snapshot_ctr(struct dm_target argv++; argc--; - r = dm_exception_store_create(ti, argc, argv, &args_used, &store); - if (r) { - ti->error = "Couldn't create exception store"; - r = -EINVAL; - goto bad_args; - } + r = create_exception_store(ti, argc, argv, &args_used, &store); + if (r) + return r; argv += args_used; argc -= args_used; @@ -1408,7 +1442,7 @@ static int __init dm_snapshot_init(void) r = dm_register_target(&snapshot_target); if (r) { DMERR("snapshot target register failed %d", r); - return r; + goto bad0; } r = dm_register_target(&origin_target); @@ -1465,6 +1499,8 @@ bad2: dm_unregister_target(&origin_target); bad1: dm_unregister_target(&snapshot_target); +bad0: + dm_exception_store_exit(); return r; }