Merge pull request #530 from MarcinMiklas/avtp_pipeline_fqtss_fix

andrew-elder · web-flow · commit d5d7d2608faf · 2016-12-02T09:06:52.000-05:00
AVTP pipeline: fixed bandwitdh reservation.
diff --git a/lib/avtp_pipeline/qmgr/openavb_qmgr.c b/lib/avtp_pipeline/qmgr/openavb_qmgr.c
@@ -122,7 +122,7 @@ static bool setupHWQueue(int nClass, unsigned classBytesPerSec)
  */
 U16 openavbQmgrAddStream(SRClassIdx_t nClass, unsigned classRate, unsigned maxIntervalFrames, unsigned maxFrameSize)
 {
-	unsigned fullFrameSize = maxFrameSize + OPENAVB_AVTP_ETHER_FRAME_OVERHEAD;
+	unsigned fullFrameSize = maxFrameSize + OPENAVB_AVTP_ETHER_FRAME_OVERHEAD + 1;
 	unsigned long streamBytesPerSec = fullFrameSize * maxIntervalFrames * classRate;
 	int idx, nStream;
 	U16 fwmark = INVALID_FWMARK;

Original file line number	Diff line number	Diff line change
`@@ -122,7 +122,7 @@ static bool setupHWQueue(int nClass, unsigned classBytesPerSec)`
`122`	`122`	`*/`
`123`	`123`	`U16 openavbQmgrAddStream(SRClassIdx_t nClass, unsigned classRate, unsigned maxIntervalFrames, unsigned maxFrameSize)`
`124`	`124`	`{`
`125`		`- unsigned fullFrameSize = maxFrameSize + OPENAVB_AVTP_ETHER_FRAME_OVERHEAD;`
	`125`	`+ unsigned fullFrameSize = maxFrameSize + OPENAVB_AVTP_ETHER_FRAME_OVERHEAD + 1;`
`126`	`126`	`unsigned long streamBytesPerSec = fullFrameSize * maxIntervalFrames * classRate;`
`127`	`127`	`int idx, nStream;`
`128`	`128`	`U16 fwmark = INVALID_FWMARK;`