[{"id":1769236,"web_url":"http://patchwork.ozlabs.org/comment/1769236/","msgid":"<20170915170326.5278d578@nial.brq.redhat.com>","list_archive_url":null,"date":"2017-09-15T15:03:26","subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","submitter":{"id":11305,"url":"http://patchwork.ozlabs.org/api/people/11305/","name":"Igor Mammedov","email":"imammedo@redhat.com"},"content":"On Wed, 13 Sep 2017 18:04:52 +0200\nIgor Mammedov <imammedo@redhat.com> wrote:\n\n> Changelog since v1:\n>  * fix merge conflicts with ignore_memory_transaction_failures\n>  * fix couple merge conflicts where SoC type string where replaced by type macro\n>  * keep plain prefix string in: strncmp(cpu_type, \"pxa27\", 5)\n>  * s/\"%s\" ARM_CPU_TYPE_SUFFIX/ARM_CPU_TYPE_NAME(\"%s\")/\n>  * drop not needed assert\n>  * instead of checking error/reporting/exiting explicitly\n>    use error_fatal which will do all of it for us\n>  * squash in \"cpu: rename cpu_parse_features() to cpu_parse_cpu_model()\"\n> \n> \n> Issue 1:                                                                         \n> Some callers call CPUClass->parse_features manually to convert                   \n> '-cpu cpufoo,featurestr' string to cpu type and featurestr                       \n> into a set of global properties and then do controlled                           \n> cpu creation with setting properties and completing it with realize.             \n> That's a lot of code duplication as they all are practically                     \n> reimplement the same parsing logic.                                              \n>                                                                                  \n> Some use cpu_generic_init() instead which does the same parsing                  \n> along with creation/realizing cpu within one wrapper.                            \n>                                                                                  \n> And some trying to switch to controlled cpu creation,                            \n> implement object_new()/set properties/realize steps                              \n> but forget feature parsing logic witch leads to 'bugs'                           \n> commit (00909b585 hw/arm/integratorcp: Support specifying features via -cpu)     \n>                                                                                  \n> Issue 2:                                                                         \n> Default cpu model selection logic is spread over  all board's                    \n> machine_init() fuctions but it's basicallyi hardcodes default                    \n> cpu model string in init function.                                               \n>                                                                                  \n>  if (!cpu_model) {                                                               \n>      cpu_model = \"some cpu model string\";                                        \n>  }                                                                               \n>                                                                                  \n> and written in different ways.                                                   \n> it forces machine_init callbacks to parse cpu_model string                       \n> either by using cpu_generic_init() or by manually calling                        \n> cpu_class_by_name()/CPUClass::parse_features to perform                          \n> name to cpu type translation.\n> \n> This series moves -cpu option parsing to generic machine code                    \n> that removes some of code duplication and makes cpus creation                    \n> process more generic/simple:                                                            \n>                                                                                  \n>  * unify default (fallback) cpu type handling by replacing                       \n>    hardcoded cpu_model strings with cpu type directly in                         \n>                                                                                  \n>    machine_foo_class_init() {                                                    \n>        MachineClass::default_cpu_type = BOARD_DEFAULT_CPU_TYPE                   \n>    }                                                                             \n>                                                                                  \n>    which allows to generalize move cpu model parsing instead of                  \n>    parsing it in each board.                                                     \n>                                                                                  \n>  * make generic machine vl.c parse cpu_model into properties/cpu_type            \n>    and let boards use cpu_type without any cpu_model prasing.                    \n>    Generic parsing will kick in only if board advertises its support             \n>    by setting MachineClass::default_cpu_type to a cpu type.                      \n>                                                                                  \n> PS:                                                                              \n> I intend make tree-wide conversion but as one series it's too many patches,       \n> so I'm splitting out it into an intial series that implements generic            \n> part and several patchsets that will do per target conversion.                   \n>                                                                                  \n> As part of initial series x86 and ARM targets conversion is included             \n> to showcase generalization usage. Per target conversions are done\n> as 1 patch per target, it might be too much for targets that have lots\n> of boards (ARM) so let me know if you'd like to split it on per board\n> basis (then I'll respin it as separate series on top of generic patches)\n> \n> github tree for testing:\n> https://github.com/imammedo/qemu.git default_machine_cpu_type_PC_ARM_v2\n> \nHi Eduardo,\n\nCould you merge it via machine tree?","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","ext-mx02.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx02.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=imammedo@redhat.com"],"Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xtzFF4RRvz9sBZ\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSat, 16 Sep 2017 01:04:00 +1000 (AEST)","from localhost ([::1]:53728 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1dss9t-0003xA-EO\n\tfor incoming@patchwork.ozlabs.org; Fri, 15 Sep 2017 11:03:57 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:43087)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <imammedo@redhat.com>) id 1dss9Y-0003rQ-B4\n\tfor qemu-devel@nongnu.org; Fri, 15 Sep 2017 11:03:37 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <imammedo@redhat.com>) id 1dss9T-00020r-UF\n\tfor qemu-devel@nongnu.org; Fri, 15 Sep 2017 11:03:36 -0400","from mx1.redhat.com ([209.132.183.28]:51438)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from <imammedo@redhat.com>) id 1dss9T-0001yt-K3\n\tfor qemu-devel@nongnu.org; Fri, 15 Sep 2017 11:03:31 -0400","from smtp.corp.redhat.com\n\t(int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mx1.redhat.com (Postfix) with ESMTPS id 0701A883C4\n\tfor <qemu-devel@nongnu.org>; Fri, 15 Sep 2017 15:03:28 +0000 (UTC)","from nial.brq.redhat.com (unknown [10.43.2.209])\n\tby smtp.corp.redhat.com (Postfix) with ESMTP id 7DDC17B5AC\n\tfor <qemu-devel@nongnu.org>; Fri, 15 Sep 2017 15:03:27 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com 0701A883C4","Date":"Fri, 15 Sep 2017 17:03:26 +0200","From":"Igor Mammedov <imammedo@redhat.com>","To":"qemu-devel@nongnu.org","Message-ID":"<20170915170326.5278d578@nial.brq.redhat.com>","In-Reply-To":"<1505318697-77161-1-git-send-email-imammedo@redhat.com>","References":"<1505318697-77161-1-git-send-email-imammedo@redhat.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=US-ASCII","Content-Transfer-Encoding":"7bit","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.12","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.26]);\n\tFri, 15 Sep 2017 15:03:28 +0000 (UTC)","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic]\n\t[fuzzy]","X-Received-From":"209.132.183.28","Subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1770306,"web_url":"http://patchwork.ozlabs.org/comment/1770306/","msgid":"<20170918184213.2fd0ead5@Igors-MacBook-Pro.local>","list_archive_url":null,"date":"2017-09-18T16:42:13","subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","submitter":{"id":11305,"url":"http://patchwork.ozlabs.org/api/people/11305/","name":"Igor Mammedov","email":"imammedo@redhat.com"},"content":"On Fri, 15 Sep 2017 17:03:26 +0200\nIgor Mammedov <imammedo@redhat.com> wrote:\n\n[...]\n> > \n> Hi Eduardo,\n> \n> Could you merge it via machine tree?\nForgot to actually CC Eduardo,\n\nJust noticed a patch (1) on list adding a new soc that would break,\ndue to armv7 dropping cpu-model property in 5/5.\nIt would be better to merge this series asap so that people\ncould rebase their work on cpu-type based approach, instead of\ncontinuing copy-past old cpu-model based code.\n\n1)\n[Qemu devel v9 PATCH 4/5] msf2: Add Smartfusion2 SoC\n\nCCing Subbaraya so he could amend patch to use cpu-type instead of cpu-model.","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","ext-mx09.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx09.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=imammedo@redhat.com"],"Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xwsHs0pLsz9s72\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 19 Sep 2017 02:42:49 +1000 (AEST)","from localhost ([::1]:37834 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1dtz8B-00086j-8q\n\tfor incoming@patchwork.ozlabs.org; Mon, 18 Sep 2017 12:42:47 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:54677)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <imammedo@redhat.com>) id 1dtz7n-000852-CJ\n\tfor qemu-devel@nongnu.org; Mon, 18 Sep 2017 12:42:24 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <imammedo@redhat.com>) id 1dtz7j-0007y6-CH\n\tfor qemu-devel@nongnu.org; Mon, 18 Sep 2017 12:42:23 -0400","from mx1.redhat.com ([209.132.183.28]:45880)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from <imammedo@redhat.com>) id 1dtz7j-0007xA-5R\n\tfor qemu-devel@nongnu.org; Mon, 18 Sep 2017 12:42:19 -0400","from smtp.corp.redhat.com\n\t(int-mx03.intmail.prod.int.phx2.redhat.com [10.5.11.13])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mx1.redhat.com (Postfix) with ESMTPS id 2F6DF4A6EF;\n\tMon, 18 Sep 2017 16:42:18 +0000 (UTC)","from Igors-MacBook-Pro.local (ovpn-112-55.ams2.redhat.com\n\t[10.36.112.55])\n\tby smtp.corp.redhat.com (Postfix) with ESMTP id AE4056060A;\n\tMon, 18 Sep 2017 16:42:16 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com 2F6DF4A6EF","Date":"Mon, 18 Sep 2017 18:42:13 +0200","From":"Igor Mammedov <imammedo@redhat.com>","To":"qemu-devel@nongnu.org","Message-ID":"<20170918184213.2fd0ead5@Igors-MacBook-Pro.local>","In-Reply-To":"<20170915170326.5278d578@nial.brq.redhat.com>","References":"<1505318697-77161-1-git-send-email-imammedo@redhat.com>\n\t<20170915170326.5278d578@nial.brq.redhat.com>","MIME-Version":"1.0","Content-Type":"text/plain; charset=US-ASCII","Content-Transfer-Encoding":"7bit","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.13","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.38]);\n\tMon, 18 Sep 2017 16:42:18 +0000 (UTC)","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic]\n\t[fuzzy]","X-Received-From":"209.132.183.28","Subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Cc":"peter.maydell@linaro.org, Eduardo Habkost <ehabkost@redhat.com>,\n\tSubbaraya Sundeep <sundeep.lkml@gmail.com>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1770925,"web_url":"http://patchwork.ozlabs.org/comment/1770925/","msgid":"<20170919121125.GE21016@localhost.localdomain>","list_archive_url":null,"date":"2017-09-19T12:11:25","subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","submitter":{"id":195,"url":"http://patchwork.ozlabs.org/api/people/195/","name":"Eduardo Habkost","email":"ehabkost@redhat.com"},"content":"On Mon, Sep 18, 2017 at 06:42:13PM +0200, Igor Mammedov wrote:\n> On Fri, 15 Sep 2017 17:03:26 +0200\n> Igor Mammedov <imammedo@redhat.com> wrote:\n> \n> [...]\n> > > \n> > Hi Eduardo,\n> > \n> > Could you merge it via machine tree?\n> Forgot to actually CC Eduardo,\n> \n> Just noticed a patch (1) on list adding a new soc that would break,\n> due to armv7 dropping cpu-model property in 5/5.\n> It would be better to merge this series asap so that people\n> could rebase their work on cpu-type based approach, instead of\n> continuing copy-past old cpu-model based code.\n> \n> 1)\n> [Qemu devel v9 PATCH 4/5] msf2: Add Smartfusion2 SoC\n> \n> CCing Subbaraya so he could amend patch to use cpu-type instead of cpu-model.\n\nQueued on machine-next:\n\n https://github.com/ehabkost/qemu.git machine-next","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","ext-mx08.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx08.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=ehabkost@redhat.com"],"Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xxMGP603dz9s4q\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 19 Sep 2017 22:13:17 +1000 (AEST)","from localhost ([::1]:42504 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1duHOt-0006Ji-Ra\n\tfor incoming@patchwork.ozlabs.org; Tue, 19 Sep 2017 08:13:15 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:49481)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <ehabkost@redhat.com>) id 1duHNP-0005ay-6p\n\tfor qemu-devel@nongnu.org; Tue, 19 Sep 2017 08:11:47 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <ehabkost@redhat.com>) id 1duHNJ-0002GX-Kh\n\tfor qemu-devel@nongnu.org; Tue, 19 Sep 2017 08:11:43 -0400","from mx1.redhat.com ([209.132.183.28]:39214)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from <ehabkost@redhat.com>) id 1duHNJ-0002Fz-Eu\n\tfor qemu-devel@nongnu.org; Tue, 19 Sep 2017 08:11:37 -0400","from smtp.corp.redhat.com\n\t(int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mx1.redhat.com (Postfix) with ESMTPS id DBBE0C057F91;\n\tTue, 19 Sep 2017 12:11:35 +0000 (UTC)","from localhost (ovpn-116-23.gru2.redhat.com [10.97.116.23])\n\tby smtp.corp.redhat.com (Postfix) with ESMTP id 71CDB5D6AE;\n\tTue, 19 Sep 2017 12:11:30 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com DBBE0C057F91","Date":"Tue, 19 Sep 2017 09:11:25 -0300","From":"Eduardo Habkost <ehabkost@redhat.com>","To":"Igor Mammedov <imammedo@redhat.com>","Message-ID":"<20170919121125.GE21016@localhost.localdomain>","References":"<1505318697-77161-1-git-send-email-imammedo@redhat.com>\n\t<20170915170326.5278d578@nial.brq.redhat.com>\n\t<20170918184213.2fd0ead5@Igors-MacBook-Pro.local>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20170918184213.2fd0ead5@Igors-MacBook-Pro.local>","X-Fnord":"you can see the fnord","User-Agent":"Mutt/1.8.3 (2017-05-23)","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.15","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.32]);\n\tTue, 19 Sep 2017 12:11:36 +0000 (UTC)","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic]\n\t[fuzzy]","X-Received-From":"209.132.183.28","Subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Cc":"peter.maydell@linaro.org, qemu-devel@nongnu.org,\n\tSubbaraya Sundeep <sundeep.lkml@gmail.com>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1770979,"web_url":"http://patchwork.ozlabs.org/comment/1770979/","msgid":"<20170919144635.6c6f21b9@nial.brq.redhat.com>","list_archive_url":null,"date":"2017-09-19T12:46:35","subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","submitter":{"id":11305,"url":"http://patchwork.ozlabs.org/api/people/11305/","name":"Igor Mammedov","email":"imammedo@redhat.com"},"content":"On Tue, 19 Sep 2017 09:11:25 -0300\nEduardo Habkost <ehabkost@redhat.com> wrote:\n\n> On Mon, Sep 18, 2017 at 06:42:13PM +0200, Igor Mammedov wrote:\n> > On Fri, 15 Sep 2017 17:03:26 +0200\n> > Igor Mammedov <imammedo@redhat.com> wrote:\n> > \n> > [...]  \n> > > >   \n> > > Hi Eduardo,\n> > > \n> > > Could you merge it via machine tree?  \n> > Forgot to actually CC Eduardo,\n> > \n> > Just noticed a patch (1) on list adding a new soc that would break,\n> > due to armv7 dropping cpu-model property in 5/5.\n> > It would be better to merge this series asap so that people\n> > could rebase their work on cpu-type based approach, instead of\n> > continuing copy-past old cpu-model based code.\n> > \n> > 1)\n> > [Qemu devel v9 PATCH 4/5] msf2: Add Smartfusion2 SoC\n> > \n> > CCing Subbaraya so he could amend patch to use cpu-type instead of cpu-model.  \n> \n> Queued on machine-next:\n> \n>  https://github.com/ehabkost/qemu.git machine-next\n> \n\nThanks!","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","ext-mx03.extmail.prod.ext.phx2.redhat.com;\n\tdmarc=none (p=none dis=none) header.from=redhat.com","ext-mx03.extmail.prod.ext.phx2.redhat.com;\n\tspf=fail smtp.mailfrom=imammedo@redhat.com"],"Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xxNSm1sxfz9s4s\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 19 Sep 2017 23:07:20 +1000 (AEST)","from localhost ([::1]:42736 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1duIFC-0001II-7f\n\tfor incoming@patchwork.ozlabs.org; Tue, 19 Sep 2017 09:07:18 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:42875)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <imammedo@redhat.com>) id 1duHvH-0001Oz-KJ\n\tfor qemu-devel@nongnu.org; Tue, 19 Sep 2017 08:46:49 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <imammedo@redhat.com>) id 1duHvE-0008Ow-Ge\n\tfor qemu-devel@nongnu.org; Tue, 19 Sep 2017 08:46:43 -0400","from mx1.redhat.com ([209.132.183.28]:46470)\n\tby eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32)\n\t(Exim 4.71) (envelope-from <imammedo@redhat.com>) id 1duHvE-0008Nn-BK\n\tfor qemu-devel@nongnu.org; Tue, 19 Sep 2017 08:46:40 -0400","from smtp.corp.redhat.com\n\t(int-mx01.intmail.prod.int.phx2.redhat.com [10.5.11.11])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby mx1.redhat.com (Postfix) with ESMTPS id A493D7E431;\n\tTue, 19 Sep 2017 12:46:38 +0000 (UTC)","from nial.brq.redhat.com (unknown [10.43.2.209])\n\tby smtp.corp.redhat.com (Postfix) with ESMTP id CDE8461348;\n\tTue, 19 Sep 2017 12:46:36 +0000 (UTC)"],"DMARC-Filter":"OpenDMARC Filter v1.3.2 mx1.redhat.com A493D7E431","Date":"Tue, 19 Sep 2017 14:46:35 +0200","From":"Igor Mammedov <imammedo@redhat.com>","To":"Eduardo Habkost <ehabkost@redhat.com>","Message-ID":"<20170919144635.6c6f21b9@nial.brq.redhat.com>","In-Reply-To":"<20170919121125.GE21016@localhost.localdomain>","References":"<1505318697-77161-1-git-send-email-imammedo@redhat.com>\n\t<20170915170326.5278d578@nial.brq.redhat.com>\n\t<20170918184213.2fd0ead5@Igors-MacBook-Pro.local>\n\t<20170919121125.GE21016@localhost.localdomain>","MIME-Version":"1.0","Content-Type":"text/plain; charset=US-ASCII","Content-Transfer-Encoding":"7bit","X-Scanned-By":"MIMEDefang 2.79 on 10.5.11.11","X-Greylist":"Sender IP whitelisted, not delayed by milter-greylist-4.5.16\n\t(mx1.redhat.com [10.5.110.27]);\n\tTue, 19 Sep 2017 12:46:38 +0000 (UTC)","X-detected-operating-system":"by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic]\n\t[fuzzy]","X-Received-From":"209.132.183.28","Subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Cc":"peter.maydell@linaro.org, qemu-devel@nongnu.org,\n\tSubbaraya Sundeep <sundeep.lkml@gmail.com>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}},{"id":1771000,"web_url":"http://patchwork.ozlabs.org/comment/1771000/","msgid":"<186c6ccd-5816-6859-ceae-0f2696cededa@amsat.org>","list_archive_url":null,"date":"2017-09-19T12:56:15","subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","submitter":{"id":70924,"url":"http://patchwork.ozlabs.org/api/people/70924/","name":"Philippe Mathieu-Daudé","email":"f4bug@amsat.org"},"content":"On 09/19/2017 09:11 AM, Eduardo Habkost wrote:\n>> Igor Mammedov <imammedo@redhat.com> wrote:\n[...]\n>>> Could you merge it via machine tree?\n[...]\n> \n> Queued on machine-next:\n\nThanks Eduardo for applying the !fixup :)","headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"LqI/GX7P\"; dkim-atps=neutral"],"Received":["from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xxNth1w6qz9sBZ\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue, 19 Sep 2017 23:26:20 +1000 (AEST)","from localhost ([::1]:42866 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1duIXa-0000Oi-CG\n\tfor incoming@patchwork.ozlabs.org; Tue, 19 Sep 2017 09:26:18 -0400","from eggs.gnu.org ([2001:4830:134:3::10]:50134)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <philippe.mathieu.daude@gmail.com>)\n\tid 1duI4e-0000sh-2T\n\tfor qemu-devel@nongnu.org; Tue, 19 Sep 2017 08:56:26 -0400","from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <philippe.mathieu.daude@gmail.com>)\n\tid 1duI4Z-000729-5Z\n\tfor qemu-devel@nongnu.org; Tue, 19 Sep 2017 08:56:24 -0400","from mail-qk0-x233.google.com ([2607:f8b0:400d:c09::233]:57188)\n\tby eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.71) (envelope-from <philippe.mathieu.daude@gmail.com>)\n\tid 1duI4Z-00071f-12\n\tfor qemu-devel@nongnu.org; Tue, 19 Sep 2017 08:56:19 -0400","by mail-qk0-x233.google.com with SMTP id u7so3427011qku.13\n\tfor <qemu-devel@nongnu.org>; Tue, 19 Sep 2017 05:56:18 -0700 (PDT)","from [192.168.1.10] ([181.93.89.178])\n\tby smtp.gmail.com with ESMTPSA id\n\th40sm6760007qte.73.2017.09.19.05.56.16\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tTue, 19 Sep 2017 05:56:17 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=sender:subject:to:cc:references:from:message-id:date:user-agent\n\t:mime-version:in-reply-to:content-language:content-transfer-encoding; \n\tbh=XiT/RVE+I2A5KetTOFGREZycuwN3m8iTRoFaadRk4JA=;\n\tb=LqI/GX7PMyhWHLI5LX0hLU45DZCEogmSsM9nHPlEcXOoxoAxWpiM0dY3QSS2+pGgId\n\tXPo/LbrqsqZxxMhHLdbb6RVezDhKh/Wwn9VWHdcKs089cRdp5hFewonpz68rE54x5rVR\n\tVJe9kZ1u/Y+Sy8dcNspEAWNasmGiHIQgJ4icPD/2B7xj+BIYcEVy5RpIpFSRbAxtdSZg\n\tpiXbpRAxQhA4gc8lIGnzuZBkiD/4qhJ0JWI9LuVG5rhBoUaimR0llsXlo6yg+2aoWMWB\n\t08eWHtYPV7Rnd663sQypwlUafW8rNWqPUQ0pVAMOTAHUR1HMI/mor08txTyLsCOwGI/d\n\tfagQ==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:sender:subject:to:cc:references:from:message-id\n\t:date:user-agent:mime-version:in-reply-to:content-language\n\t:content-transfer-encoding;\n\tbh=XiT/RVE+I2A5KetTOFGREZycuwN3m8iTRoFaadRk4JA=;\n\tb=I03nwRTp1yrYVOwQP163sb8F9g5NXRxnUcJA7/Yu9XYge7c56R5otduoXV6EEgcYQa\n\tCs38QvhQ7XOzTpgedlJAy0V0EEDXX7iG0OvefRCyfbBxuNAVf8ychmJYxOQRh2CW7GHu\n\tB6O0b6xcNwCjZROFdGqI+2pMdivNBqNoIdT02x+JelqMzLa/lhJbhavPxKmk0KUXYxB9\n\tys+SfkMSczF6VX0CPRaSIexmIbvP2oe4u4rnLyzbWD2DqUc202W4PmFDL1MZDUCcRA5i\n\tQhS4oHNysBmT+iMWmMqdXo+EGGq8JumoIlCevD1GRw2j8E+C4JG5Me/1B25alr8SVeQE\n\tCyRg==","X-Gm-Message-State":"AHPjjUhqZ2I2JKm0gKxCSBI7Bqu4j/yRtHH3F+R1u/ihjcgzkhoAShnn\n\txx1g6zbVMc7KlhZo5wBbLhQ=","X-Google-Smtp-Source":"AOwi7QC6/xdNrN/YEU9qlQlf8ny0Ou6js6Z9+nSTE/ZMgF7SeeoNVHobEIfo3GA2Bprfmu9V7ed7Qw==","X-Received":"by 10.55.176.4 with SMTP id z4mr1752470qke.159.1505825778451;\n\tTue, 19 Sep 2017 05:56:18 -0700 (PDT)","To":"Eduardo Habkost <ehabkost@redhat.com>,\n\tIgor Mammedov <imammedo@redhat.com>","References":"<1505318697-77161-1-git-send-email-imammedo@redhat.com>\n\t<20170915170326.5278d578@nial.brq.redhat.com>\n\t<20170918184213.2fd0ead5@Igors-MacBook-Pro.local>\n\t<20170919121125.GE21016@localhost.localdomain>","From":"=?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= <f4bug@amsat.org>","Message-ID":"<186c6ccd-5816-6859-ceae-0f2696cededa@amsat.org>","Date":"Tue, 19 Sep 2017 09:56:15 -0300","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170919121125.GE21016@localhost.localdomain>","Content-Type":"text/plain; charset=utf-8; format=flowed","Content-Language":"en-US","Content-Transfer-Encoding":"7bit","X-detected-operating-system":"by eggs.gnu.org: Genre and OS details not\n\trecognized.","X-Received-From":"2607:f8b0:400d:c09::233","Subject":"Re: [Qemu-devel] [PATCH v2 0/5] generalize parsing of cpu_model\n\t(x86/arm)","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Id":"<qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<http://lists.nongnu.org/archive/html/qemu-devel/>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Cc":"peter.maydell@linaro.org, qemu-devel@nongnu.org,\n\tSubbaraya Sundeep <sundeep.lkml@gmail.com>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>"}}]