Re: [patch 25/26] mount options: fix udf
From: Cyrill Gorcunov
Date: Fri Jan 25 2008 - 05:57:37 EST
On Jan 25, 2008 12:29 PM, Miklos Szeredi <miklos@xxxxxxxxxx> wrote:
> > | + /* is this correct? */
> > | + if (sbi->s_anchor[2] != 0)
> > | + seq_printf(seq, ",anchor=%u", sbi->s_anchor[2]);
> >
> > you know, I would prefer to use form UDF_SB_ANCHOR(sb)[2]
> > in sake of style unification but we should wait for Jan's
> > decision (i'm not the expert in this area ;)
>
> I think UDF_SB_ANCHOR macro was removed by some patch in -mm.
>
> I'm more interested if the second element of the s_anchor array really
> does always have the value of the 'anchor=N' mount option. I haven't
> been able to verify that fully. Do you have some insight into that?
>
> Thanks,
> Miklos
>
Miklos,
I'll check this today evening (a bit busy now).
- Cyrill -
--
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/