Fw: Re: [PATCH i-g-t] tools/intel_gpu_top: Fix misaligned rendering

[anonymoustranquillity at proton.me]

I forgot to include the mailing list in the CC, I apologize. Forwarding it.




------- Forwarded Message -------
From: anonymoustranquillity at proton.me <anonymoustranquillity at proton.me>
Date: On Tuesday, January 16th, 2024 at 10:39 AM
Subject: Re: [PATCH i-g-t] tools/intel_gpu_top: Fix misaligned rendering
To: Tvrtko Ursulin <tvrtko.ursulin at linux.intel.com>


> 
> 
> 
> Hello,
> 
> I have not used send-email as my email provider (protonmail) does not seem to support SMTP (I have been meaning to switch to a self hosted provider for a while now, haven't been able to yet).
> 
> This email is the one I use for FOSS contributions, it's the same one as the one I used to sign up to the freedesktop gitlab (https://gitlab.freedesktop.org/tranquillity-codes). I am unable to include my real name due to personal reasons related to my safety (since it will be visible publicly).
> 
> Running intel_gpu_top on some terminal sizes seems to overflow a character at end of line N to the first character of line N+1, misalign the column marker for PIDs, and sometimes hide the first few lines describing the engines. I used alacritty on X11 to test this.
> 
> Thank you for your time,
> Ity
> 
> On Tuesday, January 16th, 2024 at 9:49 AM, Tvrtko Ursulin tvrtko.ursulin at linux.intel.com wrote:
> 
> 
> 
> > Hi,
> > 
> > On 15/01/2024 15:29, Kamil Konieczny wrote:
> > 
> > > Hi anonymoustranquillity,
> > > On 2024-01-14 at 21:26:36 +0000, anonymoustranquillity at proton.me wrote:
> > > 
> > > > Hello,
> > > > 
> > > > there seems to be a problem with rendering in the interactive
> > > > mode in intel_gpu_top, seems to be a simple off-by-one. Below
> > > > is a patch. I am new to contributing via email, so I apologize
> > > > if I did something wrong - I tried to follow CONTRIBUTING.md
> > > > to the best of my ability.
> > 
> > What renders incorrectly? Asking since locally things look fine for me.
> > 
> > Regards,
> > 
> > Tvrtko
> > 
> > > > Have a nice day,
> > > > - Ity.
> > > 
> > > Thank you for patch, if you would like to use git and re-send
> > > with the help of 'git format-patch -1' and then
> > > 'git send e-mail 0001-your-first.patch'
> > > 
> > > Before that setup your real name and e-mail in git configuration,
> > > like "First Last your at email-address.me"
> > > 
> > > If it is too much bother someone can take it from here
> > > but without your real name we cannot give you contribution
> > > (except for link into patchwork).
> > > 
> > > +cc Tvrtko
> > > 
> > > Regards,
> > > Kamil
> > > 
> > > > ---
> > > > tools/intel_gpu_top.c | 2 +-
> > > > 1 file changed, 1 insertion(+), 1 deletion(-)
> > > > 
> > > > diff --git a/tools/intel_gpu_top.c b/tools/intel_gpu_top.c
> > > > index 046ead15a..24363e72f 100644
> > > > --- a/tools/intel_gpu_top.c
> > > > +++ b/tools/intel_gpu_top.c
> > > > @@ -2178,7 +2178,7 @@ print_client(struct igt_drm_client *c, struct engines *engines, double t, int li
> > > > len += *class_w;
> > > > }
> > > > 
> > > > - printf(" %-*s\n", con_w - len - 1, c->print_name);
> > > > + printf(" %-*s\n", con_w - len - 2, c->print_name);
> > > > } else if (output_mode == JSON) {
> > > > char buf[64];
> > > > 
> > > > --
> > > > 2.43.0