Re: CVS commit: src/lib/librumpuser

[Antti Kantee <pooka%NetBSD.org@localhost>]

On 17/06/14 11:23, Alexander Nasonov wrote:
> > If you don't have time to wait for discussion or coordination, do everything
> > in the privacy of the sljit component.
>
> Please teach me how to create a private component.

Use RUMPCOMP_USER_SRCS, several examples under src/sys/rump

> > I'm not saying that librumpuser must be POSIX.  I'm not sure where you're
> > getting that from.
>
> rumpuser(3):
>
> DESCRIPTION
>       The rumpuser hypercall interfaces allow a rump kernel to access host
>       resources.  A hypervisor implementation must implement the routines
>       described in this document to allow a rump kernel to run on the host.
>       The implementation included in NetBSD is for POSIX hosts.
>                                                    ^^^^^
>                                                    ^^^^^

ic.  I'd change that to POSIX-like, but currently I cannot access nbcvs.

> and it indeed broke buildrump.sh builds on Linux because sysarch
> stuff isn't available.
>
> There is no way I can make this interface POSIX-compatible because
> POSIX doesn't specify icache sync as far as I know.

That's one more indication that "sync icache" is the wrong level of 
problem to represent at the interface level.  If it were a high-level, 
holistic interface, both the caller and callee would what needs to be 
done, and the caller could perhaps implement the same with a alternative 
method.  A low-level interface like "sync icache for this memory range" 
leaves no room for interpretation, even if it will ever be used for only 
one purpose.