Re: [PATCH net] selftests: pmtu: properly redirect stderr to /dev/null

2018-09-18 Thread David Miller
From: Sabrina Dubroca 
Date: Mon, 17 Sep 2018 15:30:06 +0200

> The cleanup function uses "$CMD 2 > /dev/null", which doesn't actually
> send stderr to /dev/null, so when the netns doesn't exist, the error
> message is shown. Use "2> /dev/null" instead, so that those messages
> disappear, as was intended.
> 
> Fixes: d1f1b9cbf34c ("selftests: net: Introduce first PMTU test")
> Signed-off-by: Sabrina Dubroca 

Applied.


Re: [PATCH net] selftests: pmtu: properly redirect stderr to /dev/null

2018-09-17 Thread Stefano Brivio
On Mon, 17 Sep 2018 15:30:06 +0200
Sabrina Dubroca  wrote:

> The cleanup function uses "$CMD 2 > /dev/null", which doesn't actually
> send stderr to /dev/null, so when the netns doesn't exist, the error
> message is shown. Use "2> /dev/null" instead, so that those messages
> disappear, as was intended.

Oops, thanks for catching this.

> Fixes: d1f1b9cbf34c ("selftests: net: Introduce first PMTU test")
> Signed-off-by: Sabrina Dubroca 

Acked-by: Stefano Brivio 

-- 
Stefano