On Tue, Jul 12, 2022 at 6:57 PM Felix Kuehling <felix.kuehling@xxxxxxx> wrote:
>
> Eric sent out the corresponding user mode patches to the mailing list as
> well. It looks a bit weird, because it looks like they're part of the
> same patch series. But patch 2 and 3 are actually user mode patches. The
> interesting one is patch 3.
>
>
> Do we still need a link to a user mode patch in this case?
Yeah, it's hard to include it in the patch when you send it out if
it's part of the patch set. It's just nice to have it included when
you commit it so that when I send the PR, I don't need to dig around
to find the link to the user of the new UAPI.
Alex
>
>
> Regards,
> Felix
>
>
> On 2022-07-12 09:57, Deucher, Alexander wrote:
> >
> > [AMD Official Use Only - General]
> >
> >
> > Can you please include a link to the proposed userspace in the commit
> > message when you commit this?
> >
> > Alex
> > ------------------------------------------------------------------------
> > *From:* amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> on behalf of
> > Eric Huang <jinhuieric.huang@xxxxxxx>
> > *Sent:* Monday, July 11, 2022 2:41 PM
> > *To:* amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>
> > *Cc:* Huang, JinHuiEric <JinHuiEric.Huang@xxxxxxx>; Kuehling, Felix
> > <Felix.Kuehling@xxxxxxx>
> > *Subject:* [PATCH] drm/amdkfd: bump KFD version for unified ctx
> > save/restore memory
> > To expose unified memory for ctx save/resotre area feature
> > availablity to libhsakmt.
> >
> > Signed-off-by: Eric Huang <jinhuieric.huang@xxxxxxx>
> > ---
> > include/uapi/linux/kfd_ioctl.h | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/include/uapi/linux/kfd_ioctl.h
> > b/include/uapi/linux/kfd_ioctl.h
> > index 7a423855a86e..afd8ff29c74f 100644
> > --- a/include/uapi/linux/kfd_ioctl.h
> > +++ b/include/uapi/linux/kfd_ioctl.h
> > @@ -36,9 +36,10 @@
> > * - 1.8 - CRIU - Support for SDMA transfers with GTT BOs
> > * - 1.9 - Add available memory ioctl
> > * - 1.10 - Add SMI profiler event log
> > + * - 1.11 - Add unified memory for ctx save/restore area
> > */
> > #define KFD_IOCTL_MAJOR_VERSION 1
> > -#define KFD_IOCTL_MINOR_VERSION 10
> > +#define KFD_IOCTL_MINOR_VERSION 11
> >
> > struct kfd_ioctl_get_version_args {
> > __u32 major_version; /* from KFD */
> > --
> > 2.25.1
> >
