Blaisorblade wrote:
On Monday 07 November 2005 13:20, Bodo Stroesser wrote:

Blaisorblade wrote:

On Monday 31 October 2005 05:39, Jeff Dike wrote:

From: Bodo Stroesser <[EMAIL PROTECTED]>


Or at least so I think (I must still give a proper look afterwards, and
I'll post patches). Actually it seems that this is done on purpose, but I
don't agree too much on this. I will see.


From the beginning my new code for SKAS included the checks/buffering you
later inserted for TT and SKAS. So this patch is a second version adapted
to your changes. It shifts your improvements into TT path only (where I
didn't do any changes in my old patch), while it uses my own stuff for
SKAS. Thus the patch doesn't really revert your improvements, but restricts
it to TT. As in SKAS0 UML now holds its own LDT data, there is no need for
buffering in this case. So I think it makes sense to have separate code for
SKAS.

Yep, ok - I'm undecided about the new code for SKAS3, but it may make sense (i.e. no opinion).

Instead, I have another question: is there a proper reason for using the processor format for storing the info and translating it back to (struct user_desc)? I am planning to avoid this double translation because I don't like it. Any opinion?

In my opinion there is no reason to change the current implementation for 
SAKS3/0.
Note: if someone reads LDT via [sys_]modify_ldt(), he will receive the requested
data in "processor format", that is LDT-descriptors. He will receive a list of
descriptors starting at the first descriptor of the LDT, thus no entry number is
needed in the enties.
The only case that uses user_desc is when writing one desriptor via 
modify_ldt().
modify_ldt(WRITE) exactly writes one LDT-descriptor, so user_desc must contain
the number of the entry to write. Thus user_desc is bigger than LDT descriptor.
It also uses an other data layout resulting in double the size of 
LDT-descriptor.
So I think it doesn't make sense to store user_desc. We save memory storing the
resulting LDT-descriptors, which then are copied transparently on 
modify_ldt(READ).
Conversion between user_desc and LDT-entry is done on modify_ldt(WRITE) in SKAS0
only. No other conversions are done in UML.

        Bodo


-------------------------------------------------------
SF.Net email is sponsored by:
Tame your development challenges with Apache's Geronimo App Server. Download
it for free - -and be entered to win a 42" plasma tv or your very own
Sony(tm)PSP.  Click here to play: http://sourceforge.net/geronimo.php
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel

Reply via email to