On 12/1/23 07:43, Hangbin Liu wrote:
On Thu, Nov 30, 2023 at 02:28:51PM +0100, Justin Iurman wrote:
On 11/30/23 05:01, Hangbin Liu wrote:
Here is the test result after conversion.
]# ./ioam6.sh
OUTPUT tests
TEST: Unknown IOAM namespace (inline mode) [ OK ] TEST: Unknown IOAM namespace (encap mode) [ OK ] TEST: Missing trace room (inline mode) [ OK ] TEST: Missing trace room (encap mode) [ OK ] TEST: Trace type with bit 0 only (inline mode) [ OK ] ... TEST: Full supported trace (encap mode) [ OK ]
GLOBAL tests
TEST: Forward - Full supported trace (inline mode) [ OK ] TEST: Forward - Full supported trace (encap mode) [ OK ]
- Tests passed: 88
- Tests failed: 0
Acked-by: David Ahern dsahern@kernel.org Signed-off-by: Hangbin Liu liuhangbin@gmail.com
Reviewed-by: Justin Iurman justin.iurman@uliege.be
LGTM. Just one question though. Is there any reason not to use cleanup_ns everywhere? There is the following diff (actually, 3 times):
Hi Justin,
Thanks for your review. There is no much intend. I just use del ns for one line change. And use cleanup_ns for multi line changes. I can make all ns delete via cleanup_ns in next version.
+1, I think it would make sense to use the new API everywhere for consistency.
BTW, I will use `cleanup_ns $ns || true` in next version as cleanup_ns could return none 0 in PATCHv2.
Sounds good, thanks!
Thanks Hangbin