[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2] libs/light: pass some infos to qemu

  • To: Manuel Bouyer <bouyer@xxxxxxxxxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Fri, 29 Jan 2021 15:52:14 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=C83cUDM0fn9a/8w3uA7eVM/yslXWV+568oMTj00Qlxo=; b=DOiTNNot6e28GsnBKnlPaHT02oKQqjpD7TFgVEurVCe4W71jRxayzLKW9geqxlkmkTnYYJ1TIlflA79TP8519sErEIfz5Cl1L3K3KRDBoK1/BNl9S7C3qNvgS0qr0dTtqhwwfv5pTWbSazLfDvf69Wtzd+7d1UDUjg3H8R0+X+e/j2pOEtuTJcKY+YHnUA0HzK5HeYIxNpkE6ywHQkRC0eQrtRxAZwDj7Rr8ajNQ9qlQXBKFJB1w2Fn1oR9lF0xTpwgRb4qg6m0b5VYxTM7nPOa78mH4TqrXgQUkTNyXHNjCIopUJHZaE74eExjeAyxWK7afMTbgJlD7973qOvP0Uw==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BQ5XfFrlTh5b6zMQ4RAHhpHcFgr5PRVOA3rIGKoGn9lVwcDEW8YYRFMvqUXZnEYBfIkEf2iJ2no8p2EokhIZ07FjBWTRXC8/CVHI1FUcIXCK+dBewaqtUxsxQkxwAOkYLDWkeCrmHxJd38gM+GugWlBvHXzgxG9bU9ZovV+YxOfX+wiCMAy/DJjN04SPV9Ucpvy1aRXQV+chIdPUkoctq7GpDQFL1Vsk5Uc3WR1v2TTKjYqaxJ+G+c9DGZGQLNznWrwFfrLJ9ZW/ZZUPaqKzDuFyeOXvTx8QBCtbnvR+m0DLhXp1Sm0OFolnw/zaVTP4yDCMpSh5nw7VU0VWxmUxPQ==
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: <xen-devel@xxxxxxxxxxxxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Delivery-date: Fri, 29 Jan 2021 14:52:35 +0000
  • Ironport-sdr: W/qldZ0sGTDz+J1sUK63YWi/mst1qNTAfU4UpRT6Cga2Bb+DM/CAu6vvnPcx7eSAHHb3IgFz2R hpOReXs0o4+is8klvDyeflUk0O0yaCgAOK/9r7PEkGtAXcoU/JrpuMQnWFUXtn+1I+QMjF5eLT mSTXFBa+GvGrrzLdRbmV7zNFm9MAnoLwn7oH+iWYPNAtMDO9CuwONNw6AeiF5Zt32YeEiu4oSc ivLuEFALKmOUUbWlXjfQ7GygvtdHabPK8QxwXm33T8HP/OLaEq0LvJFzdQrzlK3V/8wWWtqvpo G8w=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Fri, Jan 29, 2021 at 11:46:53AM +0100, Manuel Bouyer wrote:
> On Thu, Jan 28, 2021 at 12:08:02PM +0100, Roger Pau Monné wrote:
> > On Tue, Jan 26, 2021 at 11:47:58PM +0100, Manuel Bouyer wrote:
> > > Pass bridge name to qemu as command line option
> > > When starting qemu, set an environnement variable XEN_DOMAIN_ID,
> > > to be used by qemu helper scripts
> > > The only functional difference of using the br parameter is that the
> > > bridge name gets passed to the QEMU script.
> > > NetBSD doesn't have the ioctl to rename network interfaces implemented, 
> > > and
> > > thus cannot rename the interface from tapX to vifX.Y-emu. Only qemu knowns
> > > the tap interface name, so we need to use the qemu script from qemu 
> > > itself.
> > > 
> > > Signed-off-by: Manuel Bouyer <bouyer@xxxxxxxxxx>
> > 
> > Reviewed-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> > 
> > If you have a moment might be worth adding a note in
> > xl-network-configuration.5.pod that NetBSD in HVM mode requires
> > bridged networking I think?
> With the default qemu-ifup script only. As you can do whatever you want
> in the script, you can support whatever network configuration you want.
> It's quite easy to do IP routing for example.

Right, but the default script provided will do bridging mode only, and
even if you add 'script=vif-ip' to the network configuration line it
won't do what you expect. Instead it will try to add the tap network
interface to the default xenbr0 bridge.

I'm not opposed to having it this way right now, as it's better to
have this than no support at all, but we should have the shortcoming
documented somewhere. Can be done as a separate patch.

> > 
> > Also, the qemu-ifup script doesn't seem to be part of the NetBSD
> > scripts that are upstream, is this something carried by the NetBSD
> > package?
> Ha maybe I overlooked this. I'll add it, but maybe it can be submitted in
> a separate patch ?

Yes, that's fine, can be added as a separate patch.

Thanks, Roger.



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.