svn commit: r238539 - head/sys/fs/smbfs

[Davide Italiano]

On Wed, Jul 18, 2012 at 2:11 PM, Christian Brueffer
<brueffer at freebsd.org> wrote:
> On 7/17/12 0:10 , Attilio Rao wrote:
>> 2012/7/16 Christian Brueffer <brueffer at freebsd.org>:
>>> Author: brueffer
>>> Date: Mon Jul 16 22:07:29 2012
>>> New Revision: 238539
>>> URL: http://svn.freebsd.org/changeset/base/238539
>>>
>>> Log:
>>>   Simply error handling by moving the allocation of np down to where it is
>>>   actually used.  While here, improve style a little.
>>
>> Too bad that the biggest bug here is still in place.
>> Right now smbfs inserts in the mount list half-constructed vnodes.
>> Maybe you are interested in fixing this?
>>
>
> I can give it a shot, no promises though.
>
> Cheers,
>
> Chris
>

This patch should fix the half-constructed vnode insertion issue.
Can you give it a look, Attilio?
http://people.freebsd.org/~davide/ix/smbfs_norollhash.diff
Moreover, it makes the code relying on the global vfs_hash table, now
it works using an home-made local hash table for vnodes.

Thanks

Davide