Re: [PATCH v2] system/memory: use ldn_he_p/stn_he_p

2023-12-04 Thread Patrick Venture
On Mon, Dec 4, 2023 at 3:24 AM Philippe Mathieu-Daudé wrote: > Hi Patrick, > > On 3/12/23 16:42, Patrick Venture wrote: > > > Friendly ping? Is this going to be applied or do I need to add another > > CC or? I do think it should go into stable. > > I'll send a PR with this patch included. >

Re: [PATCH v2] system/memory: use ldn_he_p/stn_he_p

2023-12-04 Thread Philippe Mathieu-Daudé
Hi Patrick, On 3/12/23 16:42, Patrick Venture wrote: Friendly ping? Is this going to be applied or do I need to add another CC or?  I do think it should go into stable. I'll send a PR with this patch included. Regards, Phil.

Re: [PATCH v2] system/memory: use ldn_he_p/stn_he_p

2023-12-03 Thread Patrick Venture
On Fri, Nov 17, 2023 at 12:43 AM David Hildenbrand wrote: > On 16.11.23 17:36, Patrick Venture wrote: > > Using direct pointer dereferencing can allow for unaligned accesses, > > which was seen during execution with sanitizers enabled. > > > > Reviewed-by: Chris Rauer > > Reviewed-by: Peter

Re: [PATCH v2] system/memory: use ldn_he_p/stn_he_p

2023-11-17 Thread David Hildenbrand
On 16.11.23 17:36, Patrick Venture wrote: Using direct pointer dereferencing can allow for unaligned accesses, which was seen during execution with sanitizers enabled. Reviewed-by: Chris Rauer Reviewed-by: Peter Foley Signed-off-by: Patrick Venture Cc: qemu-sta...@nongnu.org Reviewed-by:

Re: [PATCH v2] system/memory: use ldn_he_p/stn_he_p

2023-11-17 Thread Philippe Mathieu-Daudé
On 16/11/23 17:36, Patrick Venture wrote: Using direct pointer dereferencing can allow for unaligned accesses, which was seen during execution with sanitizers enabled. Reviewed-by: Chris Rauer Reviewed-by: Peter Foley Signed-off-by: Patrick Venture Cc: qemu-sta...@nongnu.org --- v2: changed

[PATCH v2] system/memory: use ldn_he_p/stn_he_p

2023-11-16 Thread Patrick Venture
Using direct pointer dereferencing can allow for unaligned accesses, which was seen during execution with sanitizers enabled. Reviewed-by: Chris Rauer Reviewed-by: Peter Foley Signed-off-by: Patrick Venture Cc: qemu-sta...@nongnu.org --- v2: changed commit mesage to be more accurate and