<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=us-ascii"> <meta name="Generator" content="Microsoft Exchange Server"> <style></style> </head> <body> <div> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> While calling ucopy* functions to copy data to image ,it passes box.width which is calculated from surface . If the image buffer created for image dimensions width=40 and u copy use surface width=48. Which is giving a wrong stride for dst buffer.<br> <br> </div> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> Thanks,<br> </div> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> Suresh G<br> <br> </div> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> Get <a href="https://aka.ms/ghei36">Outlook for Android</a></div> <br> </div> <hr tabindex="-1" style="display:inline-block; width:98%"> <div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>From:</b> Ilia Mirkin <imirkin@alum.mit.edu><br> <b>Sent:</b> Tuesday, October 2, 2018 7:53:09 AM<br> <b>To:</b> Sharma, Deepak<br> <b>Cc:</b> ML Mesa-dev; Guttula, Suresh<br> <b>Subject:</b> Re: [Mesa-dev] [PATCH] st/va:Aligned image width and height to 16.</font> <div> </div> </div> </div> <font size="2"><span style="font-size:11pt;"> <div class="PlainText">Looking at vlVaGetImage, it looks like it just copies data into the<br> texture -- I don't see how UVD is implicated in this. Perhaps the<br> transfer's stride is being set wrong? Or maybe either<br> u_copy_nv12_to_yv12 or util_copy_rect (or one of the functions they<br> call) is having trouble?<br> On Mon, Oct 1, 2018 at 10:17 PM Sharma, Deepak <Deepak.Sharma@amd.com> wrote:<br> ><br> > 16 was considering UVD.<br> ><br> > in vlVagetImage, the width,height passed to function is not used but rather the surface width/height and that causes issue in this case,not sure how driver will handle that ?<br> ><br> > ________________________________<br> > From: Ilia Mirkin <imirkin@alum.mit.edu><br> > Sent: Monday, October 1, 2018 6:49 PM<br> > To: Sharma, Deepak<br> > Cc: ML Mesa-dev; Guttula, Suresh<br> > Subject: Re: [Mesa-dev] [PATCH] st/va:Aligned image width and height to 16.<br> ><br> > On Mon, Oct 1, 2018 at 9:47 PM Sharma, Deepak <Deepak.Sharma@amd.com> wrote:<br> > ><br> > > From: suresh guttula <suresh.guttula@amd.com><br> > ><br> > > In case of decoding of resolution like 40x24, while allocating surface<br> > > video buffer is always aligned with macroblock width/height which is 16.<br> > > But when application tries to get data after decoding through vaCreateImage<br> > > /vaGetImage, image width/height aligned with 2 and result a smaller image<br> > > buffer which causes the memory stomping issue.<br> ><br> > Shouldn't the driver be allocating a larger backing texture instead?<br> > Why 16 and not 8 or 32 or 4096?<br> ><br> > -ilia<br> </div> </span></font> </body> </html>