Yes.  It should be inherited.

For the property values, I'd prefer:

"standard" - we honor the standard (posix) semantics, without saying posix.

"always" - make every system call synchronous.

"barrier" - treat fsync() only as a write barrier, which ensures that all previous
writes/updates will be on stable storage before any subsequent updates.

Jeff

On Apr 1, 2010, at 10:33 AM, Darren J Moffat wrote:

On 01/04/2010 18:28, Neil Perrin wrote:
We've flip-flopped on whether it should be inherited. It's currently
coded as inherited, and I know Robert believes it should stay that way.
Anyway, it was generally felt by the zfs group that sync=disabled
was sufficiently dangerous to require explicit setting on each dataset.
I would be ok with it being inherited.

I agree that it is dangerous but I'm not sure that the change in semantics means it shouldn't be inherited. On the other hand I can't think of any other equivalently "dangerous" (to applications view of the world) per dataset property.

--
Darren J Moffat

_______________________________________________
opensolaris-arc mailing list
[email protected]

Reply via email to