]> Zhao Yanbai Git Server - minix.git/commit
MIB/libsys: support for remote MIB (RMIB) subtrees 25/3325/1
authorDavid van Moolenbroek <david@minix3.org>
Sat, 23 Apr 2016 18:07:39 +0000 (18:07 +0000)
committerDavid van Moolenbroek <david@minix3.org>
Sat, 18 Jun 2016 12:46:59 +0000 (12:46 +0000)
commit6f3e0bcd3d3c17c74429db9d63eb808350d9a344
tree81fe4d59a4d5c3501d0d4b6848b84bd3216a63f8
parent53d2fa057ee88a709bcc793d3c777acad2c2a2ee
MIB/libsys: support for remote MIB (RMIB) subtrees

Most of the nodes in the general sysctl tree will be managed directly
by the MIB service, which obtains the necessary information as needed.
However, in certain cases, it makes more sense to let another service
manage a part of the sysctl tree itself, in order to avoid replicating
part of that other service in the MIB service.  This patch adds the
basic support for such delegation: remote services may now register
their own subtrees within the full sysctl tree with the MIB service,
which will then forward any sysctl(2) requests on such subtrees to the
remote services.

The system works much like mounting a file system, but in addition to
support for shadowing an existing node, the MIB service also supports
creating temporary mount point nodes.  Each have their own use cases.
A remote "kern.ipc" would use the former, because even when such a
subtree were not mounted, userland would still expect some of its
children to exist and return default values.  A remote "net.inet"
would use the latter, as there is no reason to precreate nodes for all
possible supported networking protocols in the MIB "net" subtree.

A standard remote MIB (RMIB) implementation is provided for services
that wish to make use of this functionality.  It is essentially a
simplified and somewhat more lightweight version of the MIB service's
internals, and works more or less the same from a programmer's point
of view.  The most important difference is the "rmib" prefix instead
of the "mib" prefix.  Documentation will hopefully follow later.

Overall, the RMIB functionality should not be used lightly, for
several reasons.  First, despite being more lightweight than the MIB
service, the RMIB module still adds substantially to the code
footprint of the containing service.  Second, the RMIB protocol not
only adds extra IPC for sysctl(2), but has also not been optimized for
performance in other ways.  Third, and most importantly, the RMIB
implementation also several limitations.  The main limitation is that
remote MIB subtrees must be fully static.  Not only may the user not
create or destroy nodes, the service itself may not either, as this
would clash with the simplified remote node versioning system and
the cached subtree root node child counts.  Other limitations exist,
such as the fact that the root of a remote subtree may only be a
node-type node, and a stricter limit on the highest node identifier
of any child in this subtree root (currently 4095).

The current implementation was born out of necessity, and therefore
it leaves several improvements to future work.  Most importantly,
support for exit and crash notification is missing, primarily in the
MIB service.  This means that remote subtrees may not be cleaned up
immediately, but instead only when the MIB service attempts to talk
to the dead remote service.  In addition, if the MIB service itself
crashes, re-registration of remote subtrees is currently left up to
the individual RMIB users.  Finally, the MIB service uses synchronous
(sendrec-based) calls to the remote services, which while convenient
may cause cascading service hangs.  The underlying protocol is ready
for conversion to an asynchronous implementation already, though.

A new test set, testrmib.sh, tests the basic RMIB functionality.  To
this end it uses a test service, rmibtest, and also reuses part of
the existing test87 MIB service test.

Change-Id: I3378fe04f2e090ab231705bde7e13d6289a9183e
25 files changed:
distrib/sets/lists/minix-comp/mi
distrib/sets/lists/minix-tests/mi
etc/mtree/NetBSD.dist.base
minix/include/minix/Makefile
minix/include/minix/com.h
minix/include/minix/drivers.h
minix/include/minix/ipc.h
minix/include/minix/rmib.h [new file with mode: 0644]
minix/include/minix/sysctl.h
minix/kernel/system/do_safecopy.c
minix/lib/libsys/Makefile
minix/lib/libsys/rmib.c [new file with mode: 0644]
minix/servers/mib/Makefile
minix/servers/mib/main.c
minix/servers/mib/mib.h
minix/servers/mib/minix.c
minix/servers/mib/remote.c [new file with mode: 0644]
minix/servers/mib/tree.c
minix/tests/Makefile
minix/tests/rmibtest/Makefile [new file with mode: 0644]
minix/tests/rmibtest/rmibtest.c [new file with mode: 0644]
minix/tests/rmibtest/rmibtest.conf [new file with mode: 0644]
minix/tests/run
minix/tests/test87.c
minix/tests/testrmib.sh [new file with mode: 0755]