mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jean Delvare <jdelvare@suse.de>
To: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: "Ivan.khoronzhuk" <ivan.khoronzhuk@globallogic.com>,
	dmidecode-devel@nongnu.org,
	Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>,
	Matt Fleming <matt.fleming@intel.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Leif Lindholm <leif.lindholm@linaro.org>,
	Mark Salter <msalter@redhat.com>,
	Grant Likely <grant.likely@linaro.org>
Subject: Re: [dmidecode] [Patch v3] firmware: dmi-sysfs: add SMBIOS entry point area raw attribute
Date: Sun, 8 Mar 2015 18:11:40 +0100	[thread overview]
Message-ID: <20150308181140.288dfa88@endymion.delvare> (raw)
In-Reply-To: <CAKv+Gu-Pit=N+9cYaY7snZtvGaEzTLS0zby4RCK0B9XmX8k8+w@mail.gmail.com>

On Sun, 8 Mar 2015 14:53:04 +0100, Ard Biesheuvel wrote:
> On 8 March 2015 at 12:31, Jean Delvare <jdelvare@suse.de> wrote:
> > On Sat, 07 Mar 2015 22:53:32 +0200, Ivan.khoronzhuk wrote:
> >> The specification doesn't oblige firmware to provide two entry points.
> >> An implementation may provide either the 32-bit entry point or the 64-bit
> >> entry point, or both. For compatibility with existing SMBIOS parsers, an
> >> implementation should provide the 32-bit entry point, but it's not required.
> >
> > I expect most implementations will do, as it's trivial to implement.
> 
> Not quite. First of all, some 64-bit ARM systems do not have any
> system RAM below 4 GB, so there is not way they can implement the
> 32-bit entry point.

I didn't know that, thanks for the notice. No big deal anyway, these
systems did not support SMBIOS before version 3.0 so there cannot be
any regression on these systems.

> Also, the 64-bit entry point does not limit the
> structure size or the entire table to 64 KB like the 32-bit one does,
> so it may be necessary to create a whole separate table with a subset
> of the contents of the real table to stay within limits for the 32-bit
> entry point.

I doubt this is an issue in practice. I have been around for quite some
time now and the largest table I've ever seen was 9043 byte long, which
is nowhere close to the limit.

> And the 32-bit entry point could well be 3.0 anyway, if
> it uses any of the new enum values for the data items that were
> undefined before 3.0.

This is true but irrelevant to the discussion.

Thanks,
-- 
Jean Delvare
SUSE L3 Support

  reply	other threads:[~2015-03-08 17:11 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-28 12:39 Ivan Khoronzhuk
2015-01-28 15:56 ` Ivan Khoronzhuk
2015-02-03 10:49   ` Matt Fleming
2015-02-03 14:47     ` Ivan Khoronzhuk
2015-02-03 14:58 ` Mark Salter
2015-02-03 15:48   ` Ivan Khoronzhuk
2015-02-26  8:50     ` [dmidecode] " Jean Delvare
2015-02-26  9:41       ` Jean Delvare
2015-03-04 12:28         ` Ivan.khoronzhuk
2015-03-05  7:46           ` Jean Delvare
2015-03-07 20:53             ` Ivan.khoronzhuk
2015-03-08 11:31               ` Jean Delvare
2015-03-08 13:53                 ` Ard Biesheuvel
2015-03-08 17:11                   ` Jean Delvare [this message]
2015-03-08 17:38                     ` Ard Biesheuvel
2015-03-08 18:21                       ` Jean Delvare
2015-03-04 12:28       ` Ivan.khoronzhuk

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150308181140.288dfa88@endymion.delvare \
    --to=jdelvare@suse.de \
    --cc=ard.biesheuvel@linaro.org \
    --cc=dmidecode-devel@nongnu.org \
    --cc=grant.likely@linaro.org \
    --cc=ivan.khoronzhuk@globallogic.com \
    --cc=ivan.khoronzhuk@linaro.org \
    --cc=leif.lindholm@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matt.fleming@intel.com \
    --cc=msalter@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®