On Thu, 07 Oct 2010 15:43:43 +1000, Dave Airlie <airlied@xxxxxxxxxx> wrote: > On Wed, 2010-10-06 at 21:24 -0700, Keith Packard wrote: > > A physically mapped hardware status page is allocated at driver load > > time but was never freed. Call the existing code to free this page at > > driver unload time on hardware which uses this kind. > > the free_hws gets called after we disappear the mmio mapping for the > registers. it then writes a register after killing the hws, and oopses. Right, freeing the hws page is a definite oversight. It looks like we should only be doing so on the KMS-only path anyway which neatly pulls the call before the mmio teardown. Applied with the amendment to -next. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
