[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12
From: |
Christian Borntraeger |
Subject: |
Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12 |
Date: |
Fri, 15 Dec 2017 10:53:59 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 12/15/2017 10:53 AM, David Hildenbrand wrote:
>
>> diff --git a/hw/s390x/s390-virtio-ccw.c b/hw/s390x/s390-virtio-ccw.c
>> index 80e753a5ef..9abb8de9f9 100644
>> --- a/hw/s390x/s390-virtio-ccw.c
>> +++ b/hw/s390x/s390-virtio-ccw.c
>> @@ -172,9 +172,10 @@ static void s390_memory_init(ram_addr_t mem_size)
>> name = g_strdup_printf("s390.ram");
>> while (mem_size) {
>> MemoryRegion *ram = g_new(MemoryRegion, 1);
>> + unsigned long long size = mem_size;
>>
>> /* KVM does not allow memslots >= 8 TB */
>> - chunk = MIN(mem_size, KVM_SLOT_MAX_BYTES);
>> + chunk = MIN(size, KVM_SLOT_MAX_BYTES);
>
> directly cast inline?
As I said, this still triggers the warning. gcc 7 is too clever.
- [qemu-s390x] [PULL 46/46] s390-ccw-virtio: allow for systems larger that 7.999TB, (continued)
- [qemu-s390x] [PULL 46/46] s390-ccw-virtio: allow for systems larger that 7.999TB, Cornelia Huck, 2017/12/14
- [qemu-s390x] [PULL 45/46] s390x: change the QEMU cpu model to a stripped down z12, Cornelia Huck, 2017/12/14
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12, Peter Maydell, 2017/12/14
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12, Christian Borntraeger, 2017/12/15
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12, Cornelia Huck, 2017/12/15
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12, Peter Maydell, 2017/12/15
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12, Cornelia Huck, 2017/12/15
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12, Peter Maydell, 2017/12/15
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12, Cornelia Huck, 2017/12/15
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12, David Hildenbrand, 2017/12/15
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12,
Christian Borntraeger <=
- Re: [qemu-s390x] [PULL 00/46] First batch of s390x patches for 2.12, David Hildenbrand, 2017/12/15