diff mbox series

[v2,02/29] accel/accel: Make TYPE_ACCEL abstract

Message ID 20200129212345.20547-3-philmd@redhat.com (mailing list archive)
State New, archived
Headers show
Series tests/acceptance/virtio_seg_max_adjust: Restrict it to Linux/X86 | expand

Commit Message

Philippe Mathieu-Daudé Jan. 29, 2020, 9:23 p.m. UTC
There is no generic acceleration, we have to use specific
implementations. Make the base class abstract.

Fixes: b14a0b7469f
Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
---
 accel/accel.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Philippe Mathieu-Daudé Jan. 29, 2020, 9:31 p.m. UTC | #1
Since this file isn't covered in MAINTAINERS, manually Cc'ing Paolo.

On 1/29/20 10:23 PM, Philippe Mathieu-Daudé wrote:
> There is no generic acceleration, we have to use specific
> implementations. Make the base class abstract.
> 
> Fixes: b14a0b7469f
> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>   accel/accel.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/accel/accel.c b/accel/accel.c
> index cb555e3b06..a0169b4e69 100644
> --- a/accel/accel.c
> +++ b/accel/accel.c
> @@ -33,6 +33,7 @@
>   static const TypeInfo accel_type = {
>       .name = TYPE_ACCEL,
>       .parent = TYPE_OBJECT,
> +    .abstract = true,
>       .class_size = sizeof(AccelClass),
>       .instance_size = sizeof(AccelState),
>   };
>
Cornelia Huck Jan. 30, 2020, 12:22 p.m. UTC | #2
On Wed, 29 Jan 2020 22:23:18 +0100
Philippe Mathieu-Daudé <philmd@redhat.com> wrote:

> There is no generic acceleration, we have to use specific
> implementations. Make the base class abstract.
> 
> Fixes: b14a0b7469f

Fixes: b14a0b7469fa ("accel: Use QOM classes for accel types")

> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
> ---
>  accel/accel.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/accel/accel.c b/accel/accel.c
> index cb555e3b06..a0169b4e69 100644
> --- a/accel/accel.c
> +++ b/accel/accel.c
> @@ -33,6 +33,7 @@
>  static const TypeInfo accel_type = {
>      .name = TYPE_ACCEL,
>      .parent = TYPE_OBJECT,
> +    .abstract = true,
>      .class_size = sizeof(AccelClass),
>      .instance_size = sizeof(AccelState),
>  };

Reviewed-by: Cornelia Huck <cohuck@redhat.com>
diff mbox series

Patch

diff --git a/accel/accel.c b/accel/accel.c
index cb555e3b06..a0169b4e69 100644
--- a/accel/accel.c
+++ b/accel/accel.c
@@ -33,6 +33,7 @@ 
 static const TypeInfo accel_type = {
     .name = TYPE_ACCEL,
     .parent = TYPE_OBJECT,
+    .abstract = true,
     .class_size = sizeof(AccelClass),
     .instance_size = sizeof(AccelState),
 };