[PR #1692] [MERGED] Updated to clearly output message about the file descriptor type(pseudo/physical) #2120

New issue

Closed

opened 2026-03-04 02:03:48 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 02:03:48 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/1692
Author: @ggtakec
Created: 6/20/2021
Status: ✅ Merged
Merged: 6/25/2021
Merged by: @gaul

Base: master ← Head: update_prn_msgs

📝 Commits (1)

bff9cdb Updated to clearly output message about the file descriptor type(pseudo/physical)

📊 Changes

5 files changed (+30 additions, -30 deletions)

View changed files

📝 src/fdcache.cpp (+2 -2)
📝 src/fdcache_auto.cpp (+1 -1)
📝 src/fdcache_entity.cpp (+16 -16)
📝 src/fdcache_page.cpp (+3 -3)
📝 src/s3fs.cpp (+8 -8)

📄 Description

Relevant Issue (if applicable)

#1666

Details

With the fix of #1666, the file descriptor returned from s3fs to FUSE has been changed from physical fd to pseudo fd.
Changed to clearly output the type of file descriptor included in the (debug)message output of s3fs.
This is more useful when debugging and when investigating bugs.
This is a message-only update, not a logic change.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/1692 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 6/20/2021 **Status:** ✅ Merged **Merged:** 6/25/2021 **Merged by:** [@gaul](https://github.com/gaul) **Base:** `master` ← **Head:** `update_prn_msgs` --- ### 📝 Commits (1) - [`bff9cdb`](https://github.com/s3fs-fuse/s3fs-fuse/commit/bff9cdb97cc645fee29fbe98591ae6b42b450105) Updated to clearly output message about the file descriptor type(pseudo/physical) ### 📊 Changes **5 files changed** (+30 additions, -30 deletions) <details> <summary>View changed files</summary> 📝 `src/fdcache.cpp` (+2 -2) 📝 `src/fdcache_auto.cpp` (+1 -1) 📝 `src/fdcache_entity.cpp` (+16 -16) 📝 `src/fdcache_page.cpp` (+3 -3) 📝 `src/s3fs.cpp` (+8 -8) </details> ### 📄 Description ### Relevant Issue (if applicable) #1666 ### Details With the fix of #1666, the file descriptor returned from s3fs to FUSE has been changed from physical fd to pseudo fd. Changed to clearly output the type of file descriptor included in the (debug)message output of s3fs. This is more useful when debugging and when investigating bugs. This is a message-only update, not a logic change. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>