Commits · f26c83d65ccd21fa18d426a7df6d9554e58a0236 · Verlässliche Systemsoftware / projects / osv

Oct 15, 2013

runjava: move runjava.jar code to a separate directory · f26c83d6

Tomasz Grabiec authored 11 years ago


Related changes previously made in java/build.xml
were reverted and moved to java/runjava/build.xml

Signed-off-by: Tomasz Grabiec <tgrabiec@cloudius-systems.com>

f26c83d6

RunJava: system class loader should contain application resources · c0a8e41a

Tomasz Grabiec authored 11 years ago


The system classloader should have classes which were
specified in the command line using -cp/-classpath/-jar.
Before this change only runjava.jar was included.

System class loader is queried explicitly by
java.util.logging framework to lookup classes
configred in logging properties file.

To solve this problem a custom system classloader
is set when starting JVM. It delegates to the application
classloader created when java program was started.

It also handles the case when a new java application
is started within the same JVM. Such application
should have its own view of system classes. However
we are able to install only one system classloader
per JVm. To solve this we start each java application
in a separate thread and use InheritableThreadLocal
to hold the application class loader which should be
used as system class loader view for the thread family.

Signed-off-by: Tomasz Grabiec <tgrabiec@cloudius-systems.com>

c0a8e41a

runjava.jar: build jar using ant · d93d6cfd

Tomasz Grabiec authored 11 years ago


Building with ant requires no changes in the build scripts
when new classes are added. It is a standrd tool to build
java programs.

Signed-off-by: Tomasz Grabiec <tgrabiec@cloudius-systems.com>

d93d6cfd

Oct 14, 2013

Add osv::run function · e15b3322

Nadav Har'El authored 11 years ago


Add a convenience function, osv::run(), to run an ELF executable.

The patch includes extensive Doxygen documentation for the new function,
also detailing the how osv::run() differs from the familiar Linux
functions like exec(2).

Signed-off-by: Nadav Har'El <nyh@cloudius-systems.com>

e15b3322

mmu: Fix mincore() for non-resident mmaps · 7d532448

Pekka Enberg authored 11 years ago

Fix mincore() to follow Linux ABI and return ENOMEM only if (part of
the) memory range is unmapped even if it's not resident. This is needed
for mincore() to work with demand paged mmaps.

Signed-off-by: Pekka Enberg <penberg@cloudius-systems.com>

7d532448

libzpool: Fix ->rw_count type in struct krwlock · bdc4a282

Pekka Enberg authored 11 years ago


Fixes the following compilation error:

  ../../bsd/cddl/contrib/opensolaris/lib/libzpool/common/kernel.c: In function ‘rw_enter’:
  ../../bsd/cddl/contrib/opensolaris/lib/libzpool/common/kernel.c:205:3: error: pointer targets in passing argument 1 of ‘atomic_add_int’ differ in signedness [-Werror=pointer-sign]
     atomic_add_int(&rwlp->rw_count, 1);
     ^
  In file included from ../../bsd/sys/cddl/compat/opensolaris/sys/atomic.h:33:0,
                   from ../../bsd/cddl/contrib/opensolaris/lib/libzpool/common/sys/zfs_context.h:66,
                   from ../../bsd/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/sys/spa.h:31,
                   from ../../bsd/cddl/contrib/opensolaris/lib/libzpool/common/kernel.c:32:
  ../../bsd/machine/atomic.h:285:1: note: expected ‘volatile u_int *’ but argument is of type ‘int *’
   ATOMIC_ASM(add,      int,   "addl %1,%0",  "ir",  v);
   ^

Signed-off-by: Pekka Enberg <penberg@cloudius-systems.com>

bdc4a282

libzpool: Fix gettimeofday() include · 4fd23254

Pekka Enberg authored 11 years ago


Fixes the following build breakage:

    CC bsd/cddl/contrib/opensolaris/lib/libzpool/common/kernel.o
  ../../bsd/cddl/contrib/opensolaris/lib/libzpool/common/kernel.c: In function ‘cv_timedwait’:
  ../../bsd/cddl/contrib/opensolaris/lib/libzpool/common/kernel.c:322:2: error: implicit declaration of function ‘gettimeofday’ [-Werror=implicit-function-declaration]
    if (gettimeofday(&tv, NULL) != 0)
  ^
  cc1: all warnings being treated as errors

Signed-off-by: Pekka Enberg <penberg@cloudius-systems.com>

4fd23254

Add forgotten dlfcn.cc · d2397b7e

Nadav Har'El authored 11 years ago


I forgot to commit the change to dlfcn.cc in the previous patch. Sorry...

Signed-off-by: Nadav Har'El <nyh@cloudius-systems.com>

d2397b7e

Rework elf::program's API · ba3e3efa

Nadav Har'El authored 11 years ago


This is patch v2, incorporating most of the comments from the previous round.

Solves issue #47:

elf::program's API - add_object() and remove_object() was problematic in
two respects:

1. It did not do reference-counting on the loaded object, so if add_object()
   was done 5 times, a single remove_object() would unmap the object and
  its 4 other users will crash.

2. It is un-C++-like.

This patch replaces these two functions by a single function get_library():

  std::shared_ptr<elf::object>
   get_library(std::string lib, std::vector<std::string> extra_path = {});

get_library() returns a shared_ptr, which is reference counting and does
proper C++-style RAII. For example in the code:

	auto lib = elf::get_program()->get_library(path);
	auto main = lib->lookup<int (int, char**)>("main");
	int rc = main(argc, argv);

once lib out of scope, the reference count of the elf::object automatically
goes down, and if nobody else holds another shared-pointer to the same
library, the object is destroyed (causing the shared library to be unloaded).

This patch also documents, with Doxygen, all the functions it touches.

IMPORTANT NOTE: This reference count is completely unrelated to the issue
of concurrent use of dlopen()/dlclose()/dl_iterate_phdr(), which is still
buggy and we need to fix (I have a patch for that, but it's too long to fit
in the margin).

Signed-off-by: Nadav Har'El <nyh@cloudius-systems.com>

ba3e3efa

Merge branch 'mkfs' · e4a948eb