On Thu, Feb 11, 2010 at 12:50 AM, Alex Deucher <alexdeuc...@gmail.com> wrote:
> On Wed, Feb 10, 2010 at 11:19 PM, Matt Turner <matts...@gmail.com> wrote:
>> Updated patch to use get_unaligned_le32, as suggested by Dave.
>>
>> On Wed, Feb 10, 2010 at 10:56 PM, Matt Turner <matts...@gmail.com> wrote:
>>> Noticed on a DEC Alpha.
>>>
>>> Start up into console mode caused 15 unaligned accesses, and starting X
>>> caused another 48.
>>>
>>> Signed-off-by: Matt Turner <matts...@gmail.com>
>>> CC: Jerome Glisse <jgli...@redhat.com>
>>> CC: Alex Deucher <alexdeuc...@gmail.com>
>>> CC: Dave Airlie <airl...@redhat.com>
>>> ---
>>>  drivers/gpu/drm/radeon/atom.c |    5 ++++-
>>>  1 files changed, 4 insertions(+), 1 deletions(-)
>>>
>>> diff --git a/drivers/gpu/drm/radeon/atom.c b/drivers/gpu/drm/radeon/atom.c
>>> index bd0c843..c1d3c24 100644
>>> --- a/drivers/gpu/drm/radeon/atom.c
>>> +++ b/drivers/gpu/drm/radeon/atom.c
>>> @@ -24,6 +24,7 @@
>>>
>>>  #include <linux/module.h>
>>>  #include <linux/sched.h>
>>> +#include <asm/unaligned.h>
>>>
>>>  #define ATOM_DEBUG
>>>
>>> @@ -212,7 +213,9 @@ static uint32_t atom_get_src_int(atom_exec_context 
>>> *ctx, uint8_t attr,
>>>        case ATOM_ARG_PS:
>>>                idx = U8(*ptr);
>>>                (*ptr)++;
>>> -               val = le32_to_cpu(ctx->ps[idx]);
>>> +               /* get_unaligned avoids unaligned accesses from atombios 
>>> tables
>>> +                * 21, 24, 42. */
>
> Not a big deal, but you might want to mention that this is needed for
> alpha in the comment.  Also, the tables with unaligned accesses will
> probably vary with the card, so there's not really any need to mention
> them here.
>
> Alex

Alright, please check out the attached patch.

Thanks,
Matt

Attachment: 0001-drm-radeon-kms-atom-use-get_unaligned_le32-for-ctx-p.patch
Description: Binary data

------------------------------------------------------------------------------
SOLARIS 10 is the OS for Data Centers - provides features such as DTrace,
Predictive Self Healing and Award Winning ZFS. Get Solaris 10 NOW
http://p.sf.net/sfu/solaris-dev2dev
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to