Re: [PATCH 0/3] fix unlink/rename failure in hyper-v container

[Yoshinao Muramatsu <ysno@ac.auone-net.jp>]

Hi Corinna.
I'm Yoshinao Muramatsu. Thank you for your thoughtful guidance.
I have not modified the patch yet,
but I have made some observations and post the results.

I forgot to mention in my first post that there is a workaround
for this issue, which is to use process isolation unless
hyper-v isolation is absolutely necessary
(mainly caused by build number difference between host and container).

In the logs, here we can find some differences.
But I believe it is unclear if it will always be so.
If additional inspections are done, they will be costly.
I would also like to see better support for POSIX unlink/rename
semantics in the container mechanism.

Therefore, I think it is not a bad idea to simply retry rather than spending
time every time to check the conditions to deal with rare situations.
However, in certain situations it happens all the time.
(So if we can cache it that would be great.)
But performance in the vast majority of other situations is more important.

In the following logs, c:\ is the normal ntfs volume in the container,
andc:\opt is the volume of the bind mounted host.
Where the differences are
- 0x00000020 Supports Disk Quotas
- 0x01000000 Supports Open By FileID
- 0x02000000 Supports USN Journal

== hyper-v container (has issue)
$ mount
C:/cygwin64/bin on /usr/bin type ntfs (binary,auto)
C:/cygwin64/lib on /usr/lib type ntfs (binary,auto)
C:/cygwin64 on / type ntfs (binary,auto)
C: on /cygdrive/c type ntfs (binary,posix=0,user,noumount,auto)

$ /usr/lib/csih/getVolInfo.exe /cygdrive/c/
Device Type        : 7
Characteristics    : 20020
Volume Name        : <>
Serial Number      : 3456656850
Max Filenamelength : 255
Filesystemname     : <NTFS>
Flags              : 1c706df
...snip...

$ /usr/lib/csih/getVolInfo.exe /cygdrive/c/opt/
Device Type        : 7
Characteristics    : 20020
Volume Name        : <>
Serial Number      : 955187689
Max Filenamelength : 255
Filesystemname     : <NTFS>
Flags              : 2c706ff
...snip...

=== process container (works fine)
$ /usr/lib/csih/getVolInfo.exe /cygdrive/c
Device Type        : 7
Characteristics    : 20
Volume Name        : <>
Serial Number      : 3456656850
Max Filenamelength : 255
Filesystemname     : <NTFS>
Flags              : 1c706df
...snip...

$ /usr/lib/csih/getVolInfo.exe /cygdrive/c/opt
Device Type        : 7
Characteristics    : 20
Volume Name        : <>
Serial Number      : 955187689
Max Filenamelength : 255
Filesystemname     : <NTFS>
Flags              : 3e706ff
...snip...

=== host
Microsoft Windows [Version 10.0.20348.1547]
(c) Microsoft Corporation. All rights reserved.

C:\Users\Administrator>\cygpkgs\getVolInfo.exe c:\
Device Type        : 7
Characteristics    : 20020
Volume Name        : <>
Serial Number      : 955187689
Max Filenamelength : 255
Filesystemname     : <NTFS>
Flags              : 3e706ff
   FILE_CASE_SENSITIVE_SEARCH  : TRUE
   FILE_CASE_PRESERVED_NAMES   : TRUE
   FILE_UNICODE_ON_DISK        : TRUE
   FILE_PERSISTENT_ACLS        : TRUE
   FILE_FILE_COMPRESSION       : TRUE
   FILE_VOLUME_QUOTAS          : TRUE
   FILE_SUPPORTS_SPARSE_FILES  : TRUE
   FILE_SUPPORTS_REPARSE_POINTS: TRUE
   FILE_SUPPORTS_REMOTE_STORAGE: FALSE
   FILE_VOLUME_IS_COMPRESSED   : FALSE
   FILE_SUPPORTS_OBJECT_IDS    : TRUE
   FILE_SUPPORTS_ENCRYPTION    : TRUE
   FILE_NAMED_STREAMS          : TRUE
   FILE_READ_ONLY_VOLUME       : FALSE
   FILE_SEQUENTIAL_WRITE_ONCE  : FALSE
   FILE_SUPPORTS_TRANSACTIONS  : TRUE

C:\Users\Administrator>fsutil fsinfo volumeinfo c:
Volume Name :
Volume Serial Number : 0x38ef01e9
Max Component Length : 255
File System Name : NTFS
Is ReadWrite
Not Thinly-Provisioned
Supports Case-sensitive filenames
Preserves Case of filenames
Supports Unicode in filenames
Preserves & Enforces ACL's
Supports file-based Compression
Supports Disk Quotas
Supports Sparse files
Supports Reparse Points
Returns Handle Close Result Information
Supports POSIX-style Unlink and Rename
Supports Object Identifiers
Supports Encrypted File System
Supports Named Streams
Supports Transactions
Supports Hard Links
Supports Extended Attributes
Supports Open By FileID
Supports USN Journal

-- 
Yoshinao Muramatsu <ysno@ac.auone-net.jp>