Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <6dced1ba-c468-c88e-f861-9c202e803894@nvidia.com>
Date:   Mon, 10 Jul 2023 19:30:20 +0300
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Thunderbird/102.12.0
Subject: Re: [PATCH blktests v1 2/3] nvme/rc: Avoid triggering host nvme-cli
 autoconnect
Content-Language: en-US
To:     Daniel Wagner <dwagner@suse.de>
Cc:     linux-nvme@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-block@vger.kernel.org, Chaitanya Kulkarni <kch@nvidia.com>,
        Shin'ichiro Kawasaki <shinichiro.kawasaki@wdc.com>,
        Sagi Grimberg <sagi@grimberg.me>,
        Hannes Reinecke <hare@suse.de>,
        James Smart <jsmart2021@gmail.com>,
        Martin Belanger <Martin.Belanger@dell.com>
References: <20230620132703.20648-1-dwagner@suse.de>
 <20230620132703.20648-3-dwagner@suse.de>
 <9c07e4f6-2cf5-b53b-6b48-bd4df8798ee9@nvidia.com>
 <bz2s6nyjwl6l7yfrd4wtjdtousv23ypcdleckibps5lyjpwi4x@wqq6qqesevpk>
 <39f9977e-b34c-f2dd-d356-8c78414a60d1@nvidia.com>
 <jdsoqllqmrqu5j5bt3fudkksmjskgs4ooodynm6yh3b4oc5scn@t3hydrmbb22f>
 <972a06e0-6841-043e-fc00-db7596f664c4@nvidia.com>
 <cskolyrp3s47gnn4nwtqpfxtafzhpirn3hv7ovhnnzpbb3ll2z@oum3v3n7go6l>
From:   Max Gurtovoy <mgurtovoy@nvidia.com>
In-Reply-To: <cskolyrp3s47gnn4nwtqpfxtafzhpirn3hv7ovhnnzpbb3ll2z@oum3v3n7go6l>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?YTN2YmhFMWdPbjIzc28yTGEvQUk3MFFnOGVOVitXS0tMalZaT092RWpnMmdj?=
 =?utf-8?B?d1N4dC9FRVBzN1oyc0g0YmgrUCtJMmY1OHBFdVYxdmUwTHpqUERxcDlVL01r?=
 =?utf-8?B?TTZna1BHZTg2MTFGcnNYNllJTkxrUW5McDFVY24xSHMvSnJDeSt5bFpESDhV?=
 =?utf-8?B?TzczT0k4VzJUMU1sRHJmL2J5Q2hsbjd0cTlwRzVuUDdNYlBkeDBpejhlRXJo?=
 =?utf-8?B?OXJCTVc3SHUyeWVqbDhqMUFKd29VZjI4R0pXekVqUlJITGhUN2FRZ0pMM3Y1?=
 =?utf-8?B?TkUySGErWm5oTWlWM2lZMHUzUHFTRkVMb2U5SHVSUDM1WTlnekZ1QjdqdnBW?=
 =?utf-8?B?SEtqV0p2QmhWTlhuaWNxZk4rZUtSK21tR2s5NWpZc2J3eDgxWmVlR2dBSlFK?=
 =?utf-8?B?Sko3RE04VG5JWm55RU9HZVBlV2lVS20xUVkyZzdLNWtTRUJXd1ZLMWhwNHJN?=
 =?utf-8?B?QkFRVFZBSE94NGxua2QvanNxaFJlRGZNK1A1UTZnRmZoVGR5clU2SjlZbTZH?=
 =?utf-8?B?cDhXS0t5TFBPcFpEcHZqb0dxNGdHMzhXY25mV2JNZHVlNmdWekQ4QlI0eDlz?=
 =?utf-8?B?NXV6SDVJbnJxVElxRFZRTWpsTlQ4VlBMclMzcmQrRWpLTzIyNGNFaHNabXM5?=
 =?utf-8?B?RDI0clU3QzdycnlKQnZoUzhPWE1HalNHUEpWeGV6cFl1Z0F2Y0VudUJxdkF3?=
 =?utf-8?B?NW0rYlp5VVB0Vit1dGxlSmNjMHZIZ1QxdkVOSTBqZ1ZSd0dNV1p4OERXK215?=
 =?utf-8?B?SDFoOXd0dXVPbkhSNzRjd1BQWVQ5NXFPZERXMGhxQkI4SzFPVk1MTmt3VWRJ?=
 =?utf-8?B?Nk5oSGkxUFhtenRoSWpJc003SFpoZ2psVHI5ckhHekVPUHU0Y0RrYnRNYzl0?=
 =?utf-8?B?MW5rTnkzUjJDdFVmd3RGSTZna0tYQThkbzJMUDVuTFZ1RU9GTnBHY25VOHB2?=
 =?utf-8?B?K3NtaXJSR2hUWGxoSytraXAyQU1PbEIzL0NHWHE2ZEluN1ZhYXBrTmJHOGgx?=
 =?utf-8?B?Y1NNSHBPbStNUUh0TlVmdEIyV2ZYRXRNdUtGcW8wMFBQMHR3MDluR0NUcTZ4?=
 =?utf-8?B?UEVkR3ZDaDEzeWZySDQ3U3orZjBnVkpUenYrQk9qRWlFVkVmeG52YnJ0YndF?=
 =?utf-8?B?aTlhZEpOOTZjaTNzcytReUQ5YUtUTjBDbGVONTlBYUlGZ1ZYYWZ5SEJ3QVlk?=
 =?utf-8?B?ZnNRRlRxdWVtVEtOdzhLV1l4MllkQjg1azQ0dWV3L1N6VkRCU2luTDBiVFJz?=
 =?utf-8?B?OWdxeUxOWkRDWEFyRVNNNW9CWHJEMFR2S2hOZUwyQWh1UXQvQmZ3UUhsTndV?=
 =?utf-8?B?dVJDdEN1cno0NExpTWViY2NZWVdUbWZwR0tBbGdMNmkxdy9INGRPeFRNYW12?=
 =?utf-8?B?RVN2Ty9hNXZ4UWFCNEV5WUQxbWcxWmx1aWluZkNBdVBlVlBLaEs4V043Nm9M?=
 =?utf-8?B?Rm5OT2g2SVh1UHZTMStiaDZiVVFDL2pkZk52R2ZwT0RNK1hzd3NUZ0w1K1hB?=
 =?utf-8?B?SWVRTEwyK0FmOWgzZjd3NjBSdWZ6TFJEVUhRcEs1M3cvbFp3dUQ3UVMyY2xp?=
 =?utf-8?B?Z0hnRjNmWGQrd3RMVEhHUi9HNlNKRnJzOXJBVnZwN0gwSitHSG13MUVlQ0g4?=
 =?utf-8?B?bTBMeWZoVUh5TTRTZlA0M0xIZGNqQzR2SkFTbDh1cHg1MldHZitWTW5GWDdZ?=
 =?utf-8?B?OStqdEx3eDhlRGl5VVBPcE9CWDVkMXdLTmZZQ2NUeFdaOHpXcDVTUGw1cXhD?=
 =?utf-8?B?dWhINmo0WnJYVVNjTEhzZHE1SlErcWJ1blZYMVVmT1ZIVTNub1M3R1V6a3da?=
 =?utf-8?B?RFRZYWZld0diU05SR2Nobi9hTnVGNzhJdWN2SEhpNWdPc1cwTjhVSEF5QWFy?=
 =?utf-8?B?MXIzazdzaDJzTVAzbE9zSHpkZy9MWG5xVTMya3dCRkhxTGpqRnRKTE5MRGYw?=
 =?utf-8?B?ZFhZUHJZYnI2WXNCdlc5YUxkZ293ZHdDOVpnakNIendPVVJMRTBqSUFiWjNh?=
 =?utf-8?B?R0hCbjVubVVqSzF2M28yQ0kyMUVjZk1ZRVR4eU5SK0lKbGFia3htbkdGM2p1?=
 =?utf-8?B?eXUyNlNQaEtzZkhmYkpBZlpZQlJya2Q2Q2J1bzM4UnhRWUk5NmRVMUxXVmc1?=
 =?utf-8?B?cVU1RDF6cmJRZ1d3WitpYTVtZ1NCLzVERjE3SkVIRUU0YW1NUUpPTmpGa1Qx?=
 =?utf-8?B?ZFdEcC9wcnI1NXhiRmtLaHFSMmg2akJlb3BNN2VaNFFsQmQrMTZJZmVtY2hv?=
 =?utf-8?B?RDVibW8zemtDbDlkTWdrOVVJRGhnPT0=?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 2afb6a1e-6f25-4aff-6e96-08db8162f87a
