From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38377) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WwXkI-0005SP-DK for qemu-devel@nongnu.org; Mon, 16 Jun 2014 10:18:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WwXkC-0005w5-7P for qemu-devel@nongnu.org; Mon, 16 Jun 2014 10:18:50 -0400 Received: from mx1.redhat.com ([209.132.183.28]:5779) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WwXkB-0005s7-V0 for qemu-devel@nongnu.org; Mon, 16 Jun 2014 10:18:44 -0400 Date: Mon, 16 Jun 2014 10:18:37 -0400 From: Jeff Cody Message-ID: <20140616141837.GC20898@localhost.localdomain> References: <781eb304e07fb4a333e76cea68958a978bb0bfa3.1402683788.git.jcody@redhat.com> <20140616141041.GA12177@irqsave.net> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <20140616141041.GA12177@irqsave.net> Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v5 01/10] block: Auto-generate node_names for each BDS entry List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?iso-8859-1?Q?Beno=EEt?= Canet Cc: kwolf@redhat.com, pkrempa@redhat.com, famz@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com On Mon, Jun 16, 2014 at 04:10:41PM +0200, Beno=EEt Canet wrote: > The Friday 13 Jun 2014 =E0 14:52:29 (-0400), Jeff Cody wrote : > > Currently, node_name is only filled in when done so explicitly by the > > user. If no node_name is specified, then the node name field is not > > populated. > >=20 > > If node_names are automatically generated when not specified, that me= ans > > that all block job operations can be done by reference to the unique > > node_name field. This eliminates ambiguity in resolving filenames > > (relative filenames, or file descriptors, symlinks, mounts, etc..) th= at > > qemu currently needs to deal with. > >=20 > > If a node name is specified, then it will not be automatically > > generated for that BDS entry. > >=20 > > If it is automatically generated, it will be prefaced with "__qemu##"= , > > followed by 8 characters of a unique number, followed by 8 random > > ASCII characters in the range of 'A-Z'. Some sample generated node-n= ame > > strings: > > __qemu##00000000IAIYNXXR > > __qemu##00000002METXTRBQ > > __qemu##00000001FMBORDWG > >=20 > > The prefix is to aid in identifying it as a qemu-generated name, the > > numeric portion is to guarantee uniqueness in a given qemu session, a= nd > > the random characters are to further avoid any accidental collisions > > with user-specified node-names. >=20 > if the __qemu## determined a different name space no collision resoluti= on would be > needed. >=20 True. The random string generation was pretty trivial, however, and has an added benefit: it helps prevent the user / management software from assuming or attempting to predict node names for images. > Also Eric you must take care that these node name does not ends up in l= ibvirt > XML since a new qemu process can generate a new different set of node n= ame. > Yes.. the node names are guaranteed to be unique, but are ephemeral in nature. > >=20 > > Reviewed-by: Eric Blake > > Signed-off-by: Jeff Cody > > --- > > block.c | 16 +++++++++++++++- > > 1 file changed, 15 insertions(+), 1 deletion(-) > >=20 > > diff --git a/block.c b/block.c > > index 17f763d..fd43016 100644 > > --- a/block.c > > +++ b/block.c > > @@ -844,12 +844,26 @@ static int bdrv_open_flags(BlockDriverState *bs= , int flags) > > return open_flags; > > } > > =20 > > +#define GEN_NODE_NAME_PREFIX "__qemu##" > > +#define GEN_NODE_NAME_MAX_LEN (sizeof(GEN_NODE_NAME_PREFIX) + 8 + = 8) > > static void bdrv_assign_node_name(BlockDriverState *bs, > > const char *node_name, > > Error **errp) > > { > > + char gen_node_name[GEN_NODE_NAME_MAX_LEN]; > > + static uint32_t counter; /* simple counter to guarantee uniquene= ss */ > > + > > + /* if node_name is NULL, auto-generate a node name */ > > if (!node_name) { > > - return; > > + int len; > > + snprintf(gen_node_name, GEN_NODE_NAME_MAX_LEN, > > + "%s%08x", GEN_NODE_NAME_PREFIX, counter++); > > + len =3D strlen(gen_node_name); > > + while (len < GEN_NODE_NAME_MAX_LEN - 1) { > > + gen_node_name[len++] =3D g_random_int_range('A', 'Z'); > > + } > > + gen_node_name[GEN_NODE_NAME_MAX_LEN - 1] =3D '\0'; > > + node_name =3D gen_node_name; > > } > > =20 > > /* empty string node name is invalid */ > > --=20 > > 1.8.3.1 > >=20 > >=20