Re: [PATCH] PUFFS backend allocation (round 3)

[manu%netbsd.org@localhost (Emmanuel Dreyfus)]

J. Hannken-Illjes <hannken%eis.cs.tu-bs.de@localhost> wrote:

> - Increment PUFFSVERSION.

When is it really required? Without incrementing it, a newer kernel
works with an older libpuffs, but if I increase it I get an error about
version mismatch.

> - You should recover the error in puffs_vnop_close() too.

Right.

> - Are you sure pn_sizemtx is still needed?  VOP_GETATTR() works on
>   locked vnodes since 2011-10-30.

The problem is that I have no test case to check it works without it.

-- 
Emmanuel Dreyfus
http://hcpnet.free.fr/pubz
manu%netbsd.org@localhost