Project

General

Profile

Bug #1016

vnode setup should be in vnode_cache_constructor

Added by Albert Lee over 8 years ago. Updated about 6 years ago.

Status:
New
Priority:
Low
Assignee:
Category:
kernel
Start date:
2011-05-10
Due date:
% Done:

0%

Estimated time:
Difficulty:
Bite-size
Tags:
needs-triage

Description

vn_alloc is currently responsible for some vnode initialisation, including calling vn_reinit. I see no reason this shouldn't be in the vnode_cache constructor.

History

#1

Updated by Josef Sipek about 6 years ago

The problem here is that the vnode_cache_constructor gets called only once per object. That is, the code (vn_free) would have to free vnodes in a state that is identical to a vnode that's been just constructed. vn_free doesn't do this, hence the need for vn_alloc fixing things up.

Also available in: Atom PDF