Skip to content

increase stratum send buffer size - #1418

Merged
xmrig merged 1 commit into
xmrig:masterfrom
jtgrassie:buffer-size
Dec 15, 2019
Merged

increase stratum send buffer size#1418
xmrig merged 1 commit into
xmrig:masterfrom
jtgrassie:buffer-size

Conversation

@jtgrassie

@jtgrassie jtgrassie commented Dec 15, 2019

Copy link
Copy Markdown
Contributor

This is needed for block templates over self-select as a block template can be over 2048 bytes. [ref]

@xmrig xmrig added the bug label Dec 15, 2019
@xmrig xmrig added this to the v5 milestone Dec 15, 2019
@xmrig
xmrig merged commit 06c70a7 into xmrig:master Dec 15, 2019
@xmrig

xmrig commented Dec 15, 2019

Copy link
Copy Markdown
Owner

Just on time, I was about to make v5.3.0, so this will be included into release very soon.
Thank you.

@xmrig xmrig mentioned this pull request Dec 15, 2019
Closed
@trasherdk

trasherdk commented Dec 16, 2019 •
edited
Loading

Copy link
Copy Markdown

I guess, with dynamic block size, a fixed buffer won't work. #1421

[2019-12-16 04:22:53.240]  net  new job from localhost:8002 diff 120978 algo rx/0 height 1989902
[2019-12-16 04:23:08.503] speed 10s/60s/15m 1145.0 1129.3 1070.8 H/s max 1178.0 H/s
[2019-12-16 04:23:38.541] speed 10s/60s/15m 1162.6 1132.2 1071.4 H/s max 1178.0 H/s
[2019-12-16 04:24:08.588] speed 10s/60s/15m 1125.4 1145.6 1071.1 H/s max 1178.0 H/s
[2019-12-16 04:24:35.077] [localhost:8002] send failed: "send buffer overflow: 6286 > 4094"
[2019-12-16 04:24:35.078]  net  no active pools, stop mining
[2019-12-16 04:24:38.634] speed 10s/60s/15m 1100.6 1140.6 1071.2 H/s max 1178.0 H/s
[2019-12-16 04:24:40.679]  net  use pool localhost:8002  127.0.0.1
[2019-12-16 04:24:40.707] [localhost:8002] send failed: "send buffer overflow: 6286 > 4094"
[2019-12-16 04:24:40.707]  net  no active pools, stop mining

@jtgrassie jtgrassie mentioned this pull request Dec 16, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bug

Projects

None yet

Milestone

v5

Development

Successfully merging this pull request may close these issues.

3 participants

@jtgrassie @xmrig @trasherdk