[PATCH 1/2] drm/vmwgfx: Prevent unmapping active read buffers

Ian Forbes ian.forbes at broadcom.com
Tue Aug 13 17:55:50 UTC 2024


In that case move `map_count` above `map` which should move it to a
separate cache line and update the doc strings as needed.

Reviewed-by: Ian Forbes <ian.forbes at broadcom.com>

On Tue, Aug 13, 2024 at 12:40 PM Zack Rusin <zack.rusin at broadcom.com> wrote:
>
> On Tue, Aug 13, 2024 at 1:29 PM Ian Forbes <ian.forbes at broadcom.com> wrote:
> >
> > Remove `busy_places` now that it's unused. There's also probably a
> > better place to put `map_count` in the struct layout to avoid false
> > sharing with `cpu_writers`. I'd repack the whole struct if we're going
> > to be adding and removing fields.
>
> Those are not related to this change. They'd be two seperate changes.
> One to remove other unused members and third to relayout the struct.
>
> z


More information about the dri-devel mailing list