On Wed, 15 Feb 2023 21:36:50 -0800, Dixit, Ashutosh wrote: > > On Wed, 15 Feb 2023 21:08:43 -0800, Dixit, Ashutosh wrote: > > > > On Tue, 14 Feb 2023 16:54:13 -0800, Umesh Nerlige Ramappa wrote: > > > > > > Validate the OA sseu config after all params are parsed. > > > > Commit messages for all patches need to answer the "why" or the reason for > > the patch. In this case maybe an overkill but probably something like: > > > > Validate the OA sseu config after all params are parsed since the engine > > can be passed in as part of perf properties. > > Also, if we do this the patch should probably be later in the series after > the patch which introduces engine class/instance in the perf properties. General guidelines for submitting a patch series for review (for the future): 1. The commit message should explain "why" or reason for a patch 2. As far as possible patches should be in a logical order so the series should "tell a story" 3. The patches should be small (each patch being a single logical change if possible) So after we've done the hard part of figuring out the code and getting it to work, the above guidelines also make the review process easier. Thanks. -- Ashutosh