]> granicus.if.org Git - zfs/blobdiff - cmd/zed/zed_event.c
Merge branch 'zfsonlinux/merge-spl'
[zfs] / cmd / zed / zed_event.c
index 7d819b803ecb505cb00017730fc97628caca1e91..2a7ff16fd38e7fe734c5565fcab515d98874d064 100644 (file)
 #include <sys/zfs_ioctl.h>
 #include <time.h>
 #include <unistd.h>
+#include <sys/fm/fs/zfs.h>
 #include "zed.h"
 #include "zed_conf.h"
+#include "zed_disk_event.h"
 #include "zed_exec.h"
 #include "zed_file.h"
 #include "zed_log.h"
 #include "zed_strings.h"
 
+#include "agents/zfs_agents.h"
+
 #define        MAXBUF  4096
 
 /*
@@ -50,6 +54,11 @@ zed_event_init(struct zed_conf *zcp)
        if (zcp->zevent_fd < 0)
                zed_log_die("Failed to open \"%s\": %s",
                    ZFS_DEV, strerror(errno));
+
+       zfs_agent_init(zcp->zfs_hdl);
+
+       if (zed_disk_event_init() != 0)
+               zed_log_die("Failed to initialize disk events");
 }
 
 /*
@@ -61,6 +70,9 @@ zed_event_fini(struct zed_conf *zcp)
        if (!zcp)
                zed_log_die("Failed zed_event_fini: %s", strerror(EINVAL));
 
+       zed_disk_event_fini();
+       zfs_agent_fini();
+
        if (zcp->zevent_fd >= 0) {
                if (close(zcp->zevent_fd) < 0)
                        zed_log_msg(LOG_WARNING, "Failed to close \"%s\": %s",
@@ -252,6 +264,13 @@ _zed_event_add_var(uint64_t eid, zed_strings_t *zsp,
        *dstp++ = '=';
        buflen--;
 
+       if (buflen <= 0) {
+               errno = EMSGSIZE;
+               zed_log_msg(LOG_WARNING, "Failed to add %s for eid=%llu: %s",
+                   keybuf, eid, "Exceeded buffer size");
+               return (-1);
+       }
+
        va_start(vargs, fmt);
        n = vsnprintf(dstp, buflen, fmt, vargs);
        va_end(vargs);
@@ -477,7 +496,7 @@ _zed_event_add_int64_array(uint64_t eid, zed_strings_t *zsp,
        name = nvpair_name(nvp);
        (void) nvpair_value_int64_array(nvp, &i64p, &nelem);
        for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) {
-               n = snprintf(p, buflen, "%lld ", (u_longlong_t) i64p[i]);
+               n = snprintf(p, buflen, "%lld ", (u_longlong_t)i64p[i]);
                if ((n < 0) || (n >= buflen))
                        return (_zed_event_add_array_err(eid, name));
                p += n;
@@ -509,7 +528,7 @@ _zed_event_add_uint64_array(uint64_t eid, zed_strings_t *zsp,
        fmt = _zed_event_value_is_hex(name) ? "0x%.16llX " : "%llu ";
        (void) nvpair_value_uint64_array(nvp, &u64p, &nelem);
        for (i = 0, p = buf; (i < nelem) && (buflen > 0); i++) {
-               n = snprintf(p, buflen, fmt, (u_longlong_t) u64p[i]);
+               n = snprintf(p, buflen, fmt, (u_longlong_t)u64p[i]);
                if ((n < 0) || (n >= buflen))
                        return (_zed_event_add_array_err(eid, name));
                p += n;
@@ -591,7 +610,7 @@ _zed_event_add_nvpair(uint64_t eid, zed_strings_t *zsp, nvpair_t *nvp)
                _zed_event_add_var(eid, zsp, prefix, name, "%d", i8);
                break;
        case DATA_TYPE_INT8:
-               (void) nvpair_value_int8(nvp, (int8_t *) &i8);
+               (void) nvpair_value_int8(nvp, (int8_t *)&i8);
                _zed_event_add_var(eid, zsp, prefix, name, "%d", i8);
                break;
        case DATA_TYPE_UINT8:
@@ -599,7 +618,7 @@ _zed_event_add_nvpair(uint64_t eid, zed_strings_t *zsp, nvpair_t *nvp)
                _zed_event_add_var(eid, zsp, prefix, name, "%u", i8);
                break;
        case DATA_TYPE_INT16:
-               (void) nvpair_value_int16(nvp, (int16_t *) &i16);
+               (void) nvpair_value_int16(nvp, (int16_t *)&i16);
                _zed_event_add_var(eid, zsp, prefix, name, "%d", i16);
                break;
        case DATA_TYPE_UINT16:
@@ -607,7 +626,7 @@ _zed_event_add_nvpair(uint64_t eid, zed_strings_t *zsp, nvpair_t *nvp)
                _zed_event_add_var(eid, zsp, prefix, name, "%u", i16);
                break;
        case DATA_TYPE_INT32:
-               (void) nvpair_value_int32(nvp, (int32_t *) &i32);
+               (void) nvpair_value_int32(nvp, (int32_t *)&i32);
                _zed_event_add_var(eid, zsp, prefix, name, "%d", i32);
                break;
        case DATA_TYPE_UINT32:
@@ -615,24 +634,45 @@ _zed_event_add_nvpair(uint64_t eid, zed_strings_t *zsp, nvpair_t *nvp)
                _zed_event_add_var(eid, zsp, prefix, name, "%u", i32);
                break;
        case DATA_TYPE_INT64:
-               (void) nvpair_value_int64(nvp, (int64_t *) &i64);
+               (void) nvpair_value_int64(nvp, (int64_t *)&i64);
                _zed_event_add_var(eid, zsp, prefix, name,
-                   "%lld", (longlong_t) i64);
+                   "%lld", (longlong_t)i64);
                break;
        case DATA_TYPE_UINT64:
                (void) nvpair_value_uint64(nvp, &i64);
                _zed_event_add_var(eid, zsp, prefix, name,
                    (_zed_event_value_is_hex(name) ? "0x%.16llX" : "%llu"),
-                   (u_longlong_t) i64);
+                   (u_longlong_t)i64);
+               /*
+                * shadow readable strings for vdev state pairs
+                */
+               if (strcmp(name, FM_EREPORT_PAYLOAD_ZFS_VDEV_STATE) == 0 ||
+                   strcmp(name, FM_EREPORT_PAYLOAD_ZFS_VDEV_LASTSTATE) == 0) {
+                       char alt[32];
+
+                       (void) snprintf(alt, sizeof (alt), "%s_str", name);
+                       _zed_event_add_var(eid, zsp, prefix, alt, "%s",
+                           zpool_state_to_name(i64, VDEV_AUX_NONE));
+               } else
+               /*
+                * shadow readable strings for pool state
+                */
+               if (strcmp(name, FM_EREPORT_PAYLOAD_ZFS_POOL_STATE) == 0) {
+                       char alt[32];
+
+                       (void) snprintf(alt, sizeof (alt), "%s_str", name);
+                       _zed_event_add_var(eid, zsp, prefix, alt, "%s",
+                           zpool_pool_state_to_name(i64));
+               }
                break;
        case DATA_TYPE_DOUBLE:
                (void) nvpair_value_double(nvp, &d);
                _zed_event_add_var(eid, zsp, prefix, name, "%g", d);
                break;
        case DATA_TYPE_HRTIME:
