RE: [EXTERNAL] Re: [net-next PATCH v2 7/9] octeontx2-pf: Add support to sync link state between representor and VFs

[Geethasowjanya Akula]

> -----Original Message-----
> From: Jiri Pirko <jiri@xxxxxxxxxxx>
> Sent: Wednesday, April 24, 2024 8:12 PM
> To: Geethasowjanya Akula <gakula@xxxxxxxxxxx>
> Cc: netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; kuba@xxxxxxxxxx;
> davem@xxxxxxxxxxxxx; pabeni@xxxxxxxxxx; edumazet@xxxxxxxxxx; Sunil
> Kovvuri Goutham <sgoutham@xxxxxxxxxxx>; Subbaraya Sundeep Bhatta
> <sbhatta@xxxxxxxxxxx>; Hariprasad Kelam <hkelam@xxxxxxxxxxx>
> Subject: Re: [EXTERNAL] Re: [net-next PATCH v2 7/9] octeontx2-pf: Add
> support to sync link state between representor and VFs
> 
> Tue, Apr 23, 2024 at 06:09:02PM CEST, gakula@xxxxxxxxxxx wrote:
> >
> >
> >> -----Original Message-----
> >> From: Jiri Pirko <jiri@xxxxxxxxxxx>
> >> Sent: Tuesday, April 23, 2024 6:37 PM
> >> To: Geethasowjanya Akula <gakula@xxxxxxxxxxx>
> >> Cc: netdev@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> >> kuba@xxxxxxxxxx; davem@xxxxxxxxxxxxx; pabeni@xxxxxxxxxx;
> >> edumazet@xxxxxxxxxx; Sunil Kovvuri Goutham
> <sgoutham@xxxxxxxxxxx>;
> >> Subbaraya Sundeep Bhatta <sbhatta@xxxxxxxxxxx>; Hariprasad Kelam
> >> <hkelam@xxxxxxxxxxx>
> >> 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@xxxxxxxxxxx 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 corresponding
> representee(eth0) interface.
> >
> >
> >"
> >The representee's link state is controlled through the representor. Setting the
> representor administratively UP or DOWN should cause carrier ON or OFF at
> the representee.
> 
> Put these into patch description please.
Sure. Will rewrite commit message with commands and its output in next version.
> 
> >"
> >
> >>
> >> >- 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 of VF.
> >> >