debugfs: use the functions in misc/create_inode.c
authorRobert Yang <liezhi.yang@windriver.com>
Thu, 6 Mar 2014 16:00:25 +0000 (11:00 -0500)
committerChanho Park <chanho61.park@samsung.com>
Tue, 5 Aug 2014 04:46:26 +0000 (13:46 +0900)
* Use the functions in misc/create_inode.c, and remove the duplicated
  code.

* The CREATE_INODE_DEPS in the debugfs/Makefile.in is used for recording
  create_inode.o's depends which is from misc/Makefile.in, we have to
  recompile create_inode.o because we need it to print more messages when
  it is used by debugfs, just like we recompile e2freefrag.o, but it seems
  that the e2freefrag.o's depends in debugfs/Makefile.in is incorrect, it
  would not rebuild when its depends (e.g.: lib/config.h) is changed,
  which would cause unexpected errors. Make duplicated code in
  debugfs/Makefile.in and misc/Makefile.in is not a good idea, maybe we'd
  better define CREATE_INODE_DEPS in the top Makefile, I'd like to send
  another patch and fix the e2freefrag if you are fine with it.

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
Reviewed-by: Darren Hart <dvhart@linux.intel.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
debugfs/Makefile.in
debugfs/debugfs.c
debugfs/debugfs.h

index 3dd06f0..0628587 100644 (file)
@@ -18,7 +18,7 @@ MK_CMDS=      _SS_DIR_OVERRIDE=../lib/ss ../lib/ss/mk_cmds
 
 DEBUG_OBJS= debug_cmds.o debugfs.o util.o ncheck.o icheck.o ls.o \
        lsdel.o dump.o set_fields.o logdump.o htree.o unused.o e2freefrag.o \
-       filefrag.o extent_cmds.o extent_inode.o zap.o quota.o
+       filefrag.o extent_cmds.o extent_inode.o zap.o quota.o create_inode.o
 
 RO_DEBUG_OBJS= ro_debug_cmds.o ro_debugfs.o util.o ncheck.o icheck.o ls.o \
        lsdel.o logdump.o htree.o e2freefrag.o filefrag.o extent_cmds.o \
@@ -29,8 +29,13 @@ SRCS= debug_cmds.c $(srcdir)/debugfs.c $(srcdir)/util.c $(srcdir)/ls.c \
        $(srcdir)/dump.c $(srcdir)/set_fields.c ${srcdir}/logdump.c \
        $(srcdir)/htree.c $(srcdir)/unused.c ${srcdir}/../misc/e2freefrag.c \
        $(srcdir)/filefrag.c $(srcdir)/extent_inode.c $(srcdir)/zap.c \
-       $(srcdir)/quota.c
+       $(srcdir)/../misc/create_inode.c $(srcdir)/quota.c
 
+CREATE_INODE_DEPS= $(srcdir)/../misc/create_inode.h \
+       $(srcdir)/../misc/create_inode.c $(top_builddir)/lib/config.h \
+       $(srcdir)/../lib/ext2fs/ext2fs.h $(srcdir)/../lib/et/com_err.h \
+       $(srcdir)/../lib/e2p/e2p.h $(srcdir)/../misc/nls-enable.h
 LIBS= $(LIBQUOTA) $(LIBEXT2FS) $(LIBE2P) $(LIBSS) $(LIBCOM_ERR) $(LIBBLKID) \
        $(LIBUUID) $(SYSLIBS)
 DEPLIBS= $(DEPLIBQUOTA) $(LIBEXT2FS) $(LIBE2P) $(DEPLIBSS) $(DEPLIBCOM_ERR) \
@@ -83,6 +88,11 @@ e2freefrag.o: $(srcdir)/../misc/e2freefrag.c
        $(E) "  CC $@"
        $(Q) $(CC) -c $(ALL_CFLAGS) -I$(srcdir) $< -DDEBUGFS -o $@
 
