mbox series

[blktests,0/2] TMPDIR fixes

Message ID 20231012021152.832553-1-shinichiro.kawasaki@wdc.com (mailing list archive)
Headers show
Series TMPDIR fixes | expand

Message

Shinichiro Kawasaki Oct. 12, 2023, 2:11 a.m. UTC
Yi Zhang reported that the reference to TMPDIR in nvme/rc causes an issue
because the reference happens out of test() or test_device() context [1][2].
The first patch addresses this issue. The second patch fixes another TMPDIR
bug found during this work.

[1] https://lore.kernel.org/linux-block/20231011034832.1650797-1-yi.zhang@redhat.com/
[2] https://lore.kernel.org/linux-block/20231011072530.1659810-1-yi.zhang@redhat.com/

Shin'ichiro Kawasaki (2):
  nvme/{rc,017,031}: replace def_file_path with _nvme_def_file_path()
  block/002: fix TMPDIR path

 tests/block/002 |  2 +-
 tests/nvme/017  |  9 +++++----
 tests/nvme/031  |  6 +++---
 tests/nvme/rc   | 17 +++++++++++------
 4 files changed, 20 insertions(+), 14 deletions(-)

Comments

Shinichiro Kawasaki Oct. 13, 2023, 2:32 a.m. UTC | #1
On Oct 12, 2023 / 11:11, Shin'ichiro Kawasaki wrote:
> Yi Zhang reported that the reference to TMPDIR in nvme/rc causes an issue
> because the reference happens out of test() or test_device() context [1][2].
> The first patch addresses this issue. The second patch fixes another TMPDIR
> bug found during this work.
> 
> [1] https://lore.kernel.org/linux-block/20231011034832.1650797-1-yi.zhang@redhat.com/
> [2] https://lore.kernel.org/linux-block/20231011072530.1659810-1-yi.zhang@redhat.com/

Thanks for the reviews. I've applied the patches.