On Tue, May 11, 2021 at 05:16:38PM +0200, Daniel Vetter wrote: > On Thu, May 06, 2021 at 12:13:34PM -0700, Matthew Brost wrote: > > From: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > > > > New GuC firmware will unify format of MMIO and CTB H2G messages. > > Introduce their definitions now to allow gradual transition of > > our code to match new changes. > > > > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > > Signed-off-by: Matthew Brost <matthew.brost@xxxxxxxxx> > > Cc: Michał Winiarski <michal.winiarski@xxxxxxxxx> > > --- > > .../gpu/drm/i915/gt/uc/abi/guc_messages_abi.h | 226 ++++++++++++++++++ > > 1 file changed, 226 insertions(+) > > > > diff --git a/drivers/gpu/drm/i915/gt/uc/abi/guc_messages_abi.h b/drivers/gpu/drm/i915/gt/uc/abi/guc_messages_abi.h > > index 775e21f3058c..1c264819aa03 100644 > > --- a/drivers/gpu/drm/i915/gt/uc/abi/guc_messages_abi.h > > +++ b/drivers/gpu/drm/i915/gt/uc/abi/guc_messages_abi.h > > @@ -6,6 +6,232 @@ > > #ifndef _ABI_GUC_MESSAGES_ABI_H > > #define _ABI_GUC_MESSAGES_ABI_H > > > > +/** > > + * DOC: HXG Message > > These aren't useful if we don't pull them in somewhere in the > Documentation/gpu hierarchy. General comment, and also please check that > it all renders correctly still. > Sure. Let me figure this out before my next rev. > btw if you respin a patch not originally by you we generally add a (v1) to > the original s-o-b line (or whever the version split was) and explain in > the usual changelog in the commit message what was changed. > Still new to this process. Will do. Matt > This holds for the entire series ofc. > -Daniel > > > + * > > + * All messages exchanged with GuC are defined using 32 bit dwords. > > + * First dword is treated as a message header. Remaining dwords are optional. > > + * > > + * .. _HXG Message: > > + * > > + * +---+-------+--------------------------------------------------------------+ > > + * | | Bits | Description | > > + * +===+=======+==============================================================+ > > + * | | | | > > + * | 0 | 31 | **ORIGIN** - originator of the message | > > + * | | | - _`GUC_HXG_ORIGIN_HOST` = 0 | > > + * | | | - _`GUC_HXG_ORIGIN_GUC` = 1 | > > + * | | | | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 30:28 | **TYPE** - message type | > > + * | | | - _`GUC_HXG_TYPE_REQUEST` = 0 | > > + * | | | - _`GUC_HXG_TYPE_EVENT` = 1 | > > + * | | | - _`GUC_HXG_TYPE_NO_RESPONSE_BUSY` = 3 | > > + * | | | - _`GUC_HXG_TYPE_NO_RESPONSE_RETRY` = 5 | > > + * | | | - _`GUC_HXG_TYPE_RESPONSE_FAILURE` = 6 | > > + * | | | - _`GUC_HXG_TYPE_RESPONSE_SUCCESS` = 7 | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 27:0 | **AUX** - auxiliary data (depends TYPE) | > > + * +---+-------+--------------------------------------------------------------+ > > + * | 1 | 31:0 | optional payload (depends on TYPE) | > > + * +---+-------+ | > > + * |...| | | > > + * +---+-------+ | > > + * | n | 31:0 | | > > + * +---+-------+--------------------------------------------------------------+ > > + */ > > + > > +#define GUC_HXG_MSG_MIN_LEN 1u > > +#define GUC_HXG_MSG_0_ORIGIN (0x1 << 31) > > +#define GUC_HXG_ORIGIN_HOST 0u > > +#define GUC_HXG_ORIGIN_GUC 1u > > +#define GUC_HXG_MSG_0_TYPE (0x7 << 28) > > +#define GUC_HXG_TYPE_REQUEST 0u > > +#define GUC_HXG_TYPE_EVENT 1u > > +#define GUC_HXG_TYPE_NO_RESPONSE_BUSY 3u > > +#define GUC_HXG_TYPE_NO_RESPONSE_RETRY 5u > > +#define GUC_HXG_TYPE_RESPONSE_FAILURE 6u > > +#define GUC_HXG_TYPE_RESPONSE_SUCCESS 7u > > +#define GUC_HXG_MSG_0_AUX (0xfffffff << 0) > > + > > +/** > > + * DOC: HXG Request > > + * > > + * The `HXG Request`_ message should be used to initiate synchronous activity > > + * for which confirmation or return data is expected. > > + * > > + * The recipient of this message shall use `HXG Response`_, `HXG Failure`_ > > + * or `HXG Retry`_ message as a definite reply, and may use `HXG Busy`_ > > + * message as a intermediate reply. > > + * > > + * Format of @DATA0 and all @DATAn fields depends on the @ACTION code. > > + * > > + * _HXG Request: > > + * > > + * +---+-------+--------------------------------------------------------------+ > > + * | | Bits | Description | > > + * +===+=======+==============================================================+ > > + * | 0 | 31 | ORIGIN | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 30:28 | TYPE = GUC_HXG_TYPE_REQUEST_ | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 27:16 | **DATA0** - request data (depends on ACTION) | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 15:0 | **ACTION** - requested action code | > > + * +---+-------+--------------------------------------------------------------+ > > + * | 1 | 31:0 | **DATA1** - optional data (depends on ACTION) | > > + * +---+-------+--------------------------------------------------------------+ > > + * |...| | | > > + * +---+-------+--------------------------------------------------------------+ > > + * | n | 31:0 | **DATAn** - optional data (depends on ACTION) | > > + * +---+-------+--------------------------------------------------------------+ > > + */ > > + > > +#define GUC_HXG_REQUEST_MSG_MIN_LEN GUC_HXG_MSG_MIN_LEN > > +#define GUC_HXG_REQUEST_MSG_0_DATA0 (0xfff << 16) > > +#define GUC_HXG_REQUEST_MSG_0_ACTION (0xffff << 0) > > +#define GUC_HXG_REQUEST_MSG_n_DATAn (0xffffffff << 0) > > + > > +/** > > + * DOC: HXG Event > > + * > > + * The `HXG Event`_ message should be used to initiate asynchronous activity > > + * that does not involves immediate confirmation nor data. > > + * > > + * Format of @DATA0 and all @DATAn fields depends on the @ACTION code. > > + * > > + * .. _HXG Event: > > + * > > + * +---+-------+--------------------------------------------------------------+ > > + * | | Bits | Description | > > + * +===+=======+==============================================================+ > > + * | 0 | 31 | ORIGIN | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 30:28 | TYPE = GUC_HXG_TYPE_EVENT_ | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 27:16 | **DATA0** - event data (depends on ACTION) | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 15:0 | **ACTION** - event action code | > > + * +---+-------+--------------------------------------------------------------+ > > + * | 1 | 31:0 | **DATA1** - optional event data (depends on ACTION) | > > + * +---+-------+--------------------------------------------------------------+ > > + * |...| | | > > + * +---+-------+--------------------------------------------------------------+ > > + * | n | 31:0 | **DATAn** - optional event data (depends on ACTION) | > > + * +---+-------+--------------------------------------------------------------+ > > + */ > > + > > +#define GUC_HXG_EVENT_MSG_MIN_LEN GUC_HXG_MSG_MIN_LEN > > +#define GUC_HXG_EVENT_MSG_0_DATA0 (0xfff << 16) > > +#define GUC_HXG_EVENT_MSG_0_ACTION (0xffff << 0) > > +#define GUC_HXG_EVENT_MSG_n_DATAn (0xffffffff << 0) > > + > > +/** > > + * DOC: HXG Busy > > + * > > + * The `HXG Busy`_ message may be used to acknowledge reception of the `HXG Request`_ > > + * message if the recipient expects that it processing will be longer than default > > + * timeout. > > + * > > + * The @COUNTER field may be used as a progress indicator. > > + * > > + * .. _HXG Busy: > > + * > > + * +---+-------+--------------------------------------------------------------+ > > + * | | Bits | Description | > > + * +===+=======+==============================================================+ > > + * | 0 | 31 | ORIGIN | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 30:28 | TYPE = GUC_HXG_TYPE_NO_RESPONSE_BUSY_ | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 27:0 | **COUNTER** - progress indicator | > > + * +---+-------+--------------------------------------------------------------+ > > + */ > > + > > +#define GUC_HXG_BUSY_MSG_LEN GUC_HXG_MSG_MIN_LEN > > +#define GUC_HXG_BUSY_MSG_0_COUNTER GUC_HXG_MSG_0_AUX > > + > > +/** > > + * DOC: HXG Retry > > + * > > + * The `HXG Retry`_ message should be used by recipient to indicate that the > > + * `HXG Request`_ message was dropped and it should be resent again. > > + * > > + * The @REASON field may be used to provide additional information. > > + * > > + * .. _HXG Retry: > > + * > > + * +---+-------+--------------------------------------------------------------+ > > + * | | Bits | Description | > > + * +===+=======+==============================================================+ > > + * | 0 | 31 | ORIGIN | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 30:28 | TYPE = GUC_HXG_TYPE_NO_RESPONSE_RETRY_ | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 27:0 | **REASON** - reason for retry | > > + * | | | - _`GUC_HXG_RETRY_REASON_UNSPECIFIED` = 0 | > > + * +---+-------+--------------------------------------------------------------+ > > + */ > > + > > +#define GUC_HXG_RETRY_MSG_LEN GUC_HXG_MSG_MIN_LEN > > +#define GUC_HXG_RETRY_MSG_0_REASON GUC_HXG_MSG_0_AUX > > +#define GUC_HXG_RETRY_REASON_UNSPECIFIED 0u > > + > > +/** > > + * DOC: HXG Failure > > + * > > + * The `HXG Failure`_ message shall be used as a reply to the `HXG Request`_ > > + * message that could not be processed due to an error. > > + * > > + * .. _HXG Failure: > > + * > > + * +---+-------+--------------------------------------------------------------+ > > + * | | Bits | Description | > > + * +===+=======+==============================================================+ > > + * | 0 | 31 | ORIGIN | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 30:28 | TYPE = GUC_HXG_TYPE_RESPONSE_FAILURE_ | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 27:16 | **HINT** - additional error hint | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 15:0 | **ERROR** - error/result code | > > + * +---+-------+--------------------------------------------------------------+ > > + */ > > + > > +#define GUC_HXG_FAILURE_MSG_LEN GUC_HXG_MSG_MIN_LEN > > +#define GUC_HXG_FAILURE_MSG_0_HINT (0xfff << 16) > > +#define GUC_HXG_FAILURE_MSG_0_ERROR (0xffff << 0) > > + > > +/** > > + * DOC: HXG Response > > + * > > + * The `HXG Response`_ message SHALL be used as a reply to the `HXG Request`_ > > + * message that was successfully processed without an error. > > + * > > + * .. _HXG Response: > > + * > > + * +---+-------+--------------------------------------------------------------+ > > + * | | Bits | Description | > > + * +===+=======+==============================================================+ > > + * | 0 | 31 | ORIGIN | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 30:28 | TYPE = GUC_HXG_TYPE_RESPONSE_SUCCESS_ | > > + * | +-------+--------------------------------------------------------------+ > > + * | | 27:0 | **DATA0** - data (depends on ACTION from `HXG Request`_) | > > + * +---+-------+--------------------------------------------------------------+ > > + * | 1 | 31:0 | **DATA1** - data (depends on ACTION from `HXG Request`_) | > > + * +---+-------+--------------------------------------------------------------+ > > + * |...| | | > > + * +---+-------+--------------------------------------------------------------+ > > + * | n | 31:0 | **DATAn** - data (depends on ACTION from `HXG Request`_) | > > + * +---+-------+--------------------------------------------------------------+ > > + */ > > + > > +#define GUC_HXG_RESPONSE_MSG_MIN_LEN GUC_HXG_MSG_MIN_LEN > > +#define GUC_HXG_RESPONSE_MSG_0_DATA0 GUC_HXG_MSG_0_AUX > > +#define GUC_HXG_RESPONSE_MSG_n_DATAn (0xffffffff << 0) > > + > > +/* deprecated */ > > #define INTEL_GUC_MSG_TYPE_SHIFT 28 > > #define INTEL_GUC_MSG_TYPE_MASK (0xF << INTEL_GUC_MSG_TYPE_SHIFT) > > #define INTEL_GUC_MSG_DATA_SHIFT 16 > > -- > > 2.28.0 > > > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch
