StratifyLabs / sostest

Stratify OS Test Suite
GNU Lesser General Public License v3.0
3 stars 3 forks source link

Task Stress Test needs to check RR priority bounds #62

Open tyler-gilbert opened 6 years ago

tyler-gilbert commented 6 years ago

@shomagan The task test looks great. I found a bug in the OS (https://github.com/StratifyLabs/StratifyOS/issues/161). There was another failure caused by this line:

        int signal_thread_priority = rand()%30;

The priority needs to be between Sched::get_priority_min(Sched::RR) and Sched::get_priority_max(Sched::RR).

tyler-gilbert commented 6 years ago

@shomagan How are things going? Do you still have tests to work on?

shomagan commented 6 years ago

I made changes to the previous commit for stress test, now I added changes with a constant value without rand value use, but prio test pass failed.

task_info_test.priority() have zero value

[image: image.png]

ср, 18 июл. 2018 г. в 19:37, Tyler notifications@github.com:

@shomagan https://github.com/shomagan How are things going? Do you still have tests to work on?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/StratifyLabs/sostest/issues/62#issuecomment-405953946, or mute the thread https://github.com/notifications/unsubscribe-auth/AE6GGwBQlCo-y94pzijF2km0DwndheWaks5uH0gmgaJpZM4VQGKQ .

shomagan commented 6 years ago

Hello , i'm writing a test for the messenger and I'm not getting much result, can you help with examples or description? Thanks a lot

чт, 19 июл. 2018 г. в 10:06, Shoma Gan shomagan@gmail.com:

I made changes to the previous commit for stress test, now I added changes with a constant value without rand value use, but prio test pass failed.

task_info_test.priority() have zero value

[image: image.png]

ср, 18 июл. 2018 г. в 19:37, Tyler notifications@github.com:

@shomagan https://github.com/shomagan How are things going? Do you still have tests to work on?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/StratifyLabs/sostest/issues/62#issuecomment-405953946, or mute the thread https://github.com/notifications/unsubscribe-auth/AE6GGwBQlCo-y94pzijF2km0DwndheWaks5uH0gmgaJpZM4VQGKQ .

tyler-gilbert commented 6 years ago

The Messenger is used for exchanging data between processes using a device (usually /dev/fifo). Let's hold off on that one. Can you work on a message queue test? Mq's are similar but they don't use the device file system. The Sys::Mq class is based on POSIX message queue calls ( http://pubs.opengroup.org/onlinepubs/009695399/basedefs/mqueue.h.html).

What do you think?

Tyler

On Mon, Jul 23, 2018 at 10:00 AM shomagan notifications@github.com wrote:

Hello , i'm writing a test for the messenger and I'm not getting much result, can you help with examples or description? Thanks a lot

чт, 19 июл. 2018 г. в 10:06, Shoma Gan shomagan@gmail.com:

I made changes to the previous commit for stress test, now I added changes with a constant value without rand value use, but prio test pass failed.

task_info_test.priority() have zero value

[image: image.png]

ср, 18 июл. 2018 г. в 19:37, Tyler notifications@github.com:

@shomagan https://github.com/shomagan How are things going? Do you still have tests to work on?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub < https://github.com/StratifyLabs/sostest/issues/62#issuecomment-405953946>, or mute the thread < https://github.com/notifications/unsubscribe-auth/AE6GGwBQlCo-y94pzijF2km0DwndheWaks5uH0gmgaJpZM4VQGKQ

.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/StratifyLabs/sostest/issues/62#issuecomment-407109603, or mute the thread https://github.com/notifications/unsubscribe-auth/AF2jzTcMkVRNRTT4r1IRFxeLC0MkAmKVks5uJfMmgaJpZM4VQGKQ .

shomagan commented 6 years ago

what devices can I use? I first need to create a file?

tyler-gilbert commented 6 years ago

For Sys::Mq, you don't need any devices or files. But Messenger uses /dev/fifo (usually). Mq is better to test right now. I will see if I can get Messenger ready.

Tyler

On Mon, Jul 23, 2018 at 10:16 AM shomagan notifications@github.com wrote:

what devices can I use? I first need to create a file?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/StratifyLabs/sostest/issues/62#issuecomment-407115013, or mute the thread https://github.com/notifications/unsubscribe-auth/AF2jzXvcp5YEJqYq-GdOtuBEhw4DjnIfks5uJfbIgaJpZM4VQGKQ .

shomagan commented 6 years ago

The current mq test check failed, stress suspended, maybe test have some remarks

tyler-gilbert commented 6 years ago

ok

On Mon, Jul 23, 2018 at 10:46 AM shomagan notifications@github.com wrote:

The current mq test check failed, stress suspended, maybe test have some remarks

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/StratifyLabs/sostest/issues/62#issuecomment-407124392, or mute the thread https://github.com/notifications/unsubscribe-auth/AF2jzQSyyJZVGkHUNntHpx5mDxzuclOCks5uJf4BgaJpZM4VQGKQ .

shomagan commented 6 years ago

mq stress test passed,the problem was in unused unlink() before close()

shomagan commented 6 years ago

for messenger use need create son file, but don't created through son::create or if use Stratify Link!

shomagan commented 6 years ago

What should I now pay attention to?