From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1dTNms-0006FT-Cv for mharc-qemu-trivial@gnu.org; Fri, 07 Jul 2017 03:34:50 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59839) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dTNmp-0006E2-Tg for qemu-trivial@nongnu.org; Fri, 07 Jul 2017 03:34:48 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dTNml-0002VP-0z for qemu-trivial@nongnu.org; Fri, 07 Jul 2017 03:34:47 -0400 Received: from roura.ac.upc.es ([147.83.33.10]:43465) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dTNmk-0002UE-L7; Fri, 07 Jul 2017 03:34:42 -0400 Received: from correu-2.ac.upc.es (correu-2.ac.upc.es [147.83.30.92]) by roura.ac.upc.es (8.13.8/8.13.8) with ESMTP id v677YZnv012731; Fri, 7 Jul 2017 09:34:35 +0200 Received: from localhost (63.red-83-51-187.dynamicip.rima-tde.net [83.51.187.63]) by correu-2.ac.upc.es (Postfix) with ESMTPSA id 085AF948; Fri, 7 Jul 2017 09:34:29 +0200 (CEST) From: =?utf-8?Q?Llu=C3=ADs_Vilanova?= To: Eric Blake Cc: qemu-devel@nongnu.org, "Emilio G. Cota" , Eduardo Habkost , Stefan Hajnoczi , Michael Tokarev , Laurent Vivier , "open list\:Trivial patches" References: <149915701334.6295.1724301929524364123.stgit@frigg.lan> <149915849243.6295.4484103824675839071.stgit@frigg.lan> Mail-Followup-To: Eric Blake , qemu-devel@nongnu.org, "Emilio G. Cota" , Eduardo Habkost , Stefan Hajnoczi , Michael Tokarev , Laurent Vivier , "open list\:Trivial patches" Date: Fri, 07 Jul 2017 09:34:24 +0200 In-Reply-To: (Eric Blake's message of "Wed, 5 Jul 2017 16:14:10 -0500") Message-ID: <87tw2og0ov.fsf@frigg.lan> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6.x [fuzzy] X-Received-From: 147.83.33.10 Subject: Re: [Qemu-trivial] [PATCH v11 6/6] trace: [trivial] Statically enable all guest events X-BeenThere: qemu-trivial@nongnu.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 07 Jul 2017 07:34:48 -0000 Eric Blake writes: > On 07/04/2017 03:54 AM, Llu=C3=ADs Vilanova wrote: >> The existing optimizations makes it feasible to have them available on a= ll >> builds. > While this change may feel trivial, I think it is a misnomer to include > "[trivial]" in the subject line, and I also think it should not go in > through qemu-trivial. The ideal trivial patch is one that can be > applied in isolation, but your patch can only be applied as part of a > series that includes the earlier optimizations that made this one possibl= e. Ok. Does this mean I should send a new series without the trivial tag? Cheers, Lluis From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59850) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dTNmr-0006Ex-Ga for qemu-devel@nongnu.org; Fri, 07 Jul 2017 03:34:50 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dTNmq-0002XI-SN for qemu-devel@nongnu.org; Fri, 07 Jul 2017 03:34:49 -0400 From: =?utf-8?Q?Llu=C3=ADs_Vilanova?= References: <149915701334.6295.1724301929524364123.stgit@frigg.lan> <149915849243.6295.4484103824675839071.stgit@frigg.lan> Date: Fri, 07 Jul 2017 09:34:24 +0200 In-Reply-To: (Eric Blake's message of "Wed, 5 Jul 2017 16:14:10 -0500") Message-ID: <87tw2og0ov.fsf@frigg.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v11 6/6] trace: [trivial] Statically enable all guest events List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Eric Blake Cc: qemu-devel@nongnu.org, "Emilio G. Cota" , Eduardo Habkost , Stefan Hajnoczi , Michael Tokarev , Laurent Vivier , "open list:Trivial patches" Eric Blake writes: > On 07/04/2017 03:54 AM, Llu=C3=ADs Vilanova wrote: >> The existing optimizations makes it feasible to have them available on a= ll >> builds. > While this change may feel trivial, I think it is a misnomer to include > "[trivial]" in the subject line, and I also think it should not go in > through qemu-trivial. The ideal trivial patch is one that can be > applied in isolation, but your patch can only be applied as part of a > series that includes the earlier optimizations that made this one possibl= e. Ok. Does this mean I should send a new series without the trivial tag? Cheers, Lluis