Skip to content

Commit 98f679f

Browse files
dbortfacebook-github-bot
authored andcommitted
Reduce log level of mlock failure in mmap_data_loader (#2498)
Summary: Pull Request resolved: #2498 Some large models with a large number of segments will fail this check, spamming the log with this message every time. Drop the level to Debug so we can still see it if interested, but to keep it silenced by default (Info is the default log level). The downside of this is that people may not know that their memory isn't being locked. But in those situations, they may not be looking at the logs anyway. Reviewed By: mikekgfb Differential Revision: D55045041 fbshipit-source-id: d93e4e9b38db0ef84947d66dedfb826dafd3b10f
1 parent 4c9e3ea commit 98f679f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

extension/data_loader/mmap_data_loader.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,7 @@ Result<FreeableBuffer> MmapDataLoader::Load(size_t offset, size_t size) {
201201
if (err < 0) {
202202
if (mlock_config_ == MlockConfig::UseMlockIgnoreErrors) {
203203
ET_LOG(
204-
Info,
204+
Debug,
205205
"Ignoring mlock error for file %s (off=0x%zd): "
206206
"mlock(%p, %zu) failed: %s (%d)",
207207
file_name_,

0 commit comments

Comments
 (0)