Conversation
@kernellogger WOW that is amazing work!
0
0
0
Edited 1 year ago
@kernellogger

@sj you might be interested in adapting DAMON parameters?
btw I wonder how broad range of parameters could be covered
2
0
0
@hyeyoo @kernellogger @sj @hyeyoo @kernellogger This strikes me though as a flaw in the kernel - if bpf can sit there in the background autotuning shouldn't that just be kernel code?

My concern with stuff like this is you end up with a mechanism that should be something sat in the kernel now sat outside of it.

I mean I do think tunables are sometimes used as a lazy option for something that really OUGHT to be autotuned but dev says 'meh leave it up to the user' and washes their hands of the whole thing.

Many tunables are very unclear to a user who doesn't know deep internals how to set and a very tiny % probably EVER get modified anywhere.

I wish we could just set fire to most of them but they become part of the uAPI so I guess we can't :)
1
0
4
@ljs @kernellogger @sj

That's a perspective I didn't have, and it makes sense. So, it's like performing CPR on this messed-up situation?
1
0
1
@hyeyoo @kernellogger @sj I don't mean to shit on bpf or be a grumpy old man but I do have concerns. It is a very useful thing overall and I like it, I just think we need to be wary of not going too crazy with it :)

bpftrace is fucking useful
1
0
2
@ljs @kernellogger @sj

Ah, I added sj because IIRC he's planning implement an auto tuner for DAMON parameters - no one should be like "bpftune can tune everything and make more parameters"

it would have been horrible if i.e. active/inactive list balancing was implemented as a kernel parameter :P
1
0
1
@hyeyoo @kernellogger @sj imagine if you could just drop the entire page cache via one too that'd be awful
1
0
2
@ljs @kernellogger @sj

LOL what
you mean /proc/sys/vm/drop_caches?
1
0
1
@hyeyoo @kernellogger @sj yes, cue 1,000 confused people using that to 'free up memory'. Sigh.
1
0
2
@hyeyoo @kernellogger @sj (I'm not saying it shouldn't exist, I'm saying it should be in /sys/kernel/mm or something)
1
0
1
@ljs @kernellogger @sj

or a prompt confirming if you really know what you're doing ;)

BTW what would be different if it were in /sys/kernel/mm?
1
0
1
@hyeyoo @kernellogger @sj it wouldn't be a standard sysctl found in /proc/sys/vm along with things that are sane to change (under normal circumstances, it's useful for testing perf/debugging things etc.)
0
0
2
@hyeyoo @kernellogger Obviously very interesting topic. To my humble understanding, the point of this awesome project is the lightweight and fine-grained monitoring and adaptive tuning, which comes from the underlying use of BPF. That makes sense to me, as BPF is awesome. Nevertheless, it doesn't look like the panacea for every tunable knobs. Rather than that, it seems need per-case tuning implementation. I think it might be able to be useful for DAMON parameters, but tbh I'm not sure. Anyway, interesting work. Hope to have more chance to hear about that :)
0
0
3