[dm-devel] what to do with fls(x) (or device-mapper & swsusp-1.x.x ? )

[Daniel Phillips]

On Thursday 28 August 2003 16:31, Svetoslav Slavtchev wrote:
> Hi ,
> i'm tring to incorporate the dev-mapper 1.00.04 in a kernel that already
> has int fls(int x)
> which conflicts with the definitions in dm-io.[ch]
> (the kernel comes from mandrake cooker and the fls(x) definition is added
> by swsusp patches)
>
> so i wanted to ask you whether the attached patch looks OK,
> it removes the definitions of fls & generic_fls, changes log2_align to use
> fls(int x),
> and adjusts the only referance of  log2_align to use int instead of
> unsigned int.

That's fine, except there's no point in changing the unsigned ints to signed 
int: unsigned describes more accurately what they are.  The automatic 
conversion doesn't generate a warning for you, does it?

Regards,

Daniel