Re: [Qemu-devel] [PATCH 11/30] hw/s390x: use the BYTE-based definitions

2018-03-05 Thread Cornelia Huck
On Thu, 15 Feb 2018 01:28:41 -0300
Philippe Mathieu-Daudé  wrote:

> It ease code review, unit is explicit.

s/ease/eases/

> 
> Signed-off-by: Philippe Mathieu-Daudé 
> ---
>  hw/s390x/sclp.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/hw/s390x/sclp.c b/hw/s390x/sclp.c
> index 276972b59f..8537aa2688 100644
> --- a/hw/s390x/sclp.c
> +++ b/hw/s390x/sclp.c
> @@ -526,8 +526,8 @@ static void sclp_realize(DeviceState *dev, Error **errp)
>  
>  ret = s390_set_memory_limit(machine->maxram_size, &hw_limit);
>  if (ret == -E2BIG) {
> -error_setg(&err, "host supports a maximum of %" PRIu64 " GB",
> -   hw_limit >> 30);
> +error_setg(&err, "host supports a maximum of %llu GB",
> +   hw_limit / G_BYTE);
>  } else if (ret) {
>  error_setg(&err, "setting the guest size failed");
>  }

Acked-by: Cornelia Huck 



[Qemu-devel] [PATCH 11/30] hw/s390x: use the BYTE-based definitions

2018-02-14 Thread Philippe Mathieu-Daudé
It ease code review, unit is explicit.

Signed-off-by: Philippe Mathieu-Daudé 
---
 hw/s390x/sclp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/hw/s390x/sclp.c b/hw/s390x/sclp.c
index 276972b59f..8537aa2688 100644
--- a/hw/s390x/sclp.c
+++ b/hw/s390x/sclp.c
@@ -526,8 +526,8 @@ static void sclp_realize(DeviceState *dev, Error **errp)
 
 ret = s390_set_memory_limit(machine->maxram_size, &hw_limit);
 if (ret == -E2BIG) {
-error_setg(&err, "host supports a maximum of %" PRIu64 " GB",
-   hw_limit >> 30);
+error_setg(&err, "host supports a maximum of %llu GB",
+   hw_limit / G_BYTE);
 } else if (ret) {
 error_setg(&err, "setting the guest size failed");
 }
-- 
2.16.1