Age | Commit message (Collapse) | Author | Files | Lines |
|
Function-local function prototypes may not be static, says GCC.
|
|
If the symbol is optimized away, this causes krtld to refuse to load the
module.
|
|
|
|
GCC is insistent that we check the bounds of rid (from rcookie) before
using it as the index into srd_hmergnp. Change an ASSERT to a
non-conditional VERIFY to silence it.
|
|
GCC 4.4 would warn that we were left shifting by 32 leaving a result
greater than the width of the type (presumed to be implicit int). Cast
up to a uint64_t so we have room.
|
|
|
|
We'd previously sometimes set this to 0x80, which doesn't fit in the
7-bit qualifier field. SPC-2 and SPC-3 describe these 7 bits as
reserved, and they are unused (except in format, which prints them).
|
|
|
|
|
|
Squelches warning where efe_restart is non-static and inline but
efe_stop/start are static.
|
|
We get compiler warnings because 'halt' can never be 0.
It turns out that 'halt' is whatever random global halt symbol happens
to be in scope and the code is bogus.
|
|
Keep a count running from 0 of the array index, rather than re-using the
register address as the index thus starting out of bounds and walking
even further.
|
|
|
|
|
|
|
|
Reviewed by: Adam Leventhal <ahl@delphix.com>
Approved by: Garrett D'Amore <garrett@nexenta.com>
|
|
|
|
1348 isainfo -v potentially overflows 80 columns
Reviewed by: Richard Lowe <richlowe@richlowe.net>
Reviewed by: Robert Mustacchi <rm@joyent.com>
Approved by: Eric Schrock <Eric.Schrock@delphix.com>
|
|
Reviewed by: Igor Kozhukhov <igor.kozhukhov@nexenta.com>
Reviewed by: Gordon Ross <gwr@nexenta.com>
Reviewed by: Garrett D'Amore <garrett@nexenta.com>
Reviewed by: Eric Schrock <eric.schrock@delphix.com>
Approved by: Garrett D'Amore <garrett@nexenta.com>
|
|
Reviewed by: Garrett D'Amore <garrett@nexenta.com>
Reviewed by: Robert Mustacchi <rm@joyent.com>
Approved by: Richard Lowe <richlowe@richlowe.net>
|
|
Reviewed by: Garrett D'Amore <garrett@nexenta.com>
Reviewed by: Vineeth Pillai <vineeth.pillai@nexenta.com>
Reviewed by: Igor Kozhukhov <igor.kozhukhov@nexenta.com>
Reviewed by: Albert Lee <trisk@nexenta.com>
Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
Approved by: Garrett D'Amore <garrett@nexenta.com>
|
|
943 zio_interrupt ends up calling taskq_dispatch with TQ_SLEEP
Reviewed by: Albert Lee <trisk@nexenta.com>
Reviewed by: Richard Lowe <richlowe@richlowe.net>
Reviewed by: Alexey Zaytsev <alexey.zaytsev@nexenta.com>
Reviewed by: Jason Brian King <jason.brian.king@gmail.com>
Reviewed by: George Wilson <gwilson@zfsmail.com>
Reviewed by: Adam Leventhal <ahl@delphix.com>
Approved by: Gordon Ross <gwr@nexenta.com>
|
|
|
|
|
|
|
|
Fix a case of ->mtr_present being constant false due to truncation, and
make it harder to repeat the mistake.
|
|
|
|
|
|
|
|
|
|
Squelch casts from pointer to different sized int warnings
|
|
|
|
Squelch a GCC warning about a cast from an integer to a pointer of a
different size (presumed to be an implicit 32bits before).
|
|
As a GCC extension, GCC used to define 'extern inline' to mean that a
function was only ever emitted inline and no visible symbol was emitted.
C99, however, made 'extern inline' mean pretty much the reverse of this, and
GCC 4.3 and above in C99 or GNU99 modes follow the standard, so we have to
decorate each definition to indicate we want the non-standard behaviour.
|
|
Reviewed by: Adam Leventhal <ahl@delphix.com>
Reviewed by: Garrett D'Amore <garrett@nexenta.com>
Approved by: Richard Lowe <richlowe@richlowe.net>
|
|
|
|
is missing
|
|
|
|
Reviewed by: Garrett D'Amore <garrett@nexenta.com>
Reviewed by: Albert Lee <trisk@nexenta.com>
Approved by: Eric Schrock <Eric.Schrock@delphix.com>
|
|
Reviewed by: Jason King <jason.brian.king@gmail.com>
Reviewed by: Gordon Ross <gwr@nexenta.com>
Approved by: Eric Schrock <eric.schrock@delphix.com>
|
|
Reviewed by: Robert Mustacchi <rm@joyent.com>
Reviewed by: Garrett D'Amore <garrett@nexenta.com>
Reviewed by: Jason King <jason.brian.king@gmail.com>
Reviewed by: Richard Lowe <richlowe@richlowe.net>
Approved by: Gordon Ross <gwr@nexenta.com>
|
|
Reviewed by: Richard Lowe <richlowe@richlowe.net>
Reviewed by: Stepan Zastupov <stepan.zastupov@gmail.com>
Approved by: Garrett D'Amore <garrett@nexenta.com>
|
|
Reviewed by: jason.brian.king@gmail.com
Reviewed by: Dan Kruchinin <dkruchinin@acm.org>
Reviewed by: Richard Lowe <richlowe@richlowe.net>
Approved by: Garrett D'Amore <garrett@nexenta.com>
|
|
Reviewed by: Gordon Ross <gordon.w.ross@gmail.com>
Reviewed by: Garrett D'Amore <garrett.damore@gmail.com>
Approved by: Eric Schrock <eric.schrock@delphix.com>
|
|
|
|
Reviewed by: Gordon Ross <gordon.w.ross@gmail.com>
Reviewed by: Dan McDonald <danmcd@nexenta.com>
Approved by: Eric Schrock <Eric.Schrock@delphix.com>
|
|
|
|
|
|
|
|
|