mbox series

[00/14] bcache patches for Linux v5.14

Message ID 20210615054921.101421-1-colyli@suse.de (mailing list archive)
Headers show
Series bcache patches for Linux v5.14 | expand

Message

Coly Li June 15, 2021, 5:49 a.m. UTC
Hi Jens,

Here are the bcache patches for Linux v5.14.

The patches from Chao Yu and Ding Senjie are useful code cleanup. The
rested patches for the NVDIMM support to bcache journaling.

For the series to support NVDIMM to bache journaling, all reported
issue since last merge window are all fixed. And no more issue detected
during our testing or by the kernel test robot. If there is any issue
reported during they stay in linux-next, I, Jianpang and Qiaowei will
response and fix immediately.

Please take them for Linux v5.14.

Thank you in advance.

Coly Li
---

Chao Yu (1):
  bcache: fix error info in register_bcache()

Coly Li (7):
  bcache: add initial data structures for nvm pages
  bcache: use bucket index to set GC_MARK_METADATA for journal buckets
    in bch_btree_gc_finish()
  bcache: add BCH_FEATURE_INCOMPAT_NVDIMM_META into incompat feature set
  bcache: initialize bcache journal for NVDIMM meta device
  bcache: support storing bcache journal into NVDIMM meta device
  bcache: read jset from NVDIMM pages for journal replay
  bcache: add sysfs interface register_nvdimm_meta to register NVDIMM
    meta device

Ding Senjie (1):
  md: bcache: Fix spelling of 'acquire'

Jianpeng Ma (5):
  bcache: initialize the nvm pages allocator
  bcache: initialization of the buddy
  bcache: bch_nvm_alloc_pages() of the buddy
  bcache: bch_nvm_free_pages() of the buddy
  bcache: get allocated pages from specific owner

 drivers/md/bcache/Kconfig       |  10 +
 drivers/md/bcache/Makefile      |   1 +
 drivers/md/bcache/btree.c       |   6 +-
 drivers/md/bcache/features.h    |   9 +
 drivers/md/bcache/journal.c     | 317 ++++++++++---
 drivers/md/bcache/journal.h     |   2 +-
 drivers/md/bcache/nvm-pages.c   | 773 ++++++++++++++++++++++++++++++++
 drivers/md/bcache/nvm-pages.h   |  93 ++++
 drivers/md/bcache/super.c       |  91 +++-
 include/uapi/linux/bcache-nvm.h | 206 +++++++++
 10 files changed, 1432 insertions(+), 76 deletions(-)
 create mode 100644 drivers/md/bcache/nvm-pages.c
 create mode 100644 drivers/md/bcache/nvm-pages.h
 create mode 100644 include/uapi/linux/bcache-nvm.h

Comments

Jens Axboe June 21, 2021, 3:14 p.m. UTC | #1
On 6/14/21 11:49 PM, Coly Li wrote:
> Hi Jens,
> 
> Here are the bcache patches for Linux v5.14.
> 
> The patches from Chao Yu and Ding Senjie are useful code cleanup. The
> rested patches for the NVDIMM support to bcache journaling.
> 
> For the series to support NVDIMM to bache journaling, all reported
> issue since last merge window are all fixed. And no more issue detected
> during our testing or by the kernel test robot. If there is any issue
> reported during they stay in linux-next, I, Jianpang and Qiaowei will
> response and fix immediately.
> 
> Please take them for Linux v5.14.

I'd really like the user api bits to have some wider review. Maybe
I'm missing something, but there's a lot of weird stuff in the uapi
header that includes things like pointers etc.
Coly Li June 21, 2021, 3:25 p.m. UTC | #2
On 6/21/21 11:14 PM, Jens Axboe wrote:
> On 6/14/21 11:49 PM, Coly Li wrote:
>> Hi Jens,
>>
>> Here are the bcache patches for Linux v5.14.
>>
>> The patches from Chao Yu and Ding Senjie are useful code cleanup. The
>> rested patches for the NVDIMM support to bcache journaling.
>>
>> For the series to support NVDIMM to bache journaling, all reported
>> issue since last merge window are all fixed. And no more issue detected
>> during our testing or by the kernel test robot. If there is any issue
>> reported during they stay in linux-next, I, Jianpang and Qiaowei will
>> response and fix immediately.
>>
>> Please take them for Linux v5.14.
> I'd really like the user api bits to have some wider review. Maybe
> I'm missing something, but there's a lot of weird stuff in the uapi
> header that includes things like pointers etc.
>

Hi Jens,

As I explained 2 merge windows before, we use nvdimm as non-volatiled
memory, that is, the
memory objects are stored on nvdimm as memory object which are
non-volatiled. This is why
you see the pointers in the data structure, e.g. the list is stored on
NVDIMM area, and the code
goes through the list directly on the NVDIMM, we don't load them into
memory.

This is not block device interface.

I try to ask Dan Williams, Jan Kara, Christoph Hellwig and Hannes
Reineicke to help to review,
hope there are some experts may help to take a look.

Coly Li
Jens Axboe June 21, 2021, 3:27 p.m. UTC | #3
On 6/21/21 9:25 AM, Coly Li wrote:
> On 6/21/21 11:14 PM, Jens Axboe wrote:
>> On 6/14/21 11:49 PM, Coly Li wrote:
>>> Hi Jens,
>>>
>>> Here are the bcache patches for Linux v5.14.
>>>
>>> The patches from Chao Yu and Ding Senjie are useful code cleanup. The
>>> rested patches for the NVDIMM support to bcache journaling.
>>>
>>> For the series to support NVDIMM to bache journaling, all reported
>>> issue since last merge window are all fixed. And no more issue detected
>>> during our testing or by the kernel test robot. If there is any issue
>>> reported during they stay in linux-next, I, Jianpang and Qiaowei will
>>> response and fix immediately.
>>>
>>> Please take them for Linux v5.14.
>> I'd really like the user api bits to have some wider review. Maybe
>> I'm missing something, but there's a lot of weird stuff in the uapi
>> header that includes things like pointers etc.
>>
> 
> Hi Jens,
> 
> As I explained 2 merge windows before, we use nvdimm as non-volatiled
> memory, that is, the
> memory objects are stored on nvdimm as memory object which are
> non-volatiled. This is why
> you see the pointers in the data structure, e.g. the list is stored on
> NVDIMM area, and the code
> goes through the list directly on the NVDIMM, we don't load them into
> memory.
> 
> This is not block device interface.

Right, and I'm not oblivious to those emails. What I'm saying is that
we need more review of that, so far it seems to stand unreviewed.

> I try to ask Dan Williams, Jan Kara, Christoph Hellwig and Hannes
> Reineicke to help to review,
> hope there are some experts may help to take a look.

That'd be great.