+create_inode.o: $(CREATE_INODE_DEPS)
+       $(E) "  CC $@"
+       $(Q) $(CC) -c $(ALL_CFLAGS) -I$(srcdir) \
+                $(srcdir)/../misc/create_inode.c -DDEBUGFS -o $@
+
 debugfs.8: $(DEP_SUBSTITUTE) $(srcdir)/debugfs.8.in
        $(E) "  SUBST $@"
        $(Q) $(SUBSTITUTE_UPTIME) $(srcdir)/debugfs.8.in debugfs.8
@@ -146,7 +156,7 @@ debugfs.o: $(srcdir)/debugfs.c $(top_builddir)/lib/config.h \
  $(top_srcdir)/lib/quota/quotaio_tree.h $(top_srcdir)/lib/../e2fsck/dict.h \
  $(top_srcdir)/lib/e2p/e2p.h $(top_srcdir)/version.h $(srcdir)/jfs_user.h \
  $(top_srcdir)/lib/ext2fs/kernel-jbd.h $(top_srcdir)/lib/ext2fs/jfs_compat.h \
- $(top_srcdir)/lib/ext2fs/kernel-list.h
+ $(top_srcdir)/lib/ext2fs/kernel-list.h $(top_srcdir)/misc/util.h
 util.o: $(srcdir)/util.c $(top_builddir)/lib/config.h \
  $(top_builddir)/lib/dirpaths.h $(top_srcdir)/lib/ss/ss.h \
  $(top_builddir)/lib/ss/ss_err.h $(top_srcdir)/lib/et/com_err.h \
index 1703aba..040d6ac 100644 (file)
@@ -25,8 +25,6 @@ extern char *optarg;
 #include <errno.h>
 #endif
 #include <fcntl.h>
-#include <sys/types.h>
-#include <sys/stat.h>
 
 #include "debugfs.h"
 #include "uuid/uuid.h"
@@ -41,23 +39,12 @@ extern char *optarg;
 #define BUFSIZ 8192
 #endif
 
-/* 64KiB is the minimium blksize to best minimize system call overhead. */
-#ifndef IO_BUFSIZE
-#define IO_BUFSIZE 64*1024
-#endif
-
-/* Block size for `st_blocks' */
-#ifndef S_BLKSIZE
-#define S_BLKSIZE 512
-#endif
-
 ss_request_table *extra_cmds;
 const char *debug_prog_name;
 int sci_idx;
 
-ext2_filsys    current_fs = NULL;
 quota_ctx_t    current_qctx;
-ext2_ino_t     root, cwd;
+ext2_ino_t     cwd;
 
 static void open_filesystem(char *device, int open_flags, blk64_t superblock,
                            blk64_t blocksize, int catastrophic,
@@ -1576,181 +1563,16 @@ void do_find_free_inode(int argc, char *argv[])
 }
 
 #ifndef READ_ONLY
