Port S3 multi-threaded download demo from C-SDK#436
Merged
sukhmanm merged 12 commits intoFreeRTOS:masterfrom Dec 4, 2020
Merged
Port S3 multi-threaded download demo from C-SDK#436sukhmanm merged 12 commits intoFreeRTOS:masterfrom
sukhmanm merged 12 commits intoFreeRTOS:masterfrom
Conversation
yourslab
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Show resolved
Hide resolved
sukhmanm
force-pushed
the
demos/port-s3-multithreaded
branch
from
b474edc to
647bef3
Compare
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
reviewed
Dec 2, 2020
...dows_Simulator/HTTP_S3_Download_Multithreaded/DemoTasks/S3DownloadMultithreadedHTTPExample.c
Outdated
Show resolved
Hide resolved
leegeth
approved these changes
Dec 2, 2020
n9wxu
reviewed
Dec 4, 2020
| #define configMAX_PRIORITIES ( 7 ) | ||
| #define configTICK_RATE_HZ ( 1000 ) /* In this non-real time simulated environment the tick frequency has to be at least a multiple of the Win32 tick frequency, and therefore very slow. */ | ||
| #define configMINIMAL_STACK_SIZE ( ( unsigned short ) 60 ) /* In this simulated case, the stack only has to hold one small structure as the real stack is part of the Win32 thread. */ | ||
| #define configTOTAL_HEAP_SIZE ( ( size_t ) ( 2048U * 1024U ) ) |
Contributor
There was a problem hiding this comment.
why the addition for the heap size? I don't understand the significance of 2k + 1k.
| #define configUSE_TIMERS 1 | ||
| #define configTIMER_TASK_PRIORITY ( configMAX_PRIORITIES - 1 ) | ||
| #define configTIMER_QUEUE_LENGTH 5 | ||
| #define configTIMER_TASK_STACK_DEPTH ( configMINIMAL_STACK_SIZE * 2 ) |
Contributor
There was a problem hiding this comment.
Why is the timer stack depth for this demo 2x the minimum stack size? Is there some science behind the * 2? I would prefer to simply enter the "correct size" for these stack entries. I don't think there is any reason in the demo why the stack needs to be exactly 2x bigger than the minimum.
n9wxu
approved these changes
Dec 4, 2020
paulbartell
pushed a commit
to paulbartell/FreeRTOS
that referenced
this pull request
Mar 30, 2023
uncrustify Co-authored-by: Shreyas Balakrishna <shreyasbharath@users.noreply.github.com> Co-authored-by: Gaurav-Aggarwal-AWS <33462878+aggarg@users.noreply.github.com>
Zangetsu112
pushed a commit
to Zangetsu112/FreeRTOS-evpp
that referenced
this pull request
Aug 18, 2025
Adding demo to demonstrate the use of pre-signed URLs to connect and download from a S3 bucket. Sends multiple GET requests synchronously to download a file in chunks using range headers. Uses a request and response task in addition to the main HTTP servicing task.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.