* [LTP] [PATCH] proc/proc01.c: ignore stable_secret files
@ 2015-07-08 20:33 Jeffrey Bastian
2015-07-09 7:51 ` Jan Stancek
0 siblings, 1 reply; 5+ messages in thread
From: Jeffrey Bastian @ 2015-07-08 20:33 UTC (permalink / raw)
To: ltp-list
Linux 4.1.0 introduced a per-NIC IPv6 stable_secret file for RFC7217.
The stable_secret file for each NIC cannot be read until it is set,
thus LTP fails like this:
proc01 1 TFAIL : proc01.c:392: read failed: /proc/sys/net/ipv6/conf/all/stable_secret: errno=EIO(5): Input/output error
proc01 2 TFAIL : proc01.c:392: read failed: /proc/sys/net/ipv6/conf/default/stable_secret: errno=EIO(5): Input/output error
proc01 3 TFAIL : proc01.c:392: read failed: /proc/sys/net/ipv6/conf/eno1/stable_secret: errno=EIO(5): Input/output error
...
Ignore the stable_secret files in the proc01 test.
TODO: A future test should verify stable_secret functionality.
~]# cat /proc/sys/net/ipv6/conf/eno1/stable_secret
cat: /proc/sys/net/ipv6/conf/eno1/stable_secret: Input/output error
~]# echo "ffff:ffff:ffff:ffff:ffff:ffff:255.255.255.255" > /proc/sys/net/ipv6/conf/eno1/stable_secret
~]# cat /proc/sys/net/ipv6/conf/eno1/stable_secret
ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff
Signed-off-by: Jeffrey Bastian <jbastian@redhat.com>
---
testcases/kernel/fs/proc/proc01.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/testcases/kernel/fs/proc/proc01.c b/testcases/kernel/fs/proc/proc01.c
index f513110eca0e..b9ff3ac42fe2 100644
--- a/testcases/kernel/fs/proc/proc01.c
+++ b/testcases/kernel/fs/proc/proc01.c
@@ -290,14 +290,15 @@ static long readproc(const char *obj)
dir_ent != NULL;
dir_ent = (struct dirent *)readdir(dir)) {
- /* Ignore ".", "..", "kcore", and
- * "/proc/<pid>" (unless this is our
+ /* Ignore ".", "..", "kcore", "stable_secret",
+ * and "/proc/<pid>" (unless this is our
* starting point as directed by the
* user).
*/
if (strcmp(dir_ent->d_name, ".") &&
strcmp(dir_ent->d_name, "..") &&
strcmp(dir_ent->d_name, "kcore") &&
+ strcmp(dir_ent->d_name, "stable_secret") &&
(fnmatch("[0-9]*", dir_ent->d_name,
FNM_PATHNAME) ||
strcmp(obj, procpath))) {
--
1.9.4
------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [LTP] [PATCH] proc/proc01.c: ignore stable_secret files
2015-07-08 20:33 [LTP] [PATCH] proc/proc01.c: ignore stable_secret files Jeffrey Bastian
@ 2015-07-09 7:51 ` Jan Stancek
2015-07-09 15:12 ` Jeffrey Bastian
2015-07-09 15:14 ` [LTP] [PATCH v2] " Jeffrey Bastian
0 siblings, 2 replies; 5+ messages in thread
From: Jan Stancek @ 2015-07-09 7:51 UTC (permalink / raw)
To: Jeffrey Bastian; +Cc: ltp-list
----- Original Message -----
> From: "Jeffrey Bastian" <jbastian@redhat.com>
> To: ltp-list@lists.sourceforge.net
> Sent: Wednesday, 8 July, 2015 10:33:58 PM
> Subject: [LTP] [PATCH] proc/proc01.c: ignore stable_secret files
>
> Linux 4.1.0 introduced a per-NIC IPv6 stable_secret file for RFC7217.
> The stable_secret file for each NIC cannot be read until it is set,
> thus LTP fails like this:
>
> proc01 1 TFAIL : proc01.c:392: read failed:
> /proc/sys/net/ipv6/conf/all/stable_secret: errno=EIO(5): Input/output error
> proc01 2 TFAIL : proc01.c:392: read failed:
> /proc/sys/net/ipv6/conf/default/stable_secret: errno=EIO(5): Input/output
> error
> proc01 3 TFAIL : proc01.c:392: read failed:
> /proc/sys/net/ipv6/conf/eno1/stable_secret: errno=EIO(5): Input/output error
> ...
>
> Ignore the stable_secret files in the proc01 test.
Can we add it to known_issues array instead? It looks like there's no problem
with opening/reading the file (in sense that would cause adverse effects,
e.g. hang the test).
Regards,
Jan
>
> TODO: A future test should verify stable_secret functionality.
> ~]# cat /proc/sys/net/ipv6/conf/eno1/stable_secret
> cat: /proc/sys/net/ipv6/conf/eno1/stable_secret: Input/output error
> ~]# echo "ffff:ffff:ffff:ffff:ffff:ffff:255.255.255.255" >
> /proc/sys/net/ipv6/conf/eno1/stable_secret
> ~]# cat /proc/sys/net/ipv6/conf/eno1/stable_secret
> ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff
>
> Signed-off-by: Jeffrey Bastian <jbastian@redhat.com>
> ---
> testcases/kernel/fs/proc/proc01.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/testcases/kernel/fs/proc/proc01.c
> b/testcases/kernel/fs/proc/proc01.c
> index f513110eca0e..b9ff3ac42fe2 100644
> --- a/testcases/kernel/fs/proc/proc01.c
> +++ b/testcases/kernel/fs/proc/proc01.c
> @@ -290,14 +290,15 @@ static long readproc(const char *obj)
> dir_ent != NULL;
> dir_ent = (struct dirent *)readdir(dir)) {
>
> - /* Ignore ".", "..", "kcore", and
> - * "/proc/<pid>" (unless this is our
> + /* Ignore ".", "..", "kcore", "stable_secret",
> + * and "/proc/<pid>" (unless this is our
> * starting point as directed by the
> * user).
> */
> if (strcmp(dir_ent->d_name, ".") &&
> strcmp(dir_ent->d_name, "..") &&
> strcmp(dir_ent->d_name, "kcore") &&
> + strcmp(dir_ent->d_name, "stable_secret") &&
> (fnmatch("[0-9]*", dir_ent->d_name,
> FNM_PATHNAME) ||
> strcmp(obj, procpath))) {
> --
> 1.9.4
>
> ------------------------------------------------------------------------------
> Don't Limit Your Business. Reach for the Cloud.
> GigeNET's Cloud Solutions provide you with the tools and support that
> you need to offload your IT needs and focus on growing your business.
> Configured For All Businesses. Start Your Cloud Today.
> https://www.gigenetcloud.com/
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list
>
------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [LTP] [PATCH] proc/proc01.c: ignore stable_secret files
2015-07-09 7:51 ` Jan Stancek
@ 2015-07-09 15:12 ` Jeffrey Bastian
2015-07-09 15:14 ` [LTP] [PATCH v2] " Jeffrey Bastian
1 sibling, 0 replies; 5+ messages in thread
From: Jeffrey Bastian @ 2015-07-09 15:12 UTC (permalink / raw)
To: ltp-list
On Thu, Jul 09, 2015 at 03:51:33AM -0400, Jan Stancek wrote:
> > Ignore the stable_secret files in the proc01 test.
>
> Can we add it to known_issues array instead? It looks like there's no
> problem
> with opening/reading the file (in sense that would cause adverse
> effects,
> e.g. hang the test).
For some reason, I thought the known_issues array could not handle
wildcards like '*' when I looked at it last night, but I see now that
wildcards are perfectly fine. I'll re-work the patch.
Thanks!
Jeff
------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [LTP] [PATCH v2] proc/proc01.c: ignore stable_secret files
2015-07-09 7:51 ` Jan Stancek
2015-07-09 15:12 ` Jeffrey Bastian
@ 2015-07-09 15:14 ` Jeffrey Bastian
2015-07-09 15:49 ` Jan Stancek
1 sibling, 1 reply; 5+ messages in thread
From: Jeffrey Bastian @ 2015-07-09 15:14 UTC (permalink / raw)
To: ltp-list
Linux 4.1.0 introduced a per-NIC IPv6 stable_secret file for RFC7217.
The stable_secret file for each NIC cannot be read until it is set,
thus LTP fails like this:
proc01 1 TFAIL : proc01.c:392: read failed: /proc/sys/net/ipv6/conf/all/stable_secret: errno=EIO(5): Input/output error
proc01 2 TFAIL : proc01.c:392: read failed: /proc/sys/net/ipv6/conf/default/stable_secret: errno=EIO(5): Input/output error
proc01 3 TFAIL : proc01.c:392: read failed: /proc/sys/net/ipv6/conf/eno1/stable_secret: errno=EIO(5): Input/output error
...
Add the stable_secret files to the known_issues list in the proc01 test.
TODO: A future test should verify stable_secret functionality.
~]# cat /proc/sys/net/ipv6/conf/eno1/stable_secret
cat: /proc/sys/net/ipv6/conf/eno1/stable_secret: Input/output error
~]# echo "ffff:ffff:ffff:ffff:ffff:ffff:255.255.255.255" > /proc/sys/net/ipv6/conf/eno1/stable_secret
~]# cat /proc/sys/net/ipv6/conf/eno1/stable_secret
ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff
Signed-off-by: Jeffrey Bastian <jbastian@redhat.com>
---
testcases/kernel/fs/proc/proc01.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/testcases/kernel/fs/proc/proc01.c b/testcases/kernel/fs/proc/proc01.c
index f513110eca0e..2825b51dff95 100644
--- a/testcases/kernel/fs/proc/proc01.c
+++ b/testcases/kernel/fs/proc/proc01.c
@@ -107,6 +107,7 @@ static const struct mapping known_issues[] = {
{"read", "/proc/fs/nfsd/.getfs", EINVAL},
{"read", "/proc/fs/nfsd/.getfd", EINVAL},
{"read", "/proc/self/net/rpc/use-gss-proxy", EAGAIN},
+ {"read", "/proc/sys/net/ipv6/conf/*/stable_secret", EIO},
{"", "", 0}
};
--
1.9.4
------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [LTP] [PATCH v2] proc/proc01.c: ignore stable_secret files
2015-07-09 15:14 ` [LTP] [PATCH v2] " Jeffrey Bastian
@ 2015-07-09 15:49 ` Jan Stancek
0 siblings, 0 replies; 5+ messages in thread
From: Jan Stancek @ 2015-07-09 15:49 UTC (permalink / raw)
To: Jeffrey Bastian; +Cc: ltp-list
----- Original Message -----
> From: "Jeffrey Bastian" <jbastian@redhat.com>
> To: ltp-list@lists.sourceforge.net
> Sent: Thursday, 9 July, 2015 5:14:57 PM
> Subject: Re: [LTP] [PATCH v2] proc/proc01.c: ignore stable_secret files
>
> Linux 4.1.0 introduced a per-NIC IPv6 stable_secret file for RFC7217.
> The stable_secret file for each NIC cannot be read until it is set,
> thus LTP fails like this:
>
> proc01 1 TFAIL : proc01.c:392: read failed:
> /proc/sys/net/ipv6/conf/all/stable_secret: errno=EIO(5): Input/output error
> proc01 2 TFAIL : proc01.c:392: read failed:
> /proc/sys/net/ipv6/conf/default/stable_secret: errno=EIO(5): Input/output
> error
> proc01 3 TFAIL : proc01.c:392: read failed:
> /proc/sys/net/ipv6/conf/eno1/stable_secret: errno=EIO(5): Input/output error
> ...
>
> Add the stable_secret files to the known_issues list in the proc01 test.
>
> TODO: A future test should verify stable_secret functionality.
> ~]# cat /proc/sys/net/ipv6/conf/eno1/stable_secret
> cat: /proc/sys/net/ipv6/conf/eno1/stable_secret: Input/output error
> ~]# echo "ffff:ffff:ffff:ffff:ffff:ffff:255.255.255.255" >
> /proc/sys/net/ipv6/conf/eno1/stable_secret
> ~]# cat /proc/sys/net/ipv6/conf/eno1/stable_secret
> ffff:ffff:ffff:ffff:ffff:ffff:ffff:ffff
>
> Signed-off-by: Jeffrey Bastian <jbastian@redhat.com>
Pushed.
Thanks,
Jan
> ---
> testcases/kernel/fs/proc/proc01.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/testcases/kernel/fs/proc/proc01.c
> b/testcases/kernel/fs/proc/proc01.c
> index f513110eca0e..2825b51dff95 100644
> --- a/testcases/kernel/fs/proc/proc01.c
> +++ b/testcases/kernel/fs/proc/proc01.c
> @@ -107,6 +107,7 @@ static const struct mapping known_issues[] = {
> {"read", "/proc/fs/nfsd/.getfs", EINVAL},
> {"read", "/proc/fs/nfsd/.getfd", EINVAL},
> {"read", "/proc/self/net/rpc/use-gss-proxy", EAGAIN},
> + {"read", "/proc/sys/net/ipv6/conf/*/stable_secret", EIO},
> {"", "", 0}
> };
>
> --
> 1.9.4
>
> ------------------------------------------------------------------------------
> Don't Limit Your Business. Reach for the Cloud.
> GigeNET's Cloud Solutions provide you with the tools and support that
> you need to offload your IT needs and focus on growing your business.
> Configured For All Businesses. Start Your Cloud Today.
> https://www.gigenetcloud.com/
> _______________________________________________
> Ltp-list mailing list
> Ltp-list@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/ltp-list
>
------------------------------------------------------------------------------
Don't Limit Your Business. Reach for the Cloud.
GigeNET's Cloud Solutions provide you with the tools and support that
you need to offload your IT needs and focus on growing your business.
Configured For All Businesses. Start Your Cloud Today.
https://www.gigenetcloud.com/
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-07-09 15:49 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-08 20:33 [LTP] [PATCH] proc/proc01.c: ignore stable_secret files Jeffrey Bastian
2015-07-09 7:51 ` Jan Stancek
2015-07-09 15:12 ` Jeffrey Bastian
2015-07-09 15:14 ` [LTP] [PATCH v2] " Jeffrey Bastian
2015-07-09 15:49 ` Jan Stancek
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.