<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <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#c14">Comment # 14</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 <a class="email" href="mailto:michel@daenzer.net" title="Michel Dänzer <michel@daenzer.net>"> Michel Dänzer</a> <pre>(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?</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>