Skip to content

rust: update IoctlHandler to allow arbitrary type as well. - #388

Merged
ojeda merged 1 commit into
Rust-for-Linux:rustfrom
wedsonaf:ioctl
Jun 23, 2021
Merged

rust: update IoctlHandler to allow arbitrary type as well.#388
ojeda merged 1 commit into
Rust-for-Linux:rustfrom
wedsonaf:ioctl

Conversation

@wedsonaf

@wedsonafwedsonaf commented Jun 23, 2021

Copy link
Copy Markdown

This is similar to a previous change to FileOperations. It is in
preparation for allowing ioctl handlers to take &Ref<T> as the first
argument, which will be needed in a subsequent change to Ref<T>.

Signed-off-by: Wedson Almeida Filho wedsonaf@google.com

@ksquirrel

This comment has been minimized.

This is similar to a previous change to `FileOperations`. It is in
preparation for allowing ioctl handlers to take `&Ref<T>` as the first
argument, which will be needed in a subsequent change to `Ref<T>`.
Signed-off-by: Wedson Almeida Filho <wedsonaf@google.com>
@ksquirrel

Copy link
Copy Markdown
Member

Review of 12ab579919c3:

@wedsonaf

Copy link
Copy Markdown
Author

Rebased.

@TheSven73TheSven73 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@ojeda
ojeda merged commit a5a22ed into Rust-for-Linux:rustJun 23, 2021
@wedsonaf
wedsonaf deleted the ioctl branch June 24, 2021 06:57
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@wedsonaf@ksquirrel@TheSven73@ojeda