From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 295ECC05027 for ; Wed, 8 Feb 2023 19:23:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231804AbjBHTXR (ORCPT ); Wed, 8 Feb 2023 14:23:17 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38818 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231707AbjBHTXP (ORCPT ); Wed, 8 Feb 2023 14:23:15 -0500 Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-db3eur04on2086.outbound.protection.outlook.com [40.107.6.86]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B8A321BD1; Wed, 8 Feb 2023 11:23:14 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=oGP0Ho1EGPKZmt/5Yl42kzllmQHItuw4Seycq+P7/AQI5voPmg0A/d4Pe0NLyqxBbcTEvx9BjHkdWKYXiSKi0GPHjBU/YBHdMSGbUv+OOlCfCudGKGl9lCX+XEKAyXszZ7sm5L65XM+YAI1JMmyzy+S8pZ7IJh4yNvBHj8CpeJKsFgyzhK6TRoOtUb6WNlKjAfXSFE2pDND5hvrqvUQaDPAru0JRxgVboKN9rnI7djmpO6sHmlJP7ZUam8yi+PtcZpo8fKkk/XlVc8uNl2DENCcklLcmq2JstuAf4HqTmjEjwaSiw/04akXW0N46og4MjfGX1lc+3IdvzZ3AYpEwvw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=x3Wh+GpLZfyuo0dqBa0I7D9wEIDKdyod0XNd7GNA3Q0=; b=AOIMosozXCPH0RXZb1hNZHz8fLBhRjYkKrYx27YtPZnFOn0Cah7c6ZDpjCe8J7ZCiW5DZv1VhbH2dPyedQr3uMfm5AGMiyVWQpEHUgzvR7ZrCPmdnTS52Lsk9fklxBj6AUXd0Jl3bW8cGJWjNs8l/mBTGPAk0vy1IF/5bH4n2XJcFTpF7OqmLEWBfqYt0B3rrpJ4aE+zG3+41jmD6RZnuOr4HgnCCtNmR7029V+zhVyBnySNVqTSADZQ7sa3zUrrxRU5r9snwzmxAuWwhcu7Rs4h48udV1pThZ8UxTLa6bAzrKykaSqcth5qY6IsgcMzDAY5BIW1hjkw1m5w6qqlmA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=x3Wh+GpLZfyuo0dqBa0I7D9wEIDKdyod0XNd7GNA3Q0=; b=08o6S/XnAhs59Nr+3XxntV4S0UTFQLqEQDPlrUqe6fE/R4RSesxiYQ56uHw8lLLgiouuNj4RYKm0PwT/9XyvjKaibrVt9EIYOcfyAWXNjrFRiGiVwohduu29+uurOCP5/mEZsq7oIMVYdqCEJen74kfM+olgEcjWnHVsYXT/3oY561u90sVkAryKfsSOJeXbbs2aJdY2OdufDUQJtz/FuYTlz+bjQjZ+Mwivmy5OatG0GaDyMaHiauofgFI27aBKFmOo32NutQkY3wyBkDPsmoMVyhLEVXQAkX9R+XmNtNBnRUsRJIbkULWS9ApsdqfgJCKUsvIBuKWPVITNrvFiVg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=suse.com; Received: from AM5PR04MB3089.eurprd04.prod.outlook.com (2603:10a6:206:b::28) by PA4PR04MB7693.eurprd04.prod.outlook.com (2603:10a6:102:e0::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6064.34; Wed, 8 Feb 2023 19:23:12 +0000 Received: from AM5PR04MB3089.eurprd04.prod.outlook.com ([fe80::9a12:6b0f:b2d:661f]) by AM5PR04MB3089.eurprd04.prod.outlook.com ([fe80::9a12:6b0f:b2d:661f%7]) with mapi id 15.20.6086.017; Wed, 8 Feb 2023 19:23:12 +0000 Message-ID: Date: Wed, 8 Feb 2023 11:23:07 -0800 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.0 Subject: Re: [RFC 0/9] Make iscsid-kernel communications namespace-aware (resent) Content-Language: en-US To: Lee Duncan , linux-scsi@vger.kernel.org, open-iscsi@googlegroups.com, netdev@vger.kernel.org References: From: Lee Duncan In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-ClientProxiedBy: FR2P281CA0097.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:9c::12) To AM5PR04MB3089.eurprd04.prod.outlook.com (2603:10a6:206:b::28) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: AM5PR04MB3089:EE_|PA4PR04MB7693:EE_ X-MS-Office365-Filtering-Correlation-Id: e76a8a72-2522-4ecf-afcc-08db0a09eb17 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: bRuiOdlnjDSk1ZjU8VlWKQcMJS2WRHICQxAZYXYQVSGvghgKbHPWuE0zAHCQP6ICUGSasfoW89iKIt29V5EADdGgW8tkt2/rVah4dUE9Dj8CZqgN924OsD0i665PDi8n+hGO5Mk4+EgFDyWiQmpKDte4kcTHtgaKF83pwGea4vwu3KFp1pfgIE3kTkEsEsM3aneuAZ1Zy3a47n8QRUnqMriuX57dY7ygt/fjGw7vGxR0heXGq/vyFEHY9afIKqEELO8+LaBlUcJ+OrLmtsQN4Flt5tAVzcXPYn1h1zP1zbI90SytvReYeQJ5DULoR4uM+JCND0gpInTgvEE6rj34MXvoOFSsca06IxHV0CxcwWEv2MdPQ2kTV7okCLwqGBxiSwSWc/eoDvhMhMVgowRqVvJxKgQRDC8BHcKTcTJMzvG3uF0S7CZQTiVIkEwFjwj3Cp7fOmOIl8z8MYNw4UZeW82T1w7ZmMhUPnYgy8802T/+TCOKEkQCMutLXNEDcutcMLimZ/DN7EI3Dvs3iyr27DqMBRZw5NXm0JZVM/gg7JvMUXOep1iwr2CXk8hwrO4odZxx3BFU8VBIc7U7JeUu3Jlh7mkpsbJeu9+i7Cd7aNegJZgy/WWoSf+ROtzFCiaOujboO/ShVqs579UZ2UMo5mTtFcuBFNBTbABXhKbz5toaysDYLrIBlzVEPwST2sTQdM+ehmvHf8eACD5GzOWsgTKooCu/CQJpXCN5pmcumqc= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AM5PR04MB3089.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230025)(346002)(396003)(136003)(39860400002)(366004)(376002)(451199018)(6506007)(86362001)(5660300002)(53546011)(66556008)(26005)(6512007)(31686004)(186003)(8936002)(6666004)(2906002)(66946007)(66476007)(8676002)(66899018)(38100700002)(41300700001)(316002)(2616005)(31696002)(478600001)(6486002)(36756003)(83380400001)(45980500001)(43740500002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ajJQNjExOEVMdW9RYlhlT1NWL2YwOHhBQnZ2WDEzL3lGcmxjdEJueEFVeFlv?= =?utf-8?B?WkdHVmlpQ0ZROFBhdnF6M0poY01qZkdaM2J5dyttV2tlQzFWZkVSaVJOMXFK?= =?utf-8?B?NThoWEtOaGtaeThTanFHOThHRXlBQmRua1UxZmdkbkd6a2h1SUM4TnByWWFQ?= =?utf-8?B?OG95UVBkSGRxbC9oRFUwcmh4T01qVkNqL0NQNEtYRXV4aGt6Szl1VHF1R0RM?= =?utf-8?B?cEFrd0VIYUNWbDVXTGM5SFk3QjFRc2RFMDdESzZERDhrRVFiZmRvREpXSU0w?= =?utf-8?B?UzlPYmJSajNkZGxsR0ZVdm9zdW91QzNIT3E1SzlmdHFqTEVqYW8ya0ZHZldp?= =?utf-8?B?ZFpnS0tTZTZ5MHVDRnFaSFdtbitTM25DcjU0cGtnS1F0cEVxSmpIZUFwL29i?= =?utf-8?B?SHpnWEQvaFVwT0dpM0tjWmVxN3dGWGsyK25OWVRSaDJ0cVdSYW9ubjlZMFJM?= =?utf-8?B?VTNISjNtOVAweFA4MWlCbjA5cFBqcmIvVDhZak0xTERHRTBiZmlqMHYrRmVM?= =?utf-8?B?ejV6TVF6Q284OUJLc2dnN2JVM0V3cmIrRmVoWksvNXdzOFBKd3MybVhQNHRw?= =?utf-8?B?SWlUdzRmcm1Pd0RPYXdXVmFJNTNNMnhtNjRGSGE1Vy9XbWxyczNFUXJzWGgv?= =?utf-8?B?cGszZFcxNU9YSS9oUnB0cS9KcXRvUkFaS0tDNnkyMFRJQXp6UXJjSFQ0c29P?= =?utf-8?B?clRvaXNkTnhDeEVicHdIVCs2YU5paStEVnNNNWlLdWxRaS9jc3RyVlJLVUYr?= =?utf-8?B?U2hFYm9SSnRROGNjZ250WXRaS3NRZk85aGdQNDRWdGRLeFQvUjZ0M2RKaXpV?= =?utf-8?B?MjlaWDQ1UGVPZ0oyQzQrVURiZmF5SnR2cDNad0NPelB0RjJlcytSWWZDc1Bj?= =?utf-8?B?ZWhISnAxc1hGZXhXQTI2WGcwSHpLL0FVdE5rZUkyYTQwNm1XUUEza3UvV21u?= =?utf-8?B?WE1tY0xvditwMEozSFRqV0FIbzA2Q0dmTnhJckpiOGhqbjliN1ZnT09ubG5J?= =?utf-8?B?ZStheDFkRkpKL1RFeXdqYzVrcUNKeEgrZ3FnSGxmZHdEUmtpZU1WWVhocGhk?= =?utf-8?B?L0ttUHdjOWNaUGhzY25UcEV4ZzZTYmVBNzk2Z0c1MzBGRUZZVnV6ZWJWUGZ4?= =?utf-8?B?bTNZaTRIenNuRTlRTllLeVhOZlEvT1ExV1NYMWtseENDdjNqc2YrbVVIaHFo?= =?utf-8?B?c0FZV0xNK2d3dlY1czBGWTJ6NllWMmVSa1VwRVFOZkxCOEovRlVEanNxMENF?= =?utf-8?B?UGdtelc4QUNES1hWUzZhclBDZTZQUkRhRzU5YVErRGZ0d0NPMDF6bWt2dExZ?= =?utf-8?B?OU9vMkRnTFdHZ25vc3BjWG1IU3FPaFVtUk5HUm9sbkw3SjJra2lpZy85eVNz?= =?utf-8?B?c1d2cCtvdndycXc3elBHUWtkUVc2a3M0ZTNBZHQ4a1lPaitRck9VVWxUMzcw?= =?utf-8?B?M2hHNTVmb0ZKcURNM2FMREk4TXRQUjkrRlk2OUlrME5YbXRzNTNqbkFRd2dZ?= =?utf-8?B?QzZqUVoxdm95Z1NKSGl1cW5yRWZUdEJMVmJINmZVMjhsTExpYVNpTEVBRUxH?= =?utf-8?B?a0FnRElEQTNFQzNLTDYrdk9pbFRsOVh4dytxeUI1eVFQSUlXMmtOYlFKSHdW?= =?utf-8?B?NUk0MmphcFBoaTR1WFBDWEFOSUlJenIwS25SQ25pSFpJYkw3U2E3RFljem5O?= =?utf-8?B?WHk3cllNV0swT3NFaEdKSHVaQ3kvTlRYaDMrck1MSTNPcjQrWVF4MFRjR2g2?= =?utf-8?B?ckd5UWtkdVRPWjdnNWdPc2VROUZHVjY4K21haHB3ZkpUVGJSWEh0NzBmL2Js?= =?utf-8?B?UHpOMlNiakRFTFQ0K2FDbUtSOVp4REt4M05JbmZ4Ly9YNFJaYUxDNkZid3Qr?= =?utf-8?B?R1UyTzAySnpwOUVZdStWZ3NNRHBxMzdsWElPald3Zk9GYVptYW8xY0VjaUR6?= =?utf-8?B?S3VBd2hGd1p0K09BcUgrRzZNVVB5Q1YxUXFuT3BUYW9ZTENlVDZzT2NnUCt0?= =?utf-8?B?NlBBYkgwbSt5ZllNcnR4U3NCeTVzcjN3dXVzTjU4OC9wMlRvZTh3Z1Fmdnk1?= =?utf-8?B?OGlBTnNaT01RaWJWUXpGSlpONTJodm8vVXVIOGZjT01ia2ZSNDVkZGdkUTlq?= =?utf-8?Q?OUKJlrO+hsLq2pKIZzuJKpE4p?= X-OriginatorOrg: suse.com X-MS-Exchange-CrossTenant-Network-Message-Id: e76a8a72-2522-4ecf-afcc-08db0a09eb17 X-MS-Exchange-CrossTenant-AuthSource: AM5PR04MB3089.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Feb 2023 19:23:11.9801 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: f7a17af6-1c5c-4a36-aa8b-f5be247aa4ba X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: RyNAqGLjSCtFvrZecrCSqV0K8VRiBp1Zj2X7biLZqM6PTJWYHJ47XOpRGo88UNRgUrf5+Xzx6VywoT2UklnheQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR04MB7693 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org [Resent to include all mailing lists] I wanted to mention some issues I've discovered as part of testing this: - Currently, only some sysfs entries are going to be different per namespace - This means that the Configuration and Initiator Name are going to be common to all running daemons (this is /etc/iscsi) - This also means that the Node database (and discovery DB, and interface DB) are common to all running daemons I'm really not sure all running daemons should have the same initiator name. If we think of them as separate initiators, then this seems wrong. Sharing the Node database may not be a good idea, either. This assumes that nodes discovered (and saved) from one namespace can actually be reached from other namespaces, but this may not be true. Having the Node DB and initiatorname shared means the different iscsid instances must cooperate with each other, else their requests can collide. Also, I can imagine situations where different daemons may want to set different configuration values. Currently they cannot. On 2/8/23 09:40, Lee Duncan wrote: > From: Lee Duncan > > This is a request for comment on a set of patches that > modify the kernel iSCSI initiator communications so that > they are namespace-aware. The goal is to allow multiple > iSCSI daemon (iscsid) to run at once as long as they > are in separate namespaces, and so that iscsid can > run in containers. > > Comments and suggestions are more than welcome. I do not > expect that this code is production-ready yet, and > networking isn't my strongest suit (yet). > > These patches were originally posted in 2015 by Chris > Leech. There were some issues at the time about how > to handle namespaces going away. I hope to address > any issues raised with this patchset and then > to merge these changes upstream to address working > in working in containers. > > My contribution thus far has been to update these patches > to work with the current upstream kernel. > > Chris Leech/Lee Duncan (9): > iscsi: create per-net iscsi netlink kernel sockets > iscsi: associate endpoints with a host > iscsi: sysfs filtering by network namespace > iscsi: make all iSCSI netlink multicast namespace aware > iscsi: set netns for iscsi_tcp hosts > iscsi: check net namespace for all iscsi lookup > iscsi: convert flashnode devices from bus to class > iscsi: rename iscsi_bus_flash_* to iscsi_flash_* > iscsi: filter flashnode sysfs by net namespace > > drivers/infiniband/ulp/iser/iscsi_iser.c | 7 +- > drivers/scsi/be2iscsi/be_iscsi.c | 6 +- > drivers/scsi/bnx2i/bnx2i_iscsi.c | 6 +- > drivers/scsi/cxgbi/libcxgbi.c | 6 +- > drivers/scsi/iscsi_tcp.c | 7 + > drivers/scsi/qedi/qedi_iscsi.c | 6 +- > drivers/scsi/qla4xxx/ql4_os.c | 64 +-- > drivers/scsi/scsi_transport_iscsi.c | 625 ++++++++++++++++------- > include/scsi/scsi_transport_iscsi.h | 63 ++- > 9 files changed, 537 insertions(+), 253 deletions(-) >