All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mathias Krause <mathias.krause@secunet.com>
To: Stefan Weil <weil@mail.berlios.de>
Cc: qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH] CMOS file support
Date: Fri, 17 Sep 2010 13:16:21 +0200	[thread overview]
Message-ID: <4C934E05.3040900@secunet.com> (raw)
In-Reply-To: <4C9349CE.8080402@mail.berlios.de>

On 17.09.2010 12:58, Stefan Weil wrote:
> Am 17.09.2010 08:42, schrieb Mathias Krause:
>> Am 16.09.2010 18:49, Stefan Weil schrieb:
>>> Are there use cases where having a smaller CMOS size is better?
>>> For example, when I want to emulate a system with 128 byte CMOS?
>>> The size of CMOS could be derived from the size of the data
>>> or specified by an additional parameter.
>>
>> I cannot image cases where smaller sizes would be a benefit. The saved
>> disk space is negligible and you can always pad your CMOS configuration
>> uo to 256 bytes by filling the gap with zero bytes. If the system just
>> accesses the first 128 bytes the additional 128 zero bytes shouldn't
>> hurt.
>>
> 
> The benefit of smaller sizes is not saving RAM but precision
> of the emulation.

Well, as mentioned in my initial email the current emulation isn't
accurate as well. It is emulating 128 bytes of CMOS RAM instead of 64
bytes, the real MC146818 only has.

> A BIOS or an operating system might be designed to support
> CMOS  with 128 bytes or 256 bytes and try guessing the size
> by probing (many systems do something like this for RAM or
> other memory types).
> 
> Testing the support of the small CMOS will be impossible
> if the emulation always emulates 256 bytes. Filling the
> additional 128 bytes with zero bytes won't help because
> the system will access them successfully although writes
> should fail.

To access the upper 128 bytes of memory the operation system must be
aware that it needs to use different I/O ports to probe for those
addresses. The patch doesn't change the old behavior. Only the lower 128
bytes of memory will be accessible through the default RTC ports.
So if the operating system or BIOS in question already is aware of the
alternative method to access the upper 128 bytes, why should if fail then?

  reply	other threads:[~2010-09-17 11:15 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-16 13:58 [Qemu-devel] [PATCH] CMOS file support Mathias Krause
2010-09-16 16:49 ` Stefan Weil
2010-09-17  6:42   ` Mathias Krause
2010-09-17 10:44     ` Kevin Wolf
2010-09-17 11:28       ` Mathias Krause
2010-09-24 12:40         ` Markus Armbruster
2010-09-17 10:58     ` Stefan Weil
2010-09-17 11:16       ` Mathias Krause [this message]
2010-09-16 17:20 ` Anthony Liguori
2010-09-17  6:50   ` Mathias Krause
2010-09-17 13:27     ` Anthony Liguori
2010-09-22 19:43       ` Mathias Krause
2010-09-23 12:12         ` [Qemu-devel] " Paolo Bonzini
2010-09-24 12:47         ` [Qemu-devel] " Markus Armbruster
2010-09-26 20:44           ` Mathias Krause
2010-10-11 13:25             ` Markus Armbruster
2010-09-24 12:42   ` Markus Armbruster

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=4C934E05.3040900@secunet.com \
    --to=mathias.krause@secunet.com \
    --cc=qemu-devel@nongnu.org \
    --cc=weil@mail.berlios.de \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.