zdb patch
Pawel Jakub Dawidek
pjd at FreeBSD.org
Sat Apr 14 13:47:16 UTC 2007
On Sat, Apr 14, 2007 at 10:31:19PM +0900, Hidetoshi Shimokawa wrote:
> Here is a patch for zdb.
> Now we can 'zdb -u pool' :-)
>
> ==== //depot/user/simokawa/firewire/contrib/opensolaris/lib/libzpool/common/kernel.c#2 - /home/p4/firewire/contrib/opensolaris/lib/libzpool/common/kernel.c ====
> @@ -38,6 +38,7 @@
> #include <sys/zfs_context.h>
> #include <sys/zmod.h>
> #include <sys/utsname.h>
> +#include <sys/disk.h>
>
> /*
> * Emulation of kernel services in userland.
> @@ -291,7 +292,11 @@
>
> ASSERT(abstime > 0);
> top:
> +#if 0
> delta = abstime - lbolt;
> +#else /* for compatibility with FreeBSD kernel */
> + delta = abstime;
> +#endif
What is this change for? It works ok without it here.
> if (delta <= 0)
> return (-1);
>
> @@ -413,7 +418,11 @@
> *vpp = vp = umem_zalloc(sizeof (vnode_t), UMEM_NOFAIL);
>
> vp->v_fd = fd;
> - vp->v_size = st.st_size;
> + if (st.st_mode & S_IFCHR) {
> + ioctl(fd, DIOCGMEDIASIZE, &vp->v_size);
> + } else {
> + vp->v_size = st.st_size;
> + }
> vp->v_path = spa_strdup(path);
>
> return (0);
Please commit, but:
1. Move 'include <sys/disk.h>' to lib/libzpool/common/sys/zfs_context.h
2. Use S_ISCHR() macro and drop { }:
if (S_ISCHR(st.st_mode))
ioctl(fd, DIOCGMEDIASIZE, &vp->v_size);
else
vp->v_size = st.st_size;
Thanks!
--
Pawel Jakub Dawidek http://www.wheel.pl
pjd at FreeBSD.org http://www.FreeBSD.org
FreeBSD committer Am I Evil? Yes, I Am!
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/freebsd-current/attachments/20070414/6b1e1c7d/attachment.pgp
More information about the freebsd-current
mailing list