> > That's going to be confusing. I'll add code to stop the user from > > passing both options. > > Also kind of confusing, after reading the second patch, I realized > "mba_MBps_event=mbm_total_bytes,mba_MBps" also results in both being > set. If you're able to fail the mount operation if both flags somehow > get set, that would address this one too. Peter, Yes. That's possible too. I'll cover that case in the next version. I'll wait until this gets to the top of Reinette's review queue before posting again. -Tony