Received-SPF: pass (google.com: domain of linux-kernel+bounces-5555-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <6f56726b-8deb-4e67-bdeb-4cfbe56237b0@nvidia.com>
Date: Tue, 19 Dec 2023 18:26:42 +0200
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v6 0/6] nvme: add csi, ms and nuse to sysfs
Content-Language: en-US
To: Keith Busch <kbusch@kernel.org>
Cc: Daniel Wagner <dwagner@suse.de>, linux-nvme@lists.infradead.org,
 linux-kernel@vger.kernel.org, Christoph Hellwig <hch@lst.de>,
 Sagi Grimberg <sagi@grimberg.me>, Hannes Reinecke <hare@suse.de>,
 Chaitanya Kulkarni <chaitanyak@nvidia.com>
References: <20231218165954.29652-1-dwagner@suse.de>
 <ZYDN_G-VP2_pn3kC@kbusch-mbp.dhcp.thefacebook.com>
 <acd4f5b1-b1d7-40e0-ac80-573e2d5cfe85@nvidia.com>
 <ZYG_zlnvfLOJHEJM@kbusch-mbp>
From: Max Gurtovoy <mgurtovoy@nvidia.com>
In-Reply-To: <ZYG_zlnvfLOJHEJM@kbusch-mbp>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?L0o3VHI0ODVZYkdENGduSTlleGNEKzd3RnZCcjB1M3g0ZGZiYnphN3pKUFds?=
 =?utf-8?B?UjlqL1h6dHNUeis2ZGJZdnlFMVNzMHJtbmJvYWpzcUF6WmFVTDFQaXlldFVy?=
 =?utf-8?B?WVIrMUcwZmRxMG16TXZXVVFVUjV0M05zZXFZM05XZ0pmaXBGN3NlaDJjcmlW?=
 =?utf-8?B?bUs1bzZBL0d1M3ZCU2luOGZPYldpRHlnLzdML0hnNG9LcTlNci9sTDRiRVlQ?=
 =?utf-8?B?WVFMc2k0SXRkOXJNckRucmRiSHhIMGd2SExIZm1CWnppRzQ2OG1CWnhxelF4?=
 =?utf-8?B?UmlrVGs2WGlIejJ0SEpKdjdudktUa2lzV3pTcG1sRkpvM3dnSEtLc1c0VUZF?=
 =?utf-8?B?dmpYZWhFbFFVK2xBbmNFY2YvaHBhdC9veDVRSEJlcDNSV1k5d25rbUh6MGY1?=
 =?utf-8?B?QXRsYlp1dis1bS84MThZUGpIVWJWUlY0Z0RHS1dJZVBnYy9iaE5zMUpNTWlL?=
 =?utf-8?B?SlYxWWplSGR3aGp4d3BsWlk3WklYUzRlVCtUb08vdHlCSnkwU2FmRkZkSi9Z?=
 =?utf-8?B?dXV1N0ZuZHJOcXdMOEhBa1EvOVhwUmlpV2kwR1pSZGVYakltWFdGL1JrSm40?=
 =?utf-8?B?elJnVjRRUFNyUzVvdHIrcFE5UXB2TUlYa3c1VGJiaFhiMW9rNWkyaEsxMjZS?=
 =?utf-8?B?YUcyYk1rT2ZNYmowQVNkUGxFekJBVm9ZY1prbUpRWmllTmdub3JXZ25EQ3F4?=
 =?utf-8?B?dVpyK1JVemZtYzdRUWhJM2F5QnU4aFRmSUlOZWZrcFMxZ3dWZ2VER3hEYWJU?=
 =?utf-8?B?aEJXYUZmWE5kZDNHWGZ6cEd1K01SWUZUVlhPNXJEbVpIVldUbS9aK1VTN0tK?=
 =?utf-8?B?UWpyWmUweHRwdFBDYUtYclcyQ2JwZXVhQVZjL0ZlbEJTR2wxcWgyd0NwbVQr?=
 =?utf-8?B?SXNhOXdVWk1CbkRFUkNjYTJyTWFNN3Z0cDl2ZC9xWU5aR1I3R2dMdEFGLzBx?=
 =?utf-8?B?bnVPRkUyMExmTHVhQy9vNElhM2FxUy9FZnU2bXcxNHhUMk5xR1FJUVIrZVpo?=
 =?utf-8?B?STN0dHhQbk5SWWdING02Tm54bEJ2NmpVYmRXeURhNmx6UDBYM09sc1F3QzBl?=
 =?utf-8?B?RHgwUGw3ampHemsvTkZwQzhRSTFwdmFVZVFwUStwN0FXR3VPL0ZFc0dMRytM?=
 =?utf-8?B?V05pVTVJYnQ0eFdkUm5seXlsVzdIK2ZuUlZPa2trR29IanNYNm82R3UvaTUw?=
 =?utf-8?B?eHdUQjVDWEJoMUR1K2ZRY1dERW8yTGVGeTdvakQ1a0pmZjhjak0zWmxxYkEy?=
 =?utf-8?B?V2dreDBHMDJTaGZ5RzYyVkhOem5lUmxyU1Z5RlBsQWZmNXZxZktKQWV5RVRZ?=
 =?utf-8?B?djJYYUJqb3V6blBoV0lwMUtXS1RSU2V1L01XMGx5UWdKU1NwZ1dGWUNRVkNm?=
 =?utf-8?B?UC95UFZQeGlTM3l4S2cyWkZrWXc1N3ZwTktMZnVqOWhjYWJmSGpMWHdnRy95?=
 =?utf-8?B?UXJlSGZiOHNwbjJDdEc5UUpGeWZtQWF2ckgwV3I3Y2V0SklLRGhQUXFrWXp2?=
 =?utf-8?B?YU1nVkFNUWFZN3NPWWJFQnVRUThBWERFNVh5NnZoajhRb3hYUmhLaGt6amlT?=
 =?utf-8?B?Y002eDRwaXNCcENQSFIyZlVMMkRFalBKbzJNS0NDcGhVZW1oaFVSbndjUGps?=
 =?utf-8?B?VnQzZWMxZmQ2Wjl6eHRnTFh3cUFXRVNUbWpXM1BPSDROWFh0MUhzOU9oM0JH?=
 =?utf-8?B?US9MTGY4bEZPTXk5RFBLWGp4dmZRMTF5MU53clZBSXk5TXBsYlVkYmFEQXF3?=
 =?utf-8?B?U0U3azh1Z2FkdXp6YTRnNGFSV2NDbHp3QmhCK0ZWUkMrTDY1M3hIbURnYysz?=
 =?utf-8?B?TmdpUUVpd3Azc1FUcVVYVkhuT0RSZGt6QmhJR3NneVdnVlIvKzB6bjVaVlJQ?=
 =?utf-8?B?eGZmTzhNcHFpb0ppL3crWGIwNlFSOVd0WS9ZNFhqeDVGUXQ4cjJIdTg5cUNG?=
 =?utf-8?B?U2lUbGpwNVpBU3BrMGtBNUZIUmVibUx0REtVK0U0SENzZzBHeVFBNVVXRVY0?=
 =?utf-8?B?cmtiQ1RTeDJuWVBjN2FvVjhEd2NaV1JXU2VKZXVObTJIUGViRW13RkorOHNn?=
 =?utf-8?B?T3lDMFlFQnF3QnVyeW1Ib05FTTBuOEZ1R1BxSEE3YU9BcTNaazRPR2QvSmJj?=
 =?utf-8?B?MGVyK3hzN2hWSTFqZ3RMUXBOejlOQjlTemNmZWxlRzYweUU0UWE3ZXVidlo3?=
 =?utf-8?B?bUE9PQ==?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 98090d25-c3a5-4343-1f9e-08dc00af4c0c
