From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48610) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UotyS-0002Jd-Fk for qemu-devel@nongnu.org; Tue, 18 Jun 2013 07:21:27 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UotyM-000311-Ef for qemu-devel@nongnu.org; Tue, 18 Jun 2013 07:21:20 -0400 Received: from mx1.redhat.com ([209.132.183.28]:8602) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UotyM-00030o-7J for qemu-devel@nongnu.org; Tue, 18 Jun 2013 07:21:14 -0400 Date: Tue, 18 Jun 2013 13:21:11 +0200 From: Stefan Hajnoczi Message-ID: <20130618112111.GF7649@stefanha-thinkpad.redhat.com> References: <1371476111-4449-1-git-send-email-akong@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1371476111-4449-1-git-send-email-akong@redhat.com> Subject: Re: [Qemu-devel] [PATCH 0/2] fix macaddr allocation List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Amos Kong Cc: aliguori@us.ibm.com, qemu-devel@nongnu.org, mst@redhat.com On Mon, Jun 17, 2013 at 09:35:09PM +0800, Amos Kong wrote: > This patchset adds a check in allocating mac to new nic, > it will avoid to use repeated mac. The second extends > the address space. > > Amos Kong (2): > avoid to allcate used macaddr to to new nic > extend the macaddr space to 0xffffffff > > net/net.c | 38 ++++++++++++++++++++++++++++++++++---- > 1 file changed, 34 insertions(+), 4 deletions(-) I agree with Michael here and think neither patch should be merged. Both of these patches provide some level of convenience but they cannot solve the problem. Things already work fine automatically in small setups where the user depends on QEMU to handle MAC allocation. In bigger setups you absolutely have to manage MACs or face collisions. So I don't think it's worth trying to maintain the illusion in scenarios that are too complex to automatically allocate for. Stefan