On Fri, Jan 24, 2025 at 9:06 AM Crystal Wood <crwood@xxxxxxxxxx> wrote: > > On Thu, 2025-01-23 at 14:04 +0800, Shizhao Chen wrote: > > Different hardwares have different threasholds for usleep_val to > > reliably trigger an prio inversion, add option --usleep to allow > > specifying it at runtime, to facilitate testing of prio inheritance > > on different platforms. > > > > Signed-off-by: Shizhao Chen <shichen@xxxxxxxxxx> > > --- > > src/pi_tests/pip_stress.8 | 6 +++++- > > src/pi_tests/pip_stress.c | 10 ++++++++-- > > 2 files changed, 13 insertions(+), 3 deletions(-) > > Hmm, looks like there was an earlier attempt (53956b6712fef1, > "pip_stress: parameterize usleep value to work-around platform issues") > that was missing the actual call to getopt... and then another commit > (1325cb7e9e3af08e, "Daniel Wagner <dwagner@xxxxxxx>") that added getopt > but removed usleep -- while also adding a "-s" option that does nothing. > :-P Yeah I kinda stole the option from clark :) > > Some minor nits: > > > @@ -41,6 +42,9 @@ merely increase the time that the low priority process sleeps while > > holding the lock. (usleep); > > Also note that you have to run as a user with permission to change > > scheduling priorities. > > +.SH OPTIONS > > +.IP "\-u TIME,\-\-usleep=TIME" > > +Specify the sleep time of the low priority process. Defaults to 500(us). > > The unit should be part of the description, not the default: > > Specify the sleep time in usec of the low priority process. Defaults to > 500. I hesitated several times whether I should add the (us) at the end, your version looks much better, thanks! > > > - "-h --help Show this help menu.\n" > > + "-h --help Show this help menu.\n"\ > > + "-u TIME --usleep=TIME Specify the sleep time of the low priority process.\n"\ > > + " Defaults to 500(us).\n" > > ); > > Likewise here > > > + case 'u': > > + usleep_val = (useconds_t)strtoul(optarg, NULL, 10); > > + break; > > Why is this cast needed? It's not needed - just me being unfamiliar with C. :P I'll remove it. > > > -Crystal >
