From: Jakub Kicinski <kuba@kernel.org>
To: Rodrigo Vivi <rodrigo.vivi@intel.com>
Cc: Zack McKevitt <zachary.mckevitt@oss.qualcomm.com>,
<dri-devel@lists.freedesktop.org>, <netdev@vger.kernel.org>,
<riana.tauro@intel.com>, <aravind.iddamsetty@linux.intel.com>,
<raag.jadav@intel.com>, <simona.vetter@ffwll.ch>,
<airlied@gmail.com>, <ogabbay@kernel.org>, <lizhi.hou@amd.com>,
<karol.wachowski@linux.intel.com>, <jeff.hugo@oss.qualcomm.com>,
<carl.vanderlip@oss.qualcomm.com>, <troy.hanson@oss.qualcomm.com>,
<youssef.abdulrahman@oss.qualcomm.com>
Subject: Re: [PATCH 2/3] drm/drm_ras: Extend DRM RAS interface to include telemetry nodes
Date: Mon, 19 Oct 2026 13:28:24 -0700 [thread overview]
Message-ID: <20261019132824.0f2b3350@kernel.org> (raw)
In-Reply-To: <atYJLGHqCelhzSee@intel.com>
On Mon, 19 Oct 2026 08:12:28 -0400 Rodrigo Vivi wrote:
> On Fri, Oct 16, 2026 at 06:07:14PM -0700, Jakub Kicinski wrote:
> > On Fri, 9 Oct 2026 10:31:51 -0700 Zack McKevitt wrote:
> > > + type: u64
> >
> > prefer uint and then no pad
>
> same here, u64 aligns with hardware definition why uint should be desired?
> I noticed the review prompts, I fail to agree because we need predictability
> and align with hardware definitions.
Just because u64 is defined to be aligned and require the padding.
uint is a "modern u64"
next prev parent reply other threads:[~2026-10-19 20:28 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-10-09 17:31 [PATCH 0/3] Extend drm_ras for qaic telemetry Zack McKevitt
2026-10-09 17:31 ` [PATCH 1/3] drm/drm_ras: Remove admin-perm flag from list-nodes operation Zack McKevitt
2026-10-10 17:36 ` sashiko-bot
2026-10-14 21:11 ` Rodrigo Vivi
2026-10-09 17:31 ` [PATCH 2/3] drm/drm_ras: Extend DRM RAS interface to include telemetry nodes Zack McKevitt
2026-10-10 17:36 ` sashiko-bot
2026-10-14 21:11 ` Rodrigo Vivi
2026-10-16 20:09 ` Zack McKevitt
2026-10-16 21:32 ` Rodrigo Vivi
2026-10-17 1:07 ` Jakub Kicinski
2026-10-19 12:12 ` Rodrigo Vivi
2026-10-19 20:28 ` Jakub Kicinski [this message]
2026-10-09 17:31 ` [PATCH 3/3] accel/qaic: Add callbacks for DRM RAS netlink telemetry Zack McKevitt
2026-10-10 17:36 ` sashiko-bot
2026-10-10 9:42 ` [syzbot ci] Re: Extend drm_ras for qaic telemetry syzbot ci
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20261019132824.0f2b3350@kernel.org \
--to=kuba@kernel.org \
--cc=airlied@gmail.com \
--cc=aravind.iddamsetty@linux.intel.com \
--cc=carl.vanderlip@oss.qualcomm.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=jeff.hugo@oss.qualcomm.com \
--cc=karol.wachowski@linux.intel.com \
--cc=lizhi.hou@amd.com \
--cc=netdev@vger.kernel.org \
--cc=ogabbay@kernel.org \
--cc=raag.jadav@intel.com \
--cc=riana.tauro@intel.com \
--cc=rodrigo.vivi@intel.com \
--cc=simona.vetter@ffwll.ch \
--cc=troy.hanson@oss.qualcomm.com \
--cc=youssef.abdulrahman@oss.qualcomm.com \
--cc=zachary.mckevitt@oss.qualcomm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.