Hi Luca, On 12/03/2024 14:03, Luca Fancellu wrote: > > > From: Penny Zheng <penny.zh...@arm.com> > > Putting overlap and overflow checking in the loop is causing repetitive > operation, so this commit extracts both checking outside the loop. > > Signed-off-by: Penny Zheng <penny.zh...@arm.com> > Signed-off-by: Luca Fancellu <luca.fance...@arm.com> > --- > v1: > - Rework of > https://patchwork.kernel.org/project/xen-devel/patch/20231206090623.1932275-3-penny.zh...@arm.com/ > - use strncmp to match the branch above I don't understand why we need strncmp if we already validated that shm_id is NULL terminated within MAX_SHM_ID_LENGTH at the beginning of the function with strnlen.
Other than that: Reviewed-by: Michal Orzel <michal.or...@amd.com> ~Michal