Merge ace8238 into c32e100

Author: jerzyjamroz

.github/workflows/ci-scripts-build.yml

@@ -39,68 +39,69 @@ jobs:
       fail-fast: false
       matrix:
         include:
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
             cmp: gcc
             configuration: default
             base: "7.0"
             wine: "64"
 
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
             cmp: gcc
             configuration: static
             base: "7.0"
 
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
             cmp: gcc
             configuration: default
             base: "3.15"
 
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
             cmp: gcc
             configuration: default
             base: "3.14"
 
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
             cmp: gcc
             configuration: static
             base: "7.0"
             extra: "CMD_CXXFLAGS=-std=c++11"
 
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
             cmp: clang
             configuration: default
             base: "7.0"
             extra: "CMD_CXXFLAGS=-std=c++11"
 
-          - os: ubuntu-20.04
-            cmp: gcc
-            configuration: default
-            base: "7.0"
-            rtems: "4.10"
+          # It requires more debugging:
+          # - os: ubuntu-22.04
+          #   cmp: gcc
+          #   configuration: default
+          #   base: "7.0"
+          #   rtems: "4.10"
 
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
             cmp: gcc
             configuration: default
             base: "7.0"
             rtems: "4.9"
 
-          - os: ubuntu-20.04
-            cmp: gcc-8
-            utoolchain: "8"
+          - os: ubuntu-22.04
+            cmp: gcc-9
+            utoolchain: "9"
             configuration: default
             base: "7.0"
 
-          - os: ubuntu-20.04
+          - os: ubuntu-22.04
             cmp: clang
             configuration: default
             base: "7.0"
 
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
         with:
           submodules: true
       - name: Cache Dependencies
-        uses: actions/cache@v2
+        uses: actions/cache@v3
         with:
           path: ~/.cache
           key: ${{ matrix.base }}/${{ matrix.os }}/${{ matrix.cmp }}/${{ matrix.configuration }}/${{ matrix.wine }}${{ matrix.rtems }}/${{ matrix.extra }}

configure/CONFIG_DEVLIB2_VERSION

@@ -1,2 +1,2 @@
 DEVLIB2_MAJOR_VERSION = 2
-DEVLIB2_MINOR_VERSION = 12
+DEVLIB2_MINOR_VERSION = 13

pciApp/devLibPCI.h

@@ -451,6 +451,16 @@ int devPCIDisableInterrupt(const epicsPCIDevice *dev);
 epicsShareFunc
 const char* devPCIDeviceClassToString(int classId);
 
+/**
+ * @brief Optional callback invoked on PCI device hot-swap.
+ *
+ * This function pointer may be set to a custom handler that will be called
+ * when a PCI device is successfully reopened (e.g. after hot-swap).
+ *
+ * @param name  Identifier or name of the reopened PCI device.
+ */
+epicsShareExtern void (*devPCIonHotSwapHook)(const char*);
+
 #ifdef __cplusplus
 } /* extern "C" */
 #endif

pciApp/os/Linux/devLibPCIOSD.c

@@ -507,7 +507,7 @@ int linuxDevPCIInit(void)
         /* Read BAR info */
 
         /* Base address */
-        
+
         filename = allocPrintf(BUSBASE "resource",
                                osd->dev.domain, osd->dev.bus, osd->dev.device, osd->dev.function);
         if (!filename) {
@@ -548,10 +548,10 @@ int linuxDevPCIInit(void)
 
         osd->displayErom = start;
         osd->eromlen = (start || stop ) ? (stop - start + 1) : 0;
-        
+
         fclose(file);
         free(filename);
-        
+
         /* driver name */
         filename = allocPrintf(BUSBASE "driver",
                                osd->dev.domain, osd->dev.bus, osd->dev.device, osd->dev.function);
@@ -922,6 +922,30 @@ int linuxDevPCIConnectInterrupt(
     return ret;
 }
 
+static int reopen_uio(struct osdPCIDevice *osd)
+{
+    int uio = find_uio_number(osd);
+    if (uio < 0)
+        return -1;
+
+    char *devname = allocPrintf("/dev/uio%u", uio);
+    if (!devname)
+        return -1;
+
+    int newfd = open(devname, O_RDWR);
+    free(devname);
+    if (newfd < 0)
+        return -1;
+
+    if (osd->fd != -1)
+        close(osd->fd);
+
+    osd->fd = newfd;
+    return 0;
+}
+
+epicsShareDef void (*devPCIonHotSwapHook)(const char* name) = NULL;
+
 static
 void isrThread(void* arg)
 {
@@ -959,18 +983,36 @@ void isrThread(void* arg)
             epicsInterruptUnlock(isrflag);
         }
 
-        ret=read(osd->fd, &event, sizeof(event));
-        if (ret==-1) {
-            switch(errno) {
-            case EINTR: /* interrupted by a signal */
+        ret = read(osd->fd, &event, sizeof(event));
+        if (ret == -1) {
+            switch (errno) {
+            case EINTR:
                 break;
+
+            case EIO:
+            case EINVAL:
+            case ENODEV:
+                errlogPrintf("isrThread '%s': Device removed or UIO invalid (errno=%d: %s)\n", name, errno, strerror(errno));
+
+                epicsMutexMustLock(osd->devLock);
+                if (reopen_uio(osd) == 0) {
+                    errlogPrintf("isrThread '%s': Successfully reopened UIO device\n", name);
+                    if (devPCIonHotSwapHook) devPCIonHotSwapHook(name);
+                } else {
+                    errlogPrintf("isrThread '%s': UIO reopen failed. Will retry.\n", name);
+                }
+                epicsMutexUnlock(osd->devLock);
+                epicsThreadSleep(1);
+                continue;
+
             default:
-                errlogPrintf("isrThread '%s' read error %d\n",
-                             name,errno);
-                epicsThreadSleep(0.5);
+                errlogPrintf("isrThread '%s': read error %d (%s)\n", name, errno, strerror(errno));
+                epicsThreadSleep(1);
             }
-        } else
-            interrupted=1;
+        } else {
+            interrupted = 1;
+        }
+
 
         if (next!=event && next!=0) {
             errlogPrintf("isrThread '%s' missed %d events\n",