Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1571019 > unrolled thread

Re: [PULL] bcache: based on for-4.10/block, multiple updates

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PULL] bcache: based on for-4.10/block, multiple updates Eric Wheeler <bcache@lists.ewheeler.net> - 2017-01-31 20:20 +0100
    Re: [PULL] bcache: based on for-4.10/block, multiple updates Jens Axboe <axboe@kernel.dk> - 2017-01-31 20:30 +0100
      Re: [PULL] bcache: based on for-4.10/block, multiple updates Eric Wheeler <bcache@lists.ewheeler.net> - 2017-02-01 00:20 +0100

#1571019 — Re: [PULL] bcache: based on for-4.10/block, multiple updates

On Wed, 14 Dec 2016, Jens Axboe wrote:
> On 12/14/2016 06:50 PM, Eric Wheeler wrote:
> > Hi Jens,
> > 
> > I know you're busy, so when you get a moment:
> > 
> > I've not yet seen your ack/nack on this yet and I want to make sure it 
> > gets in before the merge window closes for v4.10.  I rebased it on 
> > for-4.10/block as you asked so its tested and ready to go as of 12/6 
> > unless you see something that I've missed.
> 
> Pulled for 4.10, sorry about the delay.

Hi Jens,

Looking at 4.10-rc6, I see these two commits were merged:

>       bcache: partition support: add 16 minors per bcacheN device
>       bcache: Make gc wakeup sane, remove set_task_state()

but these were in the same 'for-4.10-block-bcache-updates' and don't
appear to have been applied:

>       bcache: introduce per-process ioprio-based bypass/writeback hints
>       bcache: documentation for ioprio cache hinting
>       bcache: update bio->bi_opf bypass/writeback REQ_ flag hints

Did I miss an email that caused their exclusion?

Did these get bumped to 4.11 for some reason?

Original pull request is below for your reference.

Thank you for your help!

--
Eric Wheeler

[...] are available in the git repository at:

  https://bitbucket.org/ewheelerinc/linux.git for-4.10-block-bcache-updates

for you to fetch changes up to 951c0a8cc489aca1c68d80e5aaa576cd7e511191:

  bcache: partition support: add 16 minors per bcacheN device (2016-12-06 10:13:24 -0800)

----------------------------------------------------------------
Eric Wheeler (4):
      bcache: introduce per-process ioprio-based bypass/writeback hints
      bcache: documentation for ioprio cache hinting
      bcache: update bio->bi_opf bypass/writeback REQ_ flag hints
      bcache: partition support: add 16 minors per bcacheN device

Kent Overstreet (1):
      bcache: Make gc wakeup sane, remove set_task_state()

 Documentation/bcache.txt      | 80 +++++++++++++++++++++++++++++++++++++++++++
 drivers/md/bcache/bcache.h    |  7 ++--
 drivers/md/bcache/btree.c     | 39 +++++++++++----------
 drivers/md/bcache/btree.h     |  3 +-
 drivers/md/bcache/request.c   | 31 +++++++++++++++--
 drivers/md/bcache/super.c     |  7 +++-
 drivers/md/bcache/sysfs.c     | 71 ++++++++++++++++++++++++++++++++++++++
 drivers/md/bcache/writeback.c |  8 +++++
 drivers/md/bcache/writeback.h | 27 ++++++++++++++-
 9 files changed, 245 insertions(+), 28 deletions(-)




> 
> -- 
> Jens Axboe
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bcache" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

[toc] | [next] | [standalone]

#1571029

On 01/31/2017 11:14 AM, Eric Wheeler wrote:
> On Wed, 14 Dec 2016, Jens Axboe wrote:
>> On 12/14/2016 06:50 PM, Eric Wheeler wrote:
>>> Hi Jens,
>>>
>>> I know you're busy, so when you get a moment:
>>>
>>> I've not yet seen your ack/nack on this yet and I want to make sure it 
>>> gets in before the merge window closes for v4.10.  I rebased it on 
>>> for-4.10/block as you asked so its tested and ready to go as of 12/6 
>>> unless you see something that I've missed.
>>
>> Pulled for 4.10, sorry about the delay.
> 
> Hi Jens,
> 
> Looking at 4.10-rc6, I see these two commits were merged:
> 
>>       bcache: partition support: add 16 minors per bcacheN device
>>       bcache: Make gc wakeup sane, remove set_task_state()
> 
> but these were in the same 'for-4.10-block-bcache-updates' and don't
> appear to have been applied:
> 
>>       bcache: introduce per-process ioprio-based bypass/writeback hints
>>       bcache: documentation for ioprio cache hinting
>>       bcache: update bio->bi_opf bypass/writeback REQ_ flag hints
> 
> Did I miss an email that caused their exclusion?
> 
> Did these get bumped to 4.11 for some reason?

I thought I sent out that reply, maybe I didn't. I dropped the
last three for two reasons:

1) They happened after the merge window, and it's really a feature
   addon that should have gone in earlier

2) There seemed to be confusion and discussion around the interface
   that needed to be resolved first.

Hope that explains it.

-- 
Jens Axboe

[toc] | [prev] | [next] | [standalone]

#1571205

On Tue, 31 Jan 2017, Jens Axboe wrote:
> On 01/31/2017 11:14 AM, Eric Wheeler wrote:
> > On Wed, 14 Dec 2016, Jens Axboe wrote:
> >> On 12/14/2016 06:50 PM, Eric Wheeler wrote:
> >>> Hi Jens,
> >>>
> >>> I know you're busy, so when you get a moment:
> >>>
> >>> I've not yet seen your ack/nack on this yet and I want to make sure it 
> >>> gets in before the merge window closes for v4.10.  I rebased it on 
> >>> for-4.10/block as you asked so its tested and ready to go as of 12/6 
> >>> unless you see something that I've missed.
> >>
> >> Pulled for 4.10, sorry about the delay.
> > 
> > Hi Jens,
> > 
> > Looking at 4.10-rc6, I see these two commits were merged:
> > 
> >>       bcache: partition support: add 16 minors per bcacheN device
> >>       bcache: Make gc wakeup sane, remove set_task_state()
> > 
> > but these were in the same 'for-4.10-block-bcache-updates' and don't
> > appear to have been applied:
> > 
> >>       bcache: introduce per-process ioprio-based bypass/writeback hints
> >>       bcache: documentation for ioprio cache hinting
> >>       bcache: update bio->bi_opf bypass/writeback REQ_ flag hints
> > 
> > Did I miss an email that caused their exclusion?
> > 
> > Did these get bumped to 4.11 for some reason?
> 
> I thought I sent out that reply, maybe I didn't. I dropped the
> last three for two reasons:
> 
> 1) They happened after the merge window, and it's really a feature
>    addon that should have gone in earlier
  
Onto 4.11 then.   Hopefully this is early enough :)

> 
> 2) There seemed to be confusion and discussion around the interface
>    that needed to be resolved first.

None that I'm aware of.  This is a bcache-only feature and doesn't affect 
non-bcache users. In fact, people are using it as a patch with good 
results since they can control their erase-block churn.  Kent Overstreet 
acked it, too.

Can you pull this for 4.11?

Thank you for your help!

--
Eric Wheeler



> 
> Hope that explains it.
> 
> -- 
> Jens Axboe
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-bcache" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web