X-MS-Exchange-CrossTenant-AuthSource: DM4PR12MB5040.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Dec 2023 16:26:47.6955
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: S0wpMaJdVjshDFaZ5q9Q3mfFphh2Y8fh7ZmLijfwrf41SqiKpQ7FLj4MQVdObz9HUCekmCvuGgOrAHSLXKx+og==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DS7PR12MB6046


On 19/12/2023 18:07, Keith Busch wrote:
> On Tue, Dec 19, 2023 at 04:42:20PM +0200, Max Gurtovoy wrote:
>> Hi Keith,
>>
>> On 19/12/2023 0:55, Keith Busch wrote:
>>> On Mon, Dec 18, 2023 at 05:59:48PM +0100, Daniel Wagner wrote:
>>>> Rebased on the current nvme/nvme-6.8 branch and added the Review tags. Also
>>>> addressed the printk format issue pointed out by Chaitanya.
>>>
>>> Thanks, added to nvme-6.8.
>>
>> I'm rebasing my fixes to the PI bug (which are not trivial) on top of
>> nvme-6.8 and Daniel's series.
>> I've found that this branch is missing some fixes from the master that I've
>> added manually to local nvme-6.8 branch.
>> When do you plan to rebase the nvme-6.8 on top of master ?
>> It will be helpful, otherwise my patches will not apply cleanly on your
>> branch. :)
> 
> linux-nvme pulls go through Jens' linux-block repo, so we have to be
> based on the appropriate branch from there. We already have some late
> fixes in 6.7 that create minor merge conflicts for our existing 6.8
> branch, so we may provide a reference resolution branch with the pull
> request if conflicts are non-trivial. That might be an option for your
> fixes if you can make them work atop existing nvme-6.8 without the
> conflicting 6.7 changes.
> 
> Sometime after the next merge window opens, Jens will rebase his 6.8
> block tree to Linus', and we usually get a 2nd feature pull request
> within the merge window from that new baseline. That might be an option
> if you want to wait a couple more weeks.

I see. I think that after finishing the latest testing I'll send the 
version on top of nvme-6.8 + my local branch patches I took from 6.7.
We will start the reviewing process and probably will converge in a week 
or two. Then we will see what is the status of the linux-block repo and 
decide how to continue.