From mboxrd@z Thu Jan  1 00:00:00 1970
From: Andrew Cooper <andrew.cooper3@citrix.com>
Subject: Re: [PATCH v2 for Xen 4.6 0/4] Enabling XL to set
 per-VCPU parameters of a domain for RTDS scheduler
Date: Mon, 1 Jun 2015 10:06:37 +0100
Message-ID: <556C209D.2080906@citrix.com>
References: <CAGHO-iqn3-3KGuo8JgN1EhK=BNt+MnkbzG7FFWTzeUTgh89d-A@mail.gmail.com>
	<1432630782.14664.60.camel@citrix.com>	<1432898136.5077.13.camel@citrix.com>
	<CAENZ-+mf8tv7cDuhaHN6SesK-FkwTnQM3zC1VFVwFDQkewvqsg@mail.gmail.com>
	<556C1994.4090101@eu.citrix.com> <1433148506.15036.78.camel@citrix.com>
	<556C1CC8.10301@eu.citrix.com>
	<556C3AF2020000780007F952@mail.emea.novell.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <556C3AF2020000780007F952@mail.emea.novell.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: Jan Beulich <JBeulich@suse.com>, Ian Campbell <ian.campbell@citrix.com>, George Dunlap <george.dunlap@eu.citrix.com>
Cc: Wei Liu <wei.liu2@citrix.com>, Dario Faggioli <dario.faggioli@citrix.com>, Ian Jackson <Ian.Jackson@eu.citrix.com>, xen-devel <xen-devel@lists.xen.org>, Meng Xu <xumengpanda@gmail.com>, Meng Xu <mengxu@cis.upenn.edu>, Chong Li <lichong659@gmail.com>, Dagaen Golomb <dgolomb@seas.upenn.edu>
List-Id: xen-devel@lists.xenproject.org

On 01/06/15 09:58, Jan Beulich wrote:
>>>> On 01.06.15 at 10:50, <george.dunlap@eu.citrix.com> wrote:
>> On 06/01/2015 09:48 AM, Ian Campbell wrote:
>>> On Mon, 2015-06-01 at 09:36 +0100, George Dunlap wrote:
>>>> Signed-off-by: George Dunlap <george.dunlap@eu.citrix.com>
>>>> ---
>>>> CC: Ian Campbell <ian.campbell@citrix.com>
>>>> CC: Wei Liu <wei.liu@citrix.com>
>>> Most people put the Cc about the cut (---) which is fine too. It means
>>> the history ends up recording who was copied on the patch, which isn't
>>> necessarily a bad thing.
>> Right -- I would have thought that was useless information cluttering up
>> the history, but I can see how it might actually be useful.  Should I
>> start putting my CC's above the ---? :-)
> And should I stop dropping them even when above the --- for
> commit, which so far I've been doing as I don't consider this
> particularly useful information (other then e.g. who might have
> commented on a change without it being recorded in an Acked-by
> or Reviewed-by tag)?

Is the CC list useful to keep in history? It ends up being the list of
people who didn't respond to it before it got committed (or
ignored/missed the email entirely).

It is the $FOO'd-by tags which are important when it comes to judging
the acceptability of a patch.

~Andrew