All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stephen Boyd <sboyd@codeaurora.org>
To: Miche Baker-Harvey <miche@google.com>
Cc: linux-kernel@vger.kernel.org
Subject: Re: [PATCH RFC v2 0/3] Support multiple VirtioConsoles.
Date: Tue, 08 Nov 2011 16:54:04 -0800	[thread overview]
Message-ID: <4EB9CF2C.7040501@codeaurora.org> (raw)
In-Reply-To: <20111108192916.30437.47434.stgit@miche.sea.corp.google.com>

On 11/08/11 11:29, Miche Baker-Harvey wrote:
> (Amit pointed out that the patches never went out. This is a resend of
> the series meant to go out on 11/2/2011; I've marked it "v2".)  
>
> This patchset applies to linux-next/next-20111102.
>
> This series implements support for multiple virtio_consoles using KVM.
>
> This patchset addresses several issues associated with trying to
> establish multiple virtio consoles. 
>
> I'm trying to start a guest via KVM that supports multiple virtual
> consoles, with getty's on each, and with some being console devices.
>
> These patches let me establish more than one VirtioConsole (I'm
> running eight at the moment), and enable console output appearing on
> one of them.  It still doesn't successfully generate console output on
> multiple VirtioConsoles.
>
> Let me apologise for my last patch having gotten into Linus' tree, and
> leaving other people to deal with crashes.  I had meant to be asking
> for guidance, but I didn't mark it as "RFC".
>
> This series reflects the input from Konrad Rzeszutek, Amit Shah, Stephen
> Boyd, and Rusty Russell.  I think we do have to limit hvc_alloc() to one
> thread.

It might be good to Cc these people in the future. I only found this
when scanning over lkml.

>
> I would appreciate any comments or feedback, or accept if appropriate.

I tested this against next-20111102 and I see duplicate messages on the
console. What's going on?

# echo hi > /dev/kmsg
[  132.960000] hi
[  132.960000] hi

-- 
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.


  parent reply	other threads:[~2011-11-09  0:54 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-08 19:29 [PATCH RFC v2 0/3] Support multiple VirtioConsoles Miche Baker-Harvey
2011-11-08 19:29 ` [PATCH v2 1/3] virtio_console: Fix locking of vtermno Miche Baker-Harvey
2011-11-08 19:29 ` [PATCH v2 2/3] hvc_init(): Enforce one-time initialization Miche Baker-Harvey
2011-11-08 19:29 ` [PATCH v2 3/3] Use separate struct console structure for each hvc_console Miche Baker-Harvey
2011-11-09  0:54 ` Stephen Boyd [this message]
  -- strict thread matches above, loose matches on Subject: below --
2011-11-08 19:29 [PATCH RFC v2 0/3] Support multiple VirtioConsoles Miche Baker-Harvey
2011-11-08 19:29 Miche Baker-Harvey
2011-11-08 19:30 Miche Baker-Harvey

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4EB9CF2C.7040501@codeaurora.org \
    --to=sboyd@codeaurora.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=miche@google.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.