Linux Virtualization - Apr 2010 - [PULL] virtio console fixes, abi change

Hello Rusty,

To avoid sending the same patches again, this time I'm trying a git pull
request.

Please pull from

git://git.kernel.org/pub/scm/linux/kernel/git/amit/vs-kernel.git for-rusty

to get the patches that I've sent to the list previously, with a couple
of fixes:

- Create a console port when not using multiport (fix was sent as a
  reply to prev. series)

- Add hunk that got inadvertently reverted during git rebase operations
  (thanks Juan).

The diff to the v6 version is attached below:

diff --git a/drivers/char/virtio_console.c
b/drivers/char/virtio_console.c
index b06a0e5..a64558f 100644
--- a/drivers/char/virtio_console.c
+++ b/drivers/char/virtio_console.c
@@ -1083,6 +1083,8 @@ free_cdev:
 free_port:
        kfree(port);
 fail:
+       /* The host might want to notify management sw about port add
failure */
+       send_control_msg(port, VIRTIO_CONSOLE_PORT_READY, 0);
        return err;
 }
@@ -1407,17 +1513,14 @@ static int __devinit virtcons_probe(struct
virtio_device *vdev)
                        err = -ENOMEM;
                        goto free_vqs;
                }
+       } else {
+               /*
+                * For backward compatibility: Create a console port
+                * if we're running on older host.
+                */
+               add_port(portdev, 0);
        }

Thanks,
		Amit

Amit Shah <amit.shah at redhat.com> wrote:
> Hello Rusty,
>
> To avoid sending the same patches again, this time I'm trying a git
pull
> request.
>
> Please pull from
>
> git://git.kernel.org/pub/scm/linux/kernel/git/amit/vs-kernel.git for-rusty
>
> to get the patches that I've sent to the list previously, with a couple
> of fixes:
>
> - Create a console port when not using multiport (fix was sent as a
>   reply to prev. series)
>
> - Add hunk that got inadvertently reverted during git rebase operations
>   (thanks Juan).
>
> The diff to the v6 version is attached below:
Acked-by: Juan Quintela <quintela at redhat.com>