X-MS-Exchange-CrossTenant-AuthSource: DM4PR12MB5040.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Jul 2023 16:30:28.1158
 (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: TXPbRsu/jPjL3E0DQpnqIHI2VfO1zdf/2ZubKDC41P6P6duQWDyqQHdRYeiLm6aZwZhi98srS6FmdycFdU7FPg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR12MB8464
X-Spam-Status: No, score=-1.2 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        NICE_REPLY_A,RCVD_IN_DNSWL_BLOCKED,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 10/07/2023 18:03, Daniel Wagner wrote:
> On Mon, Jul 10, 2023 at 03:31:23PM +0300, Max Gurtovoy wrote:
>> I think it is more than just commit message.
> 
> Okay, starting to understand what's the problem.
> 
>> A lot of code that we can avoid was added regarding the --context cmdline
>> argument.
> 
> Correct and it's not optional to get the tests passing for the fc transport.

why the fc needs the --context to pass tests ?

> 
>> Maybe it's worth cleaning it..
> 
> It really solves the problem that the autoconnect setup of nvme-cli is
> distrubing the tests (*). The only other way I found to stop the autoconnect is
> by disabling the udev rule completely. If autoconnect isn't enabled the context
> isn't necessary. Though changing system configuration from blktests seems at bit
> excessive.

we should not stop any autoconnect during blktests. The autoconnect and 
all the system admin services should run normally.
The blktests should not interfere with regular system configuration and 
should not use any sysadmin file/services/devices.

It should create its own subsystems, nqn's, id's, namespaces, etc..

> 
> Another option is to detect if autoconect is enabled and report this when
> starting the tests. In this case we could remove the context part completely.

the --context might be used in the some sysadmin scripts or so. I don't 
see a point to do so in the blktests since we create a new association 
between initiator and target on each test (or on each group of tests).

> Obviously, I would prefer to keep it but I am certaintaly not against dropping
> it and make blktests a bit simpler if this is the preference. I just need to
> remember to disable the autoconnect stuff when using blktests.
> 
> (*) Sure we can fix this but at this point. Though I think it's a bit strange
> for a test suite to depend/interact with external components in this way.