qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] Thanks for QEMU!
@ 2006-11-07 15:09 John Stracke
  2006-11-07 17:42 ` Hetz Ben Hamo
  2006-11-08  9:30 ` wangji
  0 siblings, 2 replies; 5+ messages in thread
From: John Stracke @ 2006-11-07 15:09 UTC (permalink / raw)
  To: qemu-devel

Just wanted to send a thank-you for developing QEMU.  I'm writing an OS
for my master's thesis, and having an open-source emulation of the CPU
has been a *huge* help.  At least three times now, there's been some
subtlety that I couldn't figure out from the CPU docs, so I adding some
printf()s to QEMU to figure out what was really going on.

-- 
/===============================================================\
|John Stracke        | http://www.thibault.org |HTML OK         |
|Francois Thibault   |==========================================|
|East Kingdom        |"The struggle is always worthwhile, if the|
|francis@thibault.org|end be worthwhile and the means honorable;|
|                    |foreknowledge of defeat is not sufficient |
|                    |reason to withdraw from the contest." --  |
|                    |Adron e'Kieron, by Steven Brust           |
\===============================================================/

^ permalink raw reply	[flat|nested] 5+ messages in thread
[parent not found: <200611081431.28389.hoan@wanadoo.fr>]
* Re: [Qemu-devel] Thanks for QEMU!
@ 2006-11-08 14:44 John Stracke
  0 siblings, 0 replies; 5+ messages in thread
From: John Stracke @ 2006-11-08 14:44 UTC (permalink / raw)
  To: qemu-devel

Hetz Ben Hamo wrote:


> Perhaps you could submit your printf's so other people can learn from
> them please?

Well, most of them I no longer have; they slowed things down, so I
deleted them once I had the problems fixed.  Besides, they were probably
too idiosyncratic to help other people much.

However, there is one change that I'd recommend adding into the QEMU
distribution: in target-i386/helper2.c, in cpu_x86_handle_mmu_fault(),
right before the line starting "pde_addr = ((env->cr[3] & ~0xfff)", I
added a sanity check:

if (env->cr[3] & 0xfff) {
  fprintf(logfile,"Warning: page directory is not page-aligned.\n");
}

This gives a warning if the OS is using an invalid address for the page
directory.  You might want to put it inside #ifdef DEBUG_MMU, or
something.

-- 
/============================================================\
|John Stracke        | http://www.thibault.org |HTML OK      |
|Francois Thibault   |=======================================|
|East Kingdom        |Rope is rope, and string is string, and|
|francis@thibault.org|never the twine shall meet.            |
\============================================================/

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2006-11-08 14:44 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-11-07 15:09 [Qemu-devel] Thanks for QEMU! John Stracke
2006-11-07 17:42 ` Hetz Ben Hamo
2006-11-08  9:30 ` wangji
     [not found] <200611081431.28389.hoan@wanadoo.fr>
2006-11-08 14:34 ` John Stracke
  -- strict thread matches above, loose matches on Subject: below --
2006-11-08 14:44 John Stracke

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).