Hi, Jordan I agree with Andrew. The best fix should be "movw %es, %ax"
I searched MdePkg and MdeModulePkg, looks like only DebugSupportDxe .S file has such issue. Is it possible to make a clean-up? Best Regards Feng -----Original Message----- From: Jordan Justen [mailto:[email protected]] Sent: Saturday, October 18, 2014 00:27 To: [email protected] Subject: Re: [edk2] [PATCH] MdeModulePkg DebugSupportDxe: Fix build error with GNU assembler On Fri, Oct 17, 2014 at 9:11 AM, Andrew Fish <[email protected]> wrote: > >> On Oct 17, 2014, at 9:05 AM, Jordan Justen <[email protected]> wrote: >> >> The GNU assembler (2.24.51.20140918) is failing to build when movw is >> used on this instruction. Instead use the mov instruction, which >> matches the other cases of loading segment registers. >> >> The error message seen is: >> AsmFuncs.iii:283: Error: incorrect register `%rax' used with `w' >> suffix >> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: Jordan Justen <[email protected]> >> --- >> MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.S | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.S >> b/MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.S >> index 9783ec6..7f0919e 100644 >> --- a/MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.S >> +++ b/MdeModulePkg/Universal/DebugSupportDxe/X64/AsmFuncs.S >> @@ -280,7 +280,7 @@ ExtraPushDone: >> >> mov %ds, %rax >> pushq %rax >> - movw %es, %rax >> + mov %es, %rax > > Is the issue the w or the %rax? Do you need to use the smaller register name > with movw? I noticed the other instructions were using mov rather than movw with rax and the segment registers, so I just matched that. (See move from ds to rax just above.) -Jordan ------------------------------------------------------------------------------ Comprehensive Server Monitoring with Site24x7. Monitor 10 servers for $9/Month. Get alerted through email, SMS, voice calls or mobile push notifications. Take corrective actions from your mobile device. http://p.sf.net/sfu/Zoho _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel ------------------------------------------------------------------------------ Comprehensive Server Monitoring with Site24x7. Monitor 10 servers for $9/Month. Get alerted through email, SMS, voice calls or mobile push notifications. Take corrective actions from your mobile device. http://p.sf.net/sfu/Zoho _______________________________________________ edk2-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/edk2-devel