-static errcode_t copy_file(int fd, ext2_ino_t newfile, int bufsize,
-                          int make_holes)
-{
-       ext2_file_t     e2_file;
-       errcode_t       retval, close_ret;
-       int             got;
-       unsigned int    written;
-       char            *buf;
-       char            *ptr;
-       char            *zero_buf;
-       int             cmp;
-
-       retval = ext2fs_file_open(current_fs, newfile,
-                                 EXT2_FILE_WRITE, &e2_file);
-       if (retval)
-               return retval;
-
-       retval = ext2fs_get_mem(bufsize, &buf);
-       if (retval) {
-               com_err("copy_file", retval, "can't allocate buffer\n");
-               goto out_close;
-       }
-
-       /* This is used for checking whether the whole block is zero */
-       retval = ext2fs_get_memzero(bufsize, &zero_buf);
-       if (retval) {
-               com_err("copy_file", retval, "can't allocate zero buffer\n");
-               goto out_free_buf;
-       }
-
-       while (1) {
-               got = read(fd, buf, bufsize);
-               if (got == 0)
-                       break;
-               if (got < 0) {
-                       retval = errno;
-                       goto fail;
-               }
-               ptr = buf;
-
-               /* Sparse copy */
-               if (make_holes) {
-                       /* Check whether all is zero */
-                       cmp = memcmp(ptr, zero_buf, got);
-                       if (cmp == 0) {
-                                /* The whole block is zero, make a hole */
-                               retval = ext2fs_file_lseek(e2_file, got,
-                                                          EXT2_SEEK_CUR, NULL);
-                               if (retval)
-                                       goto fail;
-                               got = 0;
-                       }
-               }
-
-               /* Normal copy */
-               while (got > 0) {
-                       retval = ext2fs_file_write(e2_file, ptr,
-                                                  got, &written);
-                       if (retval)
-                               goto fail;
-
-                       got -= written;
-                       ptr += written;
-               }
-       }
-
-fail:
-       ext2fs_free_mem(&zero_buf);
-out_free_buf:
-       ext2fs_free_mem(&buf);
-out_close:
-       close_ret = ext2fs_file_close(e2_file);
-       if (retval == 0)
-               retval = close_ret;
-       return retval;
-}
-
-
 void do_write(int argc, char *argv[])
 {
-       int             fd;
-       struct stat     statbuf;
-       ext2_ino_t      newfile;
        errcode_t       retval;
-       struct ext2_inode inode;
-       int             bufsize = IO_BUFSIZE;
-       int             make_holes = 0;
 
        if (common_args_process(argc, argv, 3, 3, "write",
                                "<native file> <new file>", CHECK_FS_RW))
                return;
 
-       fd = open(argv[1], O_RDONLY);
-       if (fd < 0) {
-               com_err(argv[1], errno, 0);
-               return;
-       }
-       if (fstat(fd, &statbuf) < 0) {
-               com_err(argv[1], errno, 0);
-               close(fd);
-               return;
-       }
-
-       retval = ext2fs_namei(current_fs, root, cwd, argv[2], &newfile);
-       if (retval == 0) {
-               com_err(argv[0], 0, "The file '%s' already exists\n", argv[2]);
-               close(fd);
-               return;
-       }
-
-       retval = ext2fs_new_inode(current_fs, cwd, 010755, 0, &newfile);
-       if (retval) {
-               com_err(argv[0], retval, 0);
-               close(fd);
-               return;
-       }
-       printf("Allocated inode: %u\n", newfile);
-       retval = ext2fs_link(current_fs, cwd, argv[2], newfile,
-                            EXT2_FT_REG_FILE);
-       if (retval == EXT2_ET_DIR_NO_SPACE) {
-               retval = ext2fs_expand_dir(current_fs, cwd);
-               if (retval) {
-                       com_err(argv[0], retval, "while expanding directory");
-                       close(fd);
-                       return;
-               }
-               retval = ext2fs_link(current_fs, cwd, argv[2], newfile,
-                                    EXT2_FT_REG_FILE);
-       }
-       if (retval) {
-               com_err(argv[2], retval, 0);
-               close(fd);
-               return;
-       }
-        if (ext2fs_test_inode_bitmap2(current_fs->inode_map,newfile))
-               com_err(argv[0], 0, "Warning: inode already set");
-       ext2fs_inode_alloc_stats2(current_fs, newfile, +1, 0);
-       memset(&inode, 0, sizeof(inode));
-       inode.i_mode = (statbuf.st_mode & ~LINUX_S_IFMT) | LINUX_S_IFREG;
-       inode.i_atime = inode.i_ctime = inode.i_mtime =
-               current_fs->now ? current_fs->now : time(0);
-       inode.i_links_count = 1;
-       inode.i_size = statbuf.st_size;
-       if (current_fs->super->s_feature_incompat &
-           EXT3_FEATURE_INCOMPAT_EXTENTS) {
-               int i;
-               struct ext3_extent_header *eh;
-
-               eh = (struct ext3_extent_header *) &inode.i_block[0];
-               eh->eh_depth = 0;
-               eh->eh_entries = 0;
-               eh->eh_magic = ext2fs_cpu_to_le16(EXT3_EXT_MAGIC);
-               i = (sizeof(inode.i_block) - sizeof(*eh)) /
-                       sizeof(struct ext3_extent);
-               eh->eh_max = ext2fs_cpu_to_le16(i);
-               inode.i_flags |= EXT4_EXTENTS_FL;
-       }
-       if (debugfs_write_new_inode(newfile, &inode, argv[0])) {
-               close(fd);
-               return;
-       }
-       if (LINUX_S_ISREG(inode.i_mode)) {
-               if (statbuf.st_blocks < statbuf.st_size / S_BLKSIZE) {
-                       make_holes = 1;
-                       /*
-                        * Use I/O blocksize as buffer size when
-                        * copying sparse files.
-                        */
-                       bufsize = statbuf.st_blksize;
-               }
-               retval = copy_file(fd, newfile, bufsize, make_holes);
-               if (retval)
-                       com_err("copy_file", retval, 0);
-       }
-       close(fd);
+       if ((retval = do_write_internal(cwd, argv[1], argv[2])))
+               com_err(argv[0], retval, 0);
 }
 
 void do_mknod(int argc, char *argv[])
