summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndrew Walbran <qwandor@google.com>2020-10-22 21:53:49 +0000
committerAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>2020-10-22 21:53:49 +0000
commit02af803f2c2c71529fcb3d09142180bb30ec5f0a (patch)
tree63867ac016a12c0630b429023740c34460a3d58e
parent082a1c3c35b054d7ee9b92186a64a7c9a21c1707 (diff)
parent036c8e1562b0f51b36474c09ed6e57c3dcd18f6c (diff)
downloadp9-02af803f2c2c71529fcb3d09142180bb30ec5f0a.tar.gz
p9: Use std::os::unix rather than std::os::linux. am: 1025034f05 am: 036c8e1562
Original change: https://android-review.googlesource.com/c/platform/external/vm_tools/p9/+/1468199 Change-Id: I00db091d20fb55308fbd9ad9fe20cc027916aae8
-rw-r--r--src/server/mod.rs34
-rw-r--r--src/server/tests.rs44
2 files changed, 39 insertions, 39 deletions
diff --git a/src/server/mod.rs b/src/server/mod.rs
index c25cb3b..6a4a5c1 100644
--- a/src/server/mod.rs
+++ b/src/server/mod.rs
@@ -8,7 +8,7 @@ use std::ffi::CString;
use std::fs;
use std::io::{self, Cursor, Read, Write};
use std::mem;
-use std::os::linux::fs::MetadataExt;
+use std::os::unix::fs::MetadataExt;
use std::os::unix::fs::{DirBuilderExt, FileExt, OpenOptionsExt};
use std::os::unix::io::AsRawFd;
use std::path::{Component, Path, PathBuf};
@@ -126,8 +126,8 @@ fn metadata_to_qid(metadata: &fs::Metadata) -> Qid {
Qid {
ty,
// TODO: deal with the 2038 problem before 2038
- version: metadata.st_mtime() as u32,
- path: metadata.st_ino(),
+ version: metadata.mtime() as u32,
+ path: metadata.ino(),
}
}
@@ -634,20 +634,20 @@ impl Server {
Ok(Rgetattr {
valid: P9_GETATTR_BASIC,
qid: metadata_to_qid(&fid.metadata),
- mode: fid.metadata.st_mode(),
- uid: map_id_from_host(&self.uid_map, fid.metadata.st_uid()),
- gid: map_id_from_host(&self.gid_map, fid.metadata.st_gid()),
- nlink: fid.metadata.st_nlink(),
- rdev: fid.metadata.st_rdev(),
- size: fid.metadata.st_size(),
- blksize: fid.metadata.st_blksize(),
- blocks: fid.metadata.st_blocks(),
- atime_sec: fid.metadata.st_atime() as u64,
- atime_nsec: fid.metadata.st_atime_nsec() as u64,
- mtime_sec: fid.metadata.st_mtime() as u64,
- mtime_nsec: fid.metadata.st_mtime_nsec() as u64,
- ctime_sec: fid.metadata.st_ctime() as u64,
- ctime_nsec: fid.metadata.st_ctime_nsec() as u64,
+ mode: fid.metadata.mode(),
+ uid: map_id_from_host(&self.uid_map, fid.metadata.uid()),
+ gid: map_id_from_host(&self.gid_map, fid.metadata.gid()),
+ nlink: fid.metadata.nlink(),
+ rdev: fid.metadata.rdev(),
+ size: fid.metadata.size(),
+ blksize: fid.metadata.blksize(),
+ blocks: fid.metadata.blocks(),
+ atime_sec: fid.metadata.atime() as u64,
+ atime_nsec: fid.metadata.atime_nsec() as u64,
+ mtime_sec: fid.metadata.mtime() as u64,
+ mtime_nsec: fid.metadata.mtime_nsec() as u64,
+ ctime_sec: fid.metadata.ctime() as u64,
+ ctime_nsec: fid.metadata.ctime_nsec() as u64,
btime_sec: 0,
btime_nsec: 0,
gen: 0,
diff --git a/src/server/tests.rs b/src/server/tests.rs
index fa161e9..cc000b6 100644
--- a/src/server/tests.rs
+++ b/src/server/tests.rs
@@ -12,7 +12,7 @@ use std::fs::File;
use std::io::{self, Cursor};
use std::mem;
use std::ops::Deref;
-use std::os::linux::fs::MetadataExt;
+use std::os::unix::fs::MetadataExt;
use std::os::unix::ffi::OsStringExt;
use std::path::{Path, PathBuf};
use std::u32;
@@ -112,8 +112,8 @@ fn check_qid(qid: &Qid, md: &fs::Metadata) {
panic!("unknown file type: {:?}", md.file_type());
};
assert_eq!(qid.ty, ty);
- assert_eq!(qid.version, md.st_mtime() as u32);
- assert_eq!(qid.path, md.st_ino());
+ assert_eq!(qid.version, md.mtime() as u32);
+ assert_eq!(qid.path, md.ino());
}
fn check_attr(server: &mut Server, fid: u32, md: &fs::Metadata) {
@@ -135,20 +135,20 @@ fn check_attr(server: &mut Server, fid: u32, md: &fs::Metadata) {
};
assert_eq!(rgetattr.valid, P9_GETATTR_BASIC);
assert_eq!(rgetattr.qid.ty, ty);
- assert_eq!(rgetattr.qid.version, md.st_mtime() as u32);
- assert_eq!(rgetattr.qid.path, md.st_ino());
- assert_eq!(rgetattr.mode, md.st_mode());
- assert_eq!(rgetattr.uid, md.st_uid());
- assert_eq!(rgetattr.gid, md.st_gid());
- assert_eq!(rgetattr.nlink, md.st_nlink());
- assert_eq!(rgetattr.rdev, md.st_rdev());
- assert_eq!(rgetattr.size, md.st_size());
- assert_eq!(rgetattr.atime_sec, md.st_atime() as u64);
- assert_eq!(rgetattr.atime_nsec, md.st_atime_nsec() as u64);
- assert_eq!(rgetattr.mtime_sec, md.st_mtime() as u64);
- assert_eq!(rgetattr.mtime_nsec, md.st_mtime_nsec() as u64);
- assert_eq!(rgetattr.ctime_sec, md.st_ctime() as u64);
- assert_eq!(rgetattr.ctime_nsec, md.st_ctime_nsec() as u64);
+ assert_eq!(rgetattr.qid.version, md.mtime() as u32);
+ assert_eq!(rgetattr.qid.path, md.ino());
+ assert_eq!(rgetattr.mode, md.mode());
+ assert_eq!(rgetattr.uid, md.uid());
+ assert_eq!(rgetattr.gid, md.gid());
+ assert_eq!(rgetattr.nlink, md.nlink());
+ assert_eq!(rgetattr.rdev, md.rdev());
+ assert_eq!(rgetattr.size, md.size());
+ assert_eq!(rgetattr.atime_sec, md.atime() as u64);
+ assert_eq!(rgetattr.atime_nsec, md.atime_nsec() as u64);
+ assert_eq!(rgetattr.mtime_sec, md.mtime() as u64);
+ assert_eq!(rgetattr.mtime_nsec, md.mtime_nsec() as u64);
+ assert_eq!(rgetattr.ctime_sec, md.ctime() as u64);
+ assert_eq!(rgetattr.ctime_nsec, md.ctime_nsec() as u64);
assert_eq!(rgetattr.btime_sec, 0);
assert_eq!(rgetattr.btime_nsec, 0);
assert_eq!(rgetattr.gen, 0);
@@ -588,7 +588,7 @@ fn set_len() {
})
.expect("failed to run set length of file");
- assert_eq!(md.st_size(), len);
+ assert_eq!(md.size(), len);
}
#[test]
@@ -637,8 +637,8 @@ fn set_mtime() {
})
.expect("failed to set mtime");
- assert_eq!(md.st_mtime() as u64, secs);
- assert_eq!(md.st_mtime_nsec() as u64, nanos);
+ assert_eq!(md.mtime() as u64, secs);
+ assert_eq!(md.mtime_nsec() as u64, nanos);
}
#[test]
@@ -651,8 +651,8 @@ fn set_atime() {
})
.expect("failed to set atime");
- assert_eq!(md.st_atime() as u64, secs);
- assert_eq!(md.st_atime_nsec() as u64, nanos);
+ assert_eq!(md.atime() as u64, secs);
+ assert_eq!(md.atime_nsec() as u64, nanos);
}
#[test]