2012/4/26 David Airlie <airlied@xxxxxxxxxx>: > > > ----- Original Message ----- >> From: "Christian König" <deathsimple@xxxxxxxxxxx> >> To: "j glisse" <j.glisse@xxxxxxxxx> >> Cc: "Jerome Glisse" <jglisse@xxxxxxxxxx>, dri-devel@xxxxxxxxxxxxxxxxxxxxx >> Sent: Thursday, 26 April, 2012 10:11:12 AM >> Subject: Re: [PATCH 01/24] drm/radeon: remove fence/ring/ib debugfs files >> >> Hi Jerome, >> >> I totally agree that we can remove the following debugfs files, since >> everything that just prints out or modifies register informations >> doesn't belongs into the kernel driver and should be moved to >> radeontool >> instead. > > In the future with secure boot we will need a better mechanism to diagnose things > than a userspace tool mapping pci bars, which we can't allow to happen in a secure > boot environment, so dropping all the files might not be a good idea unless we > provide a mechanism for radeontool to use. We can't allow userspace to read/write > arbitrary registers and stay secure. > > Dave. Well those files are useless, each time i debug a lockup and try to use them i loose time on them as they never helped me so far and i seriously doubt they would help me in the future or anyone else. For secure boot my guess is that radeontool will become obsolete and by that time we might need to add some reg dumping. But none of the current files seems usefull to me. The fence and sa files might make sense to debug the associated code as it's nice to be able to dump associated kernel struct. Cheers, Jerome _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
