Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
J
jacobsa-fuse
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
Kirill Smelkov
jacobsa-fuse
Commits
b87ffb52
Commit
b87ffb52
authored
Dec 19, 2016
by
Aaron Jacobs
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Fix a bug in BenchmarkOutMessageReset.
I misunderstood what was being zeroed. Only the header is.
parent
d31e0a4e
Changes
1
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
4 additions
and
4 deletions
+4
-4
internal/buffer/out_message_test.go
internal/buffer/out_message_test.go
+4
-4
No files found.
internal/buffer/out_message_test.go
View file @
b87ffb52
...
...
@@ -60,18 +60,16 @@ func TestMemclr(t *testing.T) {
func
BenchmarkOutMessageReset
(
b
*
testing
.
B
)
{
// A single buffer, which should fit in some level of CPU cache.
b
.
Run
(
"Single buffer"
,
func
(
b
*
testing
.
B
)
{
b
.
SetBytes
(
int64
(
unsafe
.
Sizeof
(
OutMessage
{})))
var
om
OutMessage
for
i
:=
0
;
i
<
b
.
N
;
i
++
{
om
.
Reset
()
}
b
.
SetBytes
(
int64
(
om
.
offset
))
})
// Many megabytes worth of buffers, which should defeat the CPU cache.
b
.
Run
(
"Many buffers"
,
func
(
b
*
testing
.
B
)
{
b
.
SetBytes
(
int64
(
unsafe
.
Sizeof
(
OutMessage
{})))
// The number of messages; intentionally a power of two.
const
numMessages
=
128
...
...
@@ -83,5 +81,7 @@ func BenchmarkOutMessageReset(b *testing.B) {
for
i
:=
0
;
i
<
b
.
N
;
i
++
{
oms
[
i
%
numMessages
]
.
Reset
()
}
b
.
SetBytes
(
int64
(
oms
[
0
]
.
offset
))
})
}
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment