[kj][RFC] roll macro's in bitops.h

G'day people,

I was looking at bitops.h and noticed there were a couple of inline roll
functions to operate on 32bit variables. (left and right)

I think that is a little bit dumb as they are constant width (only
32bits) due to it being an inline function, and was wondering if anyone
thought it would be useful to have some (variable sized) roll macros
instead.

I whipped these twp up as examples, but have not tested them yet.

#define RollRight(Data, Distance) \
({      typeof(Data) __a = Data;   \
        typeof(Distance) __b = Distance%(sizeof(__a) * 8); \
	__a = ((__a>>__b) | (__a<<((sizeof(__a) * 8) - __b))) \
	__a; \
})

#define RollLeft(Data, Distance) \
({      typeof(Data) __a = Data;   \
        typeof(Distance) __b = Distance%(sizeof(__a) * 8); \
	__a = ((__a<<__b) | (__a>>((sizeof(__a) * 8) - __b))) \
	__a; \
})

Also if these are useful I was wondering how to handle the existing
functions ? #define them out for the time being ?

#define rol32(a, b) RollLeft(a, b)
#define ror32(a, b) RollRight(a, b)




Darren Jenkins


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

---

• Prev by Date: [PATCH] Remove obsolete raw device support.
• Next by Date: Kbuild problem (was: [2.6 patch] drivers/isdn/gigaset/: build asyncdata.o into the gigaset module)
• Previous by thread: [PATCH] Remove obsolete raw device support.
• Next by thread: [PATCH] Fix constant folding and poor optimization in byte swapping code
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]