On Fri, Feb 9, 2018 at 7:33 AM, Schweiss, Chip <c...@innovates.com> wrote:

> I've been following this review for a while.   Sorry, if this is not the
> correct place for this comment.
> Personally, I've been doing similar outside zpool import by doing a 'zpool
> import -N' then parallel executing 'zfs mount {folder}' followed by
> parallel executing 'zfs share {folder}'.  My zpool import times were over
> an hour and reduced to less than 2 minutes with parallel mounting and
> sharing.
> I'm sure this proposed parallel mount will outperform the multiple
> executions of 'zfs mount', but doesn't help with the subsequent sharing of
> folders. Currently, zpool import has '-N' to not mount folders.  However,
> with the addition of this parallel mount, it would be best if zpool import
> got a new feature to not share any folders upon importing.

​Could you elaborate a little more on why you think that feature should be
a prerequisite of landing this change?​

> Does this patch address unmount in a parallel fashion?
No. This only changes mount.

> Cheers!
> -Chip
> On Thu, Feb 8, 2018 at 2:02 PM, Prakash Surya <notificati...@github.com>
> wrote:
>> @prakashsurya <https://github.com/prakashsurya> pushed 1 commit.
>>    - c8a3e65 <https://github.com/openzfs/openzfs/commit/c8a3e65> Fix
>>    cstyle; use tab for indentation, not spaces
>> —
>> You are receiving this because you are subscribed to this thread.
>> View it on GitHub
>> <https://github.com/openzfs/openzfs/pull/536/files/3a83672ecf670c528f05209997189ad8c22ce836..c8a3e654b0940d3ba0914b397f2840f5e76167f1>
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/AOi0K_YrA1epjyYV82oWSyAVrn038EkKks5tS1JPgaJpZM4R73LC>
>> .
> *openzfs-developer* | Archives
> <https://openzfs.topicbox.com/groups/developer/discussions/T22334a01fda83bfd-M965c9648a227b358a076917e>
> | Powered by Topicbox <https://topicbox.com>

Powered by Topicbox: https://topicbox.com

Reply via email to