From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Michael S. Tsirkin" Subject: Re: [RFC PATCH 4/8] virtio: console: Ignore port name update request if name already set Date: Fri, 29 Jul 2011 07:27:55 +0300 Message-ID: <20110729042755.GF31888@redhat.com> References: <20110729041958.GE31888@redhat.com> <20110729042619.GC24868@amit-x200.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <20110729042619.GC24868@amit-x200.redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Amit Shah Cc: Virtualization List List-Id: virtualization@lists.linuxfoundation.org On Fri, Jul 29, 2011 at 09:56:19AM +0530, Amit Shah wrote: > On (Fri) 29 Jul 2011 [07:19:58], Michael S. Tsirkin wrote: > > On Thu, Jul 28, 2011 at 08:05:09PM +0530, Amit Shah wrote: > > > We don't allow port name changes dynamically for a port. So any > > > requests by the host to change the name are ignored. > > > > It would be nice to check the name and if > > changed, print a warning. > > But not change it, right? That's what I meant. > I don't know what applications expect, but > I think we should keep the port names stable while they're plugged in. > > > > Before this patch, if the hypervisor sent a port name while we had one > > > set already, we would leak memory equivalent to the size of the old > > > name. > > > > > > This scenario wasn't expected so far, but with the suspend-resume > > > support, we'll send the VIRTIO_CONSOLE_PORT_READY message after restore, > > > which can get us into this situation. > > > > > > Signed-off-by: Amit Shah > > > > Memleaks are bad. So this is a bugfix we probably should have anyway? > > Yes. > > Amit