All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wei Liu <wei.liu2@citrix.com>
To: Yi Sun <yi.y.sun@linux.intel.com>
Cc: Wei Liu <wei.liu2@citrix.com>,
	he.chen@linux.intel.com, andrew.cooper3@citrix.com,
	ian.jackson@eu.citrix.com, jbeulich@suse.com,
	chao.p.peng@linux.intel.com, xen-devel@lists.xenproject.org
Subject: Re: [v2 0/3] Enable L2 Cache Allocation Technology
Date: Fri, 23 Sep 2016 10:41:45 +0100	[thread overview]
Message-ID: <20160923094145.GR16004@citrix.com> (raw)
In-Reply-To: <20160923020654.GL30513@yi.y.sun>

On Fri, Sep 23, 2016 at 10:06:54AM +0800, Yi Sun wrote:
> On 16-09-22 11:18:12, Wei Liu wrote:
> > Hi Yi
> > 
> > Thanks for submitting this series.  I see that all the actual patches
> > are not chained to 0/3. It would be better if they show up as replies to
> > 0/3.
> > 
> > The workflow for sending patch series differs from person to person, so
> > we don't want to  dictate how you use your tool. But we have written a
> > wiki page on how to submit patches to xen-devel. That contains some tips
> > about git.
> > 
> > https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#Sending_the_patches_to_the_list
> > 
> > Feel free to ask questions here. I would be happy to share how I do it
> > if you're interested.
> > 
> > Wei.
> 
> Hi, Wei,
> 
> Thanks a lot for your kindness! I just went through the wiki and found I
> should use '--in-reply-to=', right? If you can share your steps that would
> be great.

Assuming you send out your cover letter separately from your patches,
yes, --in-reply-to= is the option that you need to add to `git
send-email' invocation. The <identifier> (see `git help send-email')
part is the message id of your cover letter. You should be able to get
that after sending out the cover letter.

My current preferred workflow is to have git handle everything for me.
I have a separate file for the *content* of cover letter. And to generate
everything including the cover letter *email*:

$ git format-patch <revision range> --reroll-count=$N --cover-letter

Then you will have v$N-0000-cover-letter.patch plus a few v$N-*.patch.
Use you favourite text editor to edit the cover letter, import the
content, edit it as you wish.

To send out all you patches including the cover letter:

$ git send-email *.patch --to xen-devel@... --cc maintainers@...  --dry-run

When you're confident the list of emails looks correct, delete --dry-run
to actually send them.

This way git send-email will automatically chain all actual patches to
cover letter.

Some people have built other tools on top of porcelain git to track more
stuff. I'm experimenting with a tool called git-series, but it is too
early to say if it is the right tool for me.

Anyway, read git manpage. It has a lot of information in it.

> 
> Can I send the patches again after addressing Ian's comments?
> 

It's better to wait a bit for hypervisor maintainers to comment on the
first three patches.

Wei.

> Thanks,
> Yi

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

  reply	other threads:[~2016-09-23  9:41 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-09-22  2:14 [v2 0/3] Enable L2 Cache Allocation Technology Yi Sun
2016-09-22 10:18 ` Wei Liu
2016-09-23  2:06   ` Yi Sun
2016-09-23  9:41     ` Wei Liu [this message]
2016-09-23 10:03       ` Yi Sun
2016-09-30 20:32 ` Konrad Rzeszutek Wilk
2016-10-09  1:52   ` Yi Sun

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=20160923094145.GR16004@citrix.com \
    --to=wei.liu2@citrix.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=chao.p.peng@linux.intel.com \
    --cc=he.chen@linux.intel.com \
    --cc=ian.jackson@eu.citrix.com \
    --cc=jbeulich@suse.com \
    --cc=xen-devel@lists.xenproject.org \
    --cc=yi.y.sun@linux.intel.com \
    /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.