Skip to content

Fix #36, Use OS_MAX_QUEUE_DEPTH rather than 64.#37

Merged
astrogeco merged 1 commit intonasa:integration-candidatefrom
jphickey:fix-36-queue-depth
May 19, 2020
Merged

Fix #36, Use OS_MAX_QUEUE_DEPTH rather than 64.#37
astrogeco merged 1 commit intonasa:integration-candidatefrom
jphickey:fix-36-queue-depth

Conversation

@jphickey
Copy link
Contributor

@jphickey jphickey commented May 7, 2020

Describe the contribution
Configure the maximum depth supported by OSAL, rather than a hard coded 64.

Fixes #36

Testing performed
Build with default config. Confirm normal operation of TO_LAB on system with max queue depth set to 50.

Expected behavior changes
None, except on systems where supported queue depth is less than 64.

System(s) tested on
Ubuntu 20.04

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

@jphickey jphickey added the CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) label May 7, 2020
This should configure the maximum depth supported by
OSAL, rather than a hard coded 64.

Moved both pipe depth configurations to a macro in the
header file.
@jphickey jphickey force-pushed the fix-36-queue-depth branch from 1886aaa to 76a0d31 Compare May 7, 2020 19:13
@skliper skliper added this to the 2.4.0 milestone May 7, 2020
@skliper skliper added CCB-20200513 and removed CCB:Ready Pull request is ready for discussion at the Configuration Control Board (CCB) labels May 13, 2020
@skliper
Copy link
Contributor

skliper commented May 13, 2020

CCB 20200513 - Approved

@astrogeco astrogeco added CCB:Approved Indicates approval by CCB IC-20200513 labels May 19, 2020
@astrogeco astrogeco changed the base branch from master to integration-candidate May 19, 2020 20:06
@astrogeco astrogeco merged commit d814475 into nasa:integration-candidate May 19, 2020
@jphickey jphickey deleted the fix-36-queue-depth branch June 8, 2021 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CCB:Approved Indicates approval by CCB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hardcoded queue depth of 64

3 participants