qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Alexander Graf <agraf@suse.de>
To: Gavin Shan <gwshan@linux.vnet.ibm.com>,
	Alexey Kardashevskiy <aik@ozlabs.ru>
Cc: alex.williamson@redhat.com, qemu-ppc@nongnu.org, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v12 3/4] headers: Update kernel header
Date: Mon, 28 Jul 2014 14:50:46 +0200	[thread overview]
Message-ID: <53D64726.5010108@suse.de> (raw)
In-Reply-To: <20140716014059.GA14362@shangw>


On 16.07.14 03:40, Gavin Shan wrote:
> On Wed, Jul 16, 2014 at 11:32:13AM +1000, Alexey Kardashevskiy wrote:
>> On 07/16/2014 11:16 AM, Gavin Shan wrote:
>>> On Wed, Jul 16, 2014 at 11:09:44AM +1000, Alexey Kardashevskiy wrote:
>>>> On 07/16/2014 10:20 AM, Gavin Shan wrote:
>>>>> This updates kernel header (vfio.h) for EEH support on VFIO PCI
>>>>> devices.
>>>> Has this reached kernel upstream? The way linux headers update normally
>>>> happens is you have to run scripts/update-linux-headers.sh against some
>>>> linux kernel tag which you know that it won't change (like v3.16-rc5) and
>>>> post all the changes as a single patch. It is never a header update for a
>>>> specific feature, it is just an update.
>>>>
>>> The kernel part isn't merged yet. I guess that's for 3.17 merge window.
>>> Ok, good to know scripts/update-linux-headers.sh. So this patch should
>>> be dropped and some one run the script to update QEMU (linux-headers
>>> directory) ?
>>
>> Once your changes are in upstream kernel, you wait till kernel tree gets
>> new "v3.xx-rcX" tag, then you run the script and make a separate patch for
>> QEMU. Then you wait till it reaches QEMU upstream (because I do not know
>> who will pull it to what tree, look at git history) or ppc-next (if Alex
>> pulls it and you are basing your work on ppc-next) and then repost other
>> patches.
>>
> Thanks for detailed explaining, Alexey. I guess I have to suspend a bit
> until "v3.17.rc1" is coming out.

It's also perfectly fine to keep the kernel header update inside your 
patch set, but please make sure to mention which commit it is against. 
If I pick it up, I will generate the headers myself though and drop your 
patch while applying the rest of the set.


Alex

  reply	other threads:[~2014-07-28 12:51 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-16  0:20 [Qemu-devel] [PATCH v12 0/4] EEH Support for VFIO Devices Gavin Shan
2014-07-16  0:20 ` [Qemu-devel] [PATCH v12 1/4] spapr_pci: Make find_phb()/find_dev() public Gavin Shan
2014-07-28 12:49   ` Alexander Graf
2014-07-28 13:52     ` Alexey Kardashevskiy
2014-07-16  0:20 ` [Qemu-devel] [PATCH v12 2/4] sPAPR: Implement EEH RTAS calls Gavin Shan
2014-07-16  1:06   ` Alexey Kardashevskiy
2014-07-16  1:12     ` Gavin Shan
2014-07-16  1:28       ` Alexey Kardashevskiy
2014-07-16  0:20 ` [Qemu-devel] [PATCH v12 3/4] headers: Update kernel header Gavin Shan
2014-07-16  1:09   ` Alexey Kardashevskiy
2014-07-16  1:16     ` Gavin Shan
2014-07-16  1:32       ` Alexey Kardashevskiy
2014-07-16  1:40         ` Gavin Shan
2014-07-28 12:50           ` Alexander Graf [this message]
2014-07-16  0:20 ` [Qemu-devel] [PATCH v12 4/4] sPAPR: Implement sPAPRPHBClass::eeh_handler Gavin Shan

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=53D64726.5010108@suse.de \
    --to=agraf@suse.de \
    --cc=aik@ozlabs.ru \
    --cc=alex.williamson@redhat.com \
    --cc=gwshan@linux.vnet.ibm.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.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).