Merge branch 'master' into add_stdin_lock

Author: k-nasa

.github/workflows/ci.yml

@@ -52,15 +52,10 @@ jobs:
  steps:
  - uses: actions/checkout@master
 
- - id: component
- uses: actions-rs/components-nightly@v1
- with:
- component: rustfmt
-
  - uses: actions-rs/toolchain@v1
  with:
  profile: minimal
- toolchain: ${{ steps.component.outputs.toolchain }}
+ toolchain: nightly
  override: true
  components: rustfmt
 

Cargo.toml

@@ -27,23 +27,23 @@ unstable = ["broadcaster"]
 [dependencies]
 async-macros = "1.0.0"
 async-task = "1.0.0"
+broadcaster = { version = "0.2.6", optional = true, default-features = false, features = ["default-channels"] }
 crossbeam-channel = "0.3.9"
 crossbeam-deque = "0.7.1"
 crossbeam-utils = "0.6.6"
 futures-core-preview = "=0.3.0-alpha.19"
 futures-io-preview = "=0.3.0-alpha.19"
 futures-timer = "1.0.2"
+kv-log-macro = "1.0.4"
 log = { version = "0.4.8", features = ["kv_unstable"] }
 memchr = "2.2.1"
 mio = "0.6.19"
 mio-uds = "0.6.7"
 num_cpus = "1.10.1"
 once_cell = "1.2.0"
+pin-project-lite = "0.1"
 pin-utils = "0.1.0-alpha.4"
 slab = "0.4.2"
-kv-log-macro = "1.0.4"
-broadcaster = { version = "0.2.6", optional = true, default-features = false, features = ["default-channels"] }
-pin-project-lite = "0.1"
 
 [dev-dependencies]
 femme = "1.2.0"

benches/mutex.rs

@@ -0,0 +1,42 @@
+#![feature(test)]
+
+extern crate test;
+
+use std::sync::Arc;
+
+use async_std::sync::Mutex;
+use async_std::task;
+use test::Bencher;
+
+#[bench]
+fn create(b: &mut Bencher) {
+ b.iter(|| Mutex::new(()));
+}
+
+#[bench]
+fn contention(b: &mut Bencher) {
+ b.iter(|| task::block_on(run(10, 1000)));
+}
+
+#[bench]
+fn no_contention(b: &mut Bencher) {
+ b.iter(|| task::block_on(run(1, 10000)));
+}
+
+async fn run(task: usize, iter: usize) {
+ let m = Arc::new(Mutex::new(()));
+ let mut tasks = Vec::new();
+
+ for _ in 0..task {
+ let m = m.clone();
+ tasks.push(task::spawn(async move {
+ for _ in 0..iter {
+ let _ = m.lock().await;
+ }
+ }));
+ }
+
+ for t in tasks {
+ t.await;
+ }
+}

benches/task.rs

@@ -0,0 +1,11 @@
+#![feature(test)]
+
+extern crate test;
+
+use async_std::task;
+use test::Bencher;
+
+#[bench]
+fn block_on(b: &mut Bencher) {
+ b.iter(|| task::block_on(async {}));
+}

src/fs/canonicalize.rs

@@ -1,6 +1,6 @@
 use crate::io;
 use crate::path::{Path, PathBuf};
-use crate::task::blocking;
+use crate::task::spawn_blocking;
 
 /// Returns the canonical form of a path.
 ///
@@ -32,5 +32,5 @@ use crate::task::blocking;
 /// ```
 pub async fn canonicalize<P: AsRef<Path>>(path: P) -> io::Result<PathBuf> {
  let path = path.as_ref().to_owned();
- blocking::spawn(move || std::fs::canonicalize(&path).map(Into::into)).await
+ spawn_blocking(move || std::fs::canonicalize(&path).map(Into::into)).await
 }

src/fs/copy.rs

@@ -1,6 +1,6 @@
 use crate::io;
 use crate::path::Path;
-use crate::task::blocking;
+use crate::task::spawn_blocking;
 
 /// Copies the contents and permissions of a file to a new location.
 ///
@@ -41,5 +41,5 @@ use crate::task::blocking;
 pub async fn copy<P: AsRef<Path>, Q: AsRef<Path>>(from: P, to: Q) -> io::Result<u64> {
  let from = from.as_ref().to_owned();
  let to = to.as_ref().to_owned();
- blocking::spawn(move || std::fs::copy(&from, &to)).await
+ spawn_blocking(move || std::fs::copy(&from, &to)).await
 }

src/fs/create_dir.rs

@@ -1,6 +1,6 @@
 use crate::io;
 use crate::path::Path;
-use crate::task::blocking;
+use crate::task::spawn_blocking;
 
 /// Creates a new directory.
 ///
@@ -34,5 +34,5 @@ use crate::task::blocking;
 /// ```
 pub async fn create_dir<P: AsRef<Path>>(path: P) -> io::Result<()> {
  let path = path.as_ref().to_owned();
- blocking::spawn(move || std::fs::create_dir(path)).await
+ spawn_blocking(move || std::fs::create_dir(path)).await
 }

src/fs/create_dir_all.rs

@@ -1,6 +1,6 @@
 use crate::io;
 use crate::path::Path;
-use crate::task::blocking;
+use crate::task::spawn_blocking;
 
 /// Creates a new directory and all of its parents if they are missing.
 ///
@@ -29,5 +29,5 @@ use crate::task::blocking;
 /// ```
 pub async fn create_dir_all<P: AsRef<Path>>(path: P) -> io::Result<()> {
  let path = path.as_ref().to_owned();
- blocking::spawn(move || std::fs::create_dir_all(path)).await
+ spawn_blocking(move || std::fs::create_dir_all(path)).await
 }

src/fs/dir_builder.rs

@@ -2,7 +2,7 @@ use std::future::Future;
 
 use crate::io;
 use crate::path::Path;
-use crate::task::blocking;
+use crate::task::spawn_blocking;
 
 /// A builder for creating directories with configurable options.
 ///
@@ -107,7 +107,7 @@ impl DirBuilder {
  }
 
  let path = path.as_ref().to_owned();
- async move { blocking::spawn(move || builder.create(path)).await }
+ async move { spawn_blocking(move || builder.create(path)).await }
  }
 }
 

src/fs/dir_entry.rs

@@ -5,7 +5,7 @@ use std::sync::Arc;
 use crate::fs::{FileType, Metadata};
 use crate::io;
 use crate::path::PathBuf;
-use crate::task::blocking;
+use crate::task::spawn_blocking;
 
 /// An entry in a directory.
 ///
@@ -87,7 +87,7 @@ impl DirEntry {
  /// ```
  pub async fn metadata(&self) -> io::Result<Metadata> {
  let inner = self.0.clone();
- blocking::spawn(move || inner.metadata()).await
+ spawn_blocking(move || inner.metadata()).await
  }
 
  /// Reads the file type for this entry.
@@ -125,7 +125,7 @@ impl DirEntry {
  /// ```
  pub async fn file_type(&self) -> io::Result<FileType> {
  let inner = self.0.clone();
- blocking::spawn(move || inner.file_type()).await
+ spawn_blocking(move || inner.file_type()).await
  }
 
  /// Returns the bare name of this entry without the leading path.