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.

Sign up for GitHub

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

Jump to bottom

New function to forward ACK/DONE/NOOP message to handler #22

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

New function to forward ACK/DONE/NOOP message to handler #22

wants to merge 2 commits into from

Commits on Apr 9, 2024

  1. Fix cargo clippy warnings 

    Signed-off-by: Gris Ge <[email protected]>
    @cathay4t
    cathay4t committed Apr 9, 2024
    Configuration menu
    Copy the full SHA
    8b90890 View commit details
    Browse the repository at this point in the history

  2. New function to forward ACK/DONE/NOOP message to handler 

    Introducing these public functions to allow forwarding ACK/DONE/NOOP
message to handler:
 * `Connection::set_forward_noop()`
 * `Connection::set_forward_done()`
 * `Connection::set_forward_ack()`

By default, we still drop these messages, so rtnetlink/ethtool crates
will not be impacted. If any protocol require the handle of
ACK/DONE/NOOP in handler, please invoke these functions during
connection setup( e.g. rtnetlink has function `new_connection()`).

Signed-off-by: Gris Ge <[email protected]>
    @cathay4t
    cathay4t committed Apr 9, 2024
    Configuration menu
    Copy the full SHA
    644b2fc View commit details
    Browse the repository at this point in the history