Date: Sat, 4 Feb 2017 18:26:39 -0700 From: Warner Losh <imp@bsdimp.com> To: Cy Schubert <Cy.Schubert@komquats.com> Cc: Warner Losh <imp@freebsd.org>, src-committers <src-committers@freebsd.org>, "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>, "svn-src-head@freebsd.org" <svn-src-head@freebsd.org> Subject: Re: svn commit: r313191 - head/sbin/nvmecontrol Message-ID: <CANCZdfrq5c97mY%2Bwe=whzqdSH96_8OP5oH2cnxWEfaCmHmObEA@mail.gmail.com> In-Reply-To: <201702050126.v151QAfE005740@slippy.cwsent.com> References: <imp@bsdimp.com> <CANCZdfq4tk9VJuRfK97UqnKExbDQZxNuSpAO-B_QWMhfJmTBzg@mail.gmail.com> <201702050126.v151QAfE005740@slippy.cwsent.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Sorry it took so long. Warner On Sat, Feb 4, 2017 at 6:26 PM, Cy Schubert <Cy.Schubert@komquats.com> wrote: > Thanks. > > > -- > Cheers, > Cy Schubert <Cy.Schubert@cschubert.com> > FreeBSD UNIX: <cy@FreeBSD.org> Web: http://www.FreeBSD.org > > The need of the many outweighs the greed of the few. > > > In message <CANCZdfq4tk9VJuRfK97UqnKExbDQZxNuSpAO-B_QWMhfJmTBzg@mail.gmail.c > om> > , Warner Losh writes: >> Thanks! Fixed in r313259. >> >> Warner >> >> On Sat, Feb 4, 2017 at 6:16 PM, Cy Schubert <Cy.Schubert@komquats.com> wrote: >> > In message <201702040553.v145r1wB002775@repo.freebsd.org>, Warner Losh >> > writes: >> >> Author: imp >> >> Date: Sat Feb 4 05:53:00 2017 >> >> New Revision: 313191 >> >> URL: https://svnweb.freebsd.org/changeset/base/313191 >> >> >> >> Log: >> >> Implement 5 wdc-specific nvme control options for their HGST drives: >> >> wdc cap-diag Capture diagnostic data from drive >> >> wdc drive-log Capture drive history data from drive >> >> wdc get-crash-dump Retrieve firmware crash dump from drive >> >> >> >> Added: >> >> head/sbin/nvmecontrol/wdc.c (contents, props changed) >> >> Modified: >> >> head/sbin/nvmecontrol/Makefile >> >> head/sbin/nvmecontrol/nvmecontrol.8 >> >> head/sbin/nvmecontrol/nvmecontrol.c >> >> head/sbin/nvmecontrol/nvmecontrol.h >> > [...] >> >> + while (len > 0) { >> >> + resid = len > NVME_MAX_XFER_SIZE ? NVME_MAX_XFER_SIZE : len; >> >> + wdc_get_data(fd, opcode, resid, offset, cmd, buf, resid); >> >> + if (write(fd2, buf, resid) != resid) >> > >> > Hi Warner, >> > >> > I'm seeing the following on i386. >> > >> > opt/src/svn-current/sbin/nvmecontrol/wdc.c:156:30: error: comparison of >> > integers of different signs: 'ssize_t' (aka 'int') and 'uint32_t' (aka >> > 'unsigned int') [-Werror,-Wsign-compare] >> > if (write(fd2, buf, resid) != resid) >> > ~~~~~~~~~~~~~~~~~~~~~~ ^ ~~~~~ >> > 1 error generated. >> > >> > amd64 builds okay. >> > >> >> + err(1, "write"); >> >> + offset += resid; >> >> + len -= resid; >> >> + } >> >> + free(buf); >> >> + close(fd2); >> > [...] >> > >> > >> > -- >> > Cheers, >> > Cy Schubert <Cy.Schubert@cschubert.com> >> > FreeBSD UNIX: <cy@FreeBSD.org> Web: http://www.FreeBSD.org >> > >> > The need of the many outweighs the greed of the few. >> > >> > >> >> > >
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfrq5c97mY%2Bwe=whzqdSH96_8OP5oH2cnxWEfaCmHmObEA>