On Fri, Mar 08, 2002 at 07:14:46PM -0500, Roland McGrath wrote:
> The io_bitmap_* functions can be static (and get inlined), no?

Yes, done.

> You left machine_task_zone.

Uh, removed.

> Btw, you could use a zone for the iopb's instead of
> kalloc, since they are all the same size.  It's not much difference, but
> maybe a wee bit faster.

Do you think the performance increase is worth the memory overhead of a
zone, esp if you preallocate iopb's (each being 8192 bytes huge)?

> With the alias decl, you should not need the
> #define for base_tss any more.  It's clearer to just use ktss in the function.

Done.

New patch, for now without a zone for iopb's, at
http://www.debian.org/~brinkmd/iopb-2002-03-11.patch

Thanks,
Marcus

-- 
`Rhubarb is no Egyptian god.' Debian http://www.debian.org [EMAIL PROTECTED]
Marcus Brinkmann              GNU    http://www.gnu.org    [EMAIL PROTECTED]
[EMAIL PROTECTED]
http://www.marcus-brinkmann.de

_______________________________________________
Bug-hurd mailing list
[EMAIL PROTECTED]
http://mail.gnu.org/mailman/listinfo/bug-hurd

Reply via email to