[PATCH v3 0/6] hw/vfio/ccw: hw: Use QOM macros and remove DO_UPCAST() uses

Philippe Mathieu-Daudé posted 6 patches 1 year, 2 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20230213170145.45666-1-philmd@linaro.org
Maintainers: Halil Pasic <pasic@linux.ibm.com>, Christian Borntraeger <borntraeger@linux.ibm.com>, Eric Farman <farman@linux.ibm.com>, Richard Henderson <richard.henderson@linaro.org>, David Hildenbrand <david@redhat.com>, Ilya Leoshkevich <iii@linux.ibm.com>, Thomas Huth <thuth@redhat.com>, Matthew Rosato <mjrosato@linux.ibm.com>, Alex Williamson <alex.williamson@redhat.com>
hw/s390x/event-facility.c |  3 +--
hw/vfio/ccw.c             | 40 ++++++++++++++++-----------------------
2 files changed, 17 insertions(+), 26 deletions(-)
[PATCH v3 0/6] hw/vfio/ccw: hw: Use QOM macros and remove DO_UPCAST() uses
Posted by Philippe Mathieu-Daudé 1 year, 2 months ago
QOM housekeeping series which replace the DO_UPCAST() macro
uses by equivalent QOM ones (and few more trivial cleanups).

Since v2:
- Split of bigger series
- Split in simpler patches
- Added Eric R-b tags

v2:
https://lore.kernel.org/qemu-devel/20230213070820.76881-17-philmd@linaro.org/

Philippe Mathieu-Daudé (6):
  hw/s390x/event-facility: Replace DO_UPCAST(SCLPEvent) by SCLP_EVENT()
  hw/vfio/ccw: Simplify using DEVICE() macro
  hw/vfio/ccw: Use intermediate S390CCWDevice variable
  hw/vfio/ccw: Replace DO_UPCAST(S390CCWDevice) by S390_CCW_DEVICE()
  hw/vfio/ccw: Remove pointless S390CCWDevice variable
  hw/vfio/ccw: Replace DO_UPCAST(VFIOCCWDevice) by VFIO_CCW()

 hw/s390x/event-facility.c |  3 +--
 hw/vfio/ccw.c             | 40 ++++++++++++++++-----------------------
 2 files changed, 17 insertions(+), 26 deletions(-)

-- 
2.38.1


Re: [PATCH v3 0/6] hw/vfio/ccw: hw: Use QOM macros and remove DO_UPCAST() uses
Posted by Philippe Mathieu-Daudé 1 year, 2 months ago
Hi Alex,

On 13/2/23 18:01, Philippe Mathieu-Daudé wrote:
> QOM housekeeping series which replace the DO_UPCAST() macro
> uses by equivalent QOM ones (and few more trivial cleanups).
> 
> Since v2:
> - Split of bigger series
> - Split in simpler patches
> - Added Eric R-b tags
> 
> v2:
> https://lore.kernel.org/qemu-devel/20230213070820.76881-17-philmd@linaro.org/
> 
> Philippe Mathieu-Daudé (6):
>    hw/s390x/event-facility: Replace DO_UPCAST(SCLPEvent) by SCLP_EVENT()

Thomas took the first patch,

>    hw/vfio/ccw: Simplify using DEVICE() macro
>    hw/vfio/ccw: Use intermediate S390CCWDevice variable
>    hw/vfio/ccw: Replace DO_UPCAST(S390CCWDevice) by S390_CCW_DEVICE()
>    hw/vfio/ccw: Remove pointless S390CCWDevice variable
>    hw/vfio/ccw: Replace DO_UPCAST(VFIOCCWDevice) by VFIO_CCW()

Could you take the other ones via your VFIO tree?
(all patches reviewed).

Thanks,

Phil.

Re: [PATCH v3 0/6] hw/vfio/ccw: hw: Use QOM macros and remove DO_UPCAST() uses
Posted by Thomas Huth 1 year, 2 months ago
On 27/02/2023 12.15, Philippe Mathieu-Daudé wrote:
> Hi Alex,
> 
> On 13/2/23 18:01, Philippe Mathieu-Daudé wrote:
>> QOM housekeeping series which replace the DO_UPCAST() macro
>> uses by equivalent QOM ones (and few more trivial cleanups).
>>
>> Since v2:
>> - Split of bigger series
>> - Split in simpler patches
>> - Added Eric R-b tags
>>
>> v2:
>> https://lore.kernel.org/qemu-devel/20230213070820.76881-17-philmd@linaro.org/
>>
>> Philippe Mathieu-Daudé (6):
>>    hw/s390x/event-facility: Replace DO_UPCAST(SCLPEvent) by SCLP_EVENT()
> 
> Thomas took the first patch,
> 
>>    hw/vfio/ccw: Simplify using DEVICE() macro
>>    hw/vfio/ccw: Use intermediate S390CCWDevice variable
>>    hw/vfio/ccw: Replace DO_UPCAST(S390CCWDevice) by S390_CCW_DEVICE()
>>    hw/vfio/ccw: Remove pointless S390CCWDevice variable
>>    hw/vfio/ccw: Replace DO_UPCAST(VFIOCCWDevice) by VFIO_CCW()
> 
> Could you take the other ones via your VFIO tree?
> (all patches reviewed).

I've picked the other patches now, too (since they've got a Reviewed-by by 
one of the vfio-ccw maintainers now).

  Thomas


Re: [PATCH v3 0/6] hw/vfio/ccw: hw: Use QOM macros and remove DO_UPCAST() uses
Posted by Philippe Mathieu-Daudé 1 year, 2 months ago
On 27/2/23 12:46, Thomas Huth wrote:
> On 27/02/2023 12.15, Philippe Mathieu-Daudé wrote:
>> Hi Alex,

>> Could you take the other ones via your VFIO tree?
>> (all patches reviewed).
> 
> I've picked the other patches now, too (since they've got a Reviewed-by 
> by one of the vfio-ccw maintainers now).

Thank you :)