linux-kernel - Re: [PATCH v10 1/1] virtio_console: Add support for multiple ports for generic guest and host communication

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <200911060900.49923.borntraeger@de.ibm.com>
Date:	Fri, 6 Nov 2009 09:00:49 +0100
From:	Christian Borntraeger <borntraeger@...ibm.com>
To:	Rusty Russell <rusty@...tcorp.com.au>
Cc:	Amit Shah <amit.shah@...hat.com>,
	Anthony Liguori <aliguori@...ibm.com>,
	linux-kernel@...r.kernel.org, virtualization@...ux-foundation.org,
	"Michael S. Tsirkin" <mst@...hat.com>
Subject: Re: [PATCH v10 1/1] virtio_console: Add support for multiple ports for generic guest and host communication

Am Freitag 06 November 2009 08:10:02 schrieb Rusty Russell:
> On Wed, 4 Nov 2009 03:08:39 am Amit Shah wrote:
> > Expose multiple char devices ("ports") for simple communication
> > between the host userspace and guest.
> 
> OK, I've taken the chance to audit this patch.  I started adding patches
> until I got overwhelmed.   It's a complete mess and needs a total rewrite
>  :(

I know that Anthony disagrees, but _If we start over_, I still think we should 
use that chance and leave the old virtio console untouched and add a new driver 
for the host guest communication. IMHO it turned out that there is only a tiny 
bit of commonality. (most code pathes check for use_multiport and  then do two 
completely different things).
I like simplicity. According to David A. Wheeler's SLOCCount, the old console 
has 141 lines of code and the I truly believe that a separate guest-host comm 
vehicle would also be a lot simpler if it must not take care of the old 
virtio_console interface.

On the other hand we all should agree on one driver vs. two drivers before we go 
on. Everything else would be unfair to Amit, who had the unpleasant task to 
implement conflicting review comments....

Christian
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/