swung0x48/MobileGL
1
0
Fork 0
mirror of https://github.com/MobileGL-Dev/MobileGL synced 2026-11-16 06:23:23 +09:00
Code Issues Packages Projects Releases Wiki Activity

Pipe: every verb names its record op in a verb-indexed table (static_assert: one row per verb), so a verb syncs only when its record does; the barriered fallback that quiesced the applier once per frame at the blit is gone (FCL inproc GL-thread wait 1.65 ms/frame; red-once: blit row removed aborts the run-ahead blit test, missing row fails to compile).

Browse Source
This commit is contained in:
swung0x48
2026-10-08 18:02:01 -04:00
parent 19a96929aa
commit e00db8e64f
4 changed files with 227 additions and 67 deletions
Download Patch File Download Diff File Expand all files Collapse all files
MobileGL/MG_Test/Pipe/FieldOwnershipTest.cpp
+28
View File
@@ -38,6 +38,7 @@
#include <string>
#include "Includes.h"
#include <MG_Pipe/VerbRecordOps.h>
#include <MG_Pipe/MGPipe.h>
#include <Config.h>
@@ -252,6 +253,33 @@ TEST_F(FieldOwnershipTest, TheSevenStickyForwardsAgreeWithTheirFieldRows) {
}
}
// P15: the verb -> record table (MG_Pipe/VerbRecordOps.h) is the client's whole barrier
// answer, so every row has to agree with what the server will do with the record: the server
// stamps the op as MGPipeVerbForWireOp(op), and that verb must be of the SAME fill class as the
// verb that published it - otherwise an unbarriered record would be applied under another class's
// field mask. A row whose op is not a stamp point at all would apply under the previous verb's
// serial. (Completeness - one row per verb - is a static_assert in the header.)
TEST_F(FieldOwnershipTest, EveryVerbsRecordOpIsStampedAsAVerbOfTheSameFillClass) {
Uint32 noRecord = 0;
for (SizeT i = 0; i < static_cast<SizeT>(MGPipeVerb::kVerbCount); ++i) {
const auto verb = static_cast<MGPipeVerb>(i);
const MGPWireOp op = MGPipeRecordOpForVerb(verb);
if (op == kMGPipeNoRecord) {
++noRecord;
continue;
}
const MGPipeVerb stamped = MGPipeVerbForWireOp(op);
ASSERT_NE(stamped, MGPipeVerb::kVerbCount) << kMGPipeVerbNames[i] << ": its record op stamps no verb";
EXPECT_EQ(kMGPipeVerbClass[static_cast<SizeT>(stamped)], kMGPipeVerbClass[i])
<< kMGPipeVerbNames[i] << " publishes a record stamped as " << kMGPipeVerbNames[static_cast<SizeT>(stamped)];
}
// GetIntegeri_v and IsTimerQuerySupported, answered from the caps mirror.
EXPECT_EQ(noRecord, 2u);
EXPECT_EQ(MGPipeRecordOpForVerb(MGPipeVerb::BlitNamedFramebuffer), MGPWireOp::Blit);
EXPECT_EQ(MGPipeRecordOpForVerb(MGPipeVerb::DrawElements), MGPWireOp::DrawVbo);
EXPECT_EQ(MGPipeRecordOpForVerb(MGPipeVerb::ClearBufferfv), MGPWireOp::Clear);
}
// Pin every boundary's representative verb, the complete count, and the three
// verb-shaped calls that are exempt by name.
TEST_F(FieldOwnershipTest, VerbBoundaryOpsCoverEveryVerbShapedCall) {