Also, a nit: > -static void inode_switch_wbs_work_fn(struct work_struct *work) > +static void process_inode_switch_wbs_work(struct bdi_writeback *new_wb, > + struct inode_switch_wbs_context *isw) Maybe just process_inode_switch_wbs()? It's a bit odd to remove "fn" without the "work" part as those two together was saying it was a function for a work_struct. Thanks. -- tejun