From: Masayoshi Mizuma <msys.mizuma@gmail.com>
To: qi.fuli@jp.fujitsu.com, linux-nvdimm@lists.01.org
Subject: Re: [ndctl PATCH v10 1/4] ndctl, monitor: add a new command - monitor
Date: Tue, 10 Jul 2018 16:18:48 -0400 [thread overview]
Message-ID: <4e80d658-6753-68aa-e608-8f5186e662f0@gmail.com> (raw)
In-Reply-To: <20180709141718.6205-2-qi.fuli@jp.fujitsu.com>
On 07/09/2018 10:17 AM, QI Fuli wrote:
[...]
> +static void log_file(struct ndctl_ctx *ctx, int priority, const char *file,
> + int line, const char *fn, const char *format, va_list args)
> +{
> + FILE *f;
> + char *buf;
> +
> + if (vasprintf(&buf, format, args) < 0) {
> + fail("vasprintf error\n");
> + return;
> + }
> +
> + f = fopen(monitor.logfile, "a+");
> + if (!f) {
> + ndctl_set_log_fn(ctx, log_syslog);
> + fail("open logfile %s failed\n%s", monitor.logfile, buf);
> + goto end;
> + }
> + fprintf(f, "%s\n", buf);
The unit test sometimes failed because the log file is empty.
I think fflush(f) should be needed here to complete the write.
Otherwise, if the monitor daemon stops accidentally, the all
log in buffer are gone...
Thanks,
Masa
> + fclose(f);
> +end:
> + free(buf);
> + return;
> +}
> +
> +static struct json_object *dimm_event_to_json(struct monitor_dimm *mdimm)
> +{
> + struct json_object *jevent, *jobj;
> + bool spares_flag, media_temp_flag, ctrl_temp_flag,
> + health_state_flag, unclean_shutdown_flag;
> + struct ndctl_ctx *ctx = ndctl_dimm_get_ctx(mdimm->dimm);
> +
> + jevent = json_object_new_object();
> + if (!jevent) {
> + fail("\n");
> + return NULL;
> + }
> +
> + if (monitor.event_flags & ND_EVENT_SPARES_REMAINING) {
> + spares_flag = !!(mdimm->event_flags
> + & ND_EVENT_SPARES_REMAINING);
> + jobj = json_object_new_boolean(spares_flag);
> + if (jobj)
> + json_object_object_add(jevent,
> + "dimm-spares-remaining", jobj);
> + }
> +
> + if (monitor.event_flags & ND_EVENT_MEDIA_TEMPERATURE) {
> + media_temp_flag = !!(mdimm->event_flags
> + & ND_EVENT_MEDIA_TEMPERATURE);
> + jobj = json_object_new_boolean(media_temp_flag);
> + if (jobj)
> + json_object_object_add(jevent,
> + "dimm-media-temperature", jobj);
> + }
> +
> + if (monitor.event_flags & ND_EVENT_CTRL_TEMPERATURE) {
> + ctrl_temp_flag = !!(mdimm->event_flags
> + & ND_EVENT_CTRL_TEMPERATURE);
> + jobj = json_object_new_boolean(ctrl_temp_flag);
> + if (jobj)
> + json_object_object_add(jevent,
> + "dimm-controller-temperature", jobj);
> + }
> +
> + if (monitor.event_flags & ND_EVENT_HEALTH_STATE) {
> + health_state_flag = !!(mdimm->event_flags
> + & ND_EVENT_HEALTH_STATE);
> + jobj = json_object_new_boolean(health_state_flag);
> + if (jobj)
> + json_object_object_add(jevent,
> + "dimm-health-state", jobj);
> + }
> +
> + if (monitor.event_flags & ND_EVENT_UNCLEAN_SHUTDOWN) {
> + unclean_shutdown_flag = !!(mdimm->event_flags
> + & ND_EVENT_UNCLEAN_SHUTDOWN);
> + jobj = json_object_new_boolean(unclean_shutdown_flag);
> + if (jobj)
> + json_object_object_add(jevent,
> + "dimm-unclean-shutdown", jobj);
> + }
> +
> + return jevent;
> +}
> +
> +static int notify_dimm_event(struct monitor_dimm *mdimm)
> +{
> + struct json_object *jmsg, *jdimm, *jobj;
> + struct timespec ts;
> + char timestamp[32];
> + struct ndctl_ctx *ctx = ndctl_dimm_get_ctx(mdimm->dimm);
> +
> + jmsg = json_object_new_object();
> + if (!jmsg) {
> + fail("\n");
> + return -1;
> + }
> +
> + clock_gettime(CLOCK_REALTIME, &ts);
> + sprintf(timestamp, "%10ld.%09ld", ts.tv_sec, ts.tv_nsec);
> + jobj = json_object_new_string(timestamp);
> + if (!jobj) {
> + fail("\n");
> + return -1;
> + }
> + json_object_object_add(jmsg, "timestamp", jobj);
> +
> + jobj = json_object_new_int(getpid());
> + if (!jobj) {
> + fail("\n");
> + return -1;
> + }
> + json_object_object_add(jmsg, "pid", jobj);
> +
> + jobj = dimm_event_to_json(mdimm);
> + if (!jobj) {
> + fail("\n");
> + return -1;
> + }
> + json_object_object_add(jmsg, "event", jobj);
> +
> + jdimm = util_dimm_to_json(mdimm->dimm, 0);
> + if (!jdimm) {
> + fail("\n");
> + return -1;
> + }
> + json_object_object_add(jmsg, "dimm", jdimm);
> +
> + jobj = util_dimm_health_to_json(mdimm->dimm);
> + if (!jobj) {
> + fail("\n");
> + return -1;
> + }
> + json_object_object_add(jdimm, "health", jobj);
> +
> + notice(ctx, "%s",
> + json_object_to_json_string_ext(jmsg, JSON_C_TO_STRING_PLAIN));
> +
> + free(jobj);
> + free(jdimm);
> + free(jmsg);
> + return 0;
> +}
> +
> +static struct monitor_dimm *util_dimm_event_filter(struct monitor_dimm *mdimm,
> + unsigned int event_flags)
> +{
> + unsigned int health;
> +
> + mdimm->event_flags = ndctl_dimm_get_event_flags(mdimm->dimm);
> + if (mdimm->event_flags == UINT_MAX)
> + return NULL;
> +
> + health = ndctl_dimm_get_health(mdimm->dimm);
> + if (health == UINT_MAX)
> + return NULL;
> + if (mdimm->health != health)
> + mdimm->event_flags |= ND_EVENT_HEALTH_STATE;
> +
> + if (mdimm->event_flags & event_flags)
> + return mdimm;
> + return NULL;
> +}
> +
> +static int enable_dimm_supported_threshold_alarms(struct ndctl_dimm *dimm)
> +{
> + unsigned int alarm;
> + int rc = -EOPNOTSUPP;
> + struct ndctl_cmd *st_cmd = NULL, *sst_cmd = NULL;
> + const char *name = ndctl_dimm_get_devname(dimm);
> + struct ndctl_ctx *ctx = ndctl_dimm_get_ctx(dimm);
> +
> + st_cmd = ndctl_dimm_cmd_new_smart_threshold(dimm);
> + if (!st_cmd) {
> + err(ctx, "%s: no smart threshold command support\n", name);
> + goto out;
> + }
> + if (ndctl_cmd_submit(st_cmd)) {
> + err(ctx, "%s: smart threshold command failed\n", name);
> + goto out;
> + }
> +
> + sst_cmd = ndctl_dimm_cmd_new_smart_set_threshold(st_cmd);
> + if (!sst_cmd) {
> + err(ctx, "%s: no smart set threshold command support\n", name);
> + goto out;
> + }
> +
> + alarm = ndctl_cmd_smart_threshold_get_alarm_control(st_cmd);
> + if (monitor.event_flags & ND_EVENT_SPARES_REMAINING)
> + alarm |= ND_SMART_SPARE_TRIP;
> + if (monitor.event_flags & ND_EVENT_MEDIA_TEMPERATURE)
> + alarm |= ND_SMART_TEMP_TRIP;
> + if (monitor.event_flags & ND_EVENT_CTRL_TEMPERATURE)
> + alarm |= ND_SMART_CTEMP_TRIP;
> + ndctl_cmd_smart_threshold_set_alarm_control(sst_cmd, alarm);
> +
> + rc = ndctl_cmd_submit(sst_cmd);
> + if (rc) {
> + err(ctx, "%s: smart set threshold command failed\n", name);
> + goto out;
> + }
> +
> +out:
> + ndctl_cmd_unref(sst_cmd);
> + ndctl_cmd_unref(st_cmd);
> + return rc;
> +}
> +
> +static bool filter_region(struct ndctl_region *region,
> + struct util_filter_ctx *fctx)
> +{
> + return true;
> +}
> +
> +static void filter_dimm(struct ndctl_dimm *dimm, struct util_filter_ctx *fctx)
> +{
> + struct monitor_dimm *mdimm;
> + struct monitor_filter_arg *mfa = fctx->monitor;
> + struct ndctl_ctx *ctx = ndctl_dimm_get_ctx(dimm);
> + const char *name = ndctl_dimm_get_devname(dimm);
> +
> + if (!ndctl_dimm_is_cmd_supported(dimm, ND_CMD_SMART)) {
> + err(ctx, "%s: no smart support\n", name);
> + return;
> + }
> + if (!ndctl_dimm_is_cmd_supported(dimm, ND_CMD_SMART_THRESHOLD)) {
> + err(ctx, "%s: no smart threshold support\n", name);
> + return;
> + }
> +
> + if (!ndctl_dimm_is_flag_supported(dimm, ND_SMART_ALARM_VALID)) {
> + err(ctx, "%s: smart alarm invalid\n", name);
> + return;
> + }
> +
> + if (enable_dimm_supported_threshold_alarms(dimm)) {
> + err(ctx, "%s: enable supported threshold alarms failed\n", name);
> + return;
> + }
> +
> + mdimm = calloc(1, sizeof(struct monitor_dimm));
> + if (!mdimm) {
> + err(ctx, "%s: calloc for monitor dimm failed\n", name);
> + return;
> + }
> +
> + mdimm->dimm = dimm;
> + mdimm->health_eventfd = ndctl_dimm_get_health_eventfd(dimm);
> + mdimm->health = ndctl_dimm_get_health(dimm);
> + mdimm->event_flags = ndctl_dimm_get_event_flags(dimm);
> +
> + if (mdimm->event_flags
> + && util_dimm_event_filter(mdimm, monitor.event_flags)) {
> + if (notify_dimm_event(mdimm)) {
> + err(ctx, "%s: notify dimm event failed\n", name);
> + free(mdimm);
> + return;
> + }
> + }
> +
> + list_add_tail(&mfa->dimms, &mdimm->list);
> + if (mdimm->health_eventfd > mfa->maxfd_dimm)
> + mfa->maxfd_dimm = mdimm->health_eventfd;
> + mfa->num_dimm++;
> + return;
> +}
> +
> +static bool filter_bus(struct ndctl_bus *bus, struct util_filter_ctx *fctx)
> +{
> + return true;
> +}
> +
> +static int monitor_event(struct ndctl_ctx *ctx,
> + struct monitor_filter_arg *mfa)
> +{
> + struct epoll_event ev, *events;
> + int nfds, epollfd, i, rc;
> + struct monitor_dimm *mdimm;
> + char buf;
> +
> + events = calloc(mfa->num_dimm, sizeof(struct epoll_event));
> + if (!events) {
> + err(ctx, "malloc for events error\n");
> + return 1;
> + }
> + epollfd = epoll_create1(0);
> + if (epollfd == -1) {
> + err(ctx, "epoll_create1 error\n");
> + return 1;
> + }
> + list_for_each(&mfa->dimms, mdimm, list) {
> + memset(&ev, 0, sizeof(ev));
> + rc = pread(mdimm->health_eventfd, &buf, sizeof(buf), 0);
> + if (rc < 0) {
> + err(ctx, "pread error\n");
> + return 1;
> + }
> + ev.data.ptr = mdimm;
> + if (epoll_ctl(epollfd, EPOLL_CTL_ADD,
> + mdimm->health_eventfd, &ev) != 0) {
> + err(ctx, "epoll_ctl error\n");
> + return 1;
> + }
> + }
> +
> + while (1) {
> + did_fail = 0;
> + nfds = epoll_wait(epollfd, events, mfa->num_dimm, -1);
> + if (nfds <= 0) {
> + err(ctx, "epoll_wait error\n");
> + return 1;
> + }
> + for (i = 0; i < nfds; i++) {
> + mdimm = events[i].data.ptr;
> + if (util_dimm_event_filter(mdimm, monitor.event_flags)) {
> + if (notify_dimm_event(mdimm))
> + fail("%s: notify dimm event failed\n",
> + ndctl_dimm_get_devname(mdimm->dimm));
> + }
> + rc = pread(mdimm->health_eventfd, &buf, sizeof(buf), 0);
> + if (rc < 0)
> + fail("pread error\n");
> + }
> + if (did_fail)
> + return 1;
> + }
> + return 0;
> +}
> +
> +static int parse_monitor_event(struct monitor *_monitor)
> +{
> + char *dimm_event, *save;
> + const char *event;
> +
> + if (!_monitor->dimm_event)
> + goto dimm_event_all;
> + dimm_event = strdup(_monitor->dimm_event);
> + if (!dimm_event)
> + return 1;
> +
> + for (event = strtok_r(dimm_event, " ", &save); event;
> + event = strtok_r(NULL, " ", &save)) {
> + if (strcmp(event, "all") == 0) {
> + free(dimm_event);
> + goto dimm_event_all;
> + }
> + if (strcmp(event, "dimm-spares-remaining") == 0)
> + _monitor->event_flags |= ND_EVENT_SPARES_REMAINING;
> + if (strcmp(event, "dimm-media-temperature") == 0)
> + _monitor->event_flags |= ND_EVENT_MEDIA_TEMPERATURE;
> + if (strcmp(event, "dimm-controller-temperature") == 0)
> + _monitor->event_flags |= ND_EVENT_CTRL_TEMPERATURE;
> + if (strcmp(event, "dimm-health-state") == 0)
> + _monitor->event_flags |= ND_EVENT_HEALTH_STATE;
> + if (strcmp(event, "dimm-unclean-shutdown") == 0)
> + _monitor->event_flags |= ND_EVENT_UNCLEAN_SHUTDOWN;
> + }
> +
> + free(dimm_event);
> + return 0;
> +
> +dimm_event_all:
> + _monitor->event_flags = ND_EVENT_SPARES_REMAINING
> + | ND_EVENT_MEDIA_TEMPERATURE
> + | ND_EVENT_CTRL_TEMPERATURE
> + | ND_EVENT_HEALTH_STATE
> + | ND_EVENT_UNCLEAN_SHUTDOWN;
> + return 0;
> +}
> +
> +int cmd_monitor(int argc, const char **argv, void *ctx)
> +{
> + const struct option options[] = {
> + OPT_STRING('b', "bus", ¶m.bus, "bus-id", "filter by bus"),
> + OPT_STRING('r', "region", ¶m.region, "region-id",
> + "filter by region"),
> + OPT_STRING('d', "dimm", ¶m.dimm, "dimm-id",
> + "filter by dimm"),
> + OPT_STRING('n', "namespace", ¶m.namespace,
> + "namespace-id", "filter by namespace id"),
> + OPT_FILENAME('l', "logfile", &monitor.logfile, "file | syslog",
> + "where to output the monitor's notification"),
> + OPT_BOOLEAN('f', "daemon", &monitor.daemon,
> + "run ndctl monitor as a daemon"),
> + OPT_STRING('D', "dimm-event", &monitor.dimm_event,
> + "dimm-spares-remaining | dimm-media-temperature | dimm-controller-temperature | dimm-health-state | dimm-unclean-shutdown",
> + "filter by DIMM event type"),
> + OPT_END(),
> + };
> + const char * const u[] = {
> + "ndctl monitor [<options>]",
> + NULL
> + };
> + const char *prefix = "./";
> + struct util_filter_ctx fctx = { 0 };
> + struct monitor_filter_arg mfa = { 0 };
> + int i;
> +
> + argc = parse_options_prefix(argc, argv, prefix, options, u, 0);
> + for (i = 0; i < argc; i++) {
> + error("unknown parameter \"%s\"\n", argv[i]);
> + }
> + if (argc)
> + usage_with_options(u, options);
> +
> + if (monitor.logfile && (strcmp(monitor.logfile, "./syslog") != 0))
> + ndctl_set_log_fn((struct ndctl_ctx *)ctx, log_file);
> + else
> + ndctl_set_log_fn((struct ndctl_ctx *)ctx, log_syslog);
> + ndctl_set_log_priority((struct ndctl_ctx *)ctx, LOG_NOTICE);
> +
> + if (monitor.daemon) {
> + if (daemon(0, 0) != 0) {
> + err((struct ndctl_ctx *)ctx, "daemon start failed\n");
> + goto out;
> + }
> + notice((struct ndctl_ctx *)ctx, "ndctl monitor daemon started\n");
> + }
> +
> + if (parse_monitor_event(&monitor))
> + goto out;
> +
> + fctx.filter_bus = filter_bus;
> + fctx.filter_dimm = filter_dimm;
> + fctx.filter_region = filter_region;
> + fctx.filter_namespace = NULL;
> + fctx.arg = &mfa;
> + list_head_init(&mfa.dimms);
> + mfa.num_dimm = 0;
> + mfa.maxfd_dimm = -1;
> + mfa.flags = 0;
> +
> + if (util_filter_walk(ctx, &fctx, ¶m))
> + goto out;
> +
> + if (!mfa.num_dimm) {
> + err((struct ndctl_ctx *)ctx, "no dimms to monitor\n");
> + goto out;
> + }
> +
> + if (monitor_event(ctx, &mfa))
> + goto out;
> +
> + return 0;
> +out:
> + return 1;
> +}
> diff --git a/ndctl/ndctl.c b/ndctl/ndctl.c
> index 7daadeb..73dabfa 100644
> --- a/ndctl/ndctl.c
> +++ b/ndctl/ndctl.c
> @@ -89,6 +89,7 @@ static struct cmd_struct commands[] = {
> { "wait-scrub", cmd_wait_scrub },
> { "start-scrub", cmd_start_scrub },
> { "list", cmd_list },
> + { "monitor", cmd_monitor},
> { "help", cmd_help },
> #ifdef ENABLE_TEST
> { "test", cmd_test },
> diff --git a/util/filter.h b/util/filter.h
> index effda24..c2cdddf 100644
> --- a/util/filter.h
> +++ b/util/filter.h
> @@ -13,6 +13,7 @@
> #ifndef _UTIL_FILTER_H_
> #define _UTIL_FILTER_H_
> #include <stdbool.h>
> +#include <ccan/list/list.h>
>
> struct ndctl_bus *util_bus_filter(struct ndctl_bus *bus, const char *ident);
> struct ndctl_region *util_region_filter(struct ndctl_region *region,
> @@ -50,6 +51,13 @@ struct list_filter_arg {
> unsigned long flags;
> };
>
> +struct monitor_filter_arg {
> + struct list_head dimms;
> + int maxfd_dimm;
> + int num_dimm;
> + unsigned long flags;
> +};
> +
> /*
> * struct util_filter_ctx - control and callbacks for util_filter_walk()
> * ->filter_bus() and ->filter_region() return bool because the
> @@ -67,6 +75,7 @@ struct util_filter_ctx {
> union {
> void *arg;
> struct list_filter_arg *list;
> + struct monitor_filter_arg *monitor;
> };
> };
>
>
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm
next prev parent reply other threads:[~2018-07-10 20:18 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-07-09 14:17 [ndctl PATCH v10 0/4] ndctl, monitor: add ndctl monitor daemon QI Fuli
2018-07-09 14:17 ` [ndctl PATCH v10 1/4] ndctl, monitor: add a new command - monitor QI Fuli
2018-07-10 20:18 ` Masayoshi Mizuma [this message]
2018-07-11 1:27 ` Qi, Fuli
2018-07-09 14:17 ` [ndctl PATCH v10 2/4] ndctl, monitor: add main ndctl monitor configuration file QI Fuli
2018-07-09 14:17 ` [ndctl PATCH v10 3/4] ndctl, monitor: add the unit file of systemd for ndctl-monitor service QI Fuli
2018-07-09 14:17 ` [ndctl PATCH v10 4/4] ndctl, documentation: add manpage for monitor QI Fuli
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4e80d658-6753-68aa-e608-8f5186e662f0@gmail.com \
--to=msys.mizuma@gmail.com \
--cc=linux-nvdimm@lists.01.org \
--cc=qi.fuli@jp.fujitsu.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).