diff options
| author | Ido Rosen | 2016-08-11 12:39:50 -0400 |
|---|---|---|
| committer | Ido Rosen | 2016-08-11 12:39:50 -0400 |
| commit | bf10af799bdc46fa0cdc885b48068646b9c411ba (patch) | |
| tree | 99f70cbdcf663c703979c50656e4f21213dd7000 /openonload-201606-bug63982-ilog2.patch | |
| parent | 7317d20565bc3cdebe034eb275452c0ffa4d000d (diff) | |
| download | aur-bf10af799bdc46fa0cdc885b48068646b9c411ba.tar.gz | |
Add workaround to GCC bug #72785.
There is a bug in GCC 6.1.1 (currently in Arch) related to link errors with
____ilog2_NaN. Explained further at this URL:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=72785
Solarflare has provided a workaround, added as a patch.
Diffstat (limited to 'openonload-201606-bug63982-ilog2.patch')
| -rw-r--r-- | openonload-201606-bug63982-ilog2.patch | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/openonload-201606-bug63982-ilog2.patch b/openonload-201606-bug63982-ilog2.patch new file mode 100644 index 000000000000..dc6414d51eec --- /dev/null +++ b/openonload-201606-bug63982-ilog2.patch @@ -0,0 +1,31 @@ +diff --git a/src/lib/efrm/vi_resource_alloc.c b/src/lib/efrm/vi_resource_alloc.c +--- a/src/lib/efrm/vi_resource_alloc.c ++++ b/src/lib/efrm/vi_resource_alloc.c +@@ -1432,6 +1432,15 @@ + + qso->q_len_entries = n_q_entries; + qso->q_len_bytes = efrm_vi_q_bytes(virs, q_type, n_q_entries); ++ ++ /* This value should always be positive, but if we don't check for this ++ * explicitly, some compilers will assume that undefined logarithms ++ * can be taken in get_order() and will generate code that won't link. ++ * See bug63982. */ ++ EFRM_ASSERT(qso->q_len_bytes > 0); ++ if (qso->q_len_bytes <= 0) ++ return -EINVAL; ++ + qso->q_len_page_order = get_order(qso->q_len_bytes); + return 0; + } +--- a/src/include/onload/version.h ++++ b/src/include/onload/version.h +@@ -29,7 +29,7 @@ + + + #ifndef ONLOAD_VERSION +-# define ONLOAD_VERSION "201606" ++# define ONLOAD_VERSION "201606-patch63982" + #endif + + #define ONLOAD_PRODUCT "OpenOnload" + |