| commit | a5ff7f7c40346e3f31e0c562807930d53c20cc95 | [log] [tgz] |
|---|---|---|
| author | Ken Rockot <rockot@google.com> | Thu Oct 13 22:24:23 2022 |
| committer | Takuto Ikuta <tikuta@google.com> | Fri Dec 02 02:09:10 2022 |
| tree | a9c458f1bace6d0e119726b127f32ce09f1bd8df | |
| parent | 0ad704f2862ea71db84479445e9e25457afd5d6e [diff] |
ipcz: Re-enable multibroker connection test This resolves two bugs in the MojoIpcz driver transport implementation. First, the logic to determine whether a transport can transmit handles was incorrect. A recent change further constrained requirements for handle transmission, and the new constraint was not enforced consistently for outgoing transmissions. Secondly, the same recent change broke the multibroker test due to a pre-existing use-after-move bug in the test framework. That bug was causing this test's secondary broker to misclassify its own transport endpoint as a non-broker endpoint, preventing it from sending pre-duplicated handles to other processes. This in turn broke all handle transfer from the secondary broker to any non-broker, since non-brokers must generally receive pre-duplicated handles. Both bugs are fixed here and the test is re-enabled. Fixed: 1374114 Change-Id: Ic7ae8263411dc4905da8f651ac58279d4d7c5226 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3954302 Commit-Queue: Ken Rockot <rockot@google.com> Reviewed-by: Alex Gough <ajgo@chromium.org> Cr-Commit-Position: refs/heads/main@{#1058962} NOKEYCHECK=True GitOrigin-RevId: 7e5e271fdba7b3db8c90159adfdb416a570eb519
ipcz is a fully cross-platform C library for interprocess communication (IPC) intended to address two generic problems: routing and data transfer.
With ipcz, applications create pairs of entangled portals to facilitate bidirectional communication. These are grouped into collections called nodes, which typically correspond 1:1 to OS processes.
Nodes may be explicitly connected by an application to establish portal pairs which span the node boundary.
┌───────┐
Connect │ │ Connect
┌──────────>O A O<───────────┐
│ │ │ │
│ └───────┘ │
│ │
v v
┌───O───┐ ┌───O───┐
│ │ │ │
│ B │ │ C │
│ │ │ │
└───────┘ └───────┘
Here nodes A and B are explicitly connected by the application, as are nodes A and C. B can put stuff into its portal, and that stuff will come out of the linked portal on A.
But portals may also be sent over other portals. For example, B may create a new pair of portals...
┌───────┐
Connect │ │ Connect
┌──────────>O A O<───────────┐
│ │ │ │
│ └───────┘ │
│ │
v v
┌───O───┐ ┌───O───┐
│ │ │ │
│ B │ │ C │
│ O───O │ │ │
└───────┘ └───────┘
...and send one over B's existing portal to A:
┌───────┐
Connect │ │ Connect
┌──────────>O A O<───────────┐
│ ┌────────>O │ │
│ │ └───────┘ │
│ │ │
v v v
┌───O─O─┐ ┌───O───┐
│ │ │ │
│ B │ │ C │
│ │ │ │
└───────┘ └───────┘
Node A may then forward this new portal along its own existing portal to node C:
┌───────┐
Connect │ │ Connect
┌──────────>O A O<───────────┐
│ │ │ │
│ └───────┘ │
│ │
v v
┌───O───┐ ┌───O───┐
│ │ │ │
│ B O────────────────────────O C │
│ │ │ │
└───────┘ └───────┘
As a result, the application ends up with a portal on node B linked directly to a portal on node C, despite no explicit effort by the application to connect these two nodes directly.
ipcz enables this seamless creation and transferrence of routes with minimal end-to-end latency and amortized overhead.
ipcz supports an arbitrarily large number of interconnected portals across the system, with potentially hundreds of thousands of individual portal pairs spanning any two nodes. Apart from managing how these portal pairs route their communications (i.e. which nodes they must traverse from end-to-end), ipcz is also concerned with how each communication is physically conveyed from one node to another.
In a traditional IPC system, each transmission typically corresponds to a system I/O call of some kind (e.g. POSIX writev(), or WriteFile() on Windows). These calls may require extra copies of transmitted data, incur additional context switches, and potentially elicit other forms of overhead (e.g. redundant idle CPU wakes) under various conditions.
ipcz tries to avoid such I/O operations in favor of pure userspace memory transactions, falling back onto system I/O only for signaling and less frequent edge cases. To facilitate this behavior, every pair of interconnected nodes has a private shared memory pool managed by ipcz.
To set up a new local repository, first install depot_tools and make sure it's in your PATH.
Then from within the repository root:
cp .gclient-default .gclient gclient sync
When updating a local copy of the repository, it's a good idea to rerun gclient sync to ensure that all external dependencies are up-to-date.
ipcz uses GN for builds. This is provided by the depot_tools installation.
To create a new build configuration, first create a directory for it. For example on Linux or macOS:
mkdir -p out/Debug
Then run gn args to create and edit the build configuration:
gn args out/Debug
For a typical debug build the contents may be as simple as:
is_debug = true
Now targets can be built:
ninja -C out/Debug ipcz_tests # Hope they all pass! ./ipcz_tests
ipcz may be statically linked into a project, or it may be consumed as a shared library. A shared library can be built with the ipcz_shared target.
The library is meant to be consumed exclusively through the C ABI defined in include/ipcz/ipcz.h. Applications populate an IpczAPI structure by calling IpczGetAPI(), the library's only exported symbol. From there they can create and connect nodes and establish portals for higher-level communication.
Applications must provide each node with an implementation of the IpczDriver function table to perform a variety of straightforward, platform- and environment-specific tasks such as establishing a basic I/O transport, generating random numbers, and allocating shared memory regions. See reference drivers for examples.
This directory in the Chromium tree is the source of truth for ipcz. It is not a mirror of an external repository, so there is no separate maintenance of local modifications or other versioning considerations.
The decision to place ipcz sources in //third_party/ipcz was made in light of some unique characteristics:
No dependencies on //base or other Chromium directories are allowed, with the exception of a very small number of carefully chosen APIs allowed when integrating with Chromium builds.
The library is structured and maintained to be useful as a standalone dependency, without needing any other contents of the Chromium tree or its large set of dependencies.
Certain style and dependency violations are made in service of the above two points; for example, ipcz depends on parts of Abseil disallowed in the rest of upstream Chromium, and ipcz internally uses relative include paths rather than paths rooted in Chromium's top-level directory.
Some extensive coverage of ipcz design details can be found here.