[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 6
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit |
Date: |
Tue, 30 Jun 2020 12:52:03 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.5.0 |
On 6/30/20 12:48 PM, Aleksandar Markovic wrote:
>
>
> уторак, 30. јун 2020., Philippe Mathieu-Daudé <f4bug@amsat.org
> <mailto:f4bug@amsat.org>> је написао/ла:
>
> Hi,
>
> Following Jiaxun Yang's patch and discussion:
> https://patchwork.kernel.org/patch/11416915/
> <https://patchwork.kernel.org/patch/11416915/>
>
> - Rename the current machine as 'malta-virt' (keeping 'malta' aliased)
> Suggestions for better names are welcome, maybe 'malta-unreal' or
> 'malta-unleashed' instead?
> - Add 'malta-phys' which respects hardware restrictions (on RAM so far)
> - Unleash 'malta-virt' to allow more than 2GB on 64-bit
>
> Philippe Mathieu-Daudé (7):
> hw/mips/malta: Trivial code movement
> hw/mips/malta: Register the machine as a TypeInfo
> hw/mips/malta: Rename 'malta' machine as 'malta-virt'
> hw/mips/malta: Introduce MaltaMachineClass::max_ramsize
> hw/mips/malta: Introduce the 'malta-phys' machine
> hw/mips/malta: Verify malta-phys machine uses correct DIMM sizes
> hw/mips/malta: Allow more than 2GB on 64-bit malta-virt
>
> hw/mips/malta.c | 121 +++++++++++++++++++++++++++++++++++++++---------
> 1 file changed, 99 insertions(+), 22 deletions(-)
>
> --
>
>
>
> Thank you, Philippe, for providing this series.
>
> However, in previous discussion on the patch you mention above, I
> already expressed serious reservations on the approach taken in that
> patch. These reservations stay today too.
>
> There is nothing qualitatively different between the original patch and
> this series. Naming and related stuff are just cosmetic issues.
OK, what about considering all patches except the last one?
So we can run firmware on a real Malta board, not the QEMU
imaginary one (in the discussion you said QEMU should respect
real hardware, which I agree).
>
> The good thing about this series is that one can apply it dowstream, if
> one finds it useful. However, it is not suitable for upstreaming
>
> Regards,
> Aleksandar
>
>
>
> 2.21.3
>
- [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Philippe Mathieu-Daudé, 2020/06/30
- [PATCH 3/7] hw/mips/malta: Rename 'malta' machine as 'malta-virt', Philippe Mathieu-Daudé, 2020/06/30
- [PATCH 2/7] hw/mips/malta: Register the machine as a TypeInfo, Philippe Mathieu-Daudé, 2020/06/30
- [PATCH 1/7] hw/mips/malta: Trivial code movement, Philippe Mathieu-Daudé, 2020/06/30
- [PATCH 4/7] hw/mips/malta: Introduce MaltaMachineClass::max_ramsize, Philippe Mathieu-Daudé, 2020/06/30
- [PATCH 5/7] hw/mips/malta: Introduce the 'malta-phys' machine, Philippe Mathieu-Daudé, 2020/06/30
- [PATCH 7/7] hw/mips/malta: Allow more than 2GB on 64-bit malta-virt, Philippe Mathieu-Daudé, 2020/06/30
- [PATCH 6/7] hw/mips/malta: Verify malta-phys machine uses correct DIMM sizes, Philippe Mathieu-Daudé, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Aleksandar Markovic, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit,
Philippe Mathieu-Daudé <=
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Philippe Mathieu-Daudé, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Aleksandar Markovic, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Philippe Mathieu-Daudé, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Aleksandar Markovic, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Philippe Mathieu-Daudé, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Aleksandar Markovic, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Philippe Mathieu-Daudé, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Aleksandar Markovic, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Thomas Huth, 2020/06/30
- Re: [PATCH 0/7] hw/mips/malta: Rework to allow more than 2GB of RAM on 64-bit, Philippe Mathieu-Daudé, 2020/06/30