From: George Dunlap <george.dunlap@eu.citrix.com>
To: Nate Studer <nate.studer@dornerworks.com>,
Andrew Cooper <andrew.cooper3@citrix.com>
Cc: robert.vanvossen@dornerworks.com, xen-devel@lists.xen.org
Subject: Re: [PATCH 2/2] Fix NULL pointer dereference in ARINC653 free_vdata.
Date: Mon, 4 Nov 2013 10:46:50 +0000 [thread overview]
Message-ID: <52777B1A.4070300@eu.citrix.com> (raw)
In-Reply-To: <5276D5C7.8040100@dornerworks.com>
On 03/11/13 23:01, Nate Studer wrote:
> On 11/1/2013 10:13 AM, George Dunlap wrote:
>> Both these patches can have my Ack, BTW (although I don't think they
>> need it): I just wanted to be a good citizen and do a bit of review. :-)
> Thanks Andrew and George.
>
> Just to make sure, and to avoid needlessly filling George's inbox, Robbie (The
> other ARINC653 scheduler maintainer) should usually be the one acking patches
> like this, correct?
Oh, don't worry about filling up my inbox. I do want to know what's
going on in the scheduling area, and I probably *should* review this
kind of patch anyway. I was just saying, I don't think the committers
necessarily need to wait for an Ack from me to commit it. :-)
I'm not sure exactly what the policy would be here -- I think the "need
an ack from someone else" rule is primarily for committers to commit
their own patches. If you're just submitting it, as a maintainer, it
seems like having the committer look over it (and being satisfied that
the community has had a chance to comment) should be enough, and needing
an extra Ack from the other committer is a bit over-kill, particularly
in this case.
But I could be wrong about that. :-)
-George
next prev parent reply other threads:[~2013-11-04 10:46 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-31 20:47 [PATCH 0/2] Fix crashes with arinc653 cpupools Nathan Studer
2013-10-31 20:47 ` [PATCH 1/2] Fix sched_priv corruption in ARINC653 alloc_vdata Nathan Studer
2013-11-01 13:53 ` Andrew Cooper
2013-11-03 23:07 ` Nate Studer
2013-11-04 15:49 ` George Dunlap
2013-10-31 20:47 ` [PATCH 2/2] Fix NULL pointer dereference in ARINC653 free_vdata Nathan Studer
2013-11-01 13:56 ` Andrew Cooper
2013-11-01 14:13 ` George Dunlap
2013-11-01 14:17 ` Andrew Cooper
2013-11-03 23:01 ` Nate Studer
2013-11-04 10:46 ` George Dunlap [this message]
2013-11-04 10:55 ` Ian Campbell
2013-11-04 15:15 ` Jan Beulich
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=52777B1A.4070300@eu.citrix.com \
--to=george.dunlap@eu.citrix.com \
--cc=andrew.cooper3@citrix.com \
--cc=nate.studer@dornerworks.com \
--cc=robert.vanvossen@dornerworks.com \
--cc=xen-devel@lists.xen.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.