From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44388) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dVOgw-0000Me-2Z for qemu-devel@nongnu.org; Wed, 12 Jul 2017 16:57:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dVOgr-0002DM-Uv for qemu-devel@nongnu.org; Wed, 12 Jul 2017 16:57:02 -0400 Received: from mx1.redhat.com ([209.132.183.28]:35214) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dVOgr-0002C7-LD for qemu-devel@nongnu.org; Wed, 12 Jul 2017 16:56:57 -0400 Date: Wed, 12 Jul 2017 17:56:46 -0300 From: Eduardo Habkost Message-ID: <20170712205646.GQ6020@localhost.localdomain> References: <1499847753-8513-1-git-send-email-thuth@redhat.com> <20170712145121.GF6020@localhost.localdomain> <20170712181652-mutt-send-email-mst@kernel.org> <20170712201550.GN6020@localhost.localdomain> <20170712232214-mutt-send-email-mst@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170712232214-mutt-send-email-mst@kernel.org> Subject: Re: [Qemu-devel] [PATCH v3] hw/i386: Deprecate the machines pc-0.10 to pc-1.2 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" Cc: Thomas Huth , qemu-devel@nongnu.org, Paolo Bonzini , Richard Henderson , Marcel Apfelbaum , Igor Mammedov , Gerd Hoffmann , Eric Blake On Wed, Jul 12, 2017 at 11:31:59PM +0300, Michael S. Tsirkin wrote: > On Wed, Jul 12, 2017 at 05:15:50PM -0300, Eduardo Habkost wrote: > > On Wed, Jul 12, 2017 at 06:17:55PM +0300, Michael S. Tsirkin wrote: > > > On Wed, Jul 12, 2017 at 11:51:21AM -0300, Eduardo Habkost wrote: > > > > On Wed, Jul 12, 2017 at 10:22:33AM +0200, Thomas Huth wrote: > > > > > We don't want to carry along old machine types forever. If we are able to > > > > > remove the pc machines up to 0.13 one day for example, this would allow > > > > > us to eventually kill the code for rombar=0 (i.e. where QEMU copies ROM > > > > > BARs directly to low memory). Everything up to pc-1.2 is also known to > > > > > have issues with migration. So let's start with a deprecation message > > > > > for the old machine types so that the (hopefully) few users of these old > > > > > systems start switching over to newer machine types instead. > > > > > > > > > > Signed-off-by: Thomas Huth > > > > > > > > Reviewed-by: Eduardo Habkost > > > > > > > > I suggest changing "-machine help" too. Today it looks like this: > > > > > > > > Supported machines are: > > > > pc-i440fx-2.9 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-2.8 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-2.7 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-2.6 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-2.5 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-2.4 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-2.3 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-2.2 Standard PC (i440FX + PIIX, 1996) > > > > pc Standard PC (i440FX + PIIX, 1996) (alias of pc-i440fx-2.10) > > > > pc-i440fx-2.10 Standard PC (i440FX + PIIX, 1996) (default) > > > > pc-i440fx-2.1 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-2.0 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-1.7 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-1.6 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-1.5 Standard PC (i440FX + PIIX, 1996) > > > > pc-i440fx-1.4 Standard PC (i440FX + PIIX, 1996) > > > > pc-1.3 Standard PC (i440FX + PIIX, 1996) > > > > pc-1.2 Standard PC (i440FX + PIIX, 1996) > > > > pc-1.1 Standard PC (i440FX + PIIX, 1996) > > > > pc-1.0 Standard PC (i440FX + PIIX, 1996) > > > > pc-0.15 Standard PC (i440FX + PIIX, 1996) > > > > pc-0.14 Standard PC (i440FX + PIIX, 1996) > > > > pc-0.13 Standard PC (i440FX + PIIX, 1996) > > > > pc-0.12 Standard PC (i440FX + PIIX, 1996) > > > > pc-0.11 Standard PC (i440FX + PIIX, 1996) > > > > pc-0.10 Standard PC (i440FX + PIIX, 1996) > > > > pc-q35-2.9 Standard PC (Q35 + ICH9, 2009) > > > > pc-q35-2.8 Standard PC (Q35 + ICH9, 2009) > > > > pc-q35-2.7 Standard PC (Q35 + ICH9, 2009) > > > > pc-q35-2.6 Standard PC (Q35 + ICH9, 2009) > > > > pc-q35-2.5 Standard PC (Q35 + ICH9, 2009) > > > > pc-q35-2.4 Standard PC (Q35 + ICH9, 2009) > > > > q35 Standard PC (Q35 + ICH9, 2009) (alias of pc-q35-2.10) > > > > pc-q35-2.10 Standard PC (Q35 + ICH9, 2009) > > > > isapc ISA-only PC > > > > none empty machine > > > > > > Any chance we can sort them reasonably too? > > > > If we use strverscmp(), it will be sorted in a more reasonable > > way. We could copy the gnulib version on systems without glibc. > > Life is too short for writing configure checks by hand, though; I > > will add this to the end of my wish-todo list. If somebody wants > > to volunteer, be my guest. > > > > I'm CCing Eric in case he has suggestions that would help import > > the gnulib module in an easy way. > > > > -- > > Eduardo > > As we never have leading zeroes, and input comes from QEMU so > it's safe, it's probably easier to just open-code it: > > /* compare string numerically: shorter strings give smaller numbers */ > int mstcmp(const char *s1, const char *s2) > { > int l1, l2; > > l1 = strlen(s1); > l2 = strlen(s2); > return l1 == l2 ? strcmp(s1, s2) : l1 - l2; > } > This doesn't work because strlen("pc-0.10") > strlen("pc-1.0"). -- Eduardo