[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] Re: [PATCH 1/4] Add 'set_link' monitor command
From: |
Mark McLoughlin |
Subject: |
Re: [Qemu-devel] Re: [PATCH 1/4] Add 'set_link' monitor command |
Date: |
Tue, 06 Jan 2009 14:59:27 +0000 |
Hi Anthony,
On Fri, 2008-12-12 at 14:45 +0000, Mark McLoughlin wrote:
> On Tue, 2008-12-09 at 09:01 -0600, Anthony Liguori wrote:
> > > + for(vc = vlan->first_client; vc != NULL; vc = vc->next)
> > > + if (i++ == device_idx)
> > > + break;
> > >
> >
> > I think a better way would be to allow a user to specify an id for the
> > nic that gets saved early on. info nics should print the id and a
> > default id should be assigned. I think you could argue either way as to
> > whether the ids should be vlan local or global.
> >
> > I sort of think that making them global has a number of advantages. You
> > could also use a string to identify the nics globally, it's up to you.
>
> Okay, here's a few patches to add a vlan client name.
>
> I'll repost the set_link patches once we agree on this part.
Re-posting a re-based version of these patches; also includes a fix for
the build error pointer out by Aurelien Jarno.
Cheers,
Mark.
- Re: [Qemu-devel] Re: [PATCH 1/4] Add 'set_link' monitor command,
Mark McLoughlin <=
[Qemu-devel] Re: [PATCH 1/6] Add a model string to VLANClientState, Anthony Liguori, 2009/01/07