Re: [PATCH spice v2 1/1] rename the virtio port for streaming

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> From: Lukáš Hrázký <lukkash@xxxxxxxx>
> 
> The name 'com.redhat.stream.0' is too generic and in no way denotes it
> belongs to SPICE. It is preferred to have the project's domain in the
> name and Red Hat doesn't own the project. Rename it to
> org.spice-space.stream.0.
> 
> Signed-off-by: Lukáš Hrázký <lukkash@xxxxxxxx>
> ---
>  server/reds.c                     | 2 +-
>  server/tests/test-stream-device.c | 2 +-
>  spice-common                      | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/server/reds.c b/server/reds.c
> index 998f2ffa..935448d8 100644
> --- a/server/reds.c
> +++ b/server/reds.c
> @@ -3145,7 +3145,7 @@ static int
> spice_server_char_device_add_interface(SpiceServer *reds,
>      else if (strcmp(char_device->subtype, SUBTYPE_PORT) == 0) {
>          if (strcmp(char_device->portname, "org.spice-space.webdav.0") == 0)
>          {
>              dev_state = spicevmc_device_connect(reds, char_device,
>              SPICE_CHANNEL_WEBDAV);
> -        } else if (strcmp(char_device->portname, "com.redhat.stream.0") ==
> 0) {
> +        } else if (strcmp(char_device->portname, "org.spice-space.stream.0")
> == 0) {
>              dev_state = RED_CHAR_DEVICE(stream_device_connect(reds,
>              char_device));
>          } else {
>              dev_state = spicevmc_device_connect(reds, char_device,
>              SPICE_CHANNEL_PORT);
> diff --git a/server/tests/test-stream-device.c
> b/server/tests/test-stream-device.c
> index 3c9209a4..2fdd0a39 100644
> --- a/server/tests/test-stream-device.c
> +++ b/server/tests/test-stream-device.c
> @@ -107,7 +107,7 @@ static SpiceCharDeviceInterface vmc_interface = {
>  // this specifically creates a stream device
>  static SpiceCharDeviceInstance vmc_instance = {
>      .subtype = "port",
> -    .portname = "com.redhat.stream.0",
> +    .portname = "org.spice-space.stream.0",
>  };
>  
>  static uint8_t *add_stream_hdr(uint8_t *p, StreamMsgType type, uint32_t
>  size)
> diff --git a/spice-common b/spice-common
> index 4c2d0e97..45e28448 160000
> --- a/spice-common
> +++ b/spice-common
> @@ -1 +1 @@
> -Subproject commit 4c2d0e977272c5540634d24f485dd64c424f6748
> +Subproject commit 45e2844845242b32b2bd8956da0dfffa91c0d856

I think spice-common part is accidental, besides this

Acked-by: Frediano Ziglio <fziglio@xxxxxxxxxx>

I would however wait to see what other people think about it.

As a follow up maybe a string constant in server/spice-char.h?
I noted that there's also a reference to this name in spice-prototol,
spice/stream-device.h file.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]