From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55631) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VX0D4-0005fY-45 for qemu-devel@nongnu.org; Thu, 17 Oct 2013 22:54:51 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VX0Cv-0002TZ-5L for qemu-devel@nongnu.org; Thu, 17 Oct 2013 22:54:42 -0400 Received: from e28smtp04.in.ibm.com ([122.248.162.4]:53826) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VX0Cu-0002T9-HS for qemu-devel@nongnu.org; Thu, 17 Oct 2013 22:54:33 -0400 Received: from /spool/local by e28smtp04.in.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Fri, 18 Oct 2013 08:24:19 +0530 Received: from d28relay01.in.ibm.com (d28relay01.in.ibm.com [9.184.220.58]) by d28dlp03.in.ibm.com (Postfix) with ESMTP id 47AD11258054 for ; Fri, 18 Oct 2013 08:24:46 +0530 (IST) Received: from d28av05.in.ibm.com (d28av05.in.ibm.com [9.184.220.67]) by d28relay01.in.ibm.com (8.13.8/8.13.8/NCO v10.0) with ESMTP id r9I2v1a830277674 for ; Fri, 18 Oct 2013 08:27:02 +0530 Received: from d28av05.in.ibm.com (localhost [127.0.0.1]) by d28av05.in.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id r9I2sE6i018687 for ; Fri, 18 Oct 2013 08:24:15 +0530 Message-ID: <5260A2D5.5000506@linux.vnet.ibm.com> Date: Fri, 18 Oct 2013 10:54:13 +0800 From: mike MIME-Version: 1.0 References: <1381810677-4369-1-git-send-email-qiudayu@linux.vnet.ibm.com> <525CCD98.7010707@weilnetz.de> <525D36B1.3050208@redhat.com> <525D4412.2000702@linux.vnet.ibm.com> <20131017123014.GG10774@stefanha-thinkpad.redhat.com> In-Reply-To: <20131017123014.GG10774@stefanha-thinkpad.redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v2] net/net: Change the default mac address of nic List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Stefan Weil , qemu-devel@nongnu.org, stefanha@redhat.com, aliguori@amazon.com On 10/17/2013 08:30 PM, Stefan Hajnoczi wrote: > On Tue, Oct 15, 2013 at 09:33:06PM +0800, mike wrote: >> On 10/15/2013 08:36 PM, Eric Blake wrote: >>> On 10/14/2013 11:07 PM, Stefan Weil wrote: >>>> Is it reasonable to get a random mac address in your guest? I don't >>>> think so. It would no longer be possible to connect to a guest using >>>> ssh, restart that guest and connect again with ssh. >>> Agreed - libvirt ALWAYS passes a MAC to qemu, even if the user did not >>> specify a MAC to libvirt, precisely because the MAC must be reproducible >>> rather than random to avoid changing the guest ABI. I don't think this >>> patch is needed - it's up to management to use qemu correctly. >> Yes, you are right in this condition. But qemu support Mac address unset. >> Also we can get the ip address through a lot of different ways, like use >> monitor to get the mac and then get the ip. So we can login use ssh. >> >> But as you mentioned, this patch is not needed, I don't agree with you. >> >> First, this patch just fix the Potential issue of this feature. Now libvirt >> maybe can't triggered this issue, who can promise in future will not. >> >> The second is, qemu not only be used by libvirt, lots of developers like >> to use the command line to boot up the guest. And in the future, we >> are not sure about other program will use qemu. >> >> The third is, when one feature has a issue in qemu, >> no matter when it is been triggered, should we not fix it? > NACK > > I'm not going to merge this patch: > > If you terminate QEMU and launch it again the NIC gets a different MAC > address. Some guest operating systems are sensitive to this - under For these users must use -device ,mac=XX:XX:XX:XX:XX:XX. I think no body will boot up the guest, which sensitive to this, without mac address. Actually, people use the command line without mac address, mean they mainly don't care about mac address, so give them random mac address is reasonable I think. In my opinion, if we fix this, for qemu side no any issue, we both support mac address set or unset correctly. What am I confuse is, *qemu supports mac address unset, why we force users must set the address when more than one guests*? This is unreasonable. > many Linux distros the network interfaces names change due to the MAC > address change. As a result firewall configuration will break and other > services may fail to start because they cannot find the interface. Agree, so this mac address should set in qemu command line as libvirt does :) > If you have multiple guests or want control over the MAC address, set it > explicitly using -device ,mac=XX:XX:XX:XX:XX:XX. Currently, especially for developers, people mainly use qemu command line directly, and as qemu supports mac address unset, they may try the simplest command line to boot up lots of guests, they will confuse about why all this guest use the same mac address. Thanks Mike > Stefan > > >