Re: [PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-24 Thread Joao Martins
On 2/24/22 09:01, Igor Mammedov wrote: > On Wed, 23 Feb 2022 17:18:55 + > Joao Martins wrote: > >> On 2/14/22 15:18, Joao Martins wrote: >>> On 2/14/22 15:03, Igor Mammedov wrote: On Mon, 7 Feb 2022 20:24:21 + Joao Martins wrote: > Default phys-bits on Qemu is TCG

Re: [PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-24 Thread Igor Mammedov
On Wed, 23 Feb 2022 17:18:55 + Joao Martins wrote: > On 2/14/22 15:18, Joao Martins wrote: > > On 2/14/22 15:03, Igor Mammedov wrote: > >> On Mon, 7 Feb 2022 20:24:21 + > >> Joao Martins wrote: > >> > >>> Default phys-bits on Qemu is TCG_PHYS_BITS (40) which is enough > >>> to addre

Re: [PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-23 Thread Joao Martins
On 2/14/22 15:18, Joao Martins wrote: > On 2/14/22 15:03, Igor Mammedov wrote: >> On Mon, 7 Feb 2022 20:24:21 + >> Joao Martins wrote: >> >>> Default phys-bits on Qemu is TCG_PHYS_BITS (40) which is enough >>> to address 1Tb (0xff ). On AMD platforms, if a >>> ram-above-4g relocation

Re: [PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-14 Thread Joao Martins
On 2/14/22 15:41, Igor Mammedov wrote: > On Mon, 14 Feb 2022 15:18:43 + > Joao Martins wrote: >> On 2/14/22 15:03, Igor Mammedov wrote: >>> On Mon, 7 Feb 2022 20:24:21 + >>> Joao Martins wrote: diff --git a/hw/i386/pc.c b/hw/i386/pc.c index b060aedd38f3..f8712eb8427e 100644 >>>

Re: [PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-14 Thread Joao Martins
On 2/14/22 15:03, Igor Mammedov wrote: > On Mon, 7 Feb 2022 20:24:21 + > Joao Martins wrote: > >> Default phys-bits on Qemu is TCG_PHYS_BITS (40) which is enough >> to address 1Tb (0xff ). On AMD platforms, if a >> ram-above-4g relocation happens and the CPU wasn't configured >> wit

Re: [PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-14 Thread Igor Mammedov
On Mon, 14 Feb 2022 15:18:43 + Joao Martins wrote: > On 2/14/22 15:03, Igor Mammedov wrote: > > On Mon, 7 Feb 2022 20:24:21 + > > Joao Martins wrote: > > > >> Default phys-bits on Qemu is TCG_PHYS_BITS (40) which is enough > >> to address 1Tb (0xff ). On AMD platforms, if a

Re: [PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-14 Thread Igor Mammedov
On Mon, 7 Feb 2022 20:24:21 + Joao Martins wrote: > Default phys-bits on Qemu is TCG_PHYS_BITS (40) which is enough > to address 1Tb (0xff ). On AMD platforms, if a > ram-above-4g relocation happens and the CPU wasn't configured > with a big enough phys-bits, warn the user. There is

Re: [PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-14 Thread Joao Martins
On 2/14/22 13:15, David Edmondson wrote: > On Monday, 2022-02-07 at 20:24:21 GMT, Joao Martins wrote: > >> Default phys-bits on Qemu is TCG_PHYS_BITS (40) which is enough >> to address 1Tb (0xff ). On AMD platforms, if a >> ram-above-4g relocation happens and the CPU wasn't configured >>

Re: [PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-14 Thread David Edmondson
On Monday, 2022-02-07 at 20:24:21 GMT, Joao Martins wrote: > Default phys-bits on Qemu is TCG_PHYS_BITS (40) which is enough > to address 1Tb (0xff ). On AMD platforms, if a > ram-above-4g relocation happens and the CPU wasn't configured > with a big enough phys-bits, warn the user. There

[PATCH RFCv2 3/4] i386/pc: warn if phys-bits is too low

2022-02-07 Thread Joao Martins
Default phys-bits on Qemu is TCG_PHYS_BITS (40) which is enough to address 1Tb (0xff ). On AMD platforms, if a ram-above-4g relocation happens and the CPU wasn't configured with a big enough phys-bits, warn the user. There isn't a catastrophic failure exactly, the guest will still boot, bu