Mon, 14 Jan 2013 09:22:12 +0000
More bus error fixes for FreeBee
I have fixed two more bus error handling bugs in FreeBee. First, the CPU core was executing the instruction regardless of whether a bus error occurs when fetching the opcode (which caused it to execute a bogus instruction in such cases). The other one was related to one of my previous fixes - the jump to the bus error vector was at the beginning of the main loop, so it wouldn't be called immediately after the bus error occurred if the timeslot expired, causing the return address to be off.
With these fixes, Unix now runs enough to get into userspace and run the install script (it is also possible to break out and get a shell prompt). However, many commands segfault semi-randomly (or more specifically, it seems that some child processes forked by the shell might be segfaulting before they can exec the command program), so installing the system isn't possible yet. I am not sure exactly what the bug is, but it seems to be related to some function in the shell returning null when the code calling it is assuming that it won't. What the function is, or why it is returning null, I'm not sure (the shell is built without the shared libc and is stripped, making identifying the function harder). I suspect that the function might be in libc, but that is hard to tell.
Author: Andrew Warkentin <andreww591 gmail com>
1 #ifndef _MEMORY_H
2 #define _MEMORY_H
4 /***********************************
5 * Array read/write utility macros
6 * "Don't Repeat Yourself" :)
7 ***********************************/
9 /// Array read, 32-bit
10 #define RD32(array, address, andmask) \
11 (((uint32_t)array[(address + 0) & (andmask)] << 24) | \
12 ((uint32_t)array[(address + 1) & (andmask)] << 16) | \
13 ((uint32_t)array[(address + 2) & (andmask)] << 8) | \
14 ((uint32_t)array[(address + 3) & (andmask)]))
16 /// Array read, 16-bit
17 #define RD16(array, address, andmask) \
18 (((uint32_t)array[(address + 0) & (andmask)] << 8) | \
19 ((uint32_t)array[(address + 1) & (andmask)]))
21 /// Array read, 8-bit
22 #define RD8(array, address, andmask) \
23 ((uint32_t)array[(address + 0) & (andmask)])
25 /// Array write, 32-bit
26 #define WR32(array, address, andmask, value) do { \
27 array[(address + 0) & (andmask)] = (value >> 24) & 0xff; \
28 array[(address + 1) & (andmask)] = (value >> 16) & 0xff; \
29 array[(address + 2) & (andmask)] = (value >> 8) & 0xff; \
30 array[(address + 3) & (andmask)] = value & 0xff; \
31 } while (0)
33 /// Array write, 16-bit
34 #define WR16(array, address, andmask, value) do { \
35 array[(address + 0) & (andmask)] = (value >> 8) & 0xff; \
36 array[(address + 1) & (andmask)] = value & 0xff; \
37 } while (0)
39 /// Array write, 8-bit
40 #define WR8(array, address, andmask, value) do { \
41 array[(address + 0) & (andmask)] = value & 0xff; \
42 } while (0)
44 /******************
45 * Memory mapping
46 ******************/
48 typedef enum {
49 MEM_ALLOWED = 0,
50 MEM_PAGEFAULT, // Page fault -- page not present
51 MEM_PAGE_NO_WE, // Page not write enabled
52 MEM_KERNEL, // User attempted to access kernel memory
53 MEM_UIE // User Nonmemory Location Access
54 } MEM_STATUS;
56 /**
57 * @brief Check memory access permissions for a given address.
58 * @param addr Address.
59 * @param writing true if writing to memory, false if reading.
60 * @return One of the MEM_STATUS constants, specifying whether the access is
61 * permitted, or what error occurred.
62 */
63 MEM_STATUS checkMemoryAccess(uint32_t addr, bool writing);
65 /**
66 * @brief Map a CPU memory address into physical memory space.
67 * @param addr Address.
68 * @param writing true if writing to memory, false if reading.
69 * @return Address, remapped into physical memory.
70 */
71 uint32_t mapAddr(uint32_t addr, bool writing);
73 /**
74 * @brief Check access flags for a DMA transfer and trigger an exception if
75 * the access is not permitted
76 * @param reading true if reading from memory, false if writing
77 * @return true if the access is permitted, false if not
78 */
79 bool access_check_dma(int reading);
81 #endif