On Fri, Apr 2, 2010 at 11:43 PM, Bruce Rogers <[email protected]> wrote:
> In building gpxe for openSUSE Factory (part of kvm package), there were a few 
> problems identified by the compiler.  This patch addresses them.

Is this a specific compiler version or have you customized the
compiler flags?  It would be interesting to know the details so that
we have the option of building in the same way.

> -       ns->cur_rx = ++ns->cur_rx % NR_RX_DESC;

What an odd thing to do :).

> +       ns->cur_rx = ns->cur_rx + 1 % NR_RX_DESC;

I think you mean:
ns->cur_rx = ( ns->cur_rx + 1 ) % NR_RX_DESC;

The '%' operator has higher precedence than '+'.

Stefan
_______________________________________________
gPXE-devel mailing list
[email protected]
http://etherboot.org/mailman/listinfo/gpxe-devel

Reply via email to