From: ebiederm@xmission.com (Eric W. Biederman)
To: root@chaos.analogic.com
Cc: David Christensen <David_Christensen@Phoenix.com>,
Holger Lubitz <h.lubitz@internet-factory.de>,
linux-kernel@vger.kernel.org
Subject: Re: Encrypted Swap
Date: 18 Aug 2001 04:34:28 -0600 [thread overview]
Message-ID: <m1n14xodvv.fsf@frodo.biederman.org> (raw)
In-Reply-To: <Pine.LNX.3.95.1010817151211.4584A-100000@chaos.analogic.com>
In-Reply-To: <Pine.LNX.3.95.1010817151211.4584A-100000@chaos.analogic.com>
"Richard B. Johnson" <root@chaos.analogic.com> writes:
> On 17 Aug 2001, Eric W. Biederman wrote:
>
> > "Richard B. Johnson" <root@chaos.analogic.com> writes:
> >
> > >
> > > I just posted working SDRAM controller initialization code. The SDRAM
> > > controller must be initialized in a specific step-by-step manner or
> > > else you don't even get to "restoring the memory controller settings".
> >
> > Comments frequently don't match the code. And how the SDRAM controller
> > must be initialized totally depends on the chipset and vendor. SDRAM
> > itself must be initialized in a specific matter.
> >
>
> That's what I said, and the comments match the code.
Apologies, I saw it was masm formatted code and assumed it was a
snippet taken from one of the pirated award BIOS's.
I wasn't saying that I though the comments didn't match the code but
rather that I simply don't trust comments as a general rule.
Now actually reading through the code is when I attach your
argument. Precharge does not clear memory. It is part of the refresh
process and gets executed as a normal part of memory operation, besides
being part of the SDRAM initialization. Precharge does not clear the
memory.
And then you don't have the code to clear the memory, and only a
comment about clearing the low 1 megabyte of memory. That is a
comment not matchine the code snippet at least.
> > But for the case of a warm reset there is not need to reset the SDRAM
> > controller. Memory really only needs to be cleared in the case when
> > some form of error checking is being used.
> >
>
> Memory needs to be written (with anything). It must be written before
> it's used so that there are no "almost" logic levels within the cells.
>
> The parasitic currents from having some cells "just barely on" will
> totally screw up data retention in other cells. If anybody ever
> writes memory initialization software that doesn't perform this
> extremely important step, they just don't know what the hell they are
> doing and will contribute towards a poor reputation for software
> engineering.
I can see how this could physically be, but I don't believe this
is a real situtation. Memory is continuously losing it's contents,
and the refresh cycles are upping the value. So if refresh is
operating correctly I can't see how this could occur for more than one
refresh cycle.
Further I have reviewed the SDRAM data sheets from two different
manufaturers and they don't mention anything about initializing the
contents of SDRAM.
Further if the question is what an attacker can do. Even if it isn't
a part of normal operation having a modified BIOS that doesn't clear
the memory is perfectly reasonable, to get your secure information.
> > Personally I think writing such code carries more credibility then
> > simply posting it anyway....
> >
>
> Well I wrote the code. And I have written SDRAM initialization code
> for many chip-sets.
I feel like taking a cheap shot right now and saying you are currently
contributing to a poor reputation for software engineer. But in this
case the posted code snippet is correct, so I see no failure in the
code. Just in understanding an unimportant bit of trivia of the
low-level hardware operation. That really doesn't matter unless you
manufacture a design memory controller.
I can't currenly claim many chip-sets under my belt currently. But I
certainly understand how this works as well.
Eric
next prev parent reply other threads:[~2001-08-18 10:41 UTC|newest]
Thread overview: 104+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-08-17 17:10 Encrypted Swap David Christensen
2001-08-17 17:21 ` Richard B. Johnson
2001-08-17 18:41 ` Eric W. Biederman
2001-08-17 19:05 ` Dan Hollis
2001-08-18 9:52 ` Eric W. Biederman
2001-08-18 10:24 ` Nicholas Knight
2001-08-18 12:32 ` Eric W. Biederman
2001-08-17 19:20 ` Richard B. Johnson
2001-08-18 10:34 ` Eric W. Biederman [this message]
[not found] <fa.kmbqblv.v3uvig@ifi.uio.no>
2001-08-18 14:53 ` Ted Unangst
2001-08-18 15:17 ` Mr. James W. Laferriere
2001-08-20 11:03 ` Helge Hafting
-- strict thread matches above, loose matches on Subject: below --
2001-08-07 21:40 encrypted swap David Spreen
2001-08-07 18:53 Torrey Hoffman
2001-08-07 19:15 ` Thomas Pornin
2001-08-07 19:23 ` Dan Podeanu
2001-08-07 19:48 ` Andreas Dilger
2001-08-07 20:04 ` Marty Poulin
2001-08-07 21:06 ` David Wagner
2001-08-07 21:56 ` D. Stimits
2001-08-07 21:44 ` Pavel Machek
2001-08-07 19:48 ` Justin Guyett
2001-08-07 20:05 ` Alan Cox
2001-08-07 20:17 ` Bill Rugolsky Jr.
2001-08-07 17:30 Encrypted Swap David Maynor
2001-08-07 17:27 ` Rik van Riel
2001-08-07 15:28 encrypted swap David Maynor
2001-08-07 15:51 ` Florian Weimer
2001-08-07 15:06 David Maynor
2001-08-07 15:11 ` Florian Weimer
2001-08-07 15:43 ` Joel Jaeggli
2001-08-07 15:30 ` Garett Spencley
2001-08-07 16:21 ` David Spreen
2001-08-08 8:11 ` Helge Hafting
2001-08-07 14:37 David Maynor
2001-08-07 14:48 ` Billy Harvey
2001-08-07 16:03 ` Chris Wedgwood
[not found] <no.id>
2001-08-07 14:17 ` Encrypted Swap Alan Cox
2001-08-07 15:16 ` Crutcher Dunnavant
2001-08-07 16:01 ` Chris Wedgwood
2001-08-07 2:28 David Spreen
2001-08-07 3:56 ` Justin Guyett
2001-08-07 4:01 ` Chris Wedgwood
2001-08-07 4:12 ` Steve VanDevender
2001-08-07 4:23 ` John Polyakov
2001-08-07 4:36 ` Chris Wedgwood
2001-08-07 5:12 ` Garett Spencley
2001-08-07 5:55 ` Ryan Mack
2001-08-07 6:27 ` John Polyakov
2001-08-06 23:28 ` Rob Landley
2001-08-07 10:10 ` Christopher E. Brown
2001-08-07 14:05 ` Joel Jaeggli
2001-08-07 6:41 ` Crutcher Dunnavant
2001-08-07 6:57 ` Evgeny Polyakov
2001-08-07 6:45 ` Ryan Mack
2001-08-07 7:08 ` Evgeny Polyakov
2001-08-07 7:23 ` Sean Hunter
2001-08-07 8:39 ` Ben Ford
2001-08-07 12:28 ` Kevin Krieser
2001-08-07 12:39 ` Richard B. Johnson
2001-08-07 14:21 ` Ignacio Vazquez-Abrams
2001-08-07 7:26 ` Ryan Mack
2001-08-07 7:34 ` Jeffrey Considine
2001-08-07 7:49 ` Crutcher Dunnavant
2001-08-07 9:01 ` Peter Wächtler
2001-08-07 12:37 ` Michael Bacarella
2001-08-17 14:50 ` Holger Lubitz
2001-08-17 15:39 ` Richard B. Johnson
2001-08-17 15:57 ` Holger Lubitz
2001-08-17 16:34 ` Gerhard Mack
2001-08-17 16:50 ` Richard B. Johnson
2001-08-17 17:06 ` Adrian Cox
2001-08-17 17:16 ` Richard B. Johnson
2001-08-17 17:22 ` Jacob Alifrangis
2001-08-17 17:36 ` Adrian Cox
2001-08-17 18:51 ` Nicholas Knight
2001-08-17 19:30 ` Richard B. Johnson
2001-08-18 8:51 ` Adrian Cox
2001-08-18 11:02 ` Eric W. Biederman
2001-08-19 8:51 ` Adrian Cox
2001-08-20 1:27 ` Richard B. Johnson
2001-08-20 11:08 ` Helge Hafting
2001-08-20 11:50 ` Ian Stirling
2001-08-21 13:55 ` Andreas Bombe
2001-08-17 20:00 ` Andreas Dilger
2001-08-07 20:09 ` Maciej Zenczykowski
2001-08-07 7:34 ` Steve VanDevender
2001-08-07 7:55 ` Crutcher Dunnavant
2001-08-07 15:17 ` Garett Spencley
2001-08-07 7:49 ` Helge Hafting
2001-08-07 7:58 ` Crutcher Dunnavant
2001-08-07 9:23 ` Helge Hafting
2001-08-07 13:29 ` Wichert Akkerman
2001-08-07 15:56 ` Chris Wedgwood
2001-08-07 16:54 ` Alan Cox
2001-08-07 17:10 ` Chris Wedgwood
2001-08-07 9:52 ` Brian May
2001-08-07 14:48 ` Joel Jaeggli
2001-08-07 15:59 ` Chris Wedgwood
2001-08-07 16:18 ` Joel Jaeggli
2001-08-07 16:24 ` Florian Weimer
2001-08-08 2:13 ` Dr. Kelsey Hudson
2001-08-07 20:30 ` Ian Stirling
2001-08-07 10:33 ` Andrea Arcangeli
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=m1n14xodvv.fsf@frodo.biederman.org \
--to=ebiederm@xmission.com \
--cc=David_Christensen@Phoenix.com \
--cc=h.lubitz@internet-factory.de \
--cc=linux-kernel@vger.kernel.org \
--cc=root@chaos.analogic.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