| Message ID | 20201014193355.53074-1-dgilbert@redhat.com |
|---|---|
| State | Accepted |
| Commit | 43f828e155b443641765a1e933100a96f26be3dd |
| Headers | show |
| Series | arm/trace: Fix hex printing | expand |
On 10/14/20 9:33 PM, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com> > > Use of 0x%d - make up our mind as 0x%x Uh... Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org> (hw/arm/... in subject) > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com> > --- > hw/arm/trace-events | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/arm/trace-events b/hw/arm/trace-events > index c8a4d80f6b..a335ee891d 100644 > --- a/hw/arm/trace-events > +++ b/hw/arm/trace-events > @@ -41,7 +41,7 @@ smmuv3_get_cd(uint64_t addr) "CD addr: 0x%"PRIx64 > smmuv3_decode_cd(uint32_t oas) "oas=%d" > smmuv3_decode_cd_tt(int i, uint32_t tsz, uint64_t ttb, uint32_t granule_sz, bool had) "TT[%d]:tsz:%d ttb:0x%"PRIx64" granule_sz:%d had:%d" > smmuv3_cmdq_cfgi_ste(int streamid) "streamid =%d" > -smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%d - end=0x%d" > +smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%x - end=0x%x" > smmuv3_cmdq_cfgi_cd(uint32_t sid) "streamid = %d" > smmuv3_config_cache_hit(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache HIT for sid %d (hits=%d, misses=%d, hit rate=%d)" > smmuv3_config_cache_miss(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache MISS for sid %d (hits=%d, misses=%d, hit rate=%d)" >
Hi Dave, On 10/14/20 9:33 PM, Dr. David Alan Gilbert (git) wrote: > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com> > > Use of 0x%d - make up our mind as 0x%x > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com> Acked-by: Eric Auger <eric.auger@redhat.com> Thanks Eric > --- > hw/arm/trace-events | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/arm/trace-events b/hw/arm/trace-events > index c8a4d80f6b..a335ee891d 100644 > --- a/hw/arm/trace-events > +++ b/hw/arm/trace-events > @@ -41,7 +41,7 @@ smmuv3_get_cd(uint64_t addr) "CD addr: 0x%"PRIx64 > smmuv3_decode_cd(uint32_t oas) "oas=%d" > smmuv3_decode_cd_tt(int i, uint32_t tsz, uint64_t ttb, uint32_t granule_sz, bool had) "TT[%d]:tsz:%d ttb:0x%"PRIx64" granule_sz:%d had:%d" > smmuv3_cmdq_cfgi_ste(int streamid) "streamid =%d" > -smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%d - end=0x%d" > +smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%x - end=0x%x" > smmuv3_cmdq_cfgi_cd(uint32_t sid) "streamid = %d" > smmuv3_config_cache_hit(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache HIT for sid %d (hits=%d, misses=%d, hit rate=%d)" > smmuv3_config_cache_miss(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache MISS for sid %d (hits=%d, misses=%d, hit rate=%d)" >
On Wed, 14 Oct 2020 at 20:36, Dr. David Alan Gilbert (git) <dgilbert@redhat.com> wrote: > > From: "Dr. David Alan Gilbert" <dgilbert@redhat.com> > > Use of 0x%d - make up our mind as 0x%x > > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com> > --- > hw/arm/trace-events | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/hw/arm/trace-events b/hw/arm/trace-events > index c8a4d80f6b..a335ee891d 100644 > --- a/hw/arm/trace-events > +++ b/hw/arm/trace-events > @@ -41,7 +41,7 @@ smmuv3_get_cd(uint64_t addr) "CD addr: 0x%"PRIx64 > smmuv3_decode_cd(uint32_t oas) "oas=%d" > smmuv3_decode_cd_tt(int i, uint32_t tsz, uint64_t ttb, uint32_t granule_sz, bool had) "TT[%d]:tsz:%d ttb:0x%"PRIx64" granule_sz:%d had:%d" > smmuv3_cmdq_cfgi_ste(int streamid) "streamid =%d" > -smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%d - end=0x%d" > +smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%x - end=0x%x" Ah, I missed that you'd sent this patch before. Eric, do we want to use hex here, or should we go for decimal the way we do with (almost) all the other tracing of stream IDs (eg mmuv3_cmdq_cfgi_ste in the line before)? The other odd-one-out is smmuv3_find_ste which prints a hex SID; I think the other tracing of SIDs is always decimal. thanks -- PMM
Hi Peter, On 10/19/20 9:26 PM, Peter Maydell wrote: > On Wed, 14 Oct 2020 at 20:36, Dr. David Alan Gilbert (git) > <dgilbert@redhat.com> wrote: >> >> From: "Dr. David Alan Gilbert" <dgilbert@redhat.com> >> >> Use of 0x%d - make up our mind as 0x%x >> >> Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com> >> --- >> hw/arm/trace-events | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/hw/arm/trace-events b/hw/arm/trace-events >> index c8a4d80f6b..a335ee891d 100644 >> --- a/hw/arm/trace-events >> +++ b/hw/arm/trace-events >> @@ -41,7 +41,7 @@ smmuv3_get_cd(uint64_t addr) "CD addr: 0x%"PRIx64 >> smmuv3_decode_cd(uint32_t oas) "oas=%d" >> smmuv3_decode_cd_tt(int i, uint32_t tsz, uint64_t ttb, uint32_t granule_sz, bool had) "TT[%d]:tsz:%d ttb:0x%"PRIx64" granule_sz:%d had:%d" >> smmuv3_cmdq_cfgi_ste(int streamid) "streamid =%d" >> -smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%d - end=0x%d" >> +smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%x - end=0x%x" > > Ah, I missed that you'd sent this patch before. > > Eric, do we want to use hex here, or should we go for > decimal the way we do with (almost) all the other > tracing of stream IDs (eg mmuv3_cmdq_cfgi_ste in the line before)? > > The other odd-one-out is smmuv3_find_ste which prints a hex > SID; I think the other tracing of SIDs is always decimal. I think my preference would be to use hexa here and in the other places. Thanks Eric > > thanks > -- PMM >
On Tue, 27 Oct 2020 at 09:24, Auger Eric <eric.auger@redhat.com> wrote: > > Hi Peter, > > On 10/19/20 9:26 PM, Peter Maydell wrote: > > Eric, do we want to use hex here, or should we go for > > decimal the way we do with (almost) all the other > > tracing of stream IDs (eg mmuv3_cmdq_cfgi_ste in the line before)? > > > > The other odd-one-out is smmuv3_find_ste which prints a hex > > SID; I think the other tracing of SIDs is always decimal. > I think my preference would be to use hexa here and in the other places. OK; I'll apply this patch to target-arm.next; feel free to send a patch updating the other tracepoints to hex. -- PMM
Hi Peter, On 10/27/20 11:38 AM, Peter Maydell wrote: > OK; I'll apply this patch to target-arm.next; feel free to send > a patch updating the other tracepoints to hex. sure, I will. Thanks Eric
diff --git a/hw/arm/trace-events b/hw/arm/trace-events index c8a4d80f6b..a335ee891d 100644 --- a/hw/arm/trace-events +++ b/hw/arm/trace-events @@ -41,7 +41,7 @@ smmuv3_get_cd(uint64_t addr) "CD addr: 0x%"PRIx64 smmuv3_decode_cd(uint32_t oas) "oas=%d" smmuv3_decode_cd_tt(int i, uint32_t tsz, uint64_t ttb, uint32_t granule_sz, bool had) "TT[%d]:tsz:%d ttb:0x%"PRIx64" granule_sz:%d had:%d" smmuv3_cmdq_cfgi_ste(int streamid) "streamid =%d" -smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%d - end=0x%d" +smmuv3_cmdq_cfgi_ste_range(int start, int end) "start=0x%x - end=0x%x" smmuv3_cmdq_cfgi_cd(uint32_t sid) "streamid = %d" smmuv3_config_cache_hit(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache HIT for sid %d (hits=%d, misses=%d, hit rate=%d)" smmuv3_config_cache_miss(uint32_t sid, uint32_t hits, uint32_t misses, uint32_t perc) "Config cache MISS for sid %d (hits=%d, misses=%d, hit rate=%d)"