[3/9] seccomp: Move seccomp_info into seccomp.h

Submitted by Cyrill Gorcunov on April 26, 2018, 8:14 p.m.

Details

Message ID 20180426201443.2163-4-gorcunov@gmail.com
State Rejected
Series "Per-thread seccomp support, v3"
Headers show

Commit Message

Cyrill Gorcunov April 26, 2018, 8:14 p.m. 
This header is main place for all seccomp related
structures so move seccomp_info here. This will
allow to minimize changes area when need to update
definitions and such.

Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>
---
 criu/include/proc_parse.h | 7 -------
 criu/include/seccomp.h    | 7 +++++++
 2 files changed, 7 insertions(+), 7 deletions(-)

Patch hide | download patch | download mbox 

diff --git a/criu/include/proc_parse.h b/criu/include/proc_parse.h
index c6856613031b..ca028e055449 100644
--- a/criu/include/proc_parse.h
+++ b/criu/include/proc_parse.h
@@ -4,7 +4,6 @@ 
 #include <sys/types.h>
 
 #include <compel/compel.h>
-#include "images/seccomp.pb-c.h"
 
 #define PROC_TASK_COMM_LEN	32
 #define PROC_TASK_COMM_LEN_FMT	"(%31s"
@@ -64,12 +63,6 @@  struct proc_pid_stat {
 	int			exit_code;
 };
 
-struct seccomp_info {
-	SeccompFilter filter;
-	int id;
-	struct seccomp_info *prev;
-};
-
 #define PROC_CAP_SIZE	2
 
 struct proc_status_creds {
diff --git a/criu/include/seccomp.h b/criu/include/seccomp.h
index 42ab9ab2d518..b50ea34e20bb 100644
--- a/criu/include/seccomp.h
+++ b/criu/include/seccomp.h
@@ -4,6 +4,7 @@ 
 #include <linux/seccomp.h>
 #include <linux/filter.h>
 
+#include "images/seccomp.pb-c.h"
 #include "images/core.pb-c.h"
 
 #ifndef SECCOMP_MODE_DISABLED
@@ -26,6 +27,12 @@ 
 #define SECCOMP_FILTER_FLAG_TSYNC 1
 #endif
 
+struct seccomp_info {
+	struct seccomp_info	*prev;
+	int			id;
+	SeccompFilter		filter;
+};
+
 extern int collect_seccomp_filters(void);
 extern int prepare_seccomp_filters(void);
 struct task_restore_args;

Comments

Dmitry Safonov May 4, 2018, 10:31 p.m. 
2018-04-26 21:14 GMT+01:00 Cyrill Gorcunov <gorcunov@gmail.com>:
> This header is main place for all seccomp related
> structures so move seccomp_info here. This will
> allow to minimize changes area when need to update
> definitions and such.
>
> Signed-off-by: Cyrill Gorcunov <gorcunov@gmail.com>

Reviewed-by: Dmitry Safonov <0x7f454c46@gmail.com>