Thanks Hans. I built a new kernel and tested it, now it works better, but can still report errors - less frequently. Out of 5 reboots, it failed once, so something is still there to be found.
On Mon, Aug 1, 2016 at 3:42 PM, Hans Rosenfeld <hans.rosenf...@nexenta.com> wrote: > On Mon, Aug 01, 2016 at 03:32:50PM -0400, Youzhong Yang wrote: > > This does not make sense to me, not sure if it's relevant to the issue I > am > > seeing: > > > > http://www.nvmexpress.org/wp-content/uploads/NVM-Express-1_1b.pdf > > page 42 defines 'CSTS -Controller Status', > > > > but our code defines it as > > > > typedef union { > > struct { > > uint32_t csts_rdy:1; /* Ready */ > > uint32_t csts_cfs:1; /* Controller Fatal Status */ > > uint32_t csts_shst:2; /* Shutdown Status */ > > uint32_t csts_nssro:1; /* NVM Subsystem Reset Occured */ > > uint32_t csts_rsvd:28; > > } b; > > uint32_t r; > > } nvme_reg_csts_t; > > ( > > > http://src.illumos.org/source/xref/illumos-gate/usr/src/uts/common/io/nvme/nvme_reg.h#108 > > ) > > > > Shouldn't "uint32_t csts_rsvd:28;" be "uint32_t csts_rsvd:27;"? > > Yes, it should be. Thanks for spotting this, I'll prepare a fix > tomorrow. > > Hans > > -- > %SYSTEM-F-ANARCHISM, The operating system has been overthrown > ------------------------------------------- smartos-discuss Archives: https://www.listbox.com/member/archive/184463/=now RSS Feed: https://www.listbox.com/member/archive/rss/184463/25769125-55cfbc00 Modify Your Subscription: https://www.listbox.com/member/?member_id=25769125&id_secret=25769125-7688e9fb Powered by Listbox: http://www.listbox.com
