From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2a00:1450:400c:c0c::242; helo=mail-wr0-x242.google.com; envelope-from=leif.lindholm@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-wr0-x242.google.com (mail-wr0-x242.google.com [IPv6:2a00:1450:400c:c0c::242]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 52A9221110023 for ; Fri, 1 Jun 2018 03:18:26 -0700 (PDT) Received: by mail-wr0-x242.google.com with SMTP id l10-v6so963732wrn.2 for ; Fri, 01 Jun 2018 03:18:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=w+XonAHeobM53trHYsqVjF+yLvRGCOt4HOJ7jMnwL44=; b=FYb/AHVlGvgpMJACUk/zim1l2vaOVpr1nJ7G69Zo2C+By9b0GB1UJrUWAQQBzwLPxp vPchDMlu4qkonMjJvRz+BQQdCrdSbvqk2kfAiEPYCsIj9HhbimG5HwVL0N1XmcIg7lxw yHgxJX6tIWQ0ro0V1UX4EmT4JmWM8FokvpijM= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=w+XonAHeobM53trHYsqVjF+yLvRGCOt4HOJ7jMnwL44=; b=IisMqQFpus4p9GygsFDl5mWQ48LQ0ornU2RBNMmxYldA0+cIlIjxoN/8xXdkd8EuIh +6H0ygzXxchjD2F3OmPoVgFmZfnwBTrQQBtS9GT4+Wnz1JkaNqZSFg0w3YOYvFAA3bu2 SHDagWKXxo8LHJv82DzEYlKtg9rY16LqTq02B6y7Lr9A/SZKtjrJssk0U7dC04xQnnq+ 5H6Tn7jlsXwvo61CakxsZBd1A+Fo0RQBJ0kEoys44Y6iIvpXgUZLLor7UV3mTY4ZsATo 0uFO854r5LJo4aXkcLHsITtKAPQmWCrSIpaBuRPsVu06DlaMQGiMXvFaIBkqfMe2VMfA t96w== X-Gm-Message-State: ALKqPwfRe6bcqdI/zV49fkWnFlIoy4pKxO2eOwwkkz/jhMj4TJQhC8bG zBQeDIpaxNo9VTt72svKu5wXQg== X-Google-Smtp-Source: ADUXVKJDJYPD9jBr575OWdiYtYq7JkA+EEBdyJsDo/w+J2/i5lImOOI56hMDhfUKnlCBUoMIcUgldA== X-Received: by 2002:adf:bbcd:: with SMTP id z13-v6mr8442760wrg.183.1527848304990; Fri, 01 Jun 2018 03:18:24 -0700 (PDT) Received: from bivouac.eciton.net (bivouac.eciton.net. [2a00:1098:0:86:1000:23:0:2]) by smtp.gmail.com with ESMTPSA id x73-v6sm1153524wmf.21.2018.06.01.03.18.23 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Fri, 01 Jun 2018 03:18:23 -0700 (PDT) Date: Fri, 1 Jun 2018 11:18:22 +0100 From: Leif Lindholm To: Ard Biesheuvel Cc: edk2-devel@lists.01.org, graeme.gregory@linaro.org Message-ID: <20180601101822.a4xpssqyv7ynxklm@bivouac.eciton.net> References: <20180531104634.23215-1-ard.biesheuvel@linaro.org> <20180531211608.jqwwtfv4s5e3p2ft@bivouac.eciton.net> MIME-Version: 1.0 In-Reply-To: <20180531211608.jqwwtfv4s5e3p2ft@bivouac.eciton.net> User-Agent: NeoMutt/20170113 (1.7.2) Subject: Re: [PATCH edk2-platforms 0/2] enable FPDT table for SynQuacer X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 01 Jun 2018 10:18:27 -0000 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Thu, May 31, 2018 at 10:16:08PM +0100, Leif Lindholm wrote: > On Thu, May 31, 2018 at 12:46:32PM +0200, Ard Biesheuvel wrote: > > Enable some of the status code reporting infrastructure in patch #1 so > > we can include the FPDT DXE driver in patch #2 which produces the FPDT > > diagnostic table which tells us how much time was spent in the firmware, > > in the OS loader, and in ExitBootServices(). > > > > Ard Biesheuvel (2): > > Platform/DeveloperBox: wire up status code reporting for DXE phase > > Platform/DeveloperBox: enable FPDT table > > For the series: > Reviewed-by: Leif Lindholm Hmm, after a side discussion with Ard, I have asked him _not_ to push this set for now. The reason for this is that its use under Linux is dependent on the fundamentally broken concept of directly mapping physical address regions into non-privileged processes using /dev/mem. I want to get a sane kernel interface in place for accessing this data before we start encouraging its use. If this holds anyone back for other operating systems, please speak up (off-list if needs be). The Reviewed-by still applies, and the patches are ready to go. / Leif > > Platform/Socionext/DeveloperBox/DeveloperBox.dsc | 12 ++++++++++++ > > Platform/Socionext/DeveloperBox/DeveloperBox.fdf | 2 ++ > > Silicon/Socionext/SynQuacer/Acpi.dsc.inc | 6 ++++++ > > 3 files changed, 20 insertions(+) > > > > -- > > 2.17.0 > >