From: Dario Faggioli <dario.faggioli@citrix.com>
To: Andrew Cooper <andrew.cooper3@citrix.com>,
xen-devel@lists.xenproject.org
Cc: Anshul Makkar <anshul.makkar@citrix.com>,
George Dunlap <george.dunlap@citrix.com>
Subject: Re: [PATCH 0/2] xen: Credit2: fix two issues from recently committed series.
Date: Tue, 19 Jul 2016 14:26:25 +0200 [thread overview]
Message-ID: <1468931185.13039.193.camel@citrix.com> (raw)
In-Reply-To: <cb4afd3f-78ac-00fc-9088-0d7f990ebd39@citrix.com>
[-- Attachment #1.1: Type: text/plain, Size: 966 bytes --]
On Tue, 2016-07-19 at 13:20 +0100, Andrew Cooper wrote:
> On 19/07/16 13:06, Dario Faggioli wrote:
> >
> > Series committed yesterday, to be precise, and two (not too big, at
> > least :-))
> > issues, have been found already (thanks Andrew!).
> I tend to put "Spotted by Coverity." in the commit message of these,
> even when we can't provide CID numbers.
>
> Its not like I spotted these from code inspection.
>
Ok, I see. As you say, since I could not provide a meaningful CID, that
would not seem too useful to say either, to me.
But in any case, if you feel strong enough about it, I'm ok resending
with that, or about anyone committing the patches adding it.
Regards,
Dario
--
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
[-- Attachment #2: Type: text/plain, Size: 127 bytes --]
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel
next prev parent reply other threads:[~2016-07-19 12:26 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-19 12:06 [PATCH 0/2] xen: Credit2: fix two issues from recently committed series Dario Faggioli
2016-07-19 12:07 ` [PATCH 1/2] xen: credit2: fix two s_time_t handling issues in load balancing Dario Faggioli
2016-07-19 13:39 ` Dario Faggioli
2016-07-19 12:07 ` [PATCH 2/2] xen: credit2: fix potential issues in csched2_cpu_pick with tracing enabled Dario Faggioli
2016-07-19 12:20 ` [PATCH 0/2] xen: Credit2: fix two issues from recently committed series Andrew Cooper
2016-07-19 12:26 ` Dario Faggioli [this message]
2016-07-19 12:27 ` Andrew Cooper
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1468931185.13039.193.camel@citrix.com \
--to=dario.faggioli@citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=anshul.makkar@citrix.com \
--cc=george.dunlap@citrix.com \
--cc=xen-devel@lists.xenproject.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).