mbox series

[0/2,v3] blktrace: Fix sparse annotations and warn if enabling multiple traces

Message ID 20200605145349.18454-1-jack@suse.cz (mailing list archive)
Headers show
Series blktrace: Fix sparse annotations and warn if enabling multiple traces | expand

Message

Jan Kara June 5, 2020, 2:58 p.m. UTC
Hello Jens,

this series contains a patch from Luis' blktrace series and then my patch
to fix sparse warnings in blktrace. Luis' patch stands on its own, was
reviewed, and changes what I need to change as well so I've decided it's just
simplest to pull it in with my patch.

								Honza

Comments

Luis Chamberlain June 5, 2020, 3:03 p.m. UTC | #1
On Fri, Jun 05, 2020 at 04:58:35PM +0200, Jan Kara wrote:
> Hello Jens,
> 
> this series contains a patch from Luis' blktrace series and then my patch
> to fix sparse warnings in blktrace. Luis' patch stands on its own, was
> reviewed, and changes what I need to change as well so I've decided it's just
> simplest to pull it in with my patch.

Hey Jan,

Since these patches have contexts which can affect other patches in the
series of fixes I have, if possible I'd prefer if these get addressed in
order. I was just waiting for 0-day to finish grinding on the series.
I already have a successful build results, but just waiting on the
series of other tests, part of which include blktests.

If this is agreeable, I hope to post that series later today.

  Luis
Jan Kara June 8, 2020, 8:25 a.m. UTC | #2
Hi Luis!

On Fri 05-06-20 15:03:50, Luis Chamberlain wrote:
> On Fri, Jun 05, 2020 at 04:58:35PM +0200, Jan Kara wrote:
> > Hello Jens,
> > 
> > this series contains a patch from Luis' blktrace series and then my patch
> > to fix sparse warnings in blktrace. Luis' patch stands on its own, was
> > reviewed, and changes what I need to change as well so I've decided it's just
> > simplest to pull it in with my patch.
> 
> Hey Jan,
> 
> Since these patches have contexts which can affect other patches in the
> series of fixes I have, if possible I'd prefer if these get addressed in
> order. I was just waiting for 0-day to finish grinding on the series.
> I already have a successful build results, but just waiting on the
> series of other tests, part of which include blktests.
> 
> If this is agreeable, I hope to post that series later today.

Whatever... I wasn't sure how long it's going to take to finish your series
and since Jens was asking about the sparse fixes, I've just sent them
along. BTW the two patches applied just fine separately from the rest of
the series so I don't think there's any dependency to the rest of your
series even in context.

								Honza
Luis Chamberlain June 8, 2020, 2:01 p.m. UTC | #3
On Mon, Jun 08, 2020 at 10:25:28AM +0200, Jan Kara wrote:
> BTW the two patches applied just fine separately from the rest of
> the series so I don't think there's any dependency to the rest of your
> series even in context.

The depndency was just minor contextual edits, which can be fixed with
a rebase, I've done that now, so this is good to go in first. I'll skip
these two patches fromy series then.

  Luis
Jan Kara June 17, 2020, 12:51 p.m. UTC | #4
Hello Jens!

On Fri 05-06-20 16:58:35, Jan Kara wrote:
> this series contains a patch from Luis' blktrace series and then my patch
> to fix sparse warnings in blktrace. Luis' patch stands on its own, was
> reviewed, and changes what I need to change as well so I've decided it's just
> simplest to pull it in with my patch.

Can you please merge this series?

								Honza
Jens Axboe June 17, 2020, 3:07 p.m. UTC | #5
On 6/17/20 6:51 AM, Jan Kara wrote:
> Hello Jens!
> 
> On Fri 05-06-20 16:58:35, Jan Kara wrote:
>> this series contains a patch from Luis' blktrace series and then my patch
>> to fix sparse warnings in blktrace. Luis' patch stands on its own, was
>> reviewed, and changes what I need to change as well so I've decided it's just
>> simplest to pull it in with my patch.
> 
> Can you please merge this series?

Yep done, thanks.