On Wed, Nov 12, 2025 at 10:39:03AM +0000, Alberto Faria wrote: > It seems Changqi Lu might have left ByteDance, the email is bouncing > with a user not found error. > > Any tips on how to proceed?
You can resume development of the series. Apply the patches, work on them until they are ready, and send them crediting the author: - It's easiest to add new patches that are authored by yourself, but often you'll also need to fix up the original patches. - If the change to an original patch is small, keep the authorship information and add your Signed-off-by, noting that you made changes. - If you change the majority of a patch, you might decide to make yourself author, but note that the patch is based on the original author's work. You may keep the patch series subject or pick a new one, depending on how relevant the original subject is to the scope of your new series. If you're unsure, try searching the web for advice on taking over patch series. There are multiple ways of doing it, and as long as you give credit to the author of the original code, it's up to you which approach you choose. Stefan > > Alberto > > On Wed, Nov 12, 2025 at 10:26 AM Alberto Faria <[email protected]> wrote: > > > > Hi Changqi, > > > > I wonder if you plan on following through with this series? It looks > > like it was/is very close to being merged. I can help or pick it up > > from here if you want, as I'm interested in seeing it merged. > > > > Thanks, Alberto > > > > On Fri, Oct 11, 2024 at 6:24 PM Changqi Lu <[email protected]> > > wrote: > > > > > > Patch v14 has been modified, thanks to Stefan for the code review. > > > > > > v13->v14: > > > - Fix some protocol and spelling related issues at hw/nvme/ctrl.c. > > > > > > v12->v13: > > > - Fix byte swaps at hw/nvme/ctrl.c. > > > > > > v11->v12: > > > - Fixed endian conversion during command parsing; > > > - Add the maximum number of keys, currently limited to 128. > > > > > > v10->v11: > > > - Before executing the pr operation, check whether it is supported. > > > If it is not supported, return NVME_INVALID_OPCODE directly. > > > > > > v9->v10: > > > - When the driver does not support the pr operation, the error > > > code returned by nvme changes to Invalid Command Opcode. > > > > > > v8->v9: > > > - Fix double-free and remove persistent reservation operations at > > > nvme_is_write(). > > > > > > v7->v8: > > > - Fix num_keys may be less than 0 at scsi_pr_read_keys_complete(). > > > - Fix buf memory leak at iscsi driver. > > > > > > v6->v7: > > > - Add buferlen size check at SCSI layer. > > > - Add pr_cap calculation in bdrv_merge_limits() function at block layer, > > > so the ugly bs->file->bs->bl.pr_cap in scsi and nvme layers was > > > changed to bs->bl.pr_cap. > > > - Fix memory leak at iscsi driver, and some other spelling errors. > > > > > > v5->v6: > > > - Add relevant comments in the io layer. > > > > > > v4->v5: > > > - Fixed a memory leak bug at hw/nvme/ctrl.c. > > > > > > v3->v4: > > > - At the nvme layer, the two patches of enabling the ONCS > > > function and enabling rescap are combined into one. > > > - At the nvme layer, add helper functions for pr capacity > > > conversion between the block layer and the nvme layer. > > > > > > v2->v3: > > > In v2 Persist Through Power Loss(PTPL) is enable default. > > > In v3 PTPL is supported, which is passed as a parameter. > > > > > > v1->v2: > > > - Add sg_persist --report-capabilities for SCSI protocol and enable > > > oncs and rescap for NVMe protocol. > > > - Add persistent reservation capabilities constants and helper functions > > > for > > > SCSI and NVMe protocol. > > > - Add comments for necessary APIs. > > > > > > v1: > > > - Add seven APIs about persistent reservation command for block layer. > > > These APIs including reading keys, reading reservations, registering, > > > reserving, releasing, clearing and preempting. > > > - Add the necessary pr-related operation APIs for both the > > > SCSI protocol and NVMe protocol at the device layer. > > > - Add scsi driver at the driver layer to verify the functions > > > > > > Changqi Lu (10): > > > block: add persistent reservation in/out api > > > block/raw: add persistent reservation in/out driver > > > scsi/constant: add persistent reservation in/out protocol constants > > > scsi/util: add helper functions for persistent reservation types > > > conversion > > > hw/scsi: add persistent reservation in/out api for scsi device > > > block/nvme: add reservation command protocol constants > > > hw/nvme: add helper functions for converting reservation types > > > hw/nvme: enable ONCS and rescap function > > > hw/nvme: add reservation protocal command > > > block/iscsi: add persistent reservation in/out driver > > > > > > block/block-backend.c | 403 ++++++++++++++++++++++++++++ > > > block/io.c | 164 ++++++++++++ > > > block/iscsi.c | 428 ++++++++++++++++++++++++++++++ > > > block/raw-format.c | 56 ++++ > > > hw/nvme/ctrl.c | 350 +++++++++++++++++++++++- > > > hw/nvme/ns.c | 11 + > > > hw/nvme/nvme.h | 93 +++++++ > > > hw/scsi/scsi-disk.c | 374 ++++++++++++++++++++++++++ > > > include/block/block-common.h | 40 +++ > > > include/block/block-io.h | 20 ++ > > > include/block/block_int-common.h | 84 ++++++ > > > include/block/nvme.h | 121 ++++++++- > > > include/scsi/constants.h | 52 ++++ > > > include/scsi/utils.h | 8 + > > > include/sysemu/block-backend-io.h | 24 ++ > > > scsi/utils.c | 81 ++++++ > > > 16 files changed, 2307 insertions(+), 2 deletions(-) > > > > > > -- > > > 2.20.1 > > > > > > >
signature.asc
Description: PGP signature
