[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/7] hw/misc/empty_slot: Spring cleaning
From: |
Peter Maydell |
Subject: |
Re: [PATCH 0/7] hw/misc/empty_slot: Spring cleaning |
Date: |
Sun, 24 May 2020 22:37:57 +0100 |
On Sun, 24 May 2020 at 21:21, Philippe Mathieu-Daudé <address@hidden> wrote:
> On 5/24/20 9:37 PM, Aleksandar Markovic wrote:
> > I agree with all of your patches, they absolutely make sense to me,
> > but I would like to know Peter's opinion on such treatment of empty
> > slots.
> >
> > I am going to give r-bs and integrate mips patches as soon as Peter
> > OKs the general approach. So, Peter, is Philippe's approach to empty
> > slots fine?
>
> Thanks Aleksandar for looking at this series.
>
> I expect a neutral opinion from Peter.
Yes. In particular I cannot be the bottleneck for all design
choices in QEMU: that doesn't scale. Mostly you can assume
that if I have a strong opinion then I'll provide it, and
otherwise I prefer it if the people who care enough to
maintain code and write patches to tidy it up make the choices.
(Empty slots seemed a bit odd to me last time I looked at them,
but if they work for mips and sparc that's fine with me.)
thanks
-- PMM
- [PATCH 1/7] hw/sparc/sun4m: Use UnimplementedDevice for I/O devices, (continued)
- [PATCH 1/7] hw/sparc/sun4m: Use UnimplementedDevice for I/O devices, Philippe Mathieu-Daudé, 2020/05/10
- [PATCH 2/7] hw/misc/empty_slot: Lower address space priority, Philippe Mathieu-Daudé, 2020/05/10
- [PATCH 3/7] hw/misc/empty_slot: Convert 'size' field as qdev property, Philippe Mathieu-Daudé, 2020/05/10
- [PATCH 4/7] hw/misc/empty_slot: Add a 'name' qdev property, Philippe Mathieu-Daudé, 2020/05/10
- [PATCH 5/7] hw/misc/empty_slot: Convert debug printf() to trace event, Philippe Mathieu-Daudé, 2020/05/10
- [PATCH 6/7] hw/misc/empty_slot: Move the 'hw/misc' and cover in MAINTAINERS, Philippe Mathieu-Daudé, 2020/05/10
- [PATCH 7/7] hw/misc/empty_slot: Name the slots when created, Philippe Mathieu-Daudé, 2020/05/10
- Re: [PATCH 0/7] hw/misc/empty_slot: Spring cleaning, Philippe Mathieu-Daudé, 2020/05/24