Update smoltcp to the newest master

This commit is contained in:
Paweł Jan Czochański 2023-01-18 09:56:38 +01:00 committed by Dario Nieuwenhuis
parent 65ab714fae
commit 2eae12b7f1
3 changed files with 18 additions and 27 deletions

View File

@ -51,7 +51,7 @@ atomic-polyfill = { version = "1.0" }
[dependencies.smoltcp] [dependencies.smoltcp]
version = "0.8.0" version = "0.8.0"
git = "https://github.com/smoltcp-rs/smoltcp" git = "https://github.com/smoltcp-rs/smoltcp"
rev = "b7a7c4b1c56e8d4c2524c1e3a056c745a13cc09f" rev = "7b631c27cefe9b29e322a507c076b4156f477ee2"
default-features = false default-features = false
features = [ features = [
"proto-ipv4", "proto-ipv4",

View File

@ -2,6 +2,7 @@ use core::task::Context;
use embassy_net_driver::{Capabilities, Checksum, Driver, Medium, RxToken, TxToken}; use embassy_net_driver::{Capabilities, Checksum, Driver, Medium, RxToken, TxToken};
use smoltcp::phy; use smoltcp::phy;
use smoltcp::time::Instant;
pub(crate) struct DriverAdapter<'d, 'c, T> pub(crate) struct DriverAdapter<'d, 'c, T>
where where
@ -19,14 +20,14 @@ where
type RxToken<'a> = RxTokenAdapter<T::RxToken<'a>> where Self: 'a; type RxToken<'a> = RxTokenAdapter<T::RxToken<'a>> where Self: 'a;
type TxToken<'a> = TxTokenAdapter<T::TxToken<'a>> where Self: 'a; type TxToken<'a> = TxTokenAdapter<T::TxToken<'a>> where Self: 'a;
fn receive(&mut self) -> Option<(Self::RxToken<'_>, Self::TxToken<'_>)> { fn receive(&mut self, _timestamp: Instant) -> Option<(Self::RxToken<'_>, Self::TxToken<'_>)> {
self.inner self.inner
.receive(self.cx.as_deref_mut().unwrap()) .receive(self.cx.as_deref_mut().unwrap())
.map(|(rx, tx)| (RxTokenAdapter(rx), TxTokenAdapter(tx))) .map(|(rx, tx)| (RxTokenAdapter(rx), TxTokenAdapter(tx)))
} }
/// Construct a transmit token. /// Construct a transmit token.
fn transmit(&mut self) -> Option<Self::TxToken<'_>> { fn transmit(&mut self, _timestamp: Instant) -> Option<Self::TxToken<'_>> {
self.inner.transmit(self.cx.as_deref_mut().unwrap()).map(TxTokenAdapter) self.inner.transmit(self.cx.as_deref_mut().unwrap()).map(TxTokenAdapter)
} }
@ -76,9 +77,9 @@ impl<T> phy::RxToken for RxTokenAdapter<T>
where where
T: RxToken, T: RxToken,
{ {
fn consume<R, F>(self, _timestamp: smoltcp::time::Instant, f: F) -> smoltcp::Result<R> fn consume<R, F>(self, f: F) -> R
where where
F: FnOnce(&mut [u8]) -> smoltcp::Result<R>, F: FnOnce(&mut [u8]) -> R,
{ {
self.0.consume(|buf| f(buf)) self.0.consume(|buf| f(buf))
} }
@ -92,9 +93,9 @@ impl<T> phy::TxToken for TxTokenAdapter<T>
where where
T: TxToken, T: TxToken,
{ {
fn consume<R, F>(self, _timestamp: smoltcp::time::Instant, len: usize, f: F) -> smoltcp::Result<R> fn consume<R, F>(self, len: usize, f: F) -> R
where where
F: FnOnce(&mut [u8]) -> smoltcp::Result<R>, F: FnOnce(&mut [u8]) -> R,
{ {
self.0.consume(len, |buf| f(buf)) self.0.consume(len, |buf| f(buf))
} }

View File

@ -23,11 +23,11 @@ use embassy_sync::waitqueue::WakerRegistration;
use embassy_time::{Instant, Timer}; use embassy_time::{Instant, Timer};
use futures::pin_mut; use futures::pin_mut;
use heapless::Vec; use heapless::Vec;
#[cfg(feature = "medium-ethernet")]
use smoltcp::iface::Routes;
#[cfg(feature = "dhcpv4")] #[cfg(feature = "dhcpv4")]
use smoltcp::iface::SocketHandle; use smoltcp::iface::SocketHandle;
use smoltcp::iface::{Interface, InterfaceBuilder, SocketSet, SocketStorage}; use smoltcp::iface::{Interface, InterfaceBuilder, SocketSet, SocketStorage};
#[cfg(feature = "medium-ethernet")]
use smoltcp::iface::{Neighbor, NeighborCache, Route, Routes};
#[cfg(feature = "dhcpv4")] #[cfg(feature = "dhcpv4")]
use smoltcp::socket::dhcpv4; use smoltcp::socket::dhcpv4;
// smoltcp reexports // smoltcp reexports
@ -45,25 +45,16 @@ use crate::device::DriverAdapter;
const LOCAL_PORT_MIN: u16 = 1025; const LOCAL_PORT_MIN: u16 = 1025;
const LOCAL_PORT_MAX: u16 = 65535; const LOCAL_PORT_MAX: u16 = 65535;
pub struct StackResources<const ADDR: usize, const SOCK: usize, const NEIGHBOR: usize> { pub struct StackResources<const SOCK: usize, const NEIGHBOR: usize> {
addresses: [IpCidr; ADDR], addresses: [IpCidr; 5],
sockets: [SocketStorage<'static>; SOCK], sockets: [SocketStorage<'static>; SOCK],
#[cfg(feature = "medium-ethernet")]
routes: [Option<(IpCidr, Route)>; 1],
#[cfg(feature = "medium-ethernet")]
neighbor_cache: [Option<(IpAddress, Neighbor)>; NEIGHBOR],
} }
impl<const ADDR: usize, const SOCK: usize, const NEIGHBOR: usize> StackResources<ADDR, SOCK, NEIGHBOR> { impl<const SOCK: usize, const NEIGHBOR: usize> StackResources<SOCK, NEIGHBOR> {
pub fn new() -> Self { pub fn new() -> Self {
Self { Self {
addresses: [IpCidr::new(Ipv4Address::UNSPECIFIED.into(), 32); ADDR], addresses: [IpCidr::new(Ipv4Address::UNSPECIFIED.into(), 32); 5],
sockets: [SocketStorage::EMPTY; SOCK], sockets: [SocketStorage::EMPTY; SOCK],
#[cfg(feature = "medium-ethernet")]
routes: [None; 1],
#[cfg(feature = "medium-ethernet")]
neighbor_cache: [None; NEIGHBOR],
} }
} }
} }
@ -105,21 +96,20 @@ impl<D: Driver + 'static> Stack<D> {
pub fn new<const ADDR: usize, const SOCK: usize, const NEIGH: usize>( pub fn new<const ADDR: usize, const SOCK: usize, const NEIGH: usize>(
mut device: D, mut device: D,
config: ConfigStrategy, config: ConfigStrategy,
resources: &'static mut StackResources<ADDR, SOCK, NEIGH>, resources: &'static mut StackResources<SOCK, NEIGH>,
random_seed: u64, random_seed: u64,
) -> Self { ) -> Self {
#[cfg(feature = "medium-ethernet")] #[cfg(feature = "medium-ethernet")]
let medium = device.capabilities().medium; let medium = device.capabilities().medium;
let mut b = InterfaceBuilder::new(); let mut b = InterfaceBuilder::new();
b = b.ip_addrs(&mut resources.addresses[..]); b = b.ip_addrs(Vec::<IpCidr, 5>::from_iter(resources.addresses));
b = b.random_seed(random_seed); b = b.random_seed(random_seed);
#[cfg(feature = "medium-ethernet")] #[cfg(feature = "medium-ethernet")]
if medium == Medium::Ethernet { if medium == Medium::Ethernet {
b = b.hardware_addr(HardwareAddress::Ethernet(EthernetAddress(device.ethernet_address()))); b = b.hardware_addr(HardwareAddress::Ethernet(EthernetAddress(device.ethernet_address())));
b = b.neighbor_cache(NeighborCache::new(&mut resources.neighbor_cache[..])); b = b.routes(Routes::new());
b = b.routes(Routes::new(&mut resources.routes[..]));
} }
let iface = b.finalize(&mut DriverAdapter { let iface = b.finalize(&mut DriverAdapter {
@ -266,7 +256,7 @@ impl<D: Driver + 'static> Inner<D> {
cx: Some(cx), cx: Some(cx),
inner: &mut self.device, inner: &mut self.device,
}; };
if s.iface.poll(timestamp, &mut smoldev, &mut s.sockets).is_err() { if !s.iface.poll(timestamp, &mut smoldev, &mut s.sockets) {
// If poll() returns error, it may not be done yet, so poll again later. // If poll() returns error, it may not be done yet, so poll again later.
cx.waker().wake_by_ref(); cx.waker().wake_by_ref();
return; return;