Received-SPF: pass (google.com: domain of linux-kernel+bounces-158015-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
From: Geethasowjanya Akula <gakula@marvell.com>
To: Jiri Pirko <jiri@resnulli.us>
CC: "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "kuba@kernel.org" <kuba@kernel.org>,
        "davem@davemloft.net"
	<davem@davemloft.net>,
        "pabeni@redhat.com" <pabeni@redhat.com>,
        "edumazet@google.com" <edumazet@google.com>,
        Sunil Kovvuri Goutham
	<sgoutham@marvell.com>,
        Subbaraya Sundeep Bhatta <sbhatta@marvell.com>,
        Hariprasad Kelam <hkelam@marvell.com>
Subject: RE: [EXTERNAL] Re: [net-next PATCH v2 7/9] octeontx2-pf: Add support
 to sync link state between representor and VFs
Thread-Topic: [EXTERNAL] Re: [net-next PATCH v2 7/9] octeontx2-pf: Add support
 to sync link state between representor and VFs
Thread-Index: AQHalJsVB1fMpBcrLE6y6oPYaOQD67F11OkAgAAswtCAAYApgIAA8oPg
Date: Thu, 25 Apr 2024 05:11:22 +0000
Message-ID: 
 <CH0PR18MB43397A40A3073E3F995286E5CD172@CH0PR18MB4339.namprd18.prod.outlook.com>
References: <20240422095401.14245-1-gakula@marvell.com>
 <20240422095401.14245-8-gakula@marvell.com> <ZieyWKC7ReztKRWF@nanopsycho>
 <BL1PR18MB43427F05CB1F5D153DD54907CD112@BL1PR18MB4342.namprd18.prod.outlook.com>
 <ZikaJcqEqwhN-RSE@nanopsycho>
In-Reply-To: <ZikaJcqEqwhN-RSE@nanopsycho>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 
 =?us-ascii?Q?f2Nepk1YTbIrRO+t3DXQjM/HnVwE/hS5wQcEi0oVbPWv5oa+shacVVn9Q2p3?=
 =?us-ascii?Q?tcK6hCIaZBR6n4kJfIVGv7HfhkLHqZL1ZGcqUC40moWEHB1iJGXyqn7MnRbS?=
 =?us-ascii?Q?HPR5DnLTb+0pTbKHNh1k8qKmL/Vw/49l3e0rHW/BtPVCgGAlASLwyLNK+jTX?=
 =?us-ascii?Q?7h2Q4xrIgMRWbZFpwO4VVzofjBjh8DBB93xGxv6w4xetR9C2ymslRVnAVmYr?=
 =?us-ascii?Q?HKhFdYOAdtF/6y6eoBI12kLvyC/OBzxYLFC4yCHMeVP4ke8/1Y1smVsPy6Tc?=
 =?us-ascii?Q?QKQuKIjp728j+ylaZR4PR7fc6eba/SnU59oEv+urzs25BpW3QxE9nfCpYiqB?=
 =?us-ascii?Q?bnrFJQASCQyOB1ZSn4M4dkaEy2Y+rR0FgMnQJIoKfWSYWvoTVpaOk8Du9owI?=
 =?us-ascii?Q?L01uh0XesztgFXGCkORrLlSNF5Ui1TchRtCigFnaUZ5aV5dLc3V1GOH7tjCM?=
 =?us-ascii?Q?O4bsrvIxRdNyHFKicklFzmbL7zY0S3md1nM5b5CCNfHq1jnv7CsmTxFtlEk4?=
 =?us-ascii?Q?xHEYVp9uOBYvHBpPE/6+zWIAa/B0PjxuntgoY893VL6RkWBZfJPwU9Qa1kNC?=
 =?us-ascii?Q?PbHOgpysEfyX/CoYbcy0xVXhYpH20IMZX3nd6O9w9vETNBQQ26ZUtJVN/nbL?=
 =?us-ascii?Q?dNAX7+ctLrcwuzZLI9FiKJsaZNdwMeFVMsSOBbYm5dZtrvElu6OfacoIb96p?=
 =?us-ascii?Q?uLtN0uzu6dNEnBSSEkCTqjA37TpQ0C0SAddR8AAJf9YL4mXFqad/PegrcXgP?=
 =?us-ascii?Q?kfPpY60v1L0bTJftUQ001XaBy98/w2ZoAfKok+Wtir1iR6nuDL9reRxvRVLW?=
 =?us-ascii?Q?6s2NX5ZzQzczNrHG7h/F6nsCQCKzs/dxHp+YzRPpBLqSI4qxFAB6MEku3B8S?=
 =?us-ascii?Q?Y943AF3FvSRrYRctCNTN5mh1HiYw5eSyR+jge1wAaoBRSbzEMDp0cmHlYZC4?=
 =?us-ascii?Q?ieD1bEJrYWJHgUvEmtRun+a2wLu/7udgMNf7cmWyQfLXsQEm1QrR2KSjV0dt?=
 =?us-ascii?Q?KjFi0ztL4t0b2NmRx51JgNp7AgcteI3sf8SlcHBZY4JW41QOYZ5nH6Q8/C8T?=
 =?us-ascii?Q?mHRdexum2FhklqOlEp1EIcLjWfQ+RjFdAcCi8S0PFlBQbTIsyJjreaxHATtq?=
 =?us-ascii?Q?ejkGhfnZy2M757/kRukez3RVOGjyGStWHwKvivBh3DUPnojoeHNFQ7vL41jZ?=
 =?us-ascii?Q?uQBXh6NqC97PkBZ0KhiriX7Eadax4GrVJKIk72Xr8YRO5IKXr0RsPyMHF3oI?=
 =?us-ascii?Q?/xJmlmlhAZewI03JleY70cCV4/19cvTAk7CrQMWJ4h9L7J8ffrZuFIaEn9BQ?=
 =?us-ascii?Q?9qMOEeQHmIWVLjx4dngfor6W31HWV9Xq/0XyJqrfYEGjBy0oncw5VS5PVbHF?=
 =?us-ascii?Q?GbUmJgPPqsydK7sE5fCpnnnw1T2PsCNulY24aCIgdxLvnUKmTPUATBEo3/VL?=
 =?us-ascii?Q?tA2PsWXiLcz9SC70tVqcE9W7MzT6zaaEgRosU9iX+XdfnvN0ZCmRfiNb5gHM?=
 =?us-ascii?Q?p3ax6iIMTR0U9PTHZ3HckK7SNtH0Yiz/SAJluLVmxSdvZk1qqDGOR/uF0LmU?=
 =?us-ascii?Q?O9dqcyFm4MRvI688IBw=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: marvell.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CH0PR18MB4339.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: d242269a-20ce-4a1c-e2e8-08dc64e625e4
X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Apr 2024 05:11:22.0240
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: wrF7VB1uG44wjYDEfv3h5s0PWrfG3MgLpbM5ot0zXBTEqD+VNtC7asjI5YZIurjg501IR0UNUQ4ftFQy/i2Ccg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM8PR18MB4424
X-Proofpoint-GUID: k1x6ZXk_RldlDoHW5yLLZPzJgSLYMn7A
X-Proofpoint-ORIG-GUID: k1x6ZXk_RldlDoHW5yLLZPzJgSLYMn7A
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.293,Aquarius:18.0.1011,Hydra:6.0.650,FMLib:17.11.176.26
 definitions=2024-04-25_04,2024-04-24_01,2023-05-22_02


> -----Original Message-----
> From: Jiri Pirko <jiri@resnulli.us>
> Sent: Wednesday, April 24, 2024 8:12 PM
> To: Geethasowjanya Akula <gakula@marvell.com>
> Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org; kuba@kernel.org=
;
> davem@davemloft.net; pabeni@redhat.com; edumazet@google.com; Sunil
> Kovvuri Goutham <sgoutham@marvell.com>; Subbaraya Sundeep Bhatta
> <sbhatta@marvell.com>; Hariprasad Kelam <hkelam@marvell.com>
> Subject: Re: [EXTERNAL] Re: [net-next PATCH v2 7/9] octeontx2-pf: Add
> support to sync link state between representor and VFs
>=20
> Tue, Apr 23, 2024 at 06:09:02PM CEST, gakula@marvell.com wrote:
> >
> >
> >> -----Original Message-----
> >> From: Jiri Pirko <jiri@resnulli.us>
> >> Sent: Tuesday, April 23, 2024 6:37 PM
> >> To: Geethasowjanya Akula <gakula@marvell.com>
> >> Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org;
> >> kuba@kernel.org; davem@davemloft.net; pabeni@redhat.com;
> >> edumazet@google.com; Sunil Kovvuri Goutham
> <sgoutham@marvell.com>;
> >> Subbaraya Sundeep Bhatta <sbhatta@marvell.com>; Hariprasad Kelam
> >> <hkelam@marvell.com>
> >> Subject: [EXTERNAL] Re: [net-next PATCH v2 7/9] octeontx2-pf: Add
> >> support to sync link state between representor and VFs
> >>
> >> Prioritize security for external emails: Confirm sender and content
> >> safety before clicking links or opening attachments
> >>
> >> ---------------------------------------------------------------------
> >> - Mon, Apr 22, 2024 at 11:53:59AM CEST, gakula@marvell.com wrote:
> >> >Implements mbox function to sync the link state between VFs and its
> >> >representors. Same mbox is use to notify other updates like mtu etc.
> >> >
> >> >This patch enables
> >> >- Reflecting the link state of representor based on the VF state and
> >> >link state of VF based on representor.
> >>
> >> Could you please elaborate a bit more how exactly this behaves?
> >> Examples would help.
> >>
> >We patch implement the below requirement mentioned the representors
> documentation.
> >Eg: ip link set r0p1 up/down  will disable carrier on/off of the corresp=
onding
> representee(eth0) interface.
> >
> >
> >"
> >The representee's link state is controlled through the representor. Sett=
ing the
> representor administratively UP or DOWN should cause carrier ON or OFF at
> the representee.
>=20
> Put these into patch description please.
Sure. Will rewrite commit message with commands and its output in next vers=
ion.
>=20
> >"
> >
> >>
> >> >- On VF interface up/down a notification is sent via mbox to
> >> >representor
> >> >  to update the link state.
> >> >- On representor interafce up/down will cause the link state update o=
f VF.
> >> >