@@ -1760,6 +1582,7 @@ void do_mknod(int argc, char *argv[])
        errcode_t       retval;
        struct ext2_inode inode;
        int             filetype, nr;
+       struct stat     st;
 
        if (check_fs_open(argv[0]))
                return;
@@ -1768,115 +1591,50 @@ void do_mknod(int argc, char *argv[])
                com_err(argv[0], 0, "Usage: mknod <name> [p| [c|b] <major> <minor>]");
                return;
        }
+
        mode = minor = major = 0;
        switch (argv[2][0]) {
                case 'p':
-                       mode = LINUX_S_IFIFO;
-                       filetype = EXT2_FT_FIFO;
+                       st.st_mode = S_IFIFO;
                        nr = 3;
                        break;
                case 'c':
-                       mode = LINUX_S_IFCHR;
-                       filetype = EXT2_FT_CHRDEV;
+                       st.st_mode = S_IFCHR;
                        nr = 5;
                        break;
                case 'b':
-                       mode = LINUX_S_IFBLK;
-                       filetype = EXT2_FT_BLKDEV;
+                       st.st_mode = S_IFBLK;
                        nr = 5;
                        break;
                default:
-                       filetype = 0;
                        nr = 0;
        }
+
        if (nr == 5) {
                major = strtoul(argv[3], argv+3, 0);
                minor = strtoul(argv[4], argv+4, 0);
                if (major > 65535 || minor > 65535 || argv[3][0] || argv[4][0])
                        nr = 0;
        }
+
        if (argc != nr)
                goto usage;
