No build, no run. We found several problems (memory ordering related so visible on ARM but not on x86) in the upstream code when running on ARM systems. Aren’t there any ARM systems to run on in the LNG lab?
-- Ola Ola Liljedahl, Networking System Architect, ARM Phone: +46 706 866 373 Skype: ola.liljedahl From: Maxim Uvarov <[email protected]<mailto:[email protected]>> Date: Thursday, 30 March 2017 at 16:56 To: Bill Fischofer <[email protected]<mailto:[email protected]>> Cc: Brian Brooks <[email protected]<mailto:[email protected]>>, lng-odp-forward <[email protected]<mailto:[email protected]>>, Ola Liljedahl <[email protected]<mailto:[email protected]>>, Kevin Wang <[email protected]<mailto:[email protected]>>, Honnappa Nagarahalli <[email protected]<mailto:[email protected]>> Subject: Re: [lng-odp] [API-NEXT 4/4] A scalable software scheduler I think for now we do not have build for arm v8 & clang. At least it did not capture build error. Maxim. On 30 March 2017 at 17:45, Bill Fischofer <[email protected]<mailto:[email protected]>> wrote: On Thu, Mar 30, 2017 at 8:56 AM, Brian Brooks <[email protected]<mailto:[email protected]>> wrote: > On 03/28 18:50:32, Bill Fischofer wrote: >> <html><head><meta http-equiv="Content-Type" content="text/html; >> charset=utf-8"> >> <meta name="Generator" content="Microsoft Exchange Server"> >> <!-- converted from text --> >> <style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: >> #800000 2px solid; } --></style></head> >> <body> >> <font size="2"><span style="font-size:10pt;"><div class="PlainText">This >> part generates numerous checkpatch warnings and errors. Please<br> >> run checkpatch and correct for v2.<br> >> <br> >> Also, this part introduces a number of errors that result in failure<br> >> to compile using clang. Please test with both gcc and clang to ensure<br> >> that it compiles cleanly for both (gcc looks fine)<br> >> <br> >> Specific clang issues:<br> > > Does upstream CI build with Clang on ARMv8? I don't think so because > the build is broken when I try it locally. I have the patch and will > send shortly. CI is supposed to be regression testing on ARM using clang as well as gcc. Maxim may have insight here. IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
