[PATCH 4/4] nvmet: make ver stable once connection established

Chaitanya Kulkarni Chaitanya.Kulkarni at wdc.com
Thu Apr 22 20:01:33 BST 2021


On 4/22/21 01:43, Max Gurtovoy wrote:
> On 4/20/2021 8:47 PM, Chaitanya Kulkarni wrote:
>> On 4/20/21 02:09, Max Gurtovoy wrote:
>>> -static ssize_t nvmet_subsys_attr_version_store(struct config_item *item,
>>> -					       const char *page, size_t count)
>>> +static ssize_t
>>> +nvmet_subsys_attr_version_store_locked(struct nvmet_subsys *subsys,
>>> +		const char *page, size_t count)
>>>   {
>> Seems like we lost the indentation in above line which was there previously.
> We added a new line since the line was > 80 chars.
>
> And we follow the conventions.

Original line has 5 tabs + spaces :-

-static ssize_t nvmet_subsys_attr_version_store(struct config_item *item,$
-^I^I^I^I^I       const char *page, size_t count)$

This change has 2 tabs no spaces, breaks the alignment of args:-

+nvmet_subsys_attr_version_store_locked(struct nvmet_subsys *subsys,$
+^I^Iconst char *page, size_t count)$

Following fits in 70 char(note the tabs before spaces):-

static ssize_t$
nvmet_subsys_attr_version_store_locked(struct nvmet_subsys *subsys,$
^I^I^I^I       const char *page, size_t count)$
It doesn't break convention, also keeps the alignment of args.

>




More information about the Linux-nvme mailing list