This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: ARM SIM - add support to select SWI emulations to support.


> + 
> + extern unsigned int swi_mask;
> + 
> + #define SWI_MASK_DEMON		(1 << 0)
> + #define SWI_MASK_ANGEL		(1 << 1)
> + #define SWI_MASK_REDBOOT	(1 << 2)
> 
> 
> Why make this a mask?  A major part of the problem here is that these SWI 
> sets may be mutually incompatible.  Why not just require the user to 
> select the right one?

Actually, DEMON and ANGEL are compatible (they were designed to be).  But 
that doesn't make this generally true of all SWI sets that we might want 
to emulate.

The specific case here could be handled by allowing

	--swi=demon
	--swi=angel
	--swi=angel+demon
	--swi=redboot

Note the FPE in the ARMulator requires (a subset of) DEMON SWI support in 
order to be able to handle floating point exceptions.

R.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]