On Tue, 2014-05-20 at 07:02 +0200, Fabian Frederick wrote:
> and the winner is ? :)
Either works. You're the patch submitter. Pick one.
But I'm sure you know I'm not pushing the patch upstream... :)
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a
On Mon, 19 May 2014 16:50:05 -0700
Joe Perches wrote:
> On Mon, 2014-05-19 at 16:40 -0700, Andrew Morton wrote:
> > On Mon, 19 May 2014 16:22:28 -0700 Joe Perches wrote:
> >
> > > On Mon, 2014-05-19 at 16:09 -0700, Andrew Morton wrote:
> > > > On Fri, 16 May 2014 22:38:15 +0200 Fabian
On Mon, 2014-05-19 at 16:40 -0700, Andrew Morton wrote:
> On Mon, 19 May 2014 16:22:28 -0700 Joe Perches wrote:
>
> > On Mon, 2014-05-19 at 16:09 -0700, Andrew Morton wrote:
> > > On Fri, 16 May 2014 22:38:15 +0200 Fabian Frederick
> > > wrote:
> > > > Replace while blocksize;shift by fls -1
>
On Mon, 19 May 2014 16:22:28 -0700 Joe Perches wrote:
> On Mon, 2014-05-19 at 16:09 -0700, Andrew Morton wrote:
> > On Fri, 16 May 2014 22:38:15 +0200 Fabian Frederick wrote:
> > > Replace while blocksize;shift by fls -1
> []
> > fls() always makes my brain hurt. ilog() is nicer?
>
> Maybe
On Mon, 2014-05-19 at 16:09 -0700, Andrew Morton wrote:
> On Fri, 16 May 2014 22:38:15 +0200 Fabian Frederick wrote:
> > Replace while blocksize;shift by fls -1
[]
> fls() always makes my brain hurt. ilog() is nicer?
Maybe take an aspirin?
ilog2(n) is effectively fls(n) - 1 but n can be
either
On Fri, 16 May 2014 22:38:15 +0200 Fabian Frederick wrote:
> Replace while blocksize;shift by fls -1
>
> Suggested-By: Joe Perches
> Cc: Andrew Morton
> Signed-off-by: Fabian Frederick
> ---
> v2: rebased on top of mmots
> compiles without including bitops
>
> fs/hfsplus/wrapper.c | 4
On Fri, 16 May 2014 22:38:15 +0200 Fabian Frederick f...@skynet.be wrote:
Replace while blocksize;shift by fls -1
Suggested-By: Joe Perches j...@perches.com
Cc: Andrew Morton a...@linux-foundation.org
Signed-off-by: Fabian Frederick f...@skynet.be
---
v2: rebased on top of mmots
On Mon, 2014-05-19 at 16:09 -0700, Andrew Morton wrote:
On Fri, 16 May 2014 22:38:15 +0200 Fabian Frederick f...@skynet.be wrote:
Replace while blocksize;shift by fls -1
[]
fls() always makes my brain hurt. ilog() is nicer?
Maybe take an aspirin?
ilog2(n) is effectively fls(n) - 1 but n can
On Mon, 19 May 2014 16:22:28 -0700 Joe Perches j...@perches.com wrote:
On Mon, 2014-05-19 at 16:09 -0700, Andrew Morton wrote:
On Fri, 16 May 2014 22:38:15 +0200 Fabian Frederick f...@skynet.be wrote:
Replace while blocksize;shift by fls -1
[]
fls() always makes my brain hurt. ilog() is
On Mon, 2014-05-19 at 16:40 -0700, Andrew Morton wrote:
On Mon, 19 May 2014 16:22:28 -0700 Joe Perches j...@perches.com wrote:
On Mon, 2014-05-19 at 16:09 -0700, Andrew Morton wrote:
On Fri, 16 May 2014 22:38:15 +0200 Fabian Frederick f...@skynet.be
wrote:
Replace while
On Mon, 19 May 2014 16:50:05 -0700
Joe Perches j...@perches.com wrote:
On Mon, 2014-05-19 at 16:40 -0700, Andrew Morton wrote:
On Mon, 19 May 2014 16:22:28 -0700 Joe Perches j...@perches.com wrote:
On Mon, 2014-05-19 at 16:09 -0700, Andrew Morton wrote:
On Fri, 16 May 2014 22:38:15
On Tue, 2014-05-20 at 07:02 +0200, Fabian Frederick wrote:
and the winner is ? :)
Either works. You're the patch submitter. Pick one.
But I'm sure you know I'm not pushing the patch upstream... :)
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a
Replace while blocksize;shift by fls -1
Suggested-By: Joe Perches
Cc: Andrew Morton
Signed-off-by: Fabian Frederick
---
v2: rebased on top of mmots
compiles without including bitops
fs/hfsplus/wrapper.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git
Replace while blocksize;shift by fls -1
Suggested-By: Joe Perches j...@perches.com
Cc: Andrew Morton a...@linux-foundation.org
Signed-off-by: Fabian Frederick f...@skynet.be
---
v2: rebased on top of mmots
compiles without including bitops
fs/hfsplus/wrapper.c | 4 +---
1 file changed, 1
14 matches
Mail list logo