rafs: cleanup raw os error with error macros

With error macros, debug mode can show more informations such as stack trace.

Signed-off-by: Liu Bo <liub.liubo@gmail.com>

Author: liubogithub

api/src/error.rs

@@ -63,6 +63,8 @@ define_libc_error_macro!(ealready, EALREADY);
 define_libc_error_macro!(enosys, ENOSYS);
 define_libc_error_macro!(epipe, EPIPE);
 define_libc_error_macro!(eio, EIO);
+define_libc_error_macro!(erange, ERANGE);
+define_libc_error_macro!(enodata, ENODATA);
 
 /// Return EINVAL error with formatted error message.
 #[macro_export]

rafs/src/fs.rs

@@ -735,7 +735,7 @@ impl FileSystem for Rafs {
         let mut recorder = FopRecorder::settle(Getxattr, inode, &self.ios);
 
         if !self.xattr_supported() {
-            return Err(std::io::Error::from_raw_os_error(libc::ENOSYS));
+            return Err(enosys!());
         }
 
         let name = OsStr::from_bytes(name.to_bytes());
@@ -744,15 +744,15 @@ impl FileSystem for Rafs {
         let r = match value {
             Some(value) => match size {
                 0 => Ok(GetxattrReply::Count((value.len() + 1) as u32)),
-                x if x < value.len() as u32 => Err(std::io::Error::from_raw_os_error(libc::ERANGE)),
+                x if x < value.len() as u32 => Err(erange!()),
                 _ => Ok(GetxattrReply::Value(value)),
             },
             None => {
                 // TODO: Hopefully, we can have a 'decorator' procedure macro in
                 // the future to wrap this method thus to handle different reasonable
                 // errors in a clean way.
                 recorder.mark_success(0);
-                Err(std::io::Error::from_raw_os_error(libc::ENODATA))
+                Err(enodata!())
             }
         };
 
@@ -765,7 +765,7 @@ impl FileSystem for Rafs {
     fn listxattr(&self, _ctx: &Context, inode: u64, size: u32) -> Result<ListxattrReply> {
         let mut rec = FopRecorder::settle(Listxattr, inode, &self.ios);
         if !self.xattr_supported() {
-            return Err(std::io::Error::from_raw_os_error(libc::ENOSYS));
+            return Err(enosys!());
         }
 
         let inode = self.sb.get_inode(inode, false)?;
@@ -783,7 +783,7 @@ impl FileSystem for Rafs {
 
         match size {
             0 => Ok(ListxattrReply::Count(count as u32)),
-            x if x < count as u32 => Err(std::io::Error::from_raw_os_error(libc::ERANGE)),
+            x if x < count as u32 => Err(erange!()),
             _ => Ok(ListxattrReply::Names(buf)),
         }
     }

rafs/src/metadata/direct_v6.rs

@@ -237,7 +237,7 @@ impl DirectSuperBlockV6 {
         let block_size = state.block_size();
         let unit_size = size_of::<RafsV5ChunkInfo>();
         if size % unit_size != 0 {
-            return Err(std::io::Error::from_raw_os_error(libc::EINVAL));
+            return Err(einval!());
         }
 
         for idx in 0..(size / unit_size) {