Skip to content

Commit 7b85e11

Browse files
authored
[Comgr] Make VFS default
Currently this only affects device library linking
1 parent 781ae0c commit 7b85e11

File tree

4 files changed

+9
-10
lines changed

4 files changed

+9
-10
lines changed

amd/comgr/README.md

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -198,13 +198,11 @@ improving performance by reducing on-disk file I/O. Currently, VFS is only suppo
198198
for the device library link step, but we aim to progressively add support for
199199
more actions.
200200

201-
By default, VFS is turned off. Set the environment variable `AMD_COMGR_USE_VFS=1` to
202-
enable it.
201+
By default, VFS is turned on.
203202

204-
* `AMD_COMGR_USE_VFS`: When unset or "0", VFS support is turned off.
205-
* Users may use the API `amd_comgr_action_info_set_vfs` to enable VFS for individual actions
206-
without having to modify system-wide environment variables. Note that API is effective only
207-
if `AMD_COMGR_USE_VFS` is unset.
203+
* `AMD_COMGR_USE_VFS`: When set to "0", VFS support is turned off.
204+
* Users may use the API `amd_comgr_action_info_set_vfs` to disable VFS for individual actions
205+
without having to modify system-wide environment variables.
208206
* If `AMD_COMGR_SAVE_TEMPS` is set and not "0", VFS support is turned off irrespective
209207
of `AMD_COMGR_USE_VFS` or the use of `amd_comgr_action_info_set_vfs`.
210208

amd/comgr/docs/ReleaseNotes.md

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,9 @@ Bug Fixes
3838
New APIs
3939
--------
4040
- amd\_comgr\_info\_set\_vfs\_() (v3.1)
41-
- By setting this ActionInfo property, a user can explicitly dictate if
42-
device libraries should be linked using a Virtual File System (VFS).
41+
- By setting this ActionInfo property, users can explicitly dictate if
42+
device libraries should be linked using the real file system or a
43+
Virtual File System (VFS).
4344

4445
Deprecated APIs
4546
---------------

amd/comgr/src/comgr.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -204,7 +204,7 @@ struct DataAction {
204204
amd_comgr_language_t Language;
205205
bool Logging;
206206
bool ShouldLinkDeviceLibs = false;
207-
bool ShouldUseVFS = false;
207+
bool ShouldUseVFS = true;
208208

209209
std::vector<std::string> BundleEntryIDs;
210210

amd/comgr/test-lit/vfs-tests/vfs-tests.cl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@
2323
// COM: Verify success of compilation for all scenarios
2424
// STATUS: ReturnStatus: AMD_COMGR_STATUS_SUCCESS
2525

26-
// OUT-NA-NA-NA: File System: Real
26+
// OUT-NA-NA-NA: File System: VFS
2727
// OUT-NA-VFS-NOVFS: File System: VFS
2828
// OUT-NA-VFS-NA: File System: VFS
2929
// OUT-NA-NOVFS-VFS: File System: Real

0 commit comments

Comments
 (0)