* pci-arbiter/Makefile Add machdev lib and simplify * pci-arbiter/main.c (pci_device_{open,close,shutdown}): New methods (netfs_server_func): Thread the demuxer loop (pcifs_startup): Custom startup method (main): Use machdev for server and detach worker threads * pci-arbiter/options.{c,h} Add disk-server-task and priv ports * pci-arbiter/startup-ops.c Delete unneeded file --- pci-arbiter/Makefile | 7 +- pci-arbiter/main.c | 172 ++++++++++++++++++++++++++++++++++++-- pci-arbiter/options.c | 19 +++++ pci-arbiter/options.h | 8 ++ pci-arbiter/pcifs.h | 3 + pci-arbiter/startup-ops.c | 44 ---------- 6 files changed, 197 insertions(+), 56 deletions(-) delete mode 100644 pci-arbiter/startup-ops.c
diff --git a/pci-arbiter/Makefile b/pci-arbiter/Makefile index 1d0309dc..b32bc579 100644 --- a/pci-arbiter/Makefile +++ b/pci-arbiter/Makefile @@ -22,11 +22,10 @@ PORTDIR = $(srcdir)/port SRCS = main.c pci-ops.c netfs_impl.c \ pcifs.c ncache.c options.c func_files.c startup.c \ - startup-ops.c -MIGSRCS = pciServer.c startup_notifyServer.c -OBJS = $(patsubst %.S,%.o,$(patsubst %.c,%.o, $(SRCS) $(MIGSRCS))) + pciServer.c startup_notifyServer.c +OBJS = $(SRCS:.c=.o) $(MIGSTUBS) -HURDLIBS= fshelp ports shouldbeinlibc netfs iohelp ihash +HURDLIBS= fshelp ports shouldbeinlibc netfs iohelp ihash trivfs machdev LDLIBS = -lpthread $(libpciaccess_LIBS) target = pci-arbiter diff --git a/pci-arbiter/main.c b/pci-arbiter/main.c index 181bdee2..ddea5aa7 100644 --- a/pci-arbiter/main.c +++ b/pci-arbiter/main.c @@ -24,7 +24,12 @@ #include <fcntl.h> #include <version.h> #include <argp.h> +#include <unistd.h> #include <hurd/netfs.h> +#include <hurd/ports.h> +#include <hurd/fsys.h> +#include <device/device.h> +#include <sys/mman.h> #include <pci_S.h> #include <startup_notify_S.h> @@ -34,7 +39,9 @@ #include "libnetfs/fsys_S.h" #include "libports/interrupt_S.h" #include "libnetfs/ifsock_S.h" +#include "libmachdev/machdev.h" #include <pciaccess.h> +#include <pthread.h> #include "pcifs.h" #include "startup.h" @@ -46,6 +53,87 @@ int netfs_maxsymlinks = 0; char *netfs_server_name = "pci-arbiter"; char *netfs_server_version = HURD_VERSION; +static mach_port_t pci_control_port; + + +static io_return_t +pci_device_open (mach_port_t reply_port, mach_msg_type_name_t reply_port_type, + dev_mode_t mode, char *name, device_t * devp, + mach_msg_type_name_t * devicePoly) +{ + io_return_t err = D_SUCCESS; + mach_port_t dev_master; + + if (strncmp(name, "pci", 3)) + err = D_NO_SUCH_DEVICE; + + /* Fall back to opening kernel device master */ + if (err) + { + err = get_privileged_ports(NULL, &dev_master); + if (err) + return err; + if (dev_master == MACH_PORT_NULL) + return D_NO_SUCH_DEVICE; + err = device_open (dev_master, mode, name, devp); + if (err) + return err; + *devicePoly = MACH_MSG_TYPE_MOVE_SEND; + return D_SUCCESS; + } + + *devp = pci_control_port; + *devicePoly = MACH_MSG_TYPE_COPY_SEND; + return D_SUCCESS; +} + +static io_return_t +pci_device_close (void *d) +{ + ports_port_deref (&pci_control_port); + return 0; +} + +static void +pci_device_shutdown (mach_port_t dosync_handle) +{ + struct port_info *inpi = ports_lookup_port (netfs_port_bucket, dosync_handle, + pci_shutdown_notify_class); + + if (!inpi) + return; + + // Free all libpciaccess resources + pci_system_cleanup (); + + ports_port_deref (inpi); + + ports_destroy_right (&pci_control_port); + + netfs_shutdown (FSYS_GOAWAY_FORCE); +} + +static struct machdev_device_emulation_ops pci_arbiter_emulation_ops = { + NULL, + NULL, + NULL, + NULL, + pci_device_open, + pci_device_close, + NULL, + NULL, + NULL, + NULL, + NULL, + NULL, + NULL, + NULL, + NULL, + NULL, + NULL, + pci_device_shutdown, +}; + int netfs_demuxer (mach_msg_header_t * inp, mach_msg_header_t * outp) { @@ -67,24 +155,82 @@ netfs_demuxer (mach_msg_header_t * inp, mach_msg_header_t * outp) return FALSE; } +static void * +netfs_server_func (void *arg) +{ + error_t err; + + do + { + ports_manage_port_operations_multithread (netfs_port_bucket, + netfs_demuxer, + 1000 * 60 * 2, /* two minutes thread */ + 1000 * 60 * 10,/* ten minutes server */ + 0); + err = netfs_shutdown (0); + } + while (err); + return NULL; +} + + +static mach_port_t +pcifs_startup(mach_port_t bootstrap, int flags) +{ + error_t err; + mach_port_t realnode; + struct port_info *newpi; + + err = ports_create_port (netfs_control_class, netfs_port_bucket, + sizeof (struct port_info), &newpi); + if (!err) + { + pci_control_port = ports_get_send_right (newpi); + err = fsys_startup (bootstrap, flags, pci_control_port, MACH_MSG_TYPE_COPY_SEND, + &realnode); + assert_perror_backtrace (err); + } + if (err) + error (11, err, "Translator startup failure: fsys_startup"); + + return realnode; +} + int main (int argc, char **argv) { error_t err; mach_port_t bootstrap; + mach_port_t disk_server_task; + pthread_t t, nt; /* Parse options */ alloc_file_system (&fs); argp_parse (netfs_runtime_argp, argc, argv, 0, 0, 0); + disk_server_task = fs->params.disk_server_task; - task_get_bootstrap_port (mach_task_self (), &bootstrap); - if (bootstrap == MACH_PORT_NULL) - error (1, 0, "must be started as a translator"); - + if (disk_server_task != MACH_PORT_NULL) + { + machdev_register (&pci_arbiter_emulation_ops); + machdev_device_init (); + machdev_trivfs_init (disk_server_task, "pci", "/servers/bus/pci", &bootstrap); + err = pthread_create (&t, NULL, machdev_server, NULL); + if (err) + error (1, err, "Creating machdev thread"); + pthread_detach (t); + } + else + { + task_get_bootstrap_port (mach_task_self (), &bootstrap); + if (bootstrap == MACH_PORT_NULL) + error (1, 0, "must be started as a translator"); + } /* Initialize netfs and start the translator. */ netfs_init (); err = maptime_map (0, 0, &pcifs_maptime); + if (err) + err = maptime_map (1, 0, &pcifs_maptime); if (err) error (1, err, "mapping time"); @@ -93,8 +239,12 @@ main (int argc, char **argv) if (err) error (1, err, "Starting the PCI system"); - /* Create the PCI filesystem */ - err = init_file_system (netfs_startup (bootstrap, O_READ), fs); + if (disk_server_task != MACH_PORT_NULL) + machdev_trivfs_server(bootstrap); + /* Timer started, quickly do all these next, before we call rump_init */ + + /* Create the PCI filesystem without an underlying node first */ + err = init_file_system (pcifs_startup (bootstrap, O_READ), fs); if (err) error (1, err, "Creating the PCI filesystem"); @@ -108,13 +258,19 @@ main (int argc, char **argv) if (err) error (1, err, "Setting permissions"); + err = pthread_create (&nt, NULL, netfs_server_func, NULL); + if (err) + error (1, err, "Creating netfs loop thread"); + pthread_detach (nt); + /* * Ask init to tell us when the system is going down, * so we can try to be friendly to our correspondents on the network. */ arrange_shutdown_notification (); - netfs_server_loop (); /* Never returns. */ - + /* Let the other threads do their job */ + pthread_exit(NULL); + /* Never reached */ return 0; } diff --git a/pci-arbiter/options.c b/pci-arbiter/options.c index 5767ad27..141c69e3 100644 --- a/pci-arbiter/options.c +++ b/pci-arbiter/options.c @@ -241,6 +241,15 @@ parse_opt (int opt, char *arg, struct argp_state *state) case 'n': h->ncache_len = atoi (arg); break; + case 'T': + h->disk_server_task = atoi (arg); + break; + case 'H': + h->host_priv_port = atoi (arg); + break; + case 'P': + h->dev_master_port = atoi (arg); + break; case ARGP_KEY_INIT: /* Initialize our parsing state. */ h = malloc (sizeof (struct parse_hook)); @@ -250,6 +259,9 @@ parse_opt (int opt, char *arg, struct argp_state *state) h->permsets = 0; h->num_permsets = 0; h->ncache_len = NODE_CACHE_MAX; + h->disk_server_task = MACH_PORT_NULL; + h->host_priv_port = MACH_PORT_NULL; + h->dev_master_port = MACH_PORT_NULL; err = parse_hook_add_set (h); if (err) FAIL (err, 1, err, "option parsing"); @@ -286,6 +298,11 @@ parse_opt (int opt, char *arg, struct argp_state *state) /* Set cache len */ fs->params.node_cache_max = h->ncache_len; + /* Set bootstrap ports */ + fs->params.disk_server_task = h->disk_server_task; + _hurd_host_priv = h->host_priv_port; + _hurd_device_master = h->dev_master_port; + if (fs->root) { /* @@ -364,6 +381,8 @@ netfs_append_args (char **argz, size_t * argz_len) if (fs->params.node_cache_max != NODE_CACHE_MAX) ADD_OPT ("--ncache=%u", fs->params.node_cache_max); + if (fs->params.disk_server_task != MACH_PORT_NULL) + ADD_OPT ("--disk-server-task=%lu", fs->params.disk_server_task); #undef ADD_OPT return err; } diff --git a/pci-arbiter/options.h b/pci-arbiter/options.h index 8e5a9da4..292d2963 100644 --- a/pci-arbiter/options.h +++ b/pci-arbiter/options.h @@ -45,6 +45,11 @@ struct parse_hook /* Node cache length */ size_t ncache_len; + + /* Mach ports */ + mach_port_t disk_server_task; + mach_port_t host_priv_port; + mach_port_t dev_master_port; }; /* Lwip translator options. Used for both startup and runtime. */ @@ -65,6 +70,9 @@ static const struct argp_option options[] = { {0, 0, 0, 0, "Global configuration options:", 3}, {"ncache", 'n', "LENGTH", 0, "Node cache length. " STR (NODE_CACHE_MAX) " by default"}, + {"disk-server-task", 'T', "TASK", 0, "Task for bootstrapping disk server"}, + {"host-priv-port", 'H', "PORT", 0, "Port for bootstrapping host"}, + {"dev-master-port", 'P', "PORT", 0, "Port for bootstrapping device master"}, {0} }; diff --git a/pci-arbiter/pcifs.h b/pci-arbiter/pcifs.h index 72cf2910..f19654d1 100644 --- a/pci-arbiter/pcifs.h +++ b/pci-arbiter/pcifs.h @@ -141,6 +141,9 @@ struct pcifs_params /* The size of the node cache. */ size_t node_cache_max; + /* Bootstrap disk server task */ + mach_port_t disk_server_task; + /* FS permissions. */ struct pcifs_perm *perms; size_t num_perms; diff --git a/pci-arbiter/startup-ops.c b/pci-arbiter/startup-ops.c deleted file mode 100644 index eb387fd9..00000000 --- a/pci-arbiter/startup-ops.c +++ /dev/null @@ -1,44 +0,0 @@ -/* - Copyright (C) 2017 Free Software Foundation, Inc. - Written by Michael I. Bushnell, p/BSG. - - This file is part of the GNU Hurd. - - The GNU Hurd is free software; you can redistribute it and/or - modify it under the terms of the GNU General Public License as - published by the Free Software Foundation; either version 2, or (at - your option) any later version. - - The GNU Hurd is distributed in the hope that it will be useful, but - WITHOUT ANY WARRANTY; without even the implied warranty of - MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU - General Public License for more details. - - You should have received a copy of the GNU General Public License - along with the GNU Hurd. If not, see <http://www.gnu.org/licenses/>. -*/ - -#include <startup_notify_S.h> - -#include <pciaccess.h> -#include <hurd/netfs.h> - -#include "startup.h" - -/* The system is going down. Call netfs_shutdown() */ -error_t -S_startup_dosync (mach_port_t handle) -{ - struct port_info *inpi = ports_lookup_port (netfs_port_bucket, handle, - pci_shutdown_notify_class); - - if (!inpi) - return EOPNOTSUPP; - - // Free all libpciaccess resources - pci_system_cleanup (); - - ports_port_deref (inpi); - - return netfs_shutdown (FSYS_GOAWAY_FORCE); -} -- 2.30.1