Received: by 2002:a05:6a10:22f:0:0:0:0 with SMTP id 15csp3286217pxk; Mon, 5 Oct 2020 06:10:41 -0700 (PDT) X-Google-Smtp-Source: ABdhPJx/4iC/FXHQQewsGSFv3bwx+fvb0b+t5z/diR0UBSrOb3mgCO8mzwtZbZ/6eZMuJz9tig/P X-Received: by 2002:a7b:cc89:: with SMTP id p9mr17562513wma.4.1601903441241; Mon, 05 Oct 2020 06:10:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1601903441; cv=none; d=google.com; s=arc-20160816; b=ijQEJMNiuTAfWXSu4YC5bRSZUb33kkntMWisOshPtOQZE6vkQs3BqzCRxW2gbCQawP Qy98xcteezjH4ihW063x0CV0qf9VY9TWzXrYGzOD0ar3aSuJh13JKtF5pPnlWDFIORso tS+JcoBxelVKIJ1RBUMTCPOUnfRMC58ePwerW0xsNlaIaRrZenepgDK98w/dnFVcMJP+ L3q32S98gPhyzgIgFsO3jY/wQ7Soucdb6v7Bo4ZMLCXUA5eRb7mHQQUkWA/SmsmB9oty 0j17c1yKF17S6e9Uwrfi3kAcfTm7Jn6NRW8ZDTn8DlQ16Jo4uSLvqHi93uX6FmqY6R23 xwQg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:ironport-sdr :dkim-signature; bh=B7/zyK9MhHZQHAm7ESTYiuP9Vp3XE61nN18HNBWs7V4=; b=V9jkoP1mmrOGMGT/GAocjzl3vzQHOHiMadaSLCgc/2+ZxSr5qS9Aei5GeRx4zLY/bt XcFciX9FpplJn18C0aKRokjjioR6VWNtPL6oxn5SeRZpkFP0HsEOTtyLEeV/FB6/2uD7 KlD+heefKeKJOzneWweNRcUmsZkpdaMZYVvuhj4s59Y9olqN0LdWpTLIBf26s5E2Hwnx zT9GSBkLsqyTo2X5GH3NDlvh1/83Ci9CY3uvQeMKz8asMKDrKx9VJTx0r41TKoz+/r5M BxVNsj5TjrPAL1yPtAQKwL90qAP5ilrBhXwTuWPHrFoaQGCyT/7oUVgSFxFKkrmucu8K d+Rw== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@microchip.com header.s=mchp header.b=mzY8iRKY; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=microchip.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id d17si6966081edv.375.2020.10.05.06.10.17; Mon, 05 Oct 2020 06:10:41 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=fail header.i=@microchip.com header.s=mchp header.b=mzY8iRKY; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=microchip.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726329AbgJENHP (ORCPT + 99 others); Mon, 5 Oct 2020 09:07:15 -0400 Received: from esa3.microchip.iphmx.com ([68.232.153.233]:34224 "EHLO esa3.microchip.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725981AbgJENHO (ORCPT ); Mon, 5 Oct 2020 09:07:14 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1601903234; x=1633439234; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=f3Rk8sTIRBiskNiGl0kpAdCKd2pAma8USN5R7liXYu4=; b=mzY8iRKYywYXjMcQoV/4dgV1UUp6rSrS6ovR/yEmlAtk5tGeOseQduSq SEyJ5TdUX62+6G0yv7Ld5eCKrt/lhSwQJp8W6WCbMLT2Db9MmiuhaFu5M WEj2fo3akqm1/kY4TOWGyCVFam91hmIsEUVvxObqP7k3zjhfgDZmyQI5p aXXewmbDYU3NGt5sp9GhaBTpntnP0lA90YJ6mTjP+ctGoZ8mXPdrPAH2Y 8CMK8Yq2LtLiAQltTgSuCr9IGDK3puQB9XEAEbRys4bmNi2gsSFXtwaOB bQHyCgX5cu4cW9X7kt65YoE7egh/+7WyItj9HdI+0L0Ac66LkLskKWAnN A==; IronPort-SDR: hhPTvVM0iAl3d0+AfH3apqIocAbWVfEt9mADUmRBK8hytTT2Gosvsz5ZS17y0OL5MRRjEpOFbv MJNLy/RSiVVpKVTscaufeEBV9KbTorOXZj2Gz7eCvm5fmXwdnIsX7F0aGlJ6lRGPZXwWQ+oHeO 4huPh1kxohxnOIBiZuE+BCdme2ssaBMV4OO9tLT/aFIinggVl5YWZRvfair+EkqKiM758/YCU8 7K+G4CSW0DkT+TJWCWOQGkAiIM1c3f7dyVkXqEZp0tWeR1bgGrZxRV1fMjgqpsPQAE9z0Wg4P4 htQ= X-IronPort-AV: E=Sophos;i="5.77,338,1596524400"; d="scan'208";a="94230142" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa3.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 05 Oct 2020 06:07:13 -0700 Received: from chn-vm-ex01.mchp-main.com (10.10.85.143) by chn-vm-ex04.mchp-main.com (10.10.85.152) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1979.3; Mon, 5 Oct 2020 06:07:13 -0700 Received: from localhost (10.10.115.15) by chn-vm-ex01.mchp-main.com (10.10.85.143) with Microsoft SMTP Server id 15.1.1979.3 via Frontend Transport; Mon, 5 Oct 2020 06:07:10 -0700 Date: Mon, 5 Oct 2020 15:07:12 +0200 From: "Allan W. Nielsen" To: Jiri Pirko CC: Henrik Bjoernlund , , , , , , , , , , , "Horatiu Vultur" Subject: Re: [net-next v2 10/11] bridge: switchdev: cfm: switchdev interface implementation Message-ID: <20201005130712.ybbgiddb7bnbkz6h@ws.localdomain> References: <20201001103019.1342470-1-henrik.bjoernlund@microchip.com> <20201001103019.1342470-11-henrik.bjoernlund@microchip.com> <20201001124929.GM8264@nanopsycho> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8"; format=flowed Content-Disposition: inline In-Reply-To: <20201001124929.GM8264@nanopsycho> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Jiri On 01.10.2020 14:49, Jiri Pirko wrote: >EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > >Thu, Oct 01, 2020 at 12:30:18PM CEST, henrik.bjoernlund@microchip.com wrote: >>This is the definition of the CFM switchdev interface. >> >>The interface consist of these objects: >> SWITCHDEV_OBJ_ID_MEP_CFM, >> SWITCHDEV_OBJ_ID_MEP_CONFIG_CFM, >> SWITCHDEV_OBJ_ID_CC_CONFIG_CFM, >> SWITCHDEV_OBJ_ID_CC_PEER_MEP_CFM, >> SWITCHDEV_OBJ_ID_CC_CCM_TX_CFM, >> SWITCHDEV_OBJ_ID_MEP_STATUS_CFM, >> SWITCHDEV_OBJ_ID_PEER_MEP_STATUS_CFM >> >>MEP instance add/del >> switchdev_port_obj_add(SWITCHDEV_OBJ_ID_MEP_CFM) >> switchdev_port_obj_del(SWITCHDEV_OBJ_ID_MEP_CFM) >> >>MEP cofigure >> switchdev_port_obj_add(SWITCHDEV_OBJ_ID_MEP_CONFIG_CFM) >> >>MEP CC cofigure >> switchdev_port_obj_add(SWITCHDEV_OBJ_ID_CC_CONFIG_CFM) >> >>Peer MEP add/del >> switchdev_port_obj_add(SWITCHDEV_OBJ_ID_CC_PEER_MEP_CFM) >> switchdev_port_obj_del(SWITCHDEV_OBJ_ID_CC_PEER_MEP_CFM) >> >>Start/stop CCM transmission >> switchdev_port_obj_add(SWITCHDEV_OBJ_ID_CC_CCM_TX_CFM) >> >>Get MEP status >> switchdev_port_obj_get(SWITCHDEV_OBJ_ID_MEP_STATUS_CFM) >> >>Get Peer MEP status >> switchdev_port_obj_get(SWITCHDEV_OBJ_ID_PEER_MEP_STATUS_CFM) >> >>Reviewed-by: Horatiu Vultur >>Signed-off-by: Henrik Bjoernlund > >You have to submit the driver parts as a part of this patchset. >Otherwise it is no good. Fair enough. With MRP we did it like this, and after Nik asked for details on what is being offload, we thought that adding this would help. The reason why we did not include the implementation of this interface is that it is for a new SoC which is still not fully available which is why we have not done the basic SwitchDev driver for it yet. But the basic functionality clearly needs to come first. Our preference is to continue fixing the comments we got on the pure SW implementation and then get back to the SwitchDev offloading. This will mean dropping the last 2 patches in the serie. Does that work for you Jiri, and Nik? /Allan