On Tue, Aug 18, 2009 at 11:51 AM, Pauli Nieminen<suok...@gmail.com> wrote:
> GCC did war about optimization not possible because possible forever loop.
>
> Signed-off-by: Pauli Nieminen <suok...@gmail.com>

Acked-by: Alex Deucher <alexdeuc...@gmail.com>

> ---
>  libdrm/radeon/radeon_cs_gem.c |   12 ++++++------
>  1 files changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/libdrm/radeon/radeon_cs_gem.c b/libdrm/radeon/radeon_cs_gem.c
> index 264b067..a0db53b 100644
> --- a/libdrm/radeon/radeon_cs_gem.c
> +++ b/libdrm/radeon/radeon_cs_gem.c
> @@ -354,21 +354,21 @@ static void cs_gem_print(struct radeon_cs *cs, FILE 
> *file)
>     unsigned opcode;
>     unsigned reg;
>     unsigned cnt;
> -    int i, j;
> +    unsigned int i, j;
>
>     for (i = 0; i < cs->cdw;) {
> -        cnt = CP_PACKET_GET_COUNT(cs->packets[i]);
> +        cnt = CP_PACKET_GET_COUNT(cs->packets[i]) + 1;
>         switch (CP_PACKET_GET_TYPE(cs->packets[i])) {
>         case PACKET_TYPE0:
> -            fprintf(file, "Pkt0 at %d (%d dwords):\n", i, cnt + 1);
> +            fprintf(file, "Pkt0 at %d (%d dwords):\n", i, cnt);
>             reg = CP_PACKET0_GET_REG(cs->packets[i]);
>             if (CP_PACKET0_GET_ONE_REG_WR(cs->packets[i++])) {
> -                for (j = 0; j <= cnt; j++) {
> +                for (j = 0; j < cnt; j++) {
>                     fprintf(file, "    0x%08X -> 0x%04X\n",
>                             cs->packets[i++], reg);
>                 }
>             } else {
> -                for (j = 0; j <= cnt; j++) {
> +                for (j = 0; j < cnt; j++) {
>                     fprintf(file, "    0x%08X -> 0x%04X\n",
>                             cs->packets[i++], reg);
>                     reg += 4;
> @@ -410,7 +410,7 @@ static void cs_gem_print(struct radeon_cs *cs, FILE *file)
>                 fprintf(file, "Unknow opcode 0x%02X at %d\n", opcode, i);
>                 return;
>             }
> -            for (j = 0; j <= cnt; j++) {
> +            for (j = 0; j < cnt; j++) {
>                 fprintf(file, "        0x%08X\n", cs->packets[i++]);
>             }
>             break;
> --
> 1.6.0.4
>
>
> ------------------------------------------------------------------------------
> Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day
> trial. Simplify your report design, integration and deployment - and focus on
> what you do best, core application coding. Discover what's new with
> Crystal Reports now.  http://p.sf.net/sfu/bobj-july
> --
> _______________________________________________
> Dri-devel mailing list
> Dri-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/dri-devel
>

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to