-
» gingeropolous blushes
-
UkoeHB_
is it the users are too dumb for the application, or the application isn't smart enough for the users?
-
jtgrassie
more to the point, why is an informational stacktrace a default setting
-
jtgrassie
if it's an error that the user should be concerned with, fine,
-
jtgrassie
but that doesn't appear the norm.
-
jtgrassie
As it it now, they seem like debug/trace logs.
-
jtgrassie
#6694 is case in point
-
sech1
Top 2 pools are under 50% combined for the first time
-
sech1
minexmr 27.8%, nanopool 21.7%
-
moneromooo
It is in the *log*. The *log* is here to help debug.
-
moneromooo
Of course it's there by default, otherwise what's the point of it ?
-
sech1
Wiser words have never been said ^^^
-
moneromooo
"Hey guys, I got a bug" "Any interesting expcetions in the log ?" "Well, they aren't on by default, so I dunno"
-
moneromooo
That's what why it's on the INFO level.
-
gingeropolous
I may edit the error message if I can navigate to that kind of error and make the text: "Hey! Thanks for reading your logs. The following output means blah, and nothing is actually broken"
-
moneromooo
Maybe it is.
-
moneromooo
I'm not a fan of this. There's a already a couple of these things in IIRC.
-
gingeropolous
hehehe
-
moneromooo
Meh, maybe.
-
gingeropolous
yeah, i should make it a plugin
-
moneromooo
Danger: you are looking at debug logs. If you don't know what you are doing, piss off :D
-
gingeropolous
well, noobs aren't gonna install plugins. I mean,
-
gingeropolous
lol
-
moneromooo
No user serviceable parts inside. The user is not serviceable.
-
gingeropolous
if you want to reduce those kind of bug reports, we can make the wall higher on github
-
gingeropolous
i forget which repo it was,... but i went to file an issue, and there's this huge ass list of stuff needed etc
-
gingeropolous
i said myself "i don't care this much" and moved on
-
gingeropolous
so, you'd be filtering out people that may only give like 0.25 fucks.
-
moneromooo
No. I like reports. It's just that reporting anything that you've not seen before is dumb. That particular user had a reason to though. randomx failed to allocate memory.
-
moneromooo
It's just that it's in a loop and super noisy, which is bloody annoying. That should be fixed.
-
moneromooo
I thought it was actually.
-
gingeropolous
and i dunno why i like the idea of monerod trying to talk to you like a human
-
gingeropolous
it kinda feels like one of those digital pets from the 90s
-
tevador
that exception is handled, not sure why it's necessary to log it?
-
moneromooo
s/Disk space is low/Meow! Meow! *scratches at the hard disk shop window*/
-
moneromooo
It logs all exceptions.
-
moneromooo
All exceptions are handled.
-
moneromooo
This one is just annoying because it's a common occurence.
-
tevador
it's handled inside librandomx
-
tevador
we could also disable hugepages for verification
-
tevador
but it takes the right amount of dumb to notice the error and not being able to enable hugepages at the same time
-
moneromooo
Maybe it could just disable all exception reporting coming from librandomx.
-
moneromooo
grep randomx in the trace, dump.
-
moneromooo
It'd hide any actual interesting one that might happen though.
-
moneromooo
For the record, the reason I added this is that we had a number of cases where users reported bugs where I wish I had known of exceptions happening and where.
-
tevador
btw this
monero-project/monero #6694 doesn't look like it's related to hugepages
-
tevador
[0x5651d8c98e6e]:_Z10setPagesRXPvm+0x4e) [0x5651d8c98e6e]
-
moneromooo
So this is useful to me to debug stuff in general.
-
tevador
setPagesRX is called in the JIT compiler to set the code executable if W^X is enabled
-
hyc
but we don't enable that by default
-
hyc
hm, we do.
-
hyc
fr verification anyway, not for mining
-
tevador
-
tevador
why are there two identical ifs?
-
hyc
looks like a previous step got deleted
-
hyc
there probably used to be another flag combo being tried with alloc_cache and we removed it
-
hyc
-
gethh
looks like hashrate is going crazy, there must be lotsa of the free audits currently running for the cloud providers :)
-
sech1
-
sech1
-
sech1
107.7 MH/s and 125.6 MH/s
-
sech1
On the plus side, top 2 pools are still below 50% combined
-
hyc
so which other pools have grown?
-
sech1
nanopool
-
sech1
RespectXMR also got nice 60 MH/s
-
hyc
sounds about right. we were at 1.3GH before, now at 1.6GH
-
sech1
yes, pools and network hashrate are almost the same
miningpoolstats.stream/monero
-
gethh
sech1 yep 0.25GH
-
tevador
-
selsta
tevador: how should I continue here?
tevador/RandomX #183/files
-
selsta
AFAIK I have to inlcude #include <TargetConditionals.h> to differentiate between iOS / macOS
-
gethh
my new monero node soon :) =>
kobol.io
-
tevador
sech1 hyc can you please review also this?
tevador/RandomX #187
-
tevador
selsta IIRC someone successfully built randomx for iOS here:
tevador/RandomX #153