Date: Fri, 2 Apr 2010 15:12:31 +0000 (UTC) From: Andriy Gapon <avg@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/geom geom_vfs.c src/sys/kern vfs_bio.c Message-ID: <201004021513.o32FDmPn063387@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
avg 2010-04-02 15:12:31 UTC
FreeBSD src repository
Modified files:
sys/geom geom_vfs.c
sys/kern vfs_bio.c
Log:
SVN rev 206097 on 2010-04-02 15:12:31Z by avg
bo_bsize: revert r205860 and take an alternative approch in getblk
In r205860 I missed the fact that there is code that strongly assumes
that devvp bo_bsize is equal to underlying provider's sectorsize.
In those places it is hard to obtain the sectorsize in an alternative
way if devvp bo_bsize is set to something else.
So, I am reverting bo_bsize assigment in g_vfs_open.
Instead, in getblk I use DEV_BSIZE block size for b_offset calculation
if vp is a disk vp as reported by vn_isdisk. This should coinside with
vp being a devvp.
Reported by: Mykola Dzham <i@levsha.me>
Tested by: Mykola Dzham <i@levsha.me>
Pointyhat to: avg
MFC after: 2 weeks
X-ToDo: convert bread(devvp) in all fs to use bo_bsize-d blocks
Revision Changes Path
1.18 +1 -1 src/sys/geom/geom_vfs.c
1.569 +1 -1 src/sys/kern/vfs_bio.c
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201004021513.o32FDmPn063387>