2024-04-23 16:09:50

by Geethasowjanya Akula

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



> -----Original Message-----
> From: Jiri Pirko <[email protected]>
> Sent: Tuesday, April 23, 2024 6:37 PM
> To: Geethasowjanya Akula <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected]; Sunil
> Kovvuri Goutham <[email protected]>; Subbaraya Sundeep Bhatta
> <[email protected]>; Hariprasad Kelam <[email protected]>
> 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, [email protected] 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.
"

>
> >- 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.
> >


2024-04-24 14:42:37

by Jiri Pirko

[permalink] [raw]
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, [email protected] wrote:
>
>
>> -----Original Message-----
>> From: Jiri Pirko <[email protected]>
>> Sent: Tuesday, April 23, 2024 6:37 PM
>> To: Geethasowjanya Akula <[email protected]>
>> Cc: [email protected]; [email protected]; [email protected];
>> [email protected]; [email protected]; [email protected]; Sunil
>> Kovvuri Goutham <[email protected]>; Subbaraya Sundeep Bhatta
>> <[email protected]>; Hariprasad Kelam <[email protected]>
>> 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, [email protected] 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.

>"
>
>>
>> >- 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.
>> >

2024-04-25 05:17:39

by Geethasowjanya Akula

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



> -----Original Message-----
> From: Jiri Pirko <[email protected]>
> Sent: Wednesday, April 24, 2024 8:12 PM
> To: Geethasowjanya Akula <[email protected]>
> Cc: [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected]; Sunil
> Kovvuri Goutham <[email protected]>; Subbaraya Sundeep Bhatta
> <[email protected]>; Hariprasad Kelam <[email protected]>
> 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, [email protected] wrote:
> >
> >
> >> -----Original Message-----
> >> From: Jiri Pirko <[email protected]>
> >> Sent: Tuesday, April 23, 2024 6:37 PM
> >> To: Geethasowjanya Akula <[email protected]>
> >> Cc: [email protected]; [email protected];
> >> [email protected]; [email protected]; [email protected];
> >> [email protected]; Sunil Kovvuri Goutham
> <[email protected]>;
> >> Subbaraya Sundeep Bhatta <[email protected]>; Hariprasad Kelam
> >> <[email protected]>
> >> 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, [email protected] 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.
> >> >