-               (void) nvpair_value_hrtime(nvp, (hrtime_t *) &i64);
+               (void) nvpair_value_hrtime(nvp, (hrtime_t *)&i64);
                _zed_event_add_var(eid, zsp, prefix, name,
-                   "%llu", (u_longlong_t) i64);
+                   "%llu", (u_longlong_t)i64);
                break;
        case DATA_TYPE_NVLIST:
                _zed_event_add_var(eid, zsp, prefix, name,
@@ -693,12 +733,14 @@ _zed_event_add_nvpair(uint64_t eid, zed_strings_t *zsp, nvpair_t *nvp)
 
 /*
  * Restrict various environment variables to safe and sane values
- * when constructing the environment for the child process.
+ * when constructing the environment for the child process, unless
+ * we're running with a custom $PATH (like under the ZFS test suite).
  *
  * Reference: Secure Programming Cookbook by Viega & Messier, Section 1.1.
  */
 static void
-_zed_event_add_env_restrict(uint64_t eid, zed_strings_t *zsp)
+_zed_event_add_env_restrict(uint64_t eid, zed_strings_t *zsp,
+    const char *path)
 {
        const char *env_restrict[][2] = {
                { "IFS",                " \t\n" },
@@ -713,11 +755,35 @@ _zed_event_add_env_restrict(uint64_t eid, zed_strings_t *zsp)
                { "ZFS_RELEASE",        ZFS_META_RELEASE },
                { NULL,                 NULL }
        };
+
+       /*
+        * If we have a custom $PATH, use the default ZFS binary locations
+        * instead of the hard-coded ones.
+        */
+       const char *env_path[][2] = {
+               { "IFS",                " \t\n" },
+               { "PATH",               NULL }, /* $PATH copied in later on */
+               { "ZDB",                "zdb" },
+               { "ZED",                "zed" },
+               { "ZFS",                "zfs" },
+               { "ZINJECT",            "zinject" },
+               { "ZPOOL",              "zpool" },
+               { "ZFS_ALIAS",          ZFS_META_ALIAS },
+               { "ZFS_VERSION",        ZFS_META_VERSION },
+               { "ZFS_RELEASE",        ZFS_META_RELEASE },
+               { NULL,                 NULL }
+       };
        const char *(*pa)[2];
 
        assert(zsp != NULL);
 
-       for (pa = env_restrict; *(*pa); pa++) {
+       pa = path != NULL ? env_path : env_restrict;
+
+       for (; *(*pa); pa++) {
+               /* Use our custom $PATH if we have one */
+               if (path != NULL && strcmp((*pa)[0], "PATH") == 0)
+                       (*pa)[1] = path;
+
                _zed_event_add_var(eid, zsp, NULL, (*pa)[0], "%s", (*pa)[1]);
        }
 }
@@ -853,22 +919,26 @@ zed_event_service(struct zed_conf *zcp)
                zed_log_msg(LOG_WARNING,
                    "Failed to lookup zevent class (eid=%llu)", eid);
        } else {
+               /* let internal modules see this event first */
+               zfs_agent_post_event(class, NULL, nvl);
+
                zsp = zed_strings_create();
 
                nvp = NULL;
                while ((nvp = nvlist_next_nvpair(nvl, nvp)))
                        _zed_event_add_nvpair(eid, zsp, nvp);
 
-               _zed_event_add_env_restrict(eid, zsp);
+               _zed_event_add_env_restrict(eid, zsp, zcp->path);
                _zed_event_add_env_preserve(eid, zsp);
 
                _zed_event_add_var(eid, zsp, ZED_VAR_PREFIX, "PID",
-                   "%d", (int) getpid());
+                   "%d", (int)getpid());
                _zed_event_add_var(eid, zsp, ZED_VAR_PREFIX, "ZEDLET_DIR",
                    "%s", zcp->zedlet_dir);
                subclass = _zed_event_get_subclass(class);
                _zed_event_add_var(eid, zsp, ZEVENT_VAR_PREFIX, "SUBCLASS",
                    "%s", (subclass ? subclass : class));
+
                _zed_event_add_time_strings(eid, zsp, etime);
 
                zed_exec_process(eid, class, subclass,