[PATCH blktests] nvme: Add passthru error logging tests to nvme/039

[Chaitanya Kulkarni]

> +_nvme_passthru_logging_setup()
> +{
> +	ctrl_dev_passthru_logging=$(cat /sys/class/nvme/"$2"/passthru_err_log_enabled)
> +	ns_dev_passthru_logging=$(cat /sys/class/nvme/"$2"/"$1"/passthru_err_log_enabled)
> +
> +	_nvme_disable_passthru_admin_error_logging "$2"
> +	_nvme_disable_passthru_io_error_logging "$1" "$2"
> +}
> +
> +_nvme_passthru_logging_cleanup()
> +{
> +	echo $ctrl_dev_passthru_logging > /sys/class/nvme/"$2"/passthru_err_log_enabled
> +	echo $ns_dev_passthru_logging > /sys/class/nvme/"$2"/"$1"/passthru_err_log_enabled
> +}
>   
>   _nvme_err_inject_setup()
>   {
> @@ -985,6 +1002,26 @@ _nvme_disable_err_inject()
>           echo 0 > /sys/kernel/debug/"$1"/fault_inject/times
>   }
>   
> +_nvme_enable_passthru_admin_error_logging()
> +{
> +	echo on > /sys/class/nvme/"$1"/passthru_err_log_enabled
> +}
> +
> +_nvme_enable_passthru_io_error_logging()
> +{
> +	echo on > /sys/class/nvme/"$2"/"$1"/passthru_err_log_enabled
> +}
> +
> +_nvme_disable_passthru_admin_error_logging()
> +{
> +	echo off > /sys/class/nvme/"$1"/passthru_err_log_enabled
> +}
> +
> +_nvme_disable_passthru_io_error_logging()
> +{
> +	echo off > /sys/class/nvme/"$2"/"$1"/passthru_err_log_enabled
> +}
> +
>

Thanks for the test, let's move these helper testcase itself if we get
second testcase ? I'd not bloat nvme rc file unless we have another
user for these function, also if you move these to testcase itself
then you don't really need to make these as a function ...

-ck