From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59814) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yn13w-0007DZ-Iw for qemu-devel@nongnu.org; Tue, 28 Apr 2015 04:40:17 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yn13s-0006y3-Gs for qemu-devel@nongnu.org; Tue, 28 Apr 2015 04:40:16 -0400 Received: from mx1.redhat.com ([209.132.183.28]:42272) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yn13s-0006xx-8S for qemu-devel@nongnu.org; Tue, 28 Apr 2015 04:40:12 -0400 Date: Tue, 28 Apr 2015 10:40:09 +0200 From: "Michael S. Tsirkin" Message-ID: <20150428103810-mutt-send-email-mst@redhat.com> References: <1430157117-26104-1-git-send-email-thuth@redhat.com> <20150428092647.33360532@thh440s> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150428092647.33360532@thh440s> Subject: Re: [Qemu-devel] [PATCH] pci: Clean up error reporting in pci_nic_init() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Thomas Huth Cc: Peter Crosthwaite , "qemu-devel@nongnu.org Developers" , Markus Armbruster On Tue, Apr 28, 2015 at 09:26:47AM +0200, Thomas Huth wrote: > On Mon, 27 Apr 2015 10:55:41 -0700 > Peter Crosthwaite wrote: > > > On Mon, Apr 27, 2015 at 10:51 AM, Thomas Huth wrote: > > > The error reporting in pci_nic_init() is quite erratic: Some errors > > > are printed directly with error_report(), and some are passed back > > > to the (only) caller pci_nic_init_nofail() via an Error pointer. > > > Let's fix up this inconsistency by always printing the error in > > > pci_nic_init() and by getting rid of the Error pointer this way. > > > > > > > Can it be made consistent the other way? - always propagate? Usually > > we move towards consistent error propagation rather that > > deep-call-chain error reports. > > In that case, I'd need to rework qemu_find_nic_model() in net/net.c, > too, since this is also printing errors directly. ... I could certainly > do that, too, but I slowly start wondering whether this all is worth > the effort, just to make the code for the _legacy_ "-net" option a > little bit nicer. Maybe we should rather focus on thinking about ways > to finally get rid of "-net" one day? Yes please. I think what it would take is supporting dump option in netdevs. Need some careful coding to bail out if people try to use this with vhost. Want to work on it? > Another idea: What about merging pci_nic_init() into > pci_nic_init_no_fail()? pci_nic_init() is only used by the _no_fail() > function, so there seems very few benefit by having this code in a > separate function. If they got merged, the error printing inconsistency > would be solved, too. > > Thomas This later one sounds good to me. -- MST