From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: Problems when using latest git tree to boot xen on OMAP5 Date: Wed, 9 Oct 2013 13:05:59 +0100 Message-ID: <1381320359.7600.16.camel@kazak.uk.xensource.com> References: <936C837B-7005-4CE0-8265-7B7ACA7C78FD@gmail.com> <91879A6F-B459-4D10-A691-4A04915D82AA@gmail.com> <1381135195.21562.56.camel@kazak.uk.xensource.com> <525285B3.8010902@gmail.com> <52529404.8030909@linaro.org> <20131009074627.GA6342@cbz-workstation> <1381306276.9920.21.camel@kazak.uk.xensource.com> <52553AE2.6070806@linaro.org> <1381317559.7600.3.camel@kazak.uk.xensource.com> <52553E35.6080408@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <52553E35.6080408@linaro.org> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Julien Grall Cc: Chen Baozi , List Xen Developer List-Id: xen-devel@lists.xenproject.org On Wed, 2013-10-09 at 12:29 +0100, Julien Grall wrote: > On 10/09/2013 12:19 PM, Ian Campbell wrote: > > On Wed, 2013-10-09 at 12:15 +0100, Julien Grall wrote: > >> On 10/09/2013 09:11 AM, Ian Campbell wrote: > >>> Rather than whitelisting and mapping disabled devices through perhaps we > >>> should implement them as read 0xf (or 0x0) and write ignore? > >> > >> We can't assume that these values (0x0 or 0xf) won't affect dom0 behaviour. > > > > 0xff is what you would typically get back if there was no actual > > hardware present. > > > >> What about a new quirk to map all disabled device in dom0 > >> (PLATFORM_QUIRK_DOM0_MAP_DISABLED_DEVICE)? > > > > Why is this not the default behaviour? > > From our discussion a months a ago (http://patches.linaro.org/19590/), > status = "disabled" means : "not present/wired-up on this variant of > the platform". Actually ePAPR says: Indicates that the device is not presently operational, but it might become operational in the future (for example, something is not plugged in, or switched off). Refer to the device binding for details on what disabled means for a given device. This doesn't strictly speaking say "and you must not prod any associated MMIO". > As Linux should cope with this status, why do we need to map these > devices in DOM0? Because in practice Linux doesn't actually cope like you asserted it would. > If we go further, with passthrough we may want remove device from the > device tree. If Linux has hardcoded device (but also in the device tree) > it will likely fail. If/when we come to be passing through a device which Linux is poking despite it being disabled then we have a kernel bug which needs fixing, nothing more, and there is no reason for this to be a factor in deciding what to do by default. Ian.