Re: [PATCH v5 5/7] target: srpt replace enable attr to ops.enable

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Dmitry Bogdanov <d.bogdanov@xxxxxxxxx>, Martin Petersen <martin.petersen@xxxxxxxxxx>, target-devel@xxxxxxxxxxxxxxx
Subject: Re: [PATCH v5 5/7] target: srpt replace enable attr to ops.enable
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Fri, 10 Sep 2021 19:40:41 -0700
Cc: linux-scsi@xxxxxxxxxxxxxxx, linux@xxxxxxxxx, Nilesh Javali <njavali@xxxxxxxxxxx>, Chris Boot <bootc@xxxxxxxxx>, Michael Cyr <mikecyr@xxxxxxxxxxxxx>, Felipe Balbi <balbi@xxxxxxxxxx>, Roman Bolshakov <r.bolshakov@xxxxxxxxx>
In-reply-to: <20210910084133.17956-6-d.bogdanov@yadro.com>
References: <20210910084133.17956-1-d.bogdanov@yadro.com> <20210910084133.17956-6-d.bogdanov@yadro.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.0.1

On 9/10/21 01:41, Dmitry Bogdanov wrote:

Remove tpg/enable attribute.
Add fabric ops enable_tpg implementation instead.


How about changing the prefix from "target: srpt" into "RDMA/srpt:"
since that is the prefix that is used for other ib_srpt patches?

Anyway:

Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>

References:
- [PATCH v5 0/7] target: make tpg/enable attribute
  - From: Dmitry Bogdanov
- [PATCH v5 5/7] target: srpt replace enable attr to ops.enable
  - From: Dmitry Bogdanov

Prev by Date: [PATCH v5 6/7] target: ibm_vscsi: replace enable attr to ops.enable
Next by Date: [PATCH] scsi: target: Remove unused argument of some functions
Previous by thread: [PATCH v5 5/7] target: srpt replace enable attr to ops.enable
Next by thread: [PATCH v5 6/7] target: ibm_vscsi: replace enable attr to ops.enable
Index(es):
- Date
- Thread

[Index of Archives] [Linux SCSI] [Kernel Newbies] [Linux SCSI Target Infrastructure] [Share Photos] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Device Mapper]