[PATCH v2 0/3] hw/hppa/machine: Restrict the total memory size to 3GB

Philippe Mathieu-Daudé posted 3 patches 4 years, 3 months ago
Test docker-mingw@fedora passed
Test asan failed
Test checkpatch passed
Test docker-quick@centos7 passed
Test FreeBSD passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20200109000525.24744-1-f4bug@amsat.org
hw/hppa/machine.c | 13 ++++++-------
1 file changed, 6 insertions(+), 7 deletions(-)
[PATCH v2 0/3] hw/hppa/machine: Restrict the total memory size to 3GB
Posted by Philippe Mathieu-Daudé 4 years, 3 months ago
Following the discussion of Igor's patch "hppa: allow max
ram size upto 4Gb" [1] I tried to simplify the current
code so Igor's series doesn't change the CLI with this
machine.

v2: Simplify by limiting to 3GB (Helge review)

[1] https://www.mail-archive.com/qemu-devel@nongnu.org/msg667903.html
[2] https://www.mail-archive.com/qemu-devel@nongnu.org/msg669309.html

Philippe Mathieu-Daudé (3):
  hw/hppa/machine: Correctly check the firmware is in PDC range
  hw/hppa/machine: Restrict the total memory size to 3GB
  hw/hppa/machine: Map the PDC memory region with higher priority

 hw/hppa/machine.c | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

-- 
2.21.1


Re: [PATCH v2 0/3] hw/hppa/machine: Restrict the total memory size to 3GB
Posted by Richard Henderson 4 years, 3 months ago
On 1/9/20 11:05 AM, Philippe Mathieu-Daudé wrote:
> Philippe Mathieu-Daudé (3):
>   hw/hppa/machine: Correctly check the firmware is in PDC range
>   hw/hppa/machine: Restrict the total memory size to 3GB
>   hw/hppa/machine: Map the PDC memory region with higher priority

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>


r~

Re: [PATCH v2 0/3] hw/hppa/machine: Restrict the total memory size to 3GB
Posted by Helge Deller 4 years, 3 months ago
On 09.01.20 03:20, Richard Henderson wrote:
> On 1/9/20 11:05 AM, Philippe Mathieu-Daudé wrote:
>> Philippe Mathieu-Daudé (3):
>>   hw/hppa/machine: Correctly check the firmware is in PDC range
>>   hw/hppa/machine: Restrict the total memory size to 3GB
>>   hw/hppa/machine: Map the PDC memory region with higher priority
>
> Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

Acked-by: Helge Deller <deller@gmx.de>

Re: [PATCH v2 0/3] hw/hppa/machine: Restrict the total memory size to 3GB
Posted by Richard Henderson 4 years, 3 months ago
On 1/8/20 2:05 PM, Philippe Mathieu-Daudé wrote:
> Following the discussion of Igor's patch "hppa: allow max
> ram size upto 4Gb" [1] I tried to simplify the current
> code so Igor's series doesn't change the CLI with this
> machine.
> 
> v2: Simplify by limiting to 3GB (Helge review)
> 
> [1] https://www.mail-archive.com/qemu-devel@nongnu.org/msg667903.html
> [2] https://www.mail-archive.com/qemu-devel@nongnu.org/msg669309.html
> 
> Philippe Mathieu-Daudé (3):
>   hw/hppa/machine: Correctly check the firmware is in PDC range
>   hw/hppa/machine: Restrict the total memory size to 3GB
>   hw/hppa/machine: Map the PDC memory region with higher priority
> 
>  hw/hppa/machine.c | 13 ++++++-------
>  1 file changed, 6 insertions(+), 7 deletions(-)
> 

Queued.


r~