Changeset 832cbe7 in mainline for uspace/lib/drv/generic/remote_hw_res.c

Timestamp:

2025-02-05T12:30:20Z (9 days ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

master

Children:

accdf882

Parents:

0dab4850

git-author:

Jiri Svoboda <jiri@…> (2025-02-04 21:30:06)

git-committer:

Jiri Svoboda <jiri@…> (2025-02-05 12:30:20)

Message:

Add proper IDE PCI to ISA fallback mechanism.

To determine if legacy IDE I/O ports are free, isa-ide asks isa,
who asks pciintel. pciintel waits for bus enumeration to complete,
then waits for all functions except the one who is asking
(which is ISA bus) to stabilize. During attach pci-ide will claim
the legacy IDE ports. Thus, if at this point legacy IDE ports
are unclaimed, pciintel tells ISA they are free, which tells isa-ide,
which continues to attach. If they are not free, isa-ide will not
attach.

This works for all use cases, including system without PCI bus,
system with PCI bus, but no (or disabled) PCI IDE, system with PCI
IDE with unrecognized VID/PID (which we will handle in legacy ISA mode).

File:

• uspace/lib/drv/generic/remote_hw_res.c (modified) (4 diffs)

uspace/lib/drv/generic/remote_hw_res.c

@@ -1 +1 @@
 /*
- * Copyright (c) 2024 Jiri Svoboda
+ * Copyright (c) 2025 Jiri Svoboda
  * Copyright (c) 2010 Lenka Trochtova
  * Copyright (c) 2011 Jan Vesely
@@ -48 +48 @@
 static void remote_hw_res_dma_channel_setup(ddf_fun_t *, void *, ipc_call_t *);
 static void remote_hw_res_dma_channel_remain(ddf_fun_t *, void *, ipc_call_t *);
-static void remote_hw_res_get_flags(ddf_fun_t *, void *, ipc_call_t *);
+static void remote_hw_res_query_legacy_io(ddf_fun_t *, void *, ipc_call_t *);
+static void remote_hw_res_claim_legacy_io(ddf_fun_t *, void *, ipc_call_t *);
 
 static const remote_iface_func_ptr_t remote_hw_res_iface_ops [] = {
@@ -57 +58 @@
         [HW_RES_DMA_CHANNEL_SETUP] = &remote_hw_res_dma_channel_setup,
         [HW_RES_DMA_CHANNEL_REMAIN] = &remote_hw_res_dma_channel_remain,
-        [HW_RES_GET_FLAGS] = &remote_hw_res_get_flags
+        [HW_RES_QUERY_LEGACY_IO] = &remote_hw_res_query_legacy_io,
+        [HW_RES_CLAIM_LEGACY_IO] = &remote_hw_res_claim_legacy_io
 };
 
@@ -174 +176 @@
 }
 
-static void remote_hw_res_get_flags(ddf_fun_t *fun, void *ops,
-    ipc_call_t *call)
-{
-        hw_res_ops_t *hw_res_ops = ops;
-
-        if (hw_res_ops->get_flags == NULL) {
-                async_answer_0(call, ENOTSUP);
-                return;
-        }
-
-        hw_res_flags_t flags = 0;
-        const errno_t ret = hw_res_ops->get_flags(fun, &flags);
-        async_answer_1(call, ret, flags);
+static void remote_hw_res_query_legacy_io(ddf_fun_t *fun, void *ops,
+    ipc_call_t *call)
+{
+        hw_res_ops_t *hw_res_ops = ops;
+
+        if (hw_res_ops->query_legacy_io == NULL) {
+                async_answer_0(call, ENOTSUP);
+                return;
+        }
+
+        hw_res_claims_t claims = 0;
+        const errno_t ret = hw_res_ops->query_legacy_io(fun, &claims);
+        async_answer_1(call, ret, claims);
+}
+
+static void remote_hw_res_claim_legacy_io(ddf_fun_t *fun, void *ops,
+    ipc_call_t *call)
+{
+        hw_res_ops_t *hw_res_ops = ops;
+        hw_res_claims_t claims;
+
+        if (hw_res_ops->claim_legacy_io == NULL) {
+                async_answer_0(call, ENOTSUP);
+                return;
+        }
+
+        claims = DEV_IPC_GET_ARG1(*call);
+
+        const errno_t ret = hw_res_ops->claim_legacy_io(fun, claims);
+        async_answer_0(call, ret);
 }