- User Since
- Jan 15 2018, 5:18 PM (52 w, 6 d)
Fri, Jan 18
Convert copycommon to a macro.
Adjust XLEN constant names and values.
Thu, Jan 17
Tue, Jan 15
Mon, Jan 7
Thu, Jan 3
Thank you for picking this up, I've finally had the chance to look through this whole thing. A couple of small nitpicks but otherwise looks good to me.
Dec 18 2018
Nov 26 2018
Nov 25 2018
Nov 24 2018
Did some of this work before I realized the obvious blocker that we don't have kgdb for RISC-V yet. It dumps successfully but I can't inspect it yet until we have that. Regardless, I figured I'd put this up now to get some initial looks at the changes. I will look into adding the required bits to kgdb to get it working.
Nov 13 2018
Nov 12 2018
Nov 11 2018
Oct 26 2018
Ahh I see, I misunderstood the purpose of duplicating the ACL. Much better now.
Oct 11 2018
Cleaned this up a bit. The logic is the same but should be a bit more obvious now.
Apr 27 2018
No code changes, just updated from changes made to D15060
Address review comments
Apr 13 2018
Apr 10 2018
Change to fix last review comments:
- Use size_t instead of u_int
- Provide proper path to set_acl_mask()
Apr 9 2018
Apr 3 2018
Fixed issue with reallocation size
Apr 2 2018
fts_open() requires an array of strings as an argument, so I had to modify the way input is handled from stdin. It now adds the filename inputs to an array which reallocs when necessary -- an approach that doesn't seem particularly elegant to me, but I thought was preferable to calling fts_open() on each input file individually. If anyone wants to weigh in on this choice I'd gladly hear it.
Mar 26 2018
Moved function from machine/cpufunc.h to machine/cpu.h
Mar 22 2018
@kib Yes, you are correct about the purpose of this function. This particular technique came from the brief discussion about it between yourself and @emaste, however, the (perhaps naive) assumption that it would apply equally to all architectures was mine. The coverage function also checks td->intr_nesting_level which I presume is a more MI way of avoiding some of the code in unwanted contexts, but is less effective (on x86) than checking this interrupts_enabled() function.
Mar 21 2018
Remove KIONREAD ioctl and move that data to the first entry of the coverage buffer
Mar 15 2018
Test cases would be a good idea, certainly it is something I can add. That being said this interface is still experimental so I may make that a separate patch.
Mar 14 2018
- Changed the stored PC type back to uintptr_t
Mar 8 2018
The interfaces are similar but not identical - for example I chose to expose the number of buffer items as an ioctl rather than at the beginning of the buffer as they do in the Linux patch. Conveniently, it is not required that they match exactly as syzkaller has specific executor code per OS which we can adjust to fit this one.
Mar 6 2018
Feb 8 2018
Feb 7 2018
Feb 6 2018
Updated some descriptions to match desired style.
Updated some descriptions to match the desired style
Updated some of the descriptions to match the desired style.
Feb 5 2018
Jan 18 2018
Move pti_get_default() to identcpu.c
Changed to enable for all except AMD
Jan 17 2018
Changed pointer justification in the struct at the beginning of ef.c
Use explicit conditions (error != 0).
Remove the change to ef_get_offset() return type for another patch.
Jan 16 2018
A lot of these comments address issues/inconsistencies already present in the source. I will create a subsequent cleanup patch to address these but follow what is there at the moment for this patch, since @emaste wants this merged to fix the lld build process.
Jan 15 2018
Upload diff correctly with tabs and full context