On Mon, Feb 27, 2023 at 11:24:04AM +0000, Shinichiro Kawasaki wrote:
On Feb 27, 2023 / 11:35, Kanchan Joshi wrote:
On Tue, Feb 14, 2023 at 01:47:37PM +0900, Shin'ichiro Kawasaki wrote:
> Per suggestion by Kanchan, add a new test case to test unprivileged passthrough
> of NVME character devices. The first patch adds a feature to run commands with
> normal user privilege. The second patch adds the test case using the feature.
>
> Changes from v2:
> * Added the first patch to add normal user privilege support to blktests
> * Adjusted the test case to the functions for normal user privilege support
Thanks, this looks way better. And works fine in my setup.
If required,
Tested-by: Kanchan Joshi <joshi.k@xxxxxxxxxxx>
Thanks for the confirmation. Sounds good.
I found two more minor points to improve:
1) tests/nvme/046 does not have executable mode bit. I will add it when I apply
the patch.
2) I ran the test case with kernel version v6.1 and it failed. Does the test
case require kernel version 6.2 or higher? If that is the case, one more line
change will be required as follows. If you are ok with the change, I can fold
this change in when I apply the patches.
Yes, unprivileged passthrough exists from 6.2. Changes looks good.
Thanks.
