[OpenBIOS] [PATCH v2 0/4] Build openbios on powerpc64 host
Mark Cave-Ayland
mark.cave-ayland at ilande.co.uk
Fri Nov 6 16:16:08 CET 2015
On 01/11/15 23:19, Laurent Vivier wrote:
> Le 01/11/2015 23:46, Mark Cave-Ayland a écrit :
>> On 25/10/15 15:48, Laurent Vivier wrote:
>>
>>> This series of patches allows to build openbios on a
>>> powerpc64 host. It has been tested with the following targets:
>>> ppc, sparc32, sparc64. It has also been tested on x86_64 host
>>> with the same targets.
>>>
>>> v2: remove patches 3/5 and 4/5 of the first series and replace it
>>> with patch 4/4. We don't include the system includes using
>>> full pathname but use '-iquote' intead of '-I'
>>>
>>> A preliminary patch is needed for fcode-utils, as "toke" is
>>> broken on big endian architecture:
>>>
>>> diff -ru fcode-utils/toke/scanner.c fcode-utils-new/toke/scanner.c
>>> --- fcode-utils/toke/scanner.c 2006-10-30 09:48:28.000000000 +0100
>>> +++ fcode-utils-new/toke/scanner.c 2015-10-19 00:21:43.417373690 +0200
>>> @@ -1625,7 +1625,7 @@
>>>
>>> void user_message( tic_param_t pfield )
>>> {
>>> - char delim = (char)pfield.deflt_elem ;
>>> + char delim = (char)pfield.fw_token ;
>>> handle_user_message( delim, TRUE);
>>> }
>>>
>>> @@ -5295,7 +5295,7 @@
>>>
>>> void process_remark( tic_param_t pfield )
>>> {
>>> - char until_char = (char)pfield.deflt_elem ;
>>> + char until_char = (char)pfield.fw_token ;
>>> unsigned int start_lineno = lineno;
>>>
>>> #ifdef DEBUG_SCANNER
>>>
>>> Laurent Vivier (4):
>>> switch-arch: factorize code to compute architecture properties.
>>> switch-arch: compute base arch and allow native compiler for 32bit
>>> and 64bit.
>>> switch-arch: as for powerpc64, select sparc64 compiler to compile
>>> sparc32
>>> bootstrap: don't include files from target/include as system includes.
>>>
>>> Makefile.target | 2 +-
>>> config/scripts/switch-arch | 85 +++++++++++++++++++++++++---------------------
>>> include/arch/ppc/types.h | 8 -----
>>> 3 files changed, 48 insertions(+), 47 deletions(-)
>>
>> Hi Laurent,
>
> Hi Mark,
>
>> I've done some tests here on both x86 and PPC hardware and the basic
>> patchset works well here. The only issue I found is that git-am
>> complains about whitespace errors during application - if you could
>> resubmit the series with this fixed, that would be great.
>
> Can I have more details, I've tried to "git-am" the patches I have
> generated and I had no troubles (do you have some git hooks ?)
>
>> As for the toke fix, I don't know this code particularly well but I can
>> at least confirm that when applied it still produces valid binaries.
>> Could you send a separate patch in "git style" explaining what bug this
>> patch fixes, and if it looks good then I'll apply it too.
>>
>
> Done
Hi Laurent,
Thanks a lot for this patchset. After re-applying on a fresh branch, the
whitespace errors were gone and everything still passed my local tests
so I've pushed the changes to SVN trunk.
This will definitely make testing OpenBIOS on PPC/KVM so much easier in
future :)
Many thanks,
Mark.
More information about the OpenBIOS
mailing list