On Wed, Mar 29, 2023 at 1:22 AM Conor Dooley <conor@xxxxxxxxxx> wrote: > > On Mon, Mar 27, 2023 at 04:49:30PM +0000, Andy Chiu wrote: > > Vector unit is disabled by default for all user processes. Thus, a > > process will take a trap (illegal instruction) into kernel at the first > > time when it uses Vector. Only after then, the kernel allocates V > > context and starts take care of the context for that user process. > > > > Suggested-by: Richard Henderson <richard.henderson@xxxxxxxxxx> > > Link: https://lore.kernel.org/r/3923eeee-e4dc-0911-40bf-84c34aee962d@xxxxxxxxxx > > Signed-off-by: Andy Chiu <andy.chiu@xxxxxxxxxx> > > I see you dropped two R-b from this patch, what actually changed here? > It's not immediately obvious to me (sorry!) and the cover doesn't mention > why they were dropped. Hi, the reason why I dropped R-b here is that there is a conflict in trap.c because the API has changed after converting to generic entry. And that change does not make v16's way work out of the box, so it (trap.c @ v17) requires some more reviewing I suppose. Thanks for asking! I should get better at describing these in the cover letter. Andy
