Skip to content

Re-add Jakub's docs for VSwitch#730

Open
omeh-a wants to merge 1 commit into
mainfrom
vswitch_doc_final
Open

Re-add Jakub's docs for VSwitch#730
omeh-a wants to merge 1 commit into
mainfrom
vswitch_doc_final

Conversation

@omeh-a

@omeh-a omeh-a commented Jun 9, 2026

Copy link
Copy Markdown
Member

Co-authored-by JDuchniewicz j.duchniewicz@unsw.edu.au

Re-submission of #708 with a repaired version history. Just opening this separately to be conservative, in case there might be something else in that history that was usable.

@omeh-a omeh-a force-pushed the vswitch_doc_final branch 2 times, most recently from e3ee5b2 to 7e68539 Compare June 9, 2026 01:05
@omeh-a

omeh-a commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

Aha! Co-authored by works now ...

@omeh-a omeh-a force-pushed the vswitch_doc_final branch from 7e68539 to 9556f2a Compare June 9, 2026 01:09
@omeh-a omeh-a requested a review from Courtney3141 June 9, 2026 01:18
Co-authored-by: JDuchniewicz <j.duchniewicz@unsw.edu.au>
Signed-off-by: Lesley Rossouw <lesley.rossouw@unsw.edu.au>
@Courtney3141

Copy link
Copy Markdown
Contributor

I'll take a look at this next week. I will probably need to add a few things due to the changes I made:

  • Add a README.md to the vswitch example
  • Make sure the PPC API to the vswitch for registering an IP address, checking for reachable clients and querying their IP address is well documented (I updated and standardised this API)
  • Create an issue/RFC for more fine-grained checksum offload handling (so it can be referenced below)
  • Make sure the process of handling broadcast traffic when the NIC supports checksum offload is described accurately, and reference that it is a temporary solution
  • Add a section for limitations/extensions that can be made in the future

Additionally there are a few other tricky details I want to ensure are described clearly.

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