bring pw to latest commit

MODULE.bazel

@@ -9,8 +9,8 @@ module(
 bazel_dep(name = "bazel_skylib", version = "1.8.2")
 bazel_dep(name = "pigweed")
 bazel_dep(name = "platforms", version = "1.0.0")
-bazel_dep(name = "rules_rust", version = "0.68.1")
-bazel_dep(name = "rules_rust_mdbook", version = "0.68.1")
+bazel_dep(name = "rules_rust", version = "0.70.0")
+bazel_dep(name = "rules_rust_mdbook", version = "0.70.0")
 bazel_dep(name = "rules_platform", version = "0.1.0")
 bazel_dep(name = "rules_python", version = "1.8.3")
 bazel_dep(name = "ureg")
@@ -22,15 +22,12 @@ local_path_override(
 
 git_override(
     module_name = "pigweed",
-    commit = "67563b40f2c19aec4f899afd13c00e48baf790b2",
+    commit = "7882c537d21171b0e9641a132842a373382a78c3",
     patch_args = ["-p1"],
     patches = [
-        "//third_party/pigweed:veer_pic_generic_type.patch",
         # Allow integration_tests to build for armv7-m / armv7e-m so the
         # AST10x0 unittest_runner can use it.
         "//third_party/pigweed:integration_tests_armv7m.patch",
-        # Fix lost-wakeup race in ipc/user test USER signal round-trip.
-        "//third_party/pigweed:ipc_user_signal_race.patch",
     ],
     remote = "https://pigweed.googlesource.com/pigweed/pigweed",
 )

target/ast10x0/harness/BUILD.bazel

@@ -8,12 +8,11 @@ py_binary(
     name = "qemu_runner_bin",
     srcs = ["qemu_runner.py"],
     main = "qemu_runner.py",
-    # @@pigweed++_repo_rules5+qemu is the canonical label for the qemu repo
-    # created by pigweed's cipd_repository use_repo_rule (not directly
-    # visible as @qemu from our module). If this breaks after a pigweed
-    # upgrade, run: ls $(bazel info output_base)/external/ | grep qemu
+    # @@pigweed++cipd+pigweed.qemu is the canonical label for the qemu repo
+    # created by pigweed's cipd module extension. If this breaks after a
+    # pigweed upgrade, run: bazel mod dump_repo_mapping "pigweed+" | grep qemu
     deps = [
-        "@@pigweed++_repo_rules5+qemu//:qemu-system-arm-runfiles",
+        "@@pigweed++cipd+pigweed.qemu//:qemu-system-arm-runfiles",
         "@pigweed//pw_tokenizer/py:detokenize",
         "@rules_python//python/runfiles",
     ],

target/ast10x0/tests/interrupts/kernel/target.rs

@@ -19,7 +19,7 @@ impl TargetInterface for Target {
     const NAME: &'static str = "AST10x0 Kernel Interrupts";
 
     fn main() -> ! {
-        let sentinel: &[u8] = match test_interrupts::main::<Arch>(TEST_IRQ) {
+        let sentinel: &[u8] = match test_interrupts::main::<Arch>(Arch, TEST_IRQ) {
             Ok(()) => b"TEST_RESULT:PASS\n",
             Err(_e) => b"TEST_RESULT:FAIL\n",
         };

target/ast10x0/tests/usart/client2_main.rs

@@ -10,7 +10,7 @@ use userspace::entry;
 use userspace::syscall;
 
 #[entry]
-fn entry() -> ! {
+fn entry() {
     let client = UsartClient::new(handle::USART);
 
     match client.configure(1_500_000) {
@@ -23,8 +23,6 @@ fn entry() -> ! {
         Err(_) => pw_log::error!("USART client2 write failed"),
     }
 
-    // End the test only after client2 has executed, proving multi-client app
-    // startup and request handling against the same server endpoint.
     let _ = syscall::debug_shutdown(Ok(()));
     loop {}
 }

target/ast10x0/tests/usart/client_main.rs

@@ -10,7 +10,7 @@ use userspace::entry;
 use userspace::time::Instant;
 
 #[entry]
-fn entry() -> ! {
+fn entry() {
     let client = UsartClient::new(handle::USART);
 
     match client.configure(1_500_000) {

target/ast10x0/tests/usart/server_main.rs

@@ -11,7 +11,7 @@ use userspace::entry;
 use userspace::syscall::{self, Signals};
 
 #[entry]
-fn entry() -> ! {
+fn entry() {
     let mut backend = Backend::new();
 
     let _ = syscall::wait_group_add(

target/earlgrey/syscall_latency/main.rs

@@ -50,11 +50,9 @@ fn measure_nop_syscall(rv_timer: &RvTimer, n: usize) -> Result<()> {
 }
 
 #[entry]
-fn entry() -> ! {
+fn entry() -> Result<()> {
     let rv_timer = unsafe { RvTimer::new() };
-    let result = measure_nop_syscall(&rv_timer, 100);
-    syscall::debug_shutdown(result).unwrap();
-    loop {}
+    measure_nop_syscall(&rv_timer, 100)
 }
 
 #[panic_handler]

target/earlgrey/tests/drivers/gpio/test_gpio.rs

@@ -8,7 +8,7 @@ use earlgrey_gpio::{EarlGreyGpio, EarlGreyPinConfig, GpioMask, GpioPin};
 use earlgrey_pinmux::{Pad, Pull};
 use openprot_hal_blocking::gpio_port::{GpioPort, PinMask};
 use pw_status::Result;
-use userspace::{entry, syscall};
+use userspace::entry;
 
 fn test_gpio_basic() -> Result<()> {
     let mut gpio = unsafe { EarlGreyGpio::new() };
@@ -71,7 +71,7 @@ fn test_gpio_basic() -> Result<()> {
 }
 
 #[entry]
-fn entry() -> ! {
+fn entry() -> Result<()> {
     pw_log::info!("🔄 RUNNING GPIO SMOKE TEST");
     let ret = test_gpio_basic();
 
@@ -81,8 +81,7 @@ fn entry() -> ! {
         pw_log::info!("✅ PASS");
     }
 
-    let _ = syscall::debug_shutdown(ret);
-    loop {}
+    ret
 }
 
 #[panic_handler]

target/earlgrey/tests/uart/test_uart.rs

@@ -60,20 +60,17 @@ fn test_uart_interrupts() -> Result<()> {
 }
 
 #[entry]
-fn entry() -> ! {
+fn entry() -> Result<()> {
     pw_log::info!("🔄 RUNNING");
     let ret = test_uart_interrupts();
 
-    // Log that an error occurred so that the app that caused the shutdown is logged.
     if ret.is_err() {
         pw_log::error!("❌ FAILED: {}", ret.status_code() as u32);
     } else {
         pw_log::info!("✅ PASSED");
     }
 
-    // Since this is written as a test, shut down with the return status from `main()`.
-    let _ = syscall::debug_shutdown(ret);
-    loop {}
+    ret
 }
 
 #[panic_handler]

target/earlgrey/tests/uart/test_uart_listener.rs

@@ -77,11 +77,8 @@ fn wait_for_interrupts() -> Result<()> {
 }
 
 #[entry]
-fn entry() -> ! {
-    // Since this is written as a test, shut down with the return status from `main()`.
-    let ret = wait_for_interrupts();
-    let _ = syscall::debug_shutdown(ret);
-    loop {}
+fn entry() -> Result<()> {
+    wait_for_interrupts()
 }
 
 #[panic_handler]

target/veer/syscall_latency/main.rs

@@ -43,10 +43,8 @@ fn measure_nop_syscall(n: usize) -> Result<()> {
 }
 
 #[entry]
-fn entry() -> ! {
-    let result = measure_nop_syscall(100);
-    syscall::debug_shutdown(result).unwrap();
-    loop {}
+fn entry() -> Result<()> {
+    measure_nop_syscall(100)
 }
 
 #[panic_handler]