[Qemu-devel] [PATCH 0/6] hyperv: refactor HvSintRoute management

Roman Kagan posted 6 patches 7 years, 7 months ago
Failed in applying to current master (apply log)
There is a newer version of this series
target/i386/hyperv.h     |  20 ++-----
hw/misc/hyperv_testdev.c | 116 +++++++++++++++++++--------------------
target/i386/hyperv.c     |  78 +++++++++++++++++++-------
3 files changed, 120 insertions(+), 94 deletions(-)
[Qemu-devel] [PATCH 0/6] hyperv: refactor HvSintRoute management
Posted by Roman Kagan 7 years, 7 months ago
This series modifies the management of HvSintRoute, which is an
important building block in Hyper-V emulation infrastructure, to make it
easier to maintain and enhance, and paves the way to the more complete
Synthetic Interrupt Controller (SynIC) emulation.

This series applies on top of "[PATCH 0/2] hyperv: ensure VP index equal
to QEMU cpu_index" series (message-id:
<20180702134156.13404-1-rkagan@virtuozzo.com>).

Roman Kagan (6):
  hyperv_testdev: refactor for better maintainability
  hyperv: cosmetic: g_malloc -> g_new
  hyperv: synic: only setup ack notifier if there's a callback
  hyperv: allow passing arbitrary data to sint ack callback
  hyperv: address HvSintRoute by X86CPU pointer
  hyperv: make HvSintRoute reference-counted

---
For the reference, all of the Hyper-V / VMBus stuff can be found at
https://src.openvz.org/scm/up/qemu

 target/i386/hyperv.h     |  20 ++-----
 hw/misc/hyperv_testdev.c | 116 +++++++++++++++++++--------------------
 target/i386/hyperv.c     |  78 +++++++++++++++++++-------
 3 files changed, 120 insertions(+), 94 deletions(-)

-- 
2.17.1


Re: [Qemu-devel] [PATCH 0/6] hyperv: refactor HvSintRoute management
Posted by Eric Blake 7 years, 7 months ago
On 07/02/2018 11:58 AM, Roman Kagan wrote:
> This series modifies the management of HvSintRoute, which is an
> important building block in Hyper-V emulation infrastructure, to make it
> easier to maintain and enhance, and paves the way to the more complete
> Synthetic Interrupt Controller (SynIC) emulation.
> 
> This series applies on top of "[PATCH 0/2] hyperv: ensure VP index equal
> to QEMU cpu_index" series (message-id:
> <20180702134156.13404-1-rkagan@virtuozzo.com>).

Rewriting that in a form that patchew can parse:

Based-on: 20180702134156.13404-1-rkagan@virtuozzo.com

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

[Qemu-devel] patchew header docs [was: Re: [PATCH 0/6] hyperv: refactor HvSintRoute management]
Posted by Roman Kagan 7 years, 7 months ago
On Mon, Jul 02, 2018 at 03:55:33PM -0500, Eric Blake wrote:
> On 07/02/2018 11:58 AM, Roman Kagan wrote:
> > This series modifies the management of HvSintRoute, which is an
> > important building block in Hyper-V emulation infrastructure, to make it
> > easier to maintain and enhance, and paves the way to the more complete
> > Synthetic Interrupt Controller (SynIC) emulation.
> > 
> > This series applies on top of "[PATCH 0/2] hyperv: ensure VP index equal
> > to QEMU cpu_index" series (message-id:
> > <20180702134156.13404-1-rkagan@virtuozzo.com>).
> 
> Rewriting that in a form that patchew can parse:
> 
> Based-on: 20180702134156.13404-1-rkagan@virtuozzo.com

Thanks!

Are patchew-recognized headers documented anywhere?

Roman.

Re: [Qemu-devel] patchew header docs [was: Re: [PATCH 0/6] hyperv: refactor HvSintRoute management]
Posted by Fam Zheng 7 years, 7 months ago
On Tue, 07/03 10:47, Roman Kagan wrote:
> On Mon, Jul 02, 2018 at 03:55:33PM -0500, Eric Blake wrote:
> > On 07/02/2018 11:58 AM, Roman Kagan wrote:
> > > This series modifies the management of HvSintRoute, which is an
> > > important building block in Hyper-V emulation infrastructure, to make it
> > > easier to maintain and enhance, and paves the way to the more complete
> > > Synthetic Interrupt Controller (SynIC) emulation.
> > > 
> > > This series applies on top of "[PATCH 0/2] hyperv: ensure VP index equal
> > > to QEMU cpu_index" series (message-id:
> > > <20180702134156.13404-1-rkagan@virtuozzo.com>).
> > 
> > Rewriting that in a form that patchew can parse:
> > 
> > Based-on: 20180702134156.13404-1-rkagan@virtuozzo.com
> 
> Thanks!
> 
> Are patchew-recognized headers documented anywhere?

