assignments/parallel-firewall: Add parallel firewall assignment #121
Conversation
This file contains 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
andreistan26
force-pushed
the
parallel-firewall
branch
8 times, most recently
from
8565a02
to
8648318
Compare
andreistan26
requested review from
elena19m,
razvand,
StefanJum,
gabrielpitic and
BogdanPrunescu
andreistan26
added
needs-rendering
|
Published at https://cs-pub-ro.github.io/operating-systems/121/ |
Add parallel firewall assignment for the compute chapter. The goal is to introduce students to parallel programming with a single producer multiple consumer problem that uses a thread-safe ring buffer to transfer data between threads. Signed-off-by: Andrei Stan <[email protected]>
andreistan26
force-pushed
the
parallel-firewall
branch
from
8648318
to
f6de3b5
Compare
andreistan26
added
needs-rendering
|
Published at https://cs-pub-ro.github.io/operating-systems/121/ |
github-actions
bot
added
area/assignment
fabianpatras
reviewed
Nov 4, 2024
There was a problem hiding this comment.
I've left a suggestion for the introductory part of the statement.
Please use as much (or as little) of it as you want.
I wanted to give a bit more details about what happens in a real world scenario and how this is all dumbed down scenario using made up traffic (to not have them think they also need to do networking stuff in this assignment)
fabianpatras
force-pushed
the
parallel-firewall
branch
from
b590a68
to
59612b2
Compare
fabianpatras
force-pushed
the
parallel-firewall
branch
3 times, most recently
from
ab806f8
to
e07f264
Compare
Add more context about the assignment Signed-off-by: Anton-Fabian Patras <[email protected]>
Make checker look out for the binary to run because if the compiation failed, there won't be any. Also, use `$(MAKE)` instead of plain `make` from inside the `tests/Makefile` as that is the intended way to do it. Also 2, removing `-i` from the makefile invocation make the inner makefile fail after the first error, and adding `-` makes it not stop the outter makefile rule (i.e. also run the checker) Signed-off-by: Anton-Fabian Patras <[email protected]>
fabianpatras
force-pushed
the
parallel-firewall
branch
2 times, most recently
from
a3e9eec
to
814c01e
Compare
`__packed` only exists inside the Linux Kernel. Signed-off-by: Anton-Fabian Patras <[email protected]>
fabianpatras
force-pushed
the
parallel-firewall
branch
from
814c01e
to
b83d348
Compare
fabianpatras
added
needs-rendering
|
Published at https://cs-pub-ro.github.io/operating-systems/121/ |
Alex-deVis
added
needs-rendering
|
Published at https://cs-pub-ro.github.io/operating-systems/121/ |
andreistan26
added
needs-rendering
|
Published at https://cs-pub-ro.github.io/operating-systems/121/ |
andreistan26
added
needs-rendering
|
Published at https://cs-pub-ro.github.io/operating-systems/121/ |
andreistan26
force-pushed
the
parallel-firewall
branch
2 times, most recently
from
d90b2a9
to
7dc5fb0
Compare
elena19m
approved these changes
Nov 12, 2024
andreistan26
force-pushed
the
parallel-firewall
branch
2 times, most recently
from
7207fdd
to
267bfc8
Compare
Clarify the CLI arguments accepted by the `firewall` binary. Rephrase support code comment from `processes` -> `threads`. Signed-off-by: Andrei Stan <[email protected]>
andreistan26
force-pushed
the
parallel-firewall
branch
from
267bfc8
to
5327dfa
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Add parallel firewall assignment for the compute chapter.
The goal is to introduce students to parallel programming with a single producer multiple consumer problem that uses a thread-safe ring buffer to transfer data between threads.
Prerequisite Checklist
Description of changes