linux-um archives
 help / color / mirror / Atom feed
From: BlaisorBlade <blaisorblade_spam@yahoo.it>
To: Jeff Dike <jdike@sonycom.com>
Cc: LaurentVivier@wanadoo.fr, user-mode-linux-devel@lists.sourceforge.net
Subject: [uml-devel] Re: sysemu
Date: Mon, 30 Aug 2004 20:48:45 +0200	[thread overview]
Message-ID: <200408301722.56559.blaisorblade_personal@yahoo.it> (raw)
In-Reply-To: <20040822040821.GA7183@tp.user-mode-linux.org>

Alle 06:08, domenica 22 agosto 2004, Jeff Dike ha scritto:
> Call me a lamer, but I only now got around to looking at the sysemu
> patch in detail.
>
> As it stands, it can't be merged into mainline because it adds two
> instructions to the system call fast path.  This is the sort of thing
> that make Linus and others go ballistic.

> Below, see a patch which tweaks things to eliminate both problems.
> Now, PT_SCEMU is set in conjunction with PT_TRACESYS, rather than
> one or the other, but not both, being set.

> This means that the PT_TRACESYS test in entry.S covers both cases, and
> the fast path is unchanged.

> The other change was to make syscall_trace return zero or non-zero,
> according as it should run the system call or immediately return.
> This adds a line to syscall_trace and eliminates syscall_emulate.

> The updated patch is 83 lines of diff rather than the original 128.

> I've run it with a small test program, apparently successfully, with
> good speed increases.  I'm having more trouble getting a tt mode UML
> running, but I think that's a problem with the UML rather than the
> host.

> Have a look and see if it looks reasonable.

Yes, it does. You should just rename *_SCEMU to *_SYSEMU (the guest patch uses 
already *_SYSEMU, which is nicer IMHO, so let's be coherent).

While you are looking at the SYSEMU patch, could you try to understand why 
when SYSEMU is active (at least in SKAS mode, never done anything in TT mode) 
strace does not work?

I looked at it with gdb a while ago but I wasn't able to make sense from what 
I saw.
-- 
Paolo Giarrusso, aka Blaisorblade
Linux registered user n. 292729


-------------------------------------------------------
This SF.Net email is sponsored by BEA Weblogic Workshop
FREE Java Enterprise J2EE developer tools!
Get your free copy of BEA WebLogic Workshop 8.1 today.
http://ads.osdn.com/?ad_id=5047&alloc_id=10808&op=click
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

           reply	other threads:[~2004-08-30 19:27 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <20040822040821.GA7183@tp.user-mode-linux.org>]

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=200408301722.56559.blaisorblade_personal@yahoo.it \
    --to=blaisorblade_spam@yahoo.it \
    --cc=LaurentVivier@wanadoo.fr \
    --cc=jdike@sonycom.com \
    --cc=user-mode-linux-devel@lists.sourceforge.net \
    /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