-
Notifications
You must be signed in to change notification settings - Fork 72
feat: Expose other kqueue filters #112
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
Merged
Merged
Changes from all commits
Commits
Show all changes
8 commits
Select commit
Hold shift + click to select a range
397009e
feat: Expose other kqueue filters
notgull 069126d
Fix windows build error
notgull 0a367ca
rustfmt
notgull 0962527
Split registration.rs into three files
notgull 7133b0c
Adjust debug output
notgull c080b95
Replace the Async extension with Filter
notgull 318dd73
Review comments
notgull 8209337
Inline small functions
notgull File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,54 @@ | ||
//! Uses the `async_io::os::kqueue` module to wait for a process to terminate. | ||
//! | ||
//! Run with: | ||
//! | ||
//! ``` | ||
//! cargo run --example kqueue-process | ||
//! ``` | ||
#[cfg(any( | ||
target_os = "macos", | ||
target_os = "ios", | ||
target_os = "tvos", | ||
target_os = "watchos", | ||
target_os = "freebsd", | ||
target_os = "netbsd", | ||
target_os = "openbsd", | ||
target_os = "dragonfly", | ||
))] | ||
fn main() -> std::io::Result<()> { | ||
use std::process::Command; | ||
|
||
use async_io::os::kqueue::{Exit, Filter}; | ||
use futures_lite::future; | ||
|
||
future::block_on(async { | ||
// Spawn a process. | ||
let process = Command::new("sleep") | ||
.arg("3") | ||
.spawn() | ||
.expect("failed to spawn process"); | ||
|
||
// Wrap the process in an `Async` object that waits for it to exit. | ||
let process = Filter::new(Exit::new(process))?; | ||
|
||
// Wait for the process to exit. | ||
process.ready().await?; | ||
|
||
Ok(()) | ||
}) | ||
} | ||
|
||
#[cfg(not(any( | ||
target_os = "macos", | ||
target_os = "ios", | ||
target_os = "tvos", | ||
target_os = "watchos", | ||
target_os = "freebsd", | ||
target_os = "netbsd", | ||
target_os = "openbsd", | ||
target_os = "dragonfly", | ||
)))] | ||
fn main() { | ||
println!("This example only works for kqueue-enabled platforms."); | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
//! Platform-specific functionality. | ||
#[cfg(any( | ||
target_os = "macos", | ||
target_os = "ios", | ||
target_os = "tvos", | ||
target_os = "watchos", | ||
target_os = "freebsd", | ||
target_os = "netbsd", | ||
target_os = "openbsd", | ||
target_os = "dragonfly", | ||
))] | ||
pub mod kqueue; |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,269 @@ | ||
//! Functionality that is only available for `kqueue`-based platforms. | ||
use __private::QueueableSealed; | ||
|
||
use crate::reactor::{Reactor, Readable, Registration}; | ||
use crate::Async; | ||
|
||
use std::convert::{TryFrom, TryInto}; | ||
use std::future::Future; | ||
use std::io::{Error, Result}; | ||
use std::os::unix::io::{AsRawFd, RawFd}; | ||
use std::pin::Pin; | ||
use std::process::Child; | ||
use std::task::{Context, Poll}; | ||
|
||
#[cfg(not(async_io_no_io_safety))] | ||
use std::os::unix::io::{AsFd, BorrowedFd, OwnedFd}; | ||
|
||
/// A wrapper around a queueable object that waits until it is ready. | ||
/// | ||
/// The underlying `kqueue` implementation can be used to poll for events besides file descriptor | ||
/// read/write readiness. This API makes these faculties available to the user. | ||
/// | ||
/// See the [`Queueable`] trait and its implementors for objects that currently support being registered | ||
/// into the reactor. | ||
#[derive(Debug)] | ||
pub struct Filter<T>(Async<T>); | ||
|
||
impl<T> AsRef<T> for Filter<T> { | ||
fn as_ref(&self) -> &T { | ||
self.0.as_ref() | ||
} | ||
} | ||
|
||
impl<T> AsMut<T> for Filter<T> { | ||
fn as_mut(&mut self) -> &mut T { | ||
self.0.as_mut() | ||
} | ||
} | ||
|
||
impl<T: Queueable> Filter<T> { | ||
/// Create a new [`Filter`] around a [`Queueable`]. | ||
/// | ||
/// # Examples | ||
/// | ||
/// ```no_run | ||
/// use std::process::Command; | ||
/// use async_io::os::kqueue::{Exit, Filter}; | ||
/// | ||
/// // Create a new process to wait for. | ||
/// let mut child = Command::new("sleep").arg("5").spawn().unwrap(); | ||
/// | ||
/// // Wrap the process in an `Async` object that waits for it to exit. | ||
/// let process = Filter::new(Exit::new(child)).unwrap(); | ||
/// | ||
/// // Wait for the process to exit. | ||
/// # async_io::block_on(async { | ||
/// process.ready().await.unwrap(); | ||
/// # }); | ||
/// ``` | ||
pub fn new(mut filter: T) -> Result<Self> { | ||
Ok(Self(Async { | ||
source: Reactor::get().insert_io(filter.registration())?, | ||
io: Some(filter), | ||
})) | ||
} | ||
} | ||
|
||
impl<T: AsRawFd> AsRawFd for Filter<T> { | ||
fn as_raw_fd(&self) -> RawFd { | ||
self.0.as_raw_fd() | ||
} | ||
} | ||
|
||
#[cfg(not(async_io_no_io_safety))] | ||
impl<T: AsFd> AsFd for Filter<T> { | ||
fn as_fd(&self) -> BorrowedFd<'_> { | ||
self.0.as_fd() | ||
} | ||
} | ||
|
||
#[cfg(not(async_io_no_io_safety))] | ||
impl<T: AsRawFd + From<OwnedFd>> TryFrom<OwnedFd> for Filter<T> { | ||
type Error = Error; | ||
|
||
fn try_from(fd: OwnedFd) -> Result<Self> { | ||
Ok(Self(Async::try_from(fd)?)) | ||
} | ||
} | ||
|
||
#[cfg(not(async_io_no_io_safety))] | ||
impl<T: Into<OwnedFd>> TryFrom<Filter<T>> for OwnedFd { | ||
type Error = Error; | ||
|
||
fn try_from(filter: Filter<T>) -> Result<Self> { | ||
filter.0.try_into() | ||
} | ||
} | ||
|
||
impl<T> Filter<T> { | ||
/// Gets a reference to the underlying [`Queueable`] object. | ||
/// | ||
/// # Examples | ||
/// | ||
/// ``` | ||
/// use async_io::os::kqueue::{Exit, Filter}; | ||
/// | ||
/// # futures_lite::future::block_on(async { | ||
/// let child = std::process::Command::new("sleep").arg("5").spawn().unwrap(); | ||
/// let process = Filter::new(Exit::new(child)).unwrap(); | ||
/// let inner = process.get_ref(); | ||
/// # }); | ||
/// ``` | ||
pub fn get_ref(&self) -> &T { | ||
self.0.get_ref() | ||
} | ||
|
||
/// Gets a mutable reference to the underlying [`Queueable`] object. | ||
/// | ||
/// # Examples | ||
/// | ||
/// ``` | ||
/// use async_io::os::kqueue::{Exit, Filter}; | ||
/// | ||
/// # futures_lite::future::block_on(async { | ||
/// let child = std::process::Command::new("sleep").arg("5").spawn().unwrap(); | ||
/// let mut process = Filter::new(Exit::new(child)).unwrap(); | ||
/// let inner = process.get_mut(); | ||
/// # }); | ||
/// ``` | ||
pub fn get_mut(&mut self) -> &mut T { | ||
self.0.get_mut() | ||
} | ||
|
||
/// Unwraps the inner [`Queueable`] object. | ||
/// | ||
/// # Examples | ||
/// | ||
/// ``` | ||
/// use async_io::os::kqueue::{Exit, Filter}; | ||
/// | ||
/// # futures_lite::future::block_on(async { | ||
/// let child = std::process::Command::new("sleep").arg("5").spawn().unwrap(); | ||
/// let process = Filter::new(Exit::new(child)).unwrap(); | ||
/// let inner = process.into_inner().unwrap(); | ||
/// # }); | ||
/// ``` | ||
pub fn into_inner(self) -> Result<T> { | ||
self.0.into_inner() | ||
} | ||
|
||
/// Waits until the [`Queueable`] object is ready. | ||
/// | ||
/// This method completes when the underlying [`Queueable`] object has completed. See the documentation | ||
/// for the [`Queueable`] object for more information. | ||
/// | ||
/// # Examples | ||
/// | ||
/// ```no_run | ||
/// use std::process::Command; | ||
/// use async_io::os::kqueue::{Exit, Filter}; | ||
/// | ||
/// # futures_lite::future::block_on(async { | ||
/// let child = Command::new("sleep").arg("5").spawn()?; | ||
/// let process = Filter::new(Exit::new(child))?; | ||
/// | ||
/// // Wait for the process to exit. | ||
/// process.ready().await?; | ||
/// # std::io::Result::Ok(()) }); | ||
/// ``` | ||
pub fn ready(&self) -> Ready<'_, T> { | ||
Ready(self.0.readable()) | ||
} | ||
|
||
/// Polls the I/O handle for readiness. | ||
/// | ||
/// When this method returns [`Poll::Ready`], that means that the OS has delivered a notification | ||
/// that the underlying [`Queueable`] object is ready. See the documentation for the [`Queueable`] | ||
/// object for more information. | ||
/// | ||
/// # Caveats | ||
/// | ||
/// Two different tasks should not call this method concurrently. Otherwise, conflicting tasks | ||
/// will just keep waking each other in turn, thus wasting CPU time. | ||
/// | ||
/// # Examples | ||
/// | ||
/// ```no_run | ||
/// use std::process::Command; | ||
/// use async_io::os::kqueue::{Exit, Filter}; | ||
/// use futures_lite::future; | ||
/// | ||
/// # futures_lite::future::block_on(async { | ||
/// let child = Command::new("sleep").arg("5").spawn()?; | ||
/// let process = Filter::new(Exit::new(child))?; | ||
/// | ||
/// // Wait for the process to exit. | ||
/// future::poll_fn(|cx| process.poll_ready(cx)).await?; | ||
/// # std::io::Result::Ok(()) }); | ||
/// ``` | ||
pub fn poll_ready(&self, cx: &mut Context<'_>) -> Poll<Result<()>> { | ||
self.0.poll_readable(cx) | ||
} | ||
} | ||
|
||
/// Future for [`Filter::ready`]. | ||
#[must_use = "futures do nothing unless you `.await` or poll them"] | ||
#[derive(Debug)] | ||
pub struct Ready<'a, T>(Readable<'a, T>); | ||
|
||
impl<T> Future for Ready<'_, T> { | ||
type Output = Result<()>; | ||
|
||
fn poll(mut self: Pin<&mut Self>, cx: &mut Context<'_>) -> Poll<Self::Output> { | ||
Pin::new(&mut self.0).poll(cx) | ||
} | ||
} | ||
|
||
/// Objects that can be registered into the reactor via a [`Async`](crate::Async). | ||
/// | ||
/// These objects represent other filters associated with the `kqueue` runtime aside from readability | ||
/// and writability. Rather than waiting on readable/writable, they wait on "readiness". This is | ||
/// typically used for signals and child process exits. | ||
pub trait Queueable: QueueableSealed {} | ||
fogti marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
/// An object representing a signal. | ||
/// | ||
/// When registered into [`Async`](crate::Async) via [`with_filter`](AsyncKqueueExt::with_filter), | ||
/// it will return a [`readable`](crate::Async::readable) event when the signal is received. | ||
#[derive(Debug, Copy, Clone, Eq, PartialEq, PartialOrd, Ord, Hash)] | ||
pub struct Signal(pub i32); | ||
|
||
impl QueueableSealed for Signal { | ||
fn registration(&mut self) -> Registration { | ||
Registration::Signal(*self) | ||
} | ||
} | ||
impl Queueable for Signal {} | ||
|
||
/// Wait for a child process to exit. | ||
/// | ||
/// When registered into [`Async`](crate::Async) via [`with_filter`](AsyncKqueueExt::with_filter), | ||
/// it will return a [`readable`](crate::Async::readable) event when the child process exits. | ||
#[derive(Debug)] | ||
pub struct Exit(Option<Child>); | ||
|
||
impl Exit { | ||
/// Create a new `Exit` object. | ||
pub fn new(child: Child) -> Self { | ||
Self(Some(child)) | ||
} | ||
} | ||
|
||
impl QueueableSealed for Exit { | ||
fn registration(&mut self) -> Registration { | ||
Registration::Process(self.0.take().expect("Cannot reregister child")) | ||
} | ||
} | ||
impl Queueable for Exit {} | ||
|
||
mod __private { | ||
use crate::reactor::Registration; | ||
|
||
#[doc(hidden)] | ||
pub trait QueueableSealed { | ||
/// Get a registration object for this filter. | ||
fn registration(&mut self) -> Registration; | ||
} | ||
} |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.