The control plane should use libipcc

.cargo/config

@@ -13,7 +13,7 @@ rustdocflags = "--document-private-items"
 # things critical to correctness probably don't belong here.
 [target.x86_64-unknown-illumos]
 rustflags = [
-    "-C", "link-arg=-Wl,-znocompstrtab"
+    "-C", "link-arg=-Wl,-znocompstrtab,-R/usr/platform/oxide/lib/amd64"
 ]
 
 # Set up `cargo xtask`.

Cargo.toml

@@ -33,7 +33,7 @@ members = [
     "installinator",
     "internal-dns-cli",
     "internal-dns",
-    "ipcc-key-value",
+    "ipcc",
     "key-manager",
     "nexus",
     "nexus/authz-macros",
@@ -102,7 +102,7 @@ default-members = [
     "installinator",
     "internal-dns-cli",
     "internal-dns",
-    "ipcc-key-value",
+    "ipcc",
     "key-manager",
     "nexus",
     "nexus/authz-macros",
@@ -221,7 +221,7 @@ installinator-artifactd = { path = "installinator-artifactd" }
 installinator-artifact-client = { path = "clients/installinator-artifact-client" }
 installinator-common = { path = "installinator-common" }
 internal-dns = { path = "internal-dns" }
-ipcc-key-value = { path = "ipcc-key-value" }
+ipcc = { path = "ipcc" }
 ipnetwork = { version = "0.20", features = ["schemars"] }
 itertools = "0.12.0"
 key-manager = { path = "key-manager" }

gateway/Cargo.toml

@@ -16,7 +16,7 @@ hex.workspace = true
 http.workspace = true
 hyper.workspace = true
 illumos-utils.workspace = true
-ipcc-key-value.workspace = true
+ipcc.workspace = true
 omicron-common.workspace = true
 once_cell.workspace = true
 schemars.workspace = true

gateway/src/http_entrypoints.rs

@@ -443,11 +443,11 @@ pub struct ImageVersion {
     pub version: u32,
 }
 
-// This type is a duplicate of the type in `ipcc-key-value`, and we provide a
+// This type is a duplicate of the type in `ipcc`, and we provide a
 // `From<_>` impl to convert to it. We keep these types distinct to allow us to
 // choose different representations for MGS's HTTP API (this type) and the wire
 // format passed through the SP to installinator
-// (`ipcc_key_value::InstallinatorImageId`), although _currently_ they happen to
+// (`ipcc::InstallinatorImageId`), although _currently_ they happen to
 // be defined identically.
 #[derive(
     Debug, Clone, Copy, PartialEq, Eq, Serialize, Deserialize, JsonSchema,
@@ -1228,7 +1228,7 @@ async fn sp_power_state_set(
 
 /// Set the installinator image ID the sled should use for recovery.
 ///
-/// This value can be read by the host via IPCC; see the `ipcc-key-value` crate.
+/// This value can be read by the host via IPCC; see the `ipcc` crate.
 #[endpoint {
     method = PUT,
     path = "/sp/{type}/{slot}/ipcc/installinator-image-id",
@@ -1238,13 +1238,12 @@ async fn sp_installinator_image_id_set(
     path: Path<PathSp>,
     body: TypedBody<InstallinatorImageId>,
 ) -> Result<HttpResponseUpdatedNoContent, HttpError> {
-    use ipcc_key_value::Key;
+    use ipcc::Key;
 
     let apictx = rqctx.context();
     let sp = apictx.mgmt_switch.sp(path.into_inner().sp.into())?;
 
-    let image_id =
-        ipcc_key_value::InstallinatorImageId::from(body.into_inner());
+    let image_id = ipcc::InstallinatorImageId::from(body.into_inner());
 
     sp.set_ipcc_key_lookup_value(
         Key::InstallinatorImageId as u8,
@@ -1265,7 +1264,7 @@ async fn sp_installinator_image_id_delete(
     rqctx: RequestContext<Arc<ServerContext>>,
     path: Path<PathSp>,
 ) -> Result<HttpResponseUpdatedNoContent, HttpError> {
-    use ipcc_key_value::Key;
+    use ipcc::Key;
 
     let apictx = rqctx.context();
     let sp = apictx.mgmt_switch.sp(path.into_inner().sp.into())?;

gateway/src/http_entrypoints/conversions.rs

@@ -397,7 +397,7 @@ impl From<StartupOptions> for HostStartupOptions {
     }
 }
 
-impl From<InstallinatorImageId> for ipcc_key_value::InstallinatorImageId {
+impl From<InstallinatorImageId> for ipcc::InstallinatorImageId {
     fn from(id: InstallinatorImageId) -> Self {
         Self {
             update_id: id.update_id,

installinator/Cargo.toml

@@ -20,7 +20,7 @@ http.workspace = true
 illumos-utils.workspace = true
 installinator-artifact-client.workspace = true
 installinator-common.workspace = true
-ipcc-key-value.workspace = true
+ipcc.workspace = true
 itertools.workspace = true
 libc.workspace = true
 omicron-common.workspace = true

installinator/src/artifact.rs

@@ -9,7 +9,7 @@ use clap::Args;
 use futures::StreamExt;
 use installinator_artifact_client::ClientError;
 use installinator_common::EventReport;
-use ipcc_key_value::{InstallinatorImageId, Ipcc};
+use ipcc::{InstallinatorImageId, Ipcc};
 use omicron_common::update::{ArtifactHash, ArtifactHashId};
 use tokio::sync::mpsc;
 use uuid::Uuid;
@@ -47,7 +47,7 @@ pub(crate) struct ArtifactIdOpts {
 impl ArtifactIdOpts {
     pub(crate) fn resolve(&self) -> Result<InstallinatorImageId> {
         if self.from_ipcc {
-            let ipcc = Ipcc::open().context("error opening IPCC")?;
+            let ipcc = Ipcc::new().context("error opening IPCC")?;
             ipcc.installinator_image_id()
                 .context("error retrieving installinator image ID")
         } else {

ipcc-key-value/Cargo.toml → ipcc/Cargo.toml

@@ -1,5 +1,5 @@
 [package]
-name = "ipcc-key-value"
+name = "ipcc"
 version = "0.1.0"
 edition = "2021"
 license = "MPL-2.0"
@@ -12,6 +12,7 @@ serde.workspace = true
 thiserror.workspace = true
 uuid.workspace = true
 omicron-workspace-hack.workspace = true
+cfg-if.workspace = true
 
 [dev-dependencies]
 omicron-common = { workspace = true, features = ["testing"] }