On 05/04/11 22:17, Ali Saidi wrote: > >> On 2011-05-04 21:08:25, Gabe Black wrote: > CR3 might work. Does the kernel change it on every context switch (user > program)? The main reason for having it is when tracing user code the kernel > can context switch on you. If you want to see all the code that was executed > in a process and compare it to what it should have executed you need some > kind of identifier to disambiguate processess. The ASID is an easy way to do > this, while rooting around in the Linux process structure is annoying. > > >> On 2011-05-04 21:08:25, Gabe Black wrote: >>> src/cpu/SConscript, line 188 >>> <http://reviews.m5sim.org/r/678/diff/1/?file=12418#file12418line188> >>> >>> This isn't from your change, but should this be missing ExecSymbol? We >>> don't really need this flag in any case since you can just use Exec and >>> turn off ExecTicks. > You're right, I didn't even realize. I always do Exec,-ExecTicks. Leave it, > delete it, add it?
Let's delete it in a separate change. > > - Ali > > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/678/#review1208 > ----------------------------------------------------------- > > > On 2011-05-04 18:42:30, Ali Saidi wrote: >> ----------------------------------------------------------- >> This is an automatically generated e-mail. To reply, visit: >> http://reviews.m5sim.org/r/678/ >> ----------------------------------------------------------- >> >> (Updated 2011-05-04 18:42:30) >> >> >> Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and >> Nathan Binkert. >> >> >> Summary >> ------- >> >> Trace: Allow printing ASIDs and selectively tracing based on user/kernel >> code. >> >> Debug flags are ExecUser, ExecKernel, and ExecAsid. ExecUser and >> ExecKernel are set by default when Exec is specified. Use minus >> sign with ExecUser or ExecKernel to remove user or kernel tracing >> respectively. >> >> >> Diffs >> ----- >> >> src/arch/alpha/utility.hh 5a9a639ce16f >> src/arch/alpha/utility.cc 5a9a639ce16f >> src/arch/arm/utility.hh 5a9a639ce16f >> src/arch/mips/utility.hh 5a9a639ce16f >> src/arch/power/utility.hh 5a9a639ce16f >> src/arch/sparc/utility.hh 5a9a639ce16f >> src/arch/x86/utility.hh 5a9a639ce16f >> src/cpu/SConscript 5a9a639ce16f >> src/cpu/exetrace.cc 5a9a639ce16f >> >> Diff: http://reviews.m5sim.org/r/678/diff >> >> >> Testing >> ------- >> >> >> Thanks, >> >> Ali >> >> > _______________________________________________ > m5-dev mailing list > m5-dev@m5sim.org > http://m5sim.org/mailman/listinfo/m5-dev _______________________________________________ m5-dev mailing list m5-dev@m5sim.org http://m5sim.org/mailman/listinfo/m5-dev
