Type inconsistent between struct dirent's inum(ushort) and its usage(uint).

[zhouzilong2020]

inum defined inside struct dirent is a ushort, suggesting the dirent can only reference up to 4 inode on disk.

xv6-riscv/kernel/fs.h

Lines 56 to 59 in f5b93ef

	struct dirent {
	ushort inum;
	char name[DIRSIZ];
	};

However, in function dirlink(), it tries to set inum with a unit.

xv6-riscv/kernel/fs.c

Line 601 in f5b93ef

de.inum = inum;

I further checked all the usage of the user space function link(), and it turns out none tried to open a file via its linked name.
Is this a delicate design or a bug?

[zhouzilong2020]

Since each dinode has 64 bytes right now, and there are 13 blocks for inode on disk, xv6 chould at most have 208 inode.

To make the data type of inum consistent, I suggest:

1. Changing struct dirent as follows:

#define DIRSIZ 12
struct dirent {
  uint inum;
  char name[DIRSIZ];
};

2. Changing inum to ushort everywhere, which includes much more modifications to the code.