svn commit: r329824 - in head: sbin/nvmecontrol sys/cam/nvme sys/conf sys/dev/mpr sys/dev/nvme

[Warner Losh]

I know all about losing things during the rebase....

I'd be a lot happier if you could confirm that nothing else was lost. It's
a large patch, which makes it impossible for others to do it...  It seems
to build, and seems to be working in the light testing I've done this
morning...

Warner

On Thu, Feb 22, 2018 at 9:31 AM, Wojciech Macek <wma at semihalf.com> wrote:

> Thanks and sorry for troubles!
> These lines must have got lost somewhere during rebase as we run tinderbox
> with this patch yesterday.
>
> Regards,
> Wojtek
> wma@
>
> 2018-02-22 17:22 GMT+01:00 Alan Somers <asomers at freebsd.org>:
>
>> On Thu, Feb 22, 2018 at 9:20 AM, Kyle Evans <kevans at freebsd.org> wrote:
>>
>>> On Thu, Feb 22, 2018 at 10:19 AM, Alan Somers <asomers at freebsd.org>
>>> wrote:
>>> > On Thu, Feb 22, 2018 at 6:32 AM, Wojciech Macek <wma at freebsd.org>
>>> wrote:
>>> >
>>> >> Author: wma
>>> >> Date: Thu Feb 22 13:32:31 2018
>>> >> New Revision: 329824
>>> >> URL: https://svnweb.freebsd.org/changeset/base/329824
>>> >>
>>> >> Log:
>>> >>   NVMe: Add big-endian support
>>> >>
>>> >>   Remove bitfields from defined structures as they are not portable.
>>> >>   Instead use shift and mask macros in the driver and nvmecontrol
>>> >> application.
>>> >>
>>> >>   NVMe is now working on powerpc64 host.
>>> >>
>>> >>   Submitted by:          Michal Stanek <mst at semihalf.com>
>>> >>   Obtained from:         Semihalf
>>> >>   Reviewed by:           imp, wma
>>> >>   Sponsored by:          IBM, QCM Technologies
>>> >>   Differential revision: https://reviews.freebsd.org/D13916
>>> >>
>>> >
>>> > It looks like this change broke the build with Clang:
>>> >
>>>
>>> Fixed in r329827... all little endian arch's broken, not so great. =(
>>>
>>
>> Fast work, Kyle.  Thanks a bundle.
>> -Alan
>>
>
>