systemctl: option to list units by state

This allows to show only units with specified LOAD or SUB or ACTIVE state.

diff --git a/man/systemctl.xml b/man/systemctl.xml
index f5502153add1ee3a64ebd7fd84b047246f1e8f9a..982051778ed835b117c051aa97e3d8ff37bbffc5 100644 (file)
--- a/man/systemctl.xml
+++ b/man/systemctl.xml
@@ -94,25 +94,29 @@ along with systemd; If not, see <http://www.gnu.org/licenses/>.
         <listitem>
           <para>The argument should be a comma-separated list of unit
           types such as <option>service</option> and
         <listitem>
           <para>The argument should be a comma-separated list of unit
           types such as <option>service</option> and

-          <option>socket</option>, or unit load states such as
-          <option>loaded</option> and <option>masked</option>
-          (types and states can be mixed).</para>
+          <option>socket</option>.
+          </para>

 
           <para>If one of the arguments is a unit type, when listing
           units, limit display to certain unit types. Otherwise, units
           of all types will be shown.</para>
 
 
           <para>If one of the arguments is a unit type, when listing
           units, limit display to certain unit types. Otherwise, units
           of all types will be shown.</para>
 

-          <para>If one of the arguments is a unit load state, when
-          listing units, limit display to certain unit
-          types. Otherwise, units of in all load states will be
-          shown.</para>
-

           <para>As a special case, if one of the arguments is
           <option>help</option>, a list of allowed values will be
           printed and the program will exit.</para>
         </listitem>
       </varlistentry>
 
           <para>As a special case, if one of the arguments is
           <option>help</option>, a list of allowed values will be
           printed and the program will exit.</para>
         </listitem>
       </varlistentry>
 

+      <varlistentry>
+        <term><option>--state=</option></term>
+
+        <listitem>
+        <para>Argument should be a comma-separated list of unit LOAD
+        or SUB or ACTIVE states. When listing units show only those
+        with specified LOAD or SUB or ACTIVE state.</para>
+        </listitem>
+      </varlistentry>
+

       <varlistentry>
         <term><option>-p</option></term>
         <term><option>--property=</option></term>
       <varlistentry>
         <term><option>-p</option></term>
         <term><option>--property=</option></term>


@@ -165,15 +169,6 @@ along with systemd; If not, see <http://www.gnu.org/licenses/>.
         </listitem>
       </varlistentry>
 
         </listitem>
       </varlistentry>
 

-      <varlistentry>
-        <term><option>--failed</option></term>
-
-        <listitem>
-          <para>When listing units, show only failed units. Do not
-          confuse with <option>--fail</option>.</para>
-        </listitem>
-      </varlistentry>
-

       <varlistentry>
         <term><option>-l</option></term>
         <term><option>--full</option></term>
       <varlistentry>
         <term><option>-l</option></term>
         <term><option>--full</option></term>

diff --git a/src/systemctl/systemctl.c b/src/systemctl/systemctl.c
index b3b679e0af0b5c5637c99cc81a46ee377a99aeec..c9f9981f9c7358b43b164b088cee8a811d72c1a5 100644 (file)
--- a/src/systemctl/systemctl.c
+++ b/src/systemctl/systemctl.c
@@ -69,7 +69,7 @@
 #include "fileio.h"
 
 static char **arg_types = NULL;
 #include "fileio.h"
 
 static char **arg_types = NULL;

