xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Wei Liu <wei.liu2@citrix.com>
To: Ben Sanda <Ben.Sanda@dornerworks.com>
Cc: "xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>,
	Julien Grall <julien.grall@arm.com>,
	Wei Liu <wei.liu2@citrix.com>
Subject: Re: [PATCH 0/6] xentrace/xenalyze support on ARM
Date: Fri, 18 Mar 2016 17:12:50 +0000	[thread overview]
Message-ID: <20160318171250.GD23307@citrix.com> (raw)
In-Reply-To: <A2A949C387F3A54C9B4DAC2DCD2E9A85C38D5922@Quimby.dw.local>

On Thu, Mar 17, 2016 at 05:04:50PM +0000, Ben Sanda wrote:
> Julien,
> 
> >How did you invoke get_maintainers.pl?
> >
> >I tried with the patch #4 and got the following ouput:
> >
> >42sh$ scripts/get_maintainers.pl < xentrace-ARM-platform-DOMID_XEN-mapping-support.patch
> >
> >Stefano Stabellini <stefano.stabellini@citrix.com> Julien Grall <julien.grall@arm.com> xen-devel@lists.xen.org
> 
> Since I had a multifile patch I invoked it on the folder generated by
> git send-email --format-patch
> 
> ./scripts/get_maintainer.pl -f new_patch
> 
> Reading the help usage it indicated you could run it on folders or
> directories. It would seem you have to run it on a per file basic
> though? I will be sure to correct this moving forward.
> 

I think you did it wrong -- when using get_maintainer.pl on actual
patches, you don't need -f. It's only that when you use it on
files that are not patches that you need -f.

In you case, the invocation should be
  ./scripts/get_maintainer.pl new_patch

Wei.

> Thanks,
> Ben
> 
> -----Original Message-----
> From: Julien Grall [mailto:julien.grall@arm.com] 
> Sent: 17 March, 2016 13:01
> To: Ben Sanda <Ben.Sanda@dornerworks.com>; xen-devel@lists.xenproject.org
> Subject: Re: [Xen-devel] [PATCH 0/6] xentrace/xenalyze support on ARM
> 
> Hello Benjamin,
> 
> On 17/03/16 16:50, Ben Sanda wrote:
> >> Can you please CC the relevant maintainers in the next version of this series?
> >> You can use scripts/get_maintainers.pl for this purpose.
> > 
> > I did. The list output to me was:
> > Ian Jackson <ian.jackson@eu.citrix.com> Jan Beulich 
> > <jbeulich@suse.com> Keir Fraser <keir@xen.org> Tim Deegan 
> > <tim@xen.org>
> 
> How did you invoke get_maintainers.pl?
> 
> I tried with the patch #4 and got the following ouput:
> 
> 42sh$ scripts/get_maintainers.pl < xentrace-ARM-platform-DOMID_XEN-mapping-support.patch
> 
> Stefano Stabellini <stefano.stabellini@citrix.com> Julien Grall <julien.grall@arm.com> xen-devel@lists.xen.org
> 
> Regards,
> 
> --
> Julien Grall
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xen.org
> http://lists.xen.org/xen-devel

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  reply	other threads:[~2016-03-18 17:12 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-16 20:51 [PATCH 0/6] xentrace/xenalyze support on ARM Benjamin Sanda
2016-03-16 20:51 ` [PATCH 1/6] Flask: Support for ARM xentrace Benjamin Sanda
2016-03-17 14:56   ` Julien Grall
2016-03-17 15:03   ` Julien Grall
2016-03-25 19:27   ` Konrad Rzeszutek Wilk
2016-03-28 15:52     ` Ben Sanda
2016-03-16 20:51 ` [PATCH 2/6] xenalyze: Support for ARM platform Benjamin Sanda
2016-03-16 20:59   ` Andrew Cooper
2016-03-17 10:39     ` George Dunlap
2016-03-16 20:51 ` [PATCH 3/6] xentrace: P2M lookup suport " Benjamin Sanda
2016-03-17 16:21   ` Julien Grall
2016-03-28 18:55     ` Ben Sanda
2016-03-30 18:38       ` Julien Grall
2016-03-16 20:51 ` [PATCH 4/6] xentrace: ARM platform DOMID_XEN mapping support Benjamin Sanda
2016-03-17 15:53   ` Julien Grall
2016-03-16 20:51 ` [PATCH 5/6] xentrace: Trace buffer support for ARM platform Benjamin Sanda
2016-03-16 20:51 ` [PATCH 6/6] xentrace: ARM platform timestamp support Benjamin Sanda
2016-03-25 19:31   ` Konrad Rzeszutek Wilk
2016-03-31 16:38     ` Stefano Stabellini
2016-03-31 16:44       ` Ben Sanda
2016-04-01 13:05         ` Stefano Stabellini
2016-03-17 15:00 ` [PATCH 0/6] xentrace/xenalyze support on ARM Julien Grall
2016-03-17 16:50   ` Ben Sanda
2016-03-17 17:01     ` Julien Grall
2016-03-17 17:04       ` Ben Sanda
2016-03-18 17:12         ` Wei Liu [this message]
2016-03-17 17:23 ` George Dunlap

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=20160318171250.GD23307@citrix.com \
    --to=wei.liu2@citrix.com \
    --cc=Ben.Sanda@dornerworks.com \
    --cc=julien.grall@arm.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).