Hi Virendra, The big problem with that patch is that, as Andreas properly noted in his review, the prefetch packet does not get deallocated properly, creating a big memory leak. We are planning to fix this so that the patch can be committed.
I'm not sure how closely that relates to your proposal to have separate store-address and store-data micro-ops though. I'd have to look more closely at the code, and unfortunately I don't have time to do that right now. Regards, Steve On Mon, Nov 2, 2015 at 4:12 AM Virendra Kumar Pathak < kumarvir.pat...@gmail.com> wrote: > Hi Steve Reinhardt, > > I am working on extending store functional unit to "store-address" + > "store-data" in gem5 for aarch64 arm processor. > > Looks like you had proposed a patch with similar aim (issue excl. prefetch > as soon as store address is available) > Mailing archive link - > https://www.mail-archive.com/gem5-users%40gem5.org/msg10026.html > patch review link - http://reviews.gem5.org/r/2277/ > > But looks like patch is still in open status. Could you please comment on > what changes needs to be done to fix it. > > From Mitch comments, it seems that the patch can be extended to add the > support of splitting stores instructions in "store-address" & "store-data" > micro-ops. > If I am not wrong on above, could you please give some high level guidance > on what changes need to be done to achieve it. > > Thanks for you time. > > -- > with regards, > Virendra Kumar Pathak > _______________________________________________ > gem5-users mailing list > gem5-users@gem5.org > http://m5sim.org/cgi-bin/mailman/listinfo/gem5-users
_______________________________________________ gem5-users mailing list gem5-users@gem5.org http://m5sim.org/cgi-bin/mailman/listinfo/gem5-users
