public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Pavel Roskin <proski@gnu.org>
To: Roland Kuhn <rkuhn@e18.physik.tu-muenchen.de>
Cc: Adrian Bunk <bunk@stusta.de>, Alan Cox <alan@lxorguk.ukuu.org.uk>,
	Andrew Morton <akpm@osdl.org>,
	linux-kernel@vger.kernel.org
Subject: Re: incorrect taint of ndiswrapper
Date: Fri, 27 Oct 2006 11:53:30 -0400	[thread overview]
Message-ID: <1161964410.2469.20.camel@dv> (raw)
In-Reply-To: <B5254A8D-0E60-4C51-AF71-7F76F3B8B917@e18.physik.tu-muenchen.de>

Hi Roland,

On Fri, 2006-10-27 at 14:52 +0200, Roland Kuhn wrote:

> Maybe everyone would be more happy if this "completely different API"  
> would live at lower priviledge level, e.g. ring 1, so it could not  
> screw up kernel internals? Is this technically possible? Maybe it's  
> the same thing, but another way could be to run NDIS stuff inside a  
> xen-like virtual environment... Has anyone tried yet?

I think it would be better to move this discussion to ndiswrapper
<ndiswrapper-general@lists.sourceforge.net>

I'm not familiar with the fine details of ndiswrapper implementation and
neither am I good at understanding memory management in Linux, but I
suspect it's not worth the trouble.

I believe there is no "ring 1" on x86_64 (unless it's in i386
compatibility mode).  So it would work on i386 only.  Maybe x86_64 could
use its "ring 3" equivalent, i.e. standard userspace permissions, but I
don't think it would be what you want.

Even on i386, I don't see an easy way to allocate executable memory with
ring 1 permissions.  See include/asm-i386/pgtable.h.

I suspect that there is no support for running kernel code at anything
but "ring 0".  What do you think are the chances that support for
low-privileged kernel code will be added to the kernel just for
ndiswrapper?  I think the chances are pretty slim.

In the case of the PCI driver, some critical operations would have to be
passed to the NDIS driver, such as IRQ and DMA processing.  It would be
better to make sure that the code has the necessary priority to do it
fast and correctly.

In the case of the USB driver, it may be better to go all the way to the
standard userspace.  This would require a protocol to pass network API
to the userspace, including wireless extensions.  I believe the work is
underway.

-- 
Regards,
Pavel Roskin


  reply	other threads:[~2006-10-27 15:53 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-25 20:11 incorrect taint of ndiswrapper Pavel Roskin
2006-10-25 20:30 ` Alan Cox
2006-10-25 20:40   ` Kyle Moffett
2006-10-25 21:04     ` Alan Cox
2006-10-25 21:06   ` Pavel Roskin
2006-10-25 21:33     ` David Weinehall
2006-10-25 22:02       ` Pavel Roskin
2006-10-25 22:54         ` Alan Cox
2006-10-25 22:58       ` Alan Cox
2006-10-26  3:23         ` David Weinehall
2006-10-26 13:13           ` Thierry Vignaud
2006-10-26 13:21             ` Gianluca Alberici
2006-10-26  3:59   ` Andrew Morton
2006-10-26  9:03     ` Gianluca Alberici
2006-10-26 10:39     ` Alan Cox
2006-10-26 12:21       ` Giacomo A. Catenazzi
2006-10-26 12:59         ` Gianluca Alberici
2006-10-26 14:41       ` Al Viro
2006-10-26 14:55         ` Alan Cox
2006-10-26 16:00       ` Stephen Hemminger
2006-10-26 16:26         ` Gianluca Alberici
2006-10-27 14:24           ` Arjan van de Ven
2006-10-27 15:14             ` Stephen Hemminger
2006-10-26 19:19       ` Pavel Roskin
2006-10-26 21:46         ` Adrian Bunk
2006-10-26 22:29           ` Pavel Roskin
2006-10-26 23:00             ` Adrian Bunk
2006-10-26 23:36               ` Sven-Haegar Koch
2006-10-27  0:57                 ` Adrian Bunk
2006-10-26 23:47               ` Pavel Roskin
2006-10-27 12:52             ` Roland Kuhn
2006-10-27 15:53               ` Pavel Roskin [this message]
2006-10-26 17:26     ` [PATCH ??] " Randy Dunlap
2006-10-27 14:23       ` Arjan van de Ven
2006-10-27 15:27         ` Randy Dunlap
2006-10-27 18:26           ` Andrew Morton
2006-10-27 22:56             ` Florin Malita
2006-10-27 22:56               ` Randy Dunlap
2006-10-27 23:05               ` Alan Cox
2006-10-27 23:02                 ` Randy Dunlap
2006-10-27 23:12                 ` Florin Malita
2006-10-27 23:23                 ` Oleg Verych
2006-10-29 11:27             ` Gianluca Alberici
2006-10-27 21:32         ` Florin Malita
2006-10-27  4:32     ` Florin Malita
  -- strict thread matches above, loose matches on Subject: below --
2006-10-23  5:41 Giridhar Pemmasani
2006-10-23  5:53 ` Gianluca Alberici
2006-10-23  6:25 ` Chase Venters
2006-10-23  6:41   ` Giridhar Pemmasani
2006-10-23  6:48     ` Gianluca Alberici
2006-10-23  7:12     ` Chase Venters
2006-10-23 11:07       ` Giridhar Pemmasani
2006-10-23  9:10     ` Gianluca Alberici
2006-10-23  9:39     ` Michal Schmidt
2006-10-23  8:24 ` Bernd Petrovitsch
2006-10-23 10:41 ` Alan Cox
2006-10-23 11:35   ` Giridhar Pemmasani
2006-10-23 13:00     ` Alan Cox
2006-10-24  2:43       ` Giridhar Pemmasani
2006-10-24  3:11         ` Randy Dunlap
2006-10-24 12:12           ` Pekka Enberg
2006-10-24 12:22             ` Alan Cox
2006-10-24 14:07         ` Alan Cox
2006-10-23 18:36   ` Zan Lynx
2006-10-24 11:59     ` Jan Engelhardt

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=1161964410.2469.20.camel@dv \
    --to=proski@gnu.org \
    --cc=akpm@osdl.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=bunk@stusta.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rkuhn@e18.physik.tu-muenchen.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox