Re: [Qemu-devel] [PATCH] loader: implement START_SEG_ADDR_RECORD

2018-06-18 Thread Stefan Hajnoczi
On Thu, Jun 14, 2018 at 02:43:11PM +0800, Su Hang wrote: > > Sorry, Julia and Stefan. My school mentor assign an emergency task to me, he > ask > me to finish it within two weeks. I'm afraid I may not able to implement the > function mentioned in last comments in this two weeks. If the new functi

Re: [Qemu-devel] [PATCH] loader: implement START_SEG_ADDR_RECORD

2018-06-13 Thread Su Hang
Sorry, Julia and Stefan. My school mentor assign an emergency task to me, he ask me to finish it within two weeks. I'm afraid I may not able to implement the function mentioned in last comments in this two weeks. If the new function is in urgent need, please feel free to take over the patches; if

Re: [Qemu-devel] [PATCH] loader: implement START_SEG_ADDR_RECORD

2018-06-13 Thread Stefan Hajnoczi
On Tue, Jun 12, 2018 at 01:46:34PM +0800, Su Hang wrote: > > I do have time, the function mentioned in last comments isn't difficult to > implement, but I wonder how to write corresponding qtest-case for cortex-m3. > You > know, becuase #current# QEMU doesn't surpport the cortex-m3 instruction, I

Re: [Qemu-devel] [PATCH] loader: implement START_SEG_ADDR_RECORD

2018-06-11 Thread Su Hang
I do have time, the function mentioned in last comments isn't difficult to implement, but I wonder how to write corresponding qtest-case for cortex-m3. You know, becuase #current# QEMU doesn't surpport the cortex-m3 instruction, I don't know how to prove correctness of hex loader. For example, in

Re: [Qemu-devel] [PATCH] loader: implement START_SEG_ADDR_RECORD

2018-06-11 Thread Stefan Hajnoczi
On Sun, Jun 10, 2018 at 10:04:32AM +0800, Su Hang wrote: > Sure, Thanks for remind me of this. > One thing I must point out, in current code logic, if "START_SEG_ADDR_RECORD" > occured multiple times, only the last one works. I don't know whether GNU > binutils would emit 'The Record' many times.

Re: [Qemu-devel] [PATCH] loader: implement START_SEG_ADDR_RECORD

2018-06-09 Thread Su Hang
Sure, Thanks for remind me of this. One thing I must point out, in current code logic, if "START_SEG_ADDR_RECORD" occured multiple times, only the last one works. I don't know whether GNU binutils would emit 'The Record' many times. Best, SU Hang > -Original Messages- > From: "Stefan Haj

[Qemu-devel] [PATCH] loader: implement START_SEG_ADDR_RECORD

2018-06-09 Thread Stefan Hajnoczi
It turns out that GNU binutils emits START_SEG_ADDR_RECORD when the start address is within the first megabyte (< 0x10). Therefore we must handle this record type. Originally we thought this record type was x86-specific, but binutils also emits it on non-x86 architectures. Based-on: <1527161