-       if (check_fs_read_write(argv[0]))
-               return;
-       retval = ext2fs_new_inode(current_fs, cwd, 010755, 0, &newfile);
-       if (retval) {
+
+       st.st_rdev = makedev(major, minor);
+       if ((retval = do_mknod_internal(cwd, argv[1], &st)))
                com_err(argv[0], retval, 0);
-               return;
-       }
-       printf("Allocated inode: %u\n", newfile);
-       retval = ext2fs_link(current_fs, cwd, argv[1], newfile, filetype);
-       if (retval == EXT2_ET_DIR_NO_SPACE) {
-               retval = ext2fs_expand_dir(current_fs, cwd);
-               if (retval) {
-                       com_err(argv[0], retval, "while expanding directory");
-                       return;
-               }
-               retval = ext2fs_link(current_fs, cwd, argv[1], newfile,
-                                    filetype);
-       }
-       if (retval) {
-               com_err(argv[1], retval, 0);
-               return;
-       }
-        if (ext2fs_test_inode_bitmap2(current_fs->inode_map,newfile))
-               com_err(argv[0], 0, "Warning: inode already set");
-       ext2fs_inode_alloc_stats2(current_fs, newfile, +1, 0);
-       memset(&inode, 0, sizeof(inode));
-       inode.i_mode = mode;
-       inode.i_atime = inode.i_ctime = inode.i_mtime =
-               current_fs->now ? current_fs->now : time(0);
-       if ((major < 256) && (minor < 256)) {
-               inode.i_block[0] = major*256+minor;
-               inode.i_block[1] = 0;
-       } else {
-               inode.i_block[0] = 0;
-               inode.i_block[1] = (minor & 0xff) | (major << 8) | ((minor & ~0xff) << 12);
-       }
-       inode.i_links_count = 1;
-       if (debugfs_write_new_inode(newfile, &inode, argv[0]))
-               return;
 }
 
 void do_mkdir(int argc, char *argv[])
 {
-       char    *cp;
-       ext2_ino_t      parent;
-       char    *name;
        errcode_t retval;
 
        if (common_args_process(argc, argv, 2, 2, "mkdir",
                                "<filename>", CHECK_FS_RW))
                return;
 
-       cp = strrchr(argv[1], '/');
-       if (cp) {
-               *cp = 0;
-               parent = string_to_inode(argv[1]);
-               if (!parent) {
-                       com_err(argv[1], ENOENT, 0);
-                       return;
-               }
-               name = cp+1;
-       } else {
-               parent = cwd;
-               name = argv[1];
-       }
-
-try_again:
-       retval = ext2fs_mkdir(current_fs, parent, 0, name);
-       if (retval == EXT2_ET_DIR_NO_SPACE) {
-               retval = ext2fs_expand_dir(current_fs, parent);
-               if (retval) {
-                       com_err(argv[0], retval, "while expanding directory");
-                       return;
-               }
-               goto try_again;
-       }
-       if (retval) {
-               com_err("ext2fs_mkdir", retval, 0);
-               return;
-       }
+       if ((retval = do_mkdir_internal(cwd, argv[1], NULL)))
+               com_err(argv[0], retval, 0);
 
 }
 
@@ -2271,44 +2029,14 @@ void do_punch(int argc, char *argv[])
 
 void do_symlink(int argc, char *argv[])
 {
-       char            *cp;
-       ext2_ino_t      parent;
-       char            *name, *target;
        errcode_t       retval;
 
        if (common_args_process(argc, argv, 3, 3, "symlink",
                                "<filename> <target>", CHECK_FS_RW))
                return;
 
-       cp = strrchr(argv[1], '/');
-       if (cp) {
-               *cp = 0;
-               parent = string_to_inode(argv[1]);
-               if (!parent) {
-                       com_err(argv[1], ENOENT, 0);
-                       return;
-               }
-               name = cp+1;
-       } else {
-               parent = cwd;
-               name = argv[1];
-       }
-       target = argv[2];
-
-try_again:
-       retval = ext2fs_symlink(current_fs, parent, 0, name, target);
-       if (retval == EXT2_ET_DIR_NO_SPACE) {
-               retval = ext2fs_expand_dir(current_fs, parent);
-               if (retval) {
-                       com_err(argv[0], retval, "while expanding directory");
-                       return;
-               }
-               goto try_again;
-       }
-       if (retval) {
-               com_err("ext2fs_symlink", retval, 0);
-               return;
-       }
+       if ((retval = do_symlink_internal(cwd, argv[1], argv[2])))
+               com_err(argv[0], retval, 0);
 
 }
 
index 9b67f69..dbc881c 100644 (file)
@@ -5,6 +5,7 @@
 #include "ss/ss.h"
 #include "ext2fs/ext2_fs.h"
 #include "ext2fs/ext2fs.h"
+#include "../misc/create_inode.h"
 #include "quota/quotaio.h"
 
 #ifdef __STDC__