-
Notifications
You must be signed in to change notification settings - Fork 116
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
[pdpix] Fixed Size demi_pop
#547
Conversation
e2f6bdc
to
8ce9f46
Compare
@@ -373,10 +373,20 @@ impl CatcollarLibOS { | |||
} | |||
|
|||
/// Pops data from a socket. | |||
pub fn pop(&mut self, qd: QDesc) -> Result<QToken, Fail> { | |||
trace!("pop() qd={:?}", qd); | |||
pub fn pop(&mut self, qd: QDesc, size: Option<usize>) -> Result<QToken, Fail> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably also check that the size is not above some max threshold
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure. This requires major work, thus I created a issue to keep track of this: #559
8ce9f46
to
d790e3e
Compare
This commit partially addresses #499
d790e3e
to
80a3e09
Compare
Description
This PR closes #499
Summary of Changes
demi_pop()