pci_alloc_msi is always called, is this bad?

[Konstantin Belousov]

On Fri, Oct 04, 2013 at 10:05:18AM -0600, Scott Long wrote:
> It used to be that gcc would generate code that would conditionally execute the second clause only if the first clause were true.  If that's not longer the case (with gcc and/or clang), then I bet that UDF will break due to this:
> 
>        /*
>          * Check to see if the fid is fragmented. The first test
>          * ensures that we don't wander off the end of the buffer
>          * looking for the l_iu and l_fi fields.
>          */
>         if (ds->off + UDF_FID_SIZE > ds->size ||
>             ds->off + le16toh(fid->l_iu) + fid->l_fi + UDF_FID_SIZE > ds->size){
Is this some sort of joke ?

C 'and' and 'or' logical operators are short-circuit. It is not
compiler-depended.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/freebsd-scsi/attachments/20131004/d035ee79/attachment.sig>