opensubscriber
   Find in this group all groups
 
Unknown more information…

f : freebsd-hackers@freebsd.org 27 April 2012 • 5:14PM -0400

Re: [review request] zfsboot/zfsloader: support accessing filesystems within a pool
by Andriy Gapon

REPLY TO AUTHOR
 
REPLY TO GROUP




on 23/04/2012 09:23 Andrey V. Elsukov said the following:
> On 23.04.2012 1:21, Marius Strobl wrote:
>> I can't say much about these patches as a whole as they are rather big
>> and I'm not aware of all the details of ZFS. However, one bit that makes
>> the current implementation x86-specific is zfs_dev_init(). If you could
>> move it to the MD part in the course of these patches that would be
>> great. If you could also take the second patch in PR 165025 into account,
>> which I plan to commit once the issue with the current ofw_disk.c are
>> properly solved, that would be great.
>
> I also have some WIP related to moving partition table handling into MD
> part. You can look here: http://people.freebsd.org/~ae/sys_boot.diff

I like this patch.  OTOH, I couldn't help by wonder if it is possible to
somehow directly re-use the results of dv_init probing by other drivers (or
subset of the drivers, like only 'disk').  E.g. if instead of re-examining the
partition tables, we could ask a driver for a list of devices that it discovered.
But how to do that is not something that I can answer.

> This patch have one problem, there is no way to determine disk size and i'm
> thinking about adding ioctl(DIOCGMEDIASIZE) to the "disk" devsw.
>


--
Andriy Gapon
_______________________________________________
freebsd-hackers@free... mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscribe@free..."

Bookmark with:

Delicious   Digg   reddit   Facebook   StumbleUpon

Related Messages

opensubscriber is not affiliated with the authors of this message nor responsible for its content.