[Qemu-devel] [PATCH v2 3/4] memory: Fix memory_region 'priority' property sign

Philippe Mathieu-Daudé posted 4 patches 7 years, 10 months ago
There is a newer version of this series
[Qemu-devel] [PATCH v2 3/4] memory: Fix memory_region 'priority' property sign
Posted by Philippe Mathieu-Daudé 7 years, 10 months ago
Priorities can be negative, fix this limitation.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
---
 memory.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/memory.c b/memory.c
index e77f9e4036..eaa5fa7f23 100644
--- a/memory.c
+++ b/memory.c
@@ -1258,7 +1258,7 @@ static void memory_region_initfn(Object *obj)
                         memory_region_get_addr,
                         NULL, /* memory_region_set_addr */
                         NULL, NULL, &error_abort);
-    object_property_add(OBJECT(mr), "priority", "uint32",
+    object_property_add(OBJECT(mr), "priority", "int32",
                         memory_region_get_priority,
                         NULL, /* memory_region_set_priority */
                         NULL, NULL, &error_abort);
-- 
2.16.3


Re: [PATCH v2 3/4] memory: Fix memory_region 'priority' property sign
Posted by Philippe Mathieu-Daudé 4 years, 10 months ago
ping?

On Thu, Apr 5, 2018 at 3:23 AM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>
> Priorities can be negative, fix this limitation.
>
> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
>  memory.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/memory.c b/memory.c
> index e77f9e4036..eaa5fa7f23 100644
> --- a/memory.c
> +++ b/memory.c
> @@ -1258,7 +1258,7 @@ static void memory_region_initfn(Object *obj)
>                          memory_region_get_addr,
>                          NULL, /* memory_region_set_addr */
>                          NULL, NULL, &error_abort);
> -    object_property_add(OBJECT(mr), "priority", "uint32",
> +    object_property_add(OBJECT(mr), "priority", "int32",
>                          memory_region_get_priority,
>                          NULL, /* memory_region_set_priority */
>                          NULL, NULL, &error_abort);
> --
> 2.16.3
>

Re: [PATCH v2 3/4] memory: Fix memory_region 'priority' property sign
Posted by Philippe Mathieu-Daudé 4 years, 10 months ago
On 3/27/21 2:57 PM, Philippe Mathieu-Daudé wrote:
> ping?

Bah this doesn't apply anymore (memory.c moved to softmmu/).

I'll respin.

> On Thu, Apr 5, 2018 at 3:23 AM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
>>
>> Priorities can be negative, fix this limitation.
>>
>> Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
>> ---
>>  memory.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/memory.c b/memory.c
>> index e77f9e4036..eaa5fa7f23 100644
>> --- a/memory.c
>> +++ b/memory.c
>> @@ -1258,7 +1258,7 @@ static void memory_region_initfn(Object *obj)
>>                          memory_region_get_addr,
>>                          NULL, /* memory_region_set_addr */
>>                          NULL, NULL, &error_abort);
>> -    object_property_add(OBJECT(mr), "priority", "uint32",
>> +    object_property_add(OBJECT(mr), "priority", "int32",
>>                          memory_region_get_priority,
>>                          NULL, /* memory_region_set_priority */
>>                          NULL, NULL, &error_abort);
>> --
>> 2.16.3
>>
>