On Mon, Mar 20, 2023 at 9:05 PM Conor Dooley <conor.dooley@xxxxxxxxxxxxx> wrote: > > On Fri, Mar 17, 2023 at 11:35:27AM +0000, Andy Chiu wrote: > > From: Greentime Hu <greentime.hu@xxxxxxxxxx> > > > > Add vector state context struct to be added later in thread_struct. And > > prepare low-level helper functions to save/restore vector contexts. > > > > This include Vector Regfile and CSRs holding dynamic configuration state > > (vstart, vl, vtype, vcsr). The Vec Register width could be implementation > > defined, but same for all processes, so that is saved separately. > > > > This is not yet wired into final thread_struct - will be done when > > __switch_to actually starts doing this in later patches. > > > > Given the variable (and potentially large) size of regfile, they are > > saved in dynamically allocated memory, pointed to by datap pointer in > > __riscv_v_ext_state. > > > > Co-developed-by: Vincent Chen <vincent.chen@xxxxxxxxxx> > > Signed-off-by: Vincent Chen <vincent.chen@xxxxxxxxxx> > > Signed-off-by: Greentime Hu <greentime.hu@xxxxxxxxxx> > > Signed-off-by: Vineet Gupta <vineetg@xxxxxxxxxxxx> > > Signed-off-by: Andy Chiu <andy.chiu@xxxxxxxxxx> > > I think you missed a: > Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> > > Thanks, > Conor. > Yes, removed it on purpose because I changed some inline assembly in this submission. So I think you may want to take a look in case I did something silly. Thanks, Andy.