I have tested the below changes suggested by Christoph and it is working as expected. Tested-by: <muneendra.kumar@xxxxxxxxxxxx> -----Original Message----- From: James Smart [mailto:jsmart2021@xxxxxxxxx] Sent: Tuesday, August 2, 2022 2:33 AM To: Christoph Hellwig <hch@xxxxxx>; Ewan D. Milne <emilne@xxxxxxxxxx> Cc: linux-nvme@xxxxxxxxxxxxxxxxxxx; muneendra.kumar@xxxxxxxxxxxx; james.smart@xxxxxxxxxxxx; stable@xxxxxxxxxxxxxxx Subject: Re: [PATCH] Revert "nvme-fc: fold t fc_update_appid into fc_appid_store" On 8/1/2022 11:27 AM, Christoph Hellwig wrote: > On Mon, Aug 01, 2022 at 12:27:13PM -0400, Ewan D. Milne wrote: >> This reverts commit c814153c83a892dfd42026eaa661ae2c1f298792. >> >> The commit c814153c83a8 "nvme-fc: fold t fc_update_appid into >> fc_appid_store" >> changed the userspace interface, because the code that decrements "count" >> to remove a trailing '\n' in the parsing results in the decremented >> value being incorrectly be returned from the sysfs write. Fix this by >> revering the commit. > > Wouldn't something like the patch below be much simpler and cleaner: > > > diff --git a/drivers/nvme/host/fc.c b/drivers/nvme/host/fc.c index > 9987797620b6d..e24ab688f00d5 100644 > --- a/drivers/nvme/host/fc.c > +++ b/drivers/nvme/host/fc.c > @@ -3878,6 +3878,7 @@ static int fc_parse_cgrpid(const char *buf, u64 *id) > static ssize_t fc_appid_store(struct device *dev, > struct device_attribute *attr, const char *buf, size_t count) > { > + size_t orig_count = count; > u64 cgrp_id; > int appid_len = 0; > int cgrpid_len = 0; > @@ -3902,7 +3903,7 @@ static ssize_t fc_appid_store(struct device *dev, > ret = blkcg_set_fc_appid(app_id, cgrp_id, sizeof(app_id)); > if (ret < 0) > return ret; > - return count; > + return orig_count; > } > static DEVICE_ATTR(appid_store, 0200, NULL, fc_appid_store); > #endif /* CONFIG_BLK_CGROUP_FC_APPID */ > Reviewed-by: James Smart <jsmart2021@xxxxxxxxx> looks good on my end. -- james -- This electronic communication and the information and any files transmitted with it, or attached to it, are confidential and are intended solely for the use of the individual or entity to whom it is addressed and may contain information that is confidential, legally privileged, protected by privacy laws, or otherwise restricted from disclosure to anyone else. If you are not the intended recipient or the person responsible for delivering the e-mail to the intended recipient, you are hereby notified that any use, copying, distributing, dissemination, forwarding, printing, or copying of this e-mail is strictly prohibited. If you received this e-mail in error, please return the e-mail to the sender, delete it from your computer, and destroy any printed copy of it.
Attachment:
smime.p7s
Description: S/MIME Cryptographic Signature
