Re: [PATCH 1/2] BKL: Remove BKL from default_llseek()

Jan Blunck
2009-11-18T20:02:25Z

On Wed, Nov 18, Alan Cox wrote:

> > As I said, f_pos is changed without holding BKL in the VFS already. Therefore
>
> IFF the VFS paths that change it can be invoked in parallel.
>
> > even if the driver tries to protect f_pos by holding the BKL it is racing
> > against concurrent read()/write() anyway on f_pos.
>
> It may simply be a matter of internal consistency. Anyway I've now made a
> list of the offenders so we can simply fix/delete them and continue.

Right. Thanks for that btw. I'll work on fixing them and followup with patches
tomorrow.

Thanks,
Jan

--
Jan Blunck <jblunck@suse.de>