Messages in this thread |  | | Date | Thu, 4 Sep 2025 19:54:45 +0530 | | From | Neeraj Kumar <> | | Subject | Re: [PATCH V2 07/20] nvdimm/namespace_label: Update namespace init_labels and its region_uuid |
| |
On 13/08/25 03:58PM, Jonathan Cameron wrote: >On Wed, 30 Jul 2025 17:41:56 +0530 >Neeraj Kumar <s.neeraj@samsung.com> wrote: > >> nd_mapping->labels maintains the list of labels present into LSA. >> init_labels() prepares this list while adding new label into LSA >> and updates nd_mapping->labels accordingly. During cxl region >> creation nd_mapping->labels list and LSA was updated with one >> region label. Therefore during new namespace label creation >> pre-include the previously created region label, so increase >> num_labels count by 1. >> >> Also updated nsl_set_region_uuid with region uuid with which >> namespace is associated with. > >Any reason these are in the same patch? I'd like to have >seen a bit more on why this 'Also' change is here and a separate >patch might make that easier to see.
I will create separate commit to update region_uuid and will remove 'Also' part from commit message. Yes both hunks are not associated, I just added both to avoid extra commit.
Regards, Neeraj
|  |