Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add MLDv1 support for IPv6 multicast #1205

Closed
wants to merge 1 commit into from

Conversation

vishwamartur
Copy link

Related to #1047

Add MLDv1 support for IPv6 multicast group management.

  • Add vJoinMulticastGroup and vLeaveMulticastGroup functions in source/FreeRTOS_IPv6_Utils.c to handle MLDv1 group joining and leaving.
  • Update source/FreeRTOS_IPv6.c to include vSendMLDv1Report and vSendMLDv1Done functions for sending MLDv1 report and done messages.
  • Declare vJoinMulticastGroup and vLeaveMulticastGroup functions in source/include/FreeRTOS_IPv6_Utils.h.
  • Add MLDv1 group management function declarations in source/include/FreeRTOS_IPv6.h.

@vishwamartur vishwamartur requested a review from a team as a code owner November 13, 2024 14:18
Related to FreeRTOS#1047

Add MLDv1 support for IPv6 multicast group management.

* Add `vJoinMulticastGroup` and `vLeaveMulticastGroup` functions in `source/FreeRTOS_IPv6_Utils.c` to handle MLDv1 group joining and leaving.
* Update `source/FreeRTOS_IPv6.c` to include `vSendMLDv1Report` and `vSendMLDv1Done` functions for sending MLDv1 report and done messages.
* Declare `vJoinMulticastGroup` and `vLeaveMulticastGroup` functions in `source/include/FreeRTOS_IPv6_Utils.h`.
* Add MLDv1 group management function declarations in `source/include/FreeRTOS_IPv6.h`.

Signed-off-by: Vishwanath Martur <[email protected]>
void vSendMLDv1Report( const struct xNetworkEndPoint * pxEndPoint,
const IPv6_Address_t * pxAddress )
{
/* Implementation of MLDv1 report sending. */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is the implementation left out?

void vSendMLDv1Done( const struct xNetworkEndPoint * pxEndPoint,
const IPv6_Address_t * pxAddress )
{
/* Implementation of MLDv1 done message sending. */
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above

@tony-josi-aws
Copy link
Member

Please share how you have verified these changes.

Also, PR #1019 adds support for a wider range of multicast functionalities and is already in progress. Please take a look at that PR as well.

@tony-josi-aws
Copy link
Member

Closing this PR as it's been inactive for a month.
Please feel free to reopen or create a new PR once the tests and queries are addressed.
Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants