Merge pull request #2975 from octo/issue/2954
authorFlorian Forster <ff@octo.it>
Fri, 26 Oct 2018 06:21:45 +0000 (08:21 +0200)
committerGitHub <noreply@github.com>
Fri, 26 Oct 2018 06:21:45 +0000 (08:21 +0200)
zfs_arc: Ignore the first two lines of the statistics file on Linux (5.7 cherry-pick)

src/zfs_arc.c

index af5130a..063c401 100644 (file)
@@ -212,6 +212,23 @@ static int za_read(void) {
     return (-1);
   }
 
+  // Ignore the first two lines because they contain information about
+  // the rest of the file.
+  // See kstat_seq_show_headers module/spl/spl-kstat.c of the spl kernel
+  // module.
+  if (fgets(buffer, sizeof(buffer), fh) == NULL) {
+    ERROR("zfs_arc plugin: \"%s\" does not contain a single line.",
+          ZOL_ARCSTATS_FILE);
+    fclose(fh);
+    return (-1);
+  }
+  if (fgets(buffer, sizeof(buffer), fh) == NULL) {
+    ERROR("zfs_arc plugin: \"%s\" does not contain at least two lines.",
+          ZOL_ARCSTATS_FILE);
+    fclose(fh);
+    return (-1);
+  }
+
   while (fgets(buffer, sizeof(buffer), fh) != NULL) {
     char *fields[3];
     value_t v;