On Mon, 21 Mar 2016 21:28:39 +0200 Mika Kukkonen <mikukkon@xxxxxx> wrote: > Was deleting an old github kernel tree and noticed this one line patch > I had made years back. Can't remember the compiler warning anymore but > clearly it complained about pointless assigment to cnt since it is not > used after that, although to me it looks like it should. I'll take this. Thanks Mika! -- Steve > > Signed-off-by: Mika Kukkonen <mikukkon@xxxxxx> > > diff --git a/kernel/trace/ring_buffer.c b/kernel/trace/ring_buffer.c > index 95181e3..22d1007 100644 > --- a/kernel/trace/ring_buffer.c > +++ b/kernel/trace/ring_buffer.c > @@ -4756,7 +4756,7 @@ static __init int rb_write_something(struct > rb_test_data *data, bool nested) > cnt = data->cnt + (nested ? 27 : 0); > > /* Multiply cnt by ~e, to make some unique increment */ > - size = (data->cnt * 68 / 25) % (sizeof(rb_string) - 1); > + size = (cnt * 68 / 25) % (sizeof(rb_string) - 1); > > len = size + sizeof(struct rb_item); > > --MiKu -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
