From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:51367) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VqauN-0008Qh-RF for qemu-devel@nongnu.org; Tue, 10 Dec 2013 22:56:29 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VqauD-0005xe-Rg for qemu-devel@nongnu.org; Tue, 10 Dec 2013 22:56:23 -0500 Received: from mail-pd0-f180.google.com ([209.85.192.180]:60414) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VqauD-0005xC-Kr for qemu-devel@nongnu.org; Tue, 10 Dec 2013 22:56:13 -0500 Received: by mail-pd0-f180.google.com with SMTP id q10so8673608pdj.25 for ; Tue, 10 Dec 2013 19:56:12 -0800 (PST) Message-ID: <52A7E256.4050006@ozlabs.ru> Date: Wed, 11 Dec 2013 14:56:06 +1100 From: Alexey Kardashevskiy MIME-Version: 1.0 References: <1386660757-7505-1-git-send-email-aik@ozlabs.ru> <1386660757-7505-2-git-send-email-aik@ozlabs.ru> <52A6DD35.5020302@redhat.com> In-Reply-To: <52A6DD35.5020302@redhat.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v3 1/5] boot: extend get_boot_devices_list() to ignore suffixes List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: Gleb Natapov , qemu-devel@nongnu.org, Alexander Graf , Paul Mackerras , qemu-ppc@nongnu.org, =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= On 12/10/2013 08:21 PM, Paolo Bonzini wrote: > Il 10/12/2013 08:32, Alexey Kardashevskiy ha scritto: >> As suffixes do not make sense for sPAPR's device tree and >> there is no way to filter them out on the BusState::get_fw_dev_path >> level, let's add an ability for the external caller to specify >> whether to apply suffixes or not. >> >> Signed-off-by: Alexey Kardashevskiy > > SeaBIOS simply ignores unknown suffixes once it has recognized a part of > the path. Could SLOF do the same? SeaBIOS does not need it. SLOF does not need it either. Who does need it? What is it for after all? Commit 1ca4d09ae0bcc2fdd6aeef0fdc11f82394f7e757 "Add bootindex parameter to net/block/fd device" does not explain that at all. SLOF cannot just ignore this, why? It is a part of a path, it should parse tree to find the node OR if it fails, find the closest matching node and pass the rest of the path as a package parameter. And we do not really plan to implement this (no use) as this is purely QEMU stuff which we do not really want to carry to SLOF. I'd stick to my original patch (please :) ) or add yet another Interface just to ignore the suffix or method (ugly as it will be called from a place different than hw/core/qdev.c). -- Alexey