<html>
<head>
<base href="https://bugs.freedesktop.org/" />
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [vdpau,uvd] kernel oops, Unable to handle kernel paging request at virtual address"
href="https://bugs.freedesktop.org/show_bug.cgi?id=83708#c15">Comment # 15</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW --- - [vdpau,uvd] kernel oops, Unable to handle kernel paging request at virtual address"
href="https://bugs.freedesktop.org/show_bug.cgi?id=83708">bug 83708</a>
from <span class="vcard"><a class="email" href="mailto:zduo006@163.com" title="Jack <zduo006@163.com>"> <span class="fn">Jack</span></a>
</span></b>
<pre>(In reply to <a href="show_bug.cgi?id=83708#c14">comment #14</a>)
<span class="quote">> (In reply to <a href="show_bug.cgi?id=83708#c13">comment #13</a>)
> > This patch can solve the address problem.
>
> writel() and readl() already convert to/from little endian. Does it work
> better if you remove cpu_to_le32() from all lines using those functions?
>
>
> > @@ -751,22 +748,12 @@
> > }
> >
> > /* stitch together an UVD destroy msg */
> > -#if 0
> > - msg[0] = cpu_to_le32(0x00000de4);
> > - msg[1] = cpu_to_le32(0x00000002);
> > - msg[2] = cpu_to_le32(handle);
> > - msg[3] = cpu_to_le32(0x00000000);
> > - for (i = 4; i < 1024; ++i)
> > - msg[i] = cpu_to_le32(0x0);
> > -#endif
> > -#if 1
> > writel(cpu_to_le32(0x00000de4),&msg[1]);
> > writel(cpu_to_le32(0x00000002),&msg[2]);
> > writel(cpu_to_le32(handle),&msg[3]);
> > writel(cpu_to_le32(0x00000000),&msg[4]);
> > for (i = 4; i < 1024; ++i)
> > writel(cpu_to_le32(0x0),&msg[i]);
> > -#endif
> > radeon_bo_kunmap(bo);
> > radeon_bo_unreserve(bo);
>
> Why are you enabling this code?</span >
1. I do not add cpu_to_le32() first ,but when I trace the messages from
printk,the value of msg_type is reversed. Add, all is ok.
2. #if 0 ...#endif is the original code from kernel. #if 1 ...#endif is changed
code. is the same as 1. Must do cpu_to_le32 transfer
By the way, u said writel() and readl() already convert to/from little
endian.
is based on the X86 arch implement?</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>