svn commit: r226552 - head/sys/boot/zfs
Pawel Jakub Dawidek
pjd at FreeBSD.org
Thu Oct 20 09:36:37 UTC 2011
On Thu, Oct 20, 2011 at 10:15:09AM +0300, Andriy Gapon wrote:
> I believe that this commit is incorrect.
> It does appear to do the right thing for raidz, but I believe that it breaks the
> simple case when v_read is vdev_read_phys. Now because the bp argument is not
> NULL, the latter will try to read BP_GET_PSIZE(bp) bytes instead of
> SPA_GANGBLOCKSIZE bytes.
>
> I believe that my patch that I shared (much) earlier handles this issue
> correctly and integrally:
> http://article.gmane.org/gmane.os.freebsd.devel.file-systems/13130
> Did you find anything wrong with it?
> If not, then maybe we can get back to discussing it?
You see. My memory is very short:) and I remembered that you were
working on gang blocks in ZFS boot code, but could find the commit, so I
went ahead with my changes. Those changes still didn't fix one problem
for me. Your patch seems to be much more complete and correct and it
fixes all my problems.
Please, wait no longer and commit your patch! We have to have this in 9.0.
--
Pawel Jakub Dawidek http://www.wheelsystems.com
FreeBSD committer http://www.FreeBSD.org
Am I Evil? Yes, I Am! http://yomoli.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 196 bytes
Desc: not available
Url : http://lists.freebsd.org/pipermail/svn-src-all/attachments/20111020/d7097477/attachment.pgp
More information about the svn-src-all
mailing list