________________________________________ From: Peter Maydell [peter.mayd...@linaro.org] Sent: Friday, October 11, 2013 9:10 PM To: Petar Jovanovic Cc: QEMU Developers; QEMU Trivial; Riku Voipio; Petar Jovanovic; Aurelien Jarno Subject: Re: [Qemu-devel] [PATCH] linux-user: pass correct parameter to do_shmctl()
On 11 October 2013 22:03, Petar Jovanovic <petar.jovano...@rt-rk.com> wrote: > From: Petar Jovanovic <petar.jovano...@imgtec.com> > > Fix shmctl issue by passing correct parameter buf to do_shmctl(). Reviewed-by: Peter Maydell <peter.mayd...@linaro.org> > It would be nice if you gave more detail than "shmctl issue"... Yes, indeed. However, this is a bit tricky, because it can be associated with different issues, and this particular fix is a follow up to an another change [1] that I posted previously that defines arch-specific structs target_ipc_perm and target_shmid_ds. The layout of the structs is important. This particular change will for instance, when joined with the preceding change, make the code correctly return values in the buf struct after: shmctl(shm_id, IPC_STAT, &buf) Different architectures benefit from these two patches. Regards, Petar