tests: Modify eswitch mode checking

[EdwardSro]

The devlink output behavior would be changed by having two outputs for Eswitch modes, which are legacy and switchdev, instead of having three outputs which are None, legacy, and switchdev. So, the legacy mode will be the default mode.

So, the function that checks the Eswitch mode should be adjusted, since it only checks the legacy mode in case the command returns a non zero value, but with the newer devlinks, the command will return zero value for both legacy and switchdev, so there is important to check the value of the output.

[rleon]

Do you have kernel commit that supports the claim of "going to be changed"?

Thanks

[EdwardSro]

Do you have kernel commit that supports the claim of "going to be changed"?

Thanks

The patch is going to be sent soon to upstream, once it's done I'll point to it. This patch though supports both the current implementation and the new candidate and was sent as a precautionary measurement. But we can wait for the kernel patch, no rush.

[EdwardSro]

I see that the kernel patch was accepted.

[EdwardSro]

@rleon can we merge this PR?

[jgunthorpe]

@EdwardSro please rebase it

[EdwardSro]

A similar change was already merged as part of a different PR. Will close this.