From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42883) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1agXly-0005Su-BK for qemu-devel@nongnu.org; Thu, 17 Mar 2016 09:15:31 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1agXls-0004VI-OI for qemu-devel@nongnu.org; Thu, 17 Mar 2016 09:15:30 -0400 Received: from mx1.redhat.com ([209.132.183.28]:59609) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1agXlp-0004U2-02 for qemu-devel@nongnu.org; Thu, 17 Mar 2016 09:15:24 -0400 Date: Thu, 17 Mar 2016 15:15:16 +0200 From: "Michael S. Tsirkin" Message-ID: <20160317151418-mutt-send-email-mst@redhat.com> References: <20160316181541.GG12454@HEDWIG.INI.CMU.EDU> <56E9A75D.60603@redhat.com> <20160316222436-mutt-send-email-mst@redhat.com> <56EA6F7C.6090700@redhat.com> <56EA7B88.8020004@redhat.com> <20160317133036-mutt-send-email-mst@redhat.com> <56EAAD58.9080605@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <56EAAD58.9080605@redhat.com> Subject: Re: [Qemu-devel] [PATCH v2] vl.c: disallow command line fw cfg without opt/ List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: "Gabriel L. Somlo" , kraxel@redhat.com, Laszlo Ersek , qemu-devel@nongnu.org, armbru@redhat.com On Thu, Mar 17, 2016 at 02:12:56PM +0100, Paolo Bonzini wrote: > > > On 17/03/2016 12:32, Michael S. Tsirkin wrote: > > > 1) files should actually be named etc/ovmf/foo. OVMF could optionally > > > accept both the old and the new names for a while, you would decide > > > whether this is useful. > > > > IOW if etc/ovmf exists, then ignore opt/ovmf? OK. > > > > > 2) in turn, because of (1) even the warning on opt/ should be removed. > > > > If we do 2) then users might put files in etc/ovmf/foo. > > Right, that's exactly the point. These are user-configurable knobs for > the firmware (user-configurable meaning they can't be expected to > recompile QEMU). > > Paolo Right. See Markus' answer and my proposal that attempts to be a superset of this. -- MST