> -----Original Message----- > From: intel-gvt-dev [mailto:[email protected]] On > Behalf Of [email protected] > Sent: Wednesday, June 5, 2019 6:10 PM > To: Zhang, Tina <[email protected]> > Cc: Tian, Kevin <[email protected]>; [email protected]; linux- > [email protected]; Zhenyu Wang <[email protected]>; Yuan, > Hang <[email protected]>; [email protected]; Lv, Zhiyuan > <[email protected]>; [email protected]; Wang, Zhi A > <[email protected]> > Subject: Re: [RFC PATCH v2 1/3] vfio: Use capability chains to handle device > specific irq > > Hi, > > > > Really need to split for different planes? I'd like a > > > VFIO_IRQ_SUBTYPE_GFX_DISPLAY_EVENT > > > so user space can probe change for all. > > > User space can choose to user different handlers according to the > > specific event. For example, user space might not want to handle every > > cursor event due to performance consideration. Besides, it can reduce > > the probe times, as we don't need to probe twice to make sure if both > > cursor plane and primary plane have been updated. > > I'd suggest to use the value passed via eventfd for that, i.e. instead of > sending "1" unconditionally send a mask of changed planes. If there is only one eventfd working for GFX_DISPLAY, should it be VFIO_IRQ_INFO_EVENTFD and VFIO_IRQ_INFO_AUTOMASKED? i.e. after signaling, the interrupt is automatically masked and the user space needs to unmask the line to receive new irq event.
BR, Tina > > cheers, > Gerd > > _______________________________________________ > intel-gvt-dev mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gvt-dev
