Linux MIPS Architecture development
 help / color / mirror / Atom feed
* emulate_load_store_insn
@ 2001-06-09  1:51 Carsten Langgaard
  2001-06-09  2:45 ` emulate_load_store_insn Ralf Baechle
  0 siblings, 1 reply; 2+ messages in thread
From: Carsten Langgaard @ 2001-06-09  1:51 UTC (permalink / raw)
  To: linux-mips

Can anyone please explain the whole deal with the emulate_load_store_insn 
function in arch/mips/kernel/unaligned.c.
Isn't there a potential hole there, where a user application makes an illegal 
memory access to an unaligned address and then the kernel tries to emulate that 
and crashes.
It also look like the MF_FIXADE flag is set by default, why is that ? Shouldn't 
one suppose to make a syscall setting this MF_FIXADE flag ?

/Carsten

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: emulate_load_store_insn
  2001-06-09  1:51 emulate_load_store_insn Carsten Langgaard
@ 2001-06-09  2:45 ` Ralf Baechle
  0 siblings, 0 replies; 2+ messages in thread
From: Ralf Baechle @ 2001-06-09  2:45 UTC (permalink / raw)
  To: Carsten Langgaard; +Cc: linux-mips

On Fri, Jun 08, 2001 at 06:51:01PM -0700, Carsten Langgaard wrote:

> Can anyone please explain the whole deal with the emulate_load_store_insn 
> function in arch/mips/kernel/unaligned.c.

Some software does of unaligned accesses.  Typical userspace example is fdisk
and the network stack which generally tries hard to avoid unaligned loads
and stores may make unaligned stores at times though.

> Isn't there a potential hole there, where a user application makes an illegal 
> memory access to an unaligned address and then the kernel tries to emulate
> that and crashes.

The addresses are verified the same way as any other userspace address
passed to the kernel.

> It also look like the MF_FIXADE flag is set by default, why is that ?

Two reasons 1) other MIPS OSes such as Risc/OS and IRIX also do it 2) crappy
software doesn't know how to enable this feature ...

> Shouldn't one suppose to make a syscall setting this MF_FIXADE flag ?

Sysmips(2) allows to toggle this flag.

  Ralf

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2001-06-09  2:46 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2001-06-09  1:51 emulate_load_store_insn Carsten Langgaard
2001-06-09  2:45 ` emulate_load_store_insn Ralf Baechle

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox