On 27/08/2021 12.17, Pierre Morel wrote:
We will need to look for a device inside the channel subsystem
based upon device specificities.
Let's provide a callback for an upper layer to be called during
the enumeration of the channel subsystem.
Signed-off-by: Pierre Morel <pmorel@xxxxxxxxxxxxx>
---
lib/s390x/css.h | 3 ++-
lib/s390x/css_lib.c | 4 +++-
s390x/css.c | 2 +-
3 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/lib/s390x/css.h b/lib/s390x/css.h
index d644971f..2005f4d7 100644
--- a/lib/s390x/css.h
+++ b/lib/s390x/css.h
@@ -278,7 +278,8 @@ void dump_irb(struct irb *irbp);
void dump_pmcw(struct pmcw *p);
void dump_orb(struct orb *op);
-int css_enumerate(void);
+typedef int (enumerate_cb_t)(int);
+int css_enumerate(enumerate_cb_t *f);
#define MAX_ENABLE_RETRIES 5
#define IO_SCH_ISC 3
diff --git a/lib/s390x/css_lib.c b/lib/s390x/css_lib.c
index efc70576..484f9c41 100644
--- a/lib/s390x/css_lib.c
+++ b/lib/s390x/css_lib.c
@@ -117,7 +117,7 @@ bool get_chsc_scsc(void)
* On success return the first subchannel ID found.
* On error return an invalid subchannel ID containing cc
*/
-int css_enumerate(void)
+int css_enumerate(enumerate_cb_t *f)
I'd maybe call it "cb" instead of "f" ... but that's just my personal taste,
I guess.
{
struct pmcw *pmcw = &schib.pmcw;
int scn_found = 0;
@@ -153,6 +153,8 @@ int css_enumerate(void)
schid = scn | SCHID_ONE;
report_info("Found subchannel %08x", scn | SCHID_ONE);
dev_found++;
+ if (f)
+ f(scn | SCHID_ONE);
}
out:
diff --git a/s390x/css.c b/s390x/css.c
index c340c539..b50fbc67 100644
--- a/s390x/css.c
+++ b/s390x/css.c
@@ -29,7 +29,7 @@ struct ccw1 *ccw;
static void test_enumerate(void)
{
- test_device_sid = css_enumerate();
+ test_device_sid = css_enumerate(NULL);
if (test_device_sid & SCHID_ONE) {
report(1, "Schid of first I/O device: 0x%08x", test_device_sid);
return;
Reviewed-by: Thomas Huth <thuth@xxxxxxxxxx>
