Re: Suspected bug infilesystems (UFS,ADFS,BEFS,BFS,ReiserFS) related to sector_t being unsigned, advice requested

From: Mike Fedyk
Date: Tue Jan 06 2004 - 17:26:57 EST


On Tue, Jan 06, 2004 at 11:35:10PM +0200, Oleg Drokin wrote:
> Hello!
>
> Mike Fedyk <mfedyk@xxxxxxxxxxxxx> wrote:
> MF> Did you check the locking after this is removed?
>
> Locking should be fine.
> As I remember, we take this lock near reiserfs_get_block() entrance,
> and then drop it on exit.
>

Ok, I just wondered if the sector_t merge overlooked this part, maybe it
also changed the locking in this area because of that.

> MF> Maybe after the sector_t merges, this code covered a case that is left open
> MF> now...
>
> This code was never executing anyway.

Right.

Thanks,

Mike
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/