mbox series

[PATCHv2,for-next,0/7] Misc update for rnbd

Message ID 20201210101826.29656-1-jinpu.wang@cloud.ionos.com (mailing list archive)
Headers show
Series Misc update for rnbd | expand

Message

Jinpu Wang Dec. 10, 2020, 10:18 a.m. UTC
Hi Jens,

This is the misc update for rnbd. It inlcudes:
- 2 follow-up fixes for commit 64e8a6ece1a5 ("block/rnbd-clt: Dynamically alloc buffer for pathname & blk_symlink_name")
  one warning, and one possible memleak.
- one fix for race with dev session sysfs removal.
- fix for write-back cache & FUA.
- reduce memory footprint by allocate sglist on demand and do not request pdu
  from rtrs-clt.
- Typo fix.

change since v1:
- use kstrdup as Bart suggested.
- fix return code and leaking iu in block/rnbd-clt: Dynamically allocate sglist for rnbd_iu

v1: https://lore.kernel.org/linux-block/20201209082051.12306-1-jinpu.wang@cloud.ionos.com/T/#md330070d0688bbc57325ce0a6b2181f39dca495c

The patches are based on block/for-next.

Gioh Kim (3):
  block/rnbd: Set write-back cache and fua same to the target device
  block/rnbd-clt: Dynamically allocate sglist for rnbd_iu
  block/rnbd-clt: Does not request pdu to rtrs-clt

Jack Wang (2):
  block/rnbd-clt: Fix possible memleak
  block/rnbd: Fix typos

Md Haris Iqbal (2):
  block/rnbd-clt: Get rid of warning regarding size argument in strlcpy
  block/rnbd-srv: Protect dev session sysfs removal

 drivers/block/rnbd/rnbd-clt-sysfs.c    |  5 +-
 drivers/block/rnbd/rnbd-clt.c          | 94 +++++++++++++++-----------
 drivers/block/rnbd/rnbd-clt.h          | 12 +++-
 drivers/block/rnbd/rnbd-proto.h        |  9 ++-
 drivers/block/rnbd/rnbd-srv.c          | 12 +++-
 drivers/infiniband/ulp/rtrs/rtrs-clt.c |  6 --
 drivers/infiniband/ulp/rtrs/rtrs.h     |  7 --
 7 files changed, 87 insertions(+), 58 deletions(-)

Comments

Jinpu Wang Dec. 16, 2020, 6:28 a.m. UTC | #1
On Thu, Dec 10, 2020 at 11:18 AM Jack Wang <jinpu.wang@cloud.ionos.com> wrote:
>
> Hi Jens,
>
> This is the misc update for rnbd. It inlcudes:
> - 2 follow-up fixes for commit 64e8a6ece1a5 ("block/rnbd-clt: Dynamically alloc buffer for pathname & blk_symlink_name")
>   one warning, and one possible memleak.
> - one fix for race with dev session sysfs removal.
> - fix for write-back cache & FUA.
> - reduce memory footprint by allocate sglist on demand and do not request pdu
>   from rtrs-clt.
> - Typo fix.

Ping?
>
> change since v1:
> - use kstrdup as Bart suggested.
> - fix return code and leaking iu in block/rnbd-clt: Dynamically allocate sglist for rnbd_iu
>
> v1: https://lore.kernel.org/linux-block/20201209082051.12306-1-jinpu.wang@cloud.ionos.com/T/#md330070d0688bbc57325ce0a6b2181f39dca495c
>
> The patches are based on block/for-next.
>
> Gioh Kim (3):
>   block/rnbd: Set write-back cache and fua same to the target device
>   block/rnbd-clt: Dynamically allocate sglist for rnbd_iu
>   block/rnbd-clt: Does not request pdu to rtrs-clt
>
> Jack Wang (2):
>   block/rnbd-clt: Fix possible memleak
>   block/rnbd: Fix typos
>
> Md Haris Iqbal (2):
>   block/rnbd-clt: Get rid of warning regarding size argument in strlcpy
>   block/rnbd-srv: Protect dev session sysfs removal
>
>  drivers/block/rnbd/rnbd-clt-sysfs.c    |  5 +-
>  drivers/block/rnbd/rnbd-clt.c          | 94 +++++++++++++++-----------
>  drivers/block/rnbd/rnbd-clt.h          | 12 +++-
>  drivers/block/rnbd/rnbd-proto.h        |  9 ++-
>  drivers/block/rnbd/rnbd-srv.c          | 12 +++-
>  drivers/infiniband/ulp/rtrs/rtrs-clt.c |  6 --
>  drivers/infiniband/ulp/rtrs/rtrs.h     |  7 --
>  7 files changed, 87 insertions(+), 58 deletions(-)
>
> --
> 2.25.1
>
Jens Axboe Dec. 16, 2020, 2:20 p.m. UTC | #2
On 12/15/20 11:28 PM, Jinpu Wang wrote:
> On Thu, Dec 10, 2020 at 11:18 AM Jack Wang <jinpu.wang@cloud.ionos.com> wrote:
>>
>> Hi Jens,
>>
>> This is the misc update for rnbd. It inlcudes:
>> - 2 follow-up fixes for commit 64e8a6ece1a5 ("block/rnbd-clt: Dynamically alloc buffer for pathname & blk_symlink_name")
>>   one warning, and one possible memleak.
>> - one fix for race with dev session sysfs removal.
>> - fix for write-back cache & FUA.
>> - reduce memory footprint by allocate sglist on demand and do not request pdu
>>   from rtrs-clt.
>> - Typo fix.
> 
> Ping?

Huh, I thought I had grabbed v2. In any case, it'll go in later in the merge
window, I do have it queued now.