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
pgpwCpxFh8RtM.pgp
Description: PGP signature
