On 11/30/2018 03:52 PM, Eric Blake wrote:
On 11/30/18 2:37 PM, Farhan Ali wrote:
Test 232 creates image files with read-only permission and
expects an error message when trying to access the image
files with read-only and auto-read-only turned off.

Don't run as root user, since root can open files with read/write
access for read-only files.

Signed-off-by: Farhan Ali <al...@linux.ibm.com>
---
  tests/qemu-iotests/232 | 5 +++++
  1 file changed, 5 insertions(+)

Did you mean to keep RFC in the subject line?  Will patchew even spot it without PATCH in the subject line?

Reviewed-by: Eric Blake <ebl...@redhat.com>

Thanks for reviewing the patch. I am not sure if patchew will pick it up; I have no problem in spinning another patch but I will wait till Monday to see if anyone else has any feedback.

Thanks
Farhan


Reply via email to