Re: [PATCH v4 27/53] semihosting: Split out semihost_sys_open

2022-06-27 Thread Luc Michel
On 08:05 Wed 22 Jun , Richard Henderson wrote: > On 6/22/22 02:35, Luc Michel wrote: > > On 13:45 Tue 07 Jun , Richard Henderson wrote: > > > Split out the non-ARM specific portions of SYS_OPEN to a > > > reusable function. This handles gdb and host file i/o. > > > > > > Add helpers to va

Re: [PATCH v4 27/53] semihosting: Split out semihost_sys_open

2022-06-22 Thread Richard Henderson
On 6/22/22 02:35, Luc Michel wrote: On 13:45 Tue 07 Jun , Richard Henderson wrote: Split out the non-ARM specific portions of SYS_OPEN to a reusable function. This handles gdb and host file i/o. Add helpers to validate the length of the filename string. Prepare for usage by other semihosti

Re: [PATCH v4 27/53] semihosting: Split out semihost_sys_open

2022-06-22 Thread Luc Michel
On 13:45 Tue 07 Jun , Richard Henderson wrote: > Split out the non-ARM specific portions of SYS_OPEN to a > reusable function. This handles gdb and host file i/o. > > Add helpers to validate the length of the filename string. > Prepare for usage by other semihosting by allowing the > filename

[PATCH v4 27/53] semihosting: Split out semihost_sys_open

2022-06-07 Thread Richard Henderson
Split out the non-ARM specific portions of SYS_OPEN to a reusable function. This handles gdb and host file i/o. Add helpers to validate the length of the filename string. Prepare for usage by other semihosting by allowing the filename length parameter to be 0, and calling strlen. Signed-off-by: