On 03/02/21 11:59 pm, Sandipan Das wrote:
> Hello Boris,
>
> On 03/02/21 2:34 am, Boris Shingarov wrote:
>> Hi Sandipan,
>>
>>> This makes it possible
>>> to run both 32-bit and 64-bit big and little endian PowerPC binaries
>>> in SE mode. If its okay with you, we can start by working on trying to
>>> get these changes reviewed and merged first.
>>
>> Yes, yes! This aligns very well with both what sequence I think is the most
>> realistic technically, and with my project's priorities.
>>
>
> Great!
>
>>> You can find them here: [develop-power]
>>
>> Some of these commits just scream as the perfect first candidates, for
>> example
>> https://github.com/sandip4n/gem5/commit/3dd0438159d67fe2b5bcc777b43046fdca90f6b9
>> and related. Because right now the Decoder is in a really sad state,
>> crashing even on simple, perfectly valid programs (and that's even within
>> BE, 32-bit SE), see for example https://gem5.atlassian.net/browse/GEM5-819.
>> So, how about we start from that one? How would you like to proceed -- do
>> you want to submit on Gerrit using the regular process and I will review it?
>>
>
> I think I had come across that problem too but I am sure
> that one of my patches will fix that. Probably this one:
> https://github.com/sandip4n/gem5/commit/3b0203e63d892c94fd0e67cfe79f5e6662504c4a
>
Sorry I missed the other question. Yes, I can submit it via gerrit.
As a kernel developer, I am more used to mailing list based reviews
but feel free to let me know what works best for you.
That reminds me, I'll also need to rebase my code against the latest
"develop" branch.
- Sandipan
_______________________________________________
gem5-dev mailing list -- [email protected]
To unsubscribe send an email to [email protected]
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