No dedicated documentation so far becasue the only recognized syntax is this
"^Based-on: $MESSAGE_ID$" line, which is mentioned in

https://wiki.qemu.org/Contribute/SubmitAPatch

There are requests to support more, such as the git base tree information:

https://github.com/patchew-project/patchew/issues/79

and once we get there, we'll find a better place to document these features so
that people can read about it more easily.

Fam

Re: [Qemu-devel] patchew header docs [was: Re: [PATCH 0/6] hyperv: refactor HvSintRoute management]
Posted by Eric Blake 7 years, 7 months ago
On 07/03/2018 03:00 AM, Fam Zheng wrote:
> On Tue, 07/03 10:47, Roman Kagan wrote:
>> On Mon, Jul 02, 2018 at 03:55:33PM -0500, Eric Blake wrote:
>>> On 07/02/2018 11:58 AM, Roman Kagan wrote:
>>>> This series modifies the management of HvSintRoute, which is an
>>>> important building block in Hyper-V emulation infrastructure, to make it
>>>> easier to maintain and enhance, and paves the way to the more complete
>>>> Synthetic Interrupt Controller (SynIC) emulation.
>>>>
>>>> This series applies on top of "[PATCH 0/2] hyperv: ensure VP index equal
>>>> to QEMU cpu_index" series (message-id:
>>>> <20180702134156.13404-1-rkagan@virtuozzo.com>).
>>>
>>> Rewriting that in a form that patchew can parse:
>>>
>>> Based-on: 20180702134156.13404-1-rkagan@virtuozzo.com
>>
>> Thanks!
>>
>> Are patchew-recognized headers documented anywhere?
> 
> No dedicated documentation so far becasue the only recognized syntax is this
> "^Based-on: $MESSAGE_ID$" line, which is mentioned in

I seem to recall that "^Based-on: $URL$" also works, when $URL points to 
a message within https://lists.gnu.org/archive/html/qemu-devel/ (at 
least, I've used it in a few cases, rather than further looking up the 
translation from $URL to $MESSAGE_ID myself).

> 
> https://wiki.qemu.org/Contribute/SubmitAPatch
> 
> There are requests to support more, such as the git base tree information:
> 
> https://github.com/patchew-project/patchew/issues/79

And if my recollection was wrong, adding $URL support would be one of 
those requests, and perhaps ought to be given an issue so we don't 
forget it.

> 
> and once we get there, we'll find a better place to document these features so
> that people can read about it more easily.
> 
> Fam
> 

-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Re: [Qemu-devel] patchew header docs [was: Re: [PATCH 0/6] hyperv: refactor HvSintRoute management]
Posted by Fam Zheng 7 years, 7 months ago
On Tue, 07/03 07:48, Eric Blake wrote:
> On 07/03/2018 03:00 AM, Fam Zheng wrote:
> > On Tue, 07/03 10:47, Roman Kagan wrote:
> > > On Mon, Jul 02, 2018 at 03:55:33PM -0500, Eric Blake wrote:
> > > > On 07/02/2018 11:58 AM, Roman Kagan wrote:
> > > > > This series modifies the management of HvSintRoute, which is an
> > > > > important building block in Hyper-V emulation infrastructure, to make it
> > > > > easier to maintain and enhance, and paves the way to the more complete
> > > > > Synthetic Interrupt Controller (SynIC) emulation.
> > > > > 
> > > > > This series applies on top of "[PATCH 0/2] hyperv: ensure VP index equal
> > > > > to QEMU cpu_index" series (message-id:
> > > > > <20180702134156.13404-1-rkagan@virtuozzo.com>).
> > > > 
> > > > Rewriting that in a form that patchew can parse:
> > > > 
> > > > Based-on: 20180702134156.13404-1-rkagan@virtuozzo.com
> > > 
> > > Thanks!
> > > 
> > > Are patchew-recognized headers documented anywhere?
> > 
> > No dedicated documentation so far becasue the only recognized syntax is this
> > "^Based-on: $MESSAGE_ID$" line, which is mentioned in
> 
> I seem to recall that "^Based-on: $URL$" also works, when $URL points to a
> message within https://lists.gnu.org/archive/html/qemu-devel/ (at least,
> I've used it in a few cases, rather than further looking up the translation
> from $URL to $MESSAGE_ID myself).
> 
> > 
> > https://wiki.qemu.org/Contribute/SubmitAPatch
> > 
> > There are requests to support more, such as the git base tree information:
> > 
> > https://github.com/patchew-project/patchew/issues/79
> 
> And if my recollection was wrong, adding $URL support would be one of those
> requests, and perhaps ought to be given an issue so we don't forget it.

I've created one now:

https://github.com/patchew-project/patchew/issues/84

Thanks,
Fam