-static char **arg_load_states = NULL;
+static char **arg_states = NULL;

 static char **arg_properties = NULL;
 static bool arg_all = false;
 static enum dependency {
 static char **arg_properties = NULL;
 static bool arg_all = false;
 static enum dependency {


@@ -93,7 +93,6 @@ static bool arg_quiet = false;
 static bool arg_full = false;
 static int arg_force = 0;
 static bool arg_ask_password = true;
 static bool arg_full = false;
 static int arg_force = 0;
 static bool arg_ask_password = true;

-static bool arg_failed = false;

 static bool arg_runtime = false;
 static char **arg_wall = NULL;
 static const char *arg_kill_who = NULL;
 static bool arg_runtime = false;
 static char **arg_wall = NULL;
 static const char *arg_kill_who = NULL;


@@ -301,12 +300,11 @@ static int compare_unit_info(const void *a, const void *b) {
 static bool output_show_unit(const struct unit_info *u) {
         const char *dot;
 
 static bool output_show_unit(const struct unit_info *u) {
         const char *dot;
 

-        if (arg_failed)
-                return streq(u->active_state, "failed");
+        if (!strv_isempty(arg_states))
+                return strv_contains(arg_states, u->load_state) || strv_contains(arg_states, u->sub_state) || strv_contains(arg_states, u->active_state);

 
         return (!arg_types || ((dot = strrchr(u->id, '.')) &&
                                strv_find(arg_types, dot+1))) &&
 
         return (!arg_types || ((dot = strrchr(u->id, '.')) &&
                                strv_find(arg_types, dot+1))) &&

-                (!arg_load_states || strv_find(arg_load_states, u->load_state)) &&

                 (arg_all || !(streq(u->active_state, "inactive")
                               || u->following[0]) || u->job_id > 0);
 }
                 (arg_all || !(streq(u->active_state, "inactive")
                               || u->following[0]) || u->job_id > 0);
 }


@@ -4705,12 +4703,12 @@ static int systemctl_help(void) {
                "  -h --help           Show this help\n"
                "     --version        Show package version\n"
                "  -t --type=TYPE      List only units of a particular type\n"
                "  -h --help           Show this help\n"
                "     --version        Show package version\n"
                "  -t --type=TYPE      List only units of a particular type\n"

+               "     --state=STATE    Show only units with particular LOAD or SUB or ACTIVE state\n"

                "  -p --property=NAME  Show only properties by this name\n"
                "  -a --all            Show all loaded units/properties, including dead/empty\n"
                "                      ones. To list all units installed on the system, use\n"
                "                      the 'list-unit-files' command instead.\n"
                "     --reverse        Show reverse dependencies with 'list-dependencies'\n"
                "  -p --property=NAME  Show only properties by this name\n"
                "  -a --all            Show all loaded units/properties, including dead/empty\n"
                "                      ones. To list all units installed on the system, use\n"
                "                      the 'list-unit-files' command instead.\n"
                "     --reverse        Show reverse dependencies with 'list-dependencies'\n"

-               "     --failed         Show only failed units\n"

                "  -l --full           Don't ellipsize unit names on output\n"
                "     --fail           When queueing a new job, fail if conflicting jobs are\n"
                "                      pending\n"
                "  -l --full           Don't ellipsize unit names on output\n"
                "     --fail           When queueing a new job, fail if conflicting jobs are\n"
                "                      pending\n"


@@ -4896,13 +4894,6 @@ static int help_types(void) {
                         puts(t);
         }
 
                         puts(t);
         }
 

-        puts("\nAvailable unit load states: ");
-        for(i = 0; i < _UNIT_LOAD_STATE_MAX; i++) {
-                t = unit_load_state_to_string(i);
-                if (t)
-                        puts(t);
-        }
-

         return 0;
 }
 
         return 0;
 }
 


@@ -4931,7 +4922,8 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
                 ARG_FAILED,
                 ARG_RUNTIME,
                 ARG_FORCE,
                 ARG_FAILED,
                 ARG_RUNTIME,
                 ARG_FORCE,

-                ARG_PLAIN
+                ARG_PLAIN,
+                ARG_STATE

         };
 
         static const struct option options[] = {
         };
 
         static const struct option options[] = {


@@ -4970,6 +4962,7 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
                 { "lines",     required_argument, NULL, 'n'           },
                 { "output",    required_argument, NULL, 'o'           },
                 { "plain",     no_argument,       NULL, ARG_PLAIN     },
                 { "lines",     required_argument, NULL, 'n'           },
                 { "output",    required_argument, NULL, 'o'           },
                 { "plain",     no_argument,       NULL, ARG_PLAIN     },

+                { "state",     required_argument, NULL, ARG_STATE     },

                 { NULL,        0,                 NULL, 0             }
         };
 
                 { NULL,        0,                 NULL, 0             }
         };
 


@@ -5014,8 +5007,12 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
                                         continue;
                                 }
 
                                         continue;
                                 }
 

+                                /* It's much nicer to use --state= for
+                                 * load states, but let's support this
+                                 * in --types= too for compatibility
+                                 * with old versions */

                                 if (unit_load_state_from_string(optarg) >= 0) {
                                 if (unit_load_state_from_string(optarg) >= 0) {

-                                        if (strv_push(&arg_load_states, type))
+                                        if (strv_push(&arg_states, type) < 0)

                                                 return log_oom();
                                         type = NULL;
                                         continue;
                                                 return log_oom();
                                         type = NULL;
                                         continue;


@@ -5047,7 +5044,7 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
                                         if (!prop)
                                                 return log_oom();
 
                                         if (!prop)
                                                 return log_oom();
 

-                                        if (strv_push(&arg_properties, prop)) {
+                                        if (strv_push(&arg_properties, prop) < 0) {

                                                 free(prop);
                                                 return log_oom();
                                         }
                                                 free(prop);
                                                 return log_oom();
                                         }


@@ -5131,7 +5128,9 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
                         break;
 
                 case ARG_FAILED:
                         break;
 
                 case ARG_FAILED:

-                        arg_failed = true;
+                        if (strv_extend(&arg_states, "failed") < 0)
+                                return log_oom();
+

                         break;
 
                 case 'q':
                         break;
 
                 case 'q':


@@ -5201,6 +5200,25 @@ static int systemctl_parse_argv(int argc, char *argv[]) {
                         arg_plain = true;
                         break;
 
                         arg_plain = true;
                         break;
 

+                case ARG_STATE: {
+                        char *word, *state;
+                        size_t size;
+
+                        FOREACH_WORD_SEPARATOR(word, size, optarg, ",", state) {
+                                char *s;
+
+                                s = strndup(word, size);
+                                if (!s)
+                                        return log_oom();
+
+                                if (strv_push(&arg_states, s) < 0) {
+                                        free(s);
+                                        return log_oom();
+                                }
+                        }
+                        break;
+                }
+

                 case '?':
                         return -EINVAL;
 
                 case '?':
                         return -EINVAL;
 


@@ -6257,7 +6275,7 @@ finish:
         dbus_shutdown();
 
         strv_free(arg_types);
         dbus_shutdown();
 
         strv_free(arg_types);

-        strv_free(arg_load_states);
+        strv_free(arg_states);

         strv_free(arg_properties);
 
         pager_close();
         strv_free(arg_properties);
 
         pager_close();