Skip to content
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

Thread task #152

Open
wants to merge 8 commits into
base: Bogdan.Shchogolev
Choose a base branch
from

Conversation

Derivability
Copy link

Implemented multi-threading processing of task, that user inputs through procfs.

Derivability added 6 commits May 15, 2019 23:27
Created Makefile.

Signed-off-by: Derivability <[email protected]>
Created basic module with procfs read/write interface.

Signed-off-by: Derivability <[email protected]>
Added error handling.

Signed-off-by: Derivability <[email protected]>
Created functions for analyzing and executing task.

Signed-off-by: Derivability <[email protected]>
Implemented task proccesing using threads.

Signed-off-by: Derivability <[email protected]>
Implemented pool of available workers to procces accepted task.

Signed-off-by: Derivability <[email protected]>
@Derivability Derivability added the ready The PR is ready for review label May 16, 2019
@Derivability Derivability requested a review from oleg-h2o May 16, 2019 20:30
@Derivability Derivability self-assigned this May 16, 2019
Copy link

@galkinletter galkinletter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GL_COMP2

Please see file comparison

@galkinletter galkinletter added the question Further information is requested label May 31, 2019
@Derivability Derivability requested a review from galkinletter May 31, 2019 15:40
@Derivability Derivability removed the question Further information is requested label May 31, 2019
@galkinletter galkinletter added the question Further information is requested label May 31, 2019
@Derivability
Copy link
Author

Derivability commented May 31, 2019

@galkinletter, this Pull Request was opened first, so code from this PR is original and written by specified in commit signs author. I decided to remove "question" label, as there's no point in it.

@Derivability Derivability removed the question Further information is requested label May 31, 2019
Reworked and optimized block output.

Signed-off-by: Derivability <[email protected]>
@galkinletter galkinletter added changes requested The PR review revealed issues which should be fixed (set by reviewer). and removed ready The PR is ready for review labels Jun 3, 2019
@oleg-h2o
Copy link

oleg-h2o commented Jun 3, 2019

Please use checkpatch!

@galkinletter
Copy link

Please use checkpatch (Linux Coding style check). For run the check patch for an individual file use
./checkpatch.pl –terse –file –no-tree

Fixed codestyle errors using checkpatch.pl.

Signed-off-by: Derivability <[email protected]>
@Derivability Derivability added ready The PR is ready for review and removed changes requested The PR review revealed issues which should be fixed (set by reviewer). labels Jun 3, 2019
@Derivability
Copy link
Author

Derivability commented Jun 3, 2019

Please use checkpatch!

Done! Please checkout

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready The PR is ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants