Re: [igt-dev] [Intel-xe] [PATCH igt 1/4] lib: Add xe_pciids.h

["Vivi, Rodrigo" <rodrigo.vivi@intel.com>]

On Fri, 2023-09-01 at 02:21 -0700, Lucas De Marchi wrote:
> +Rodrigo
> 
> On Thu, Aug 31, 2023 at 11:58:23AM +0530, Balasubramani Vivekanandan
> wrote:
> > On 29.08.2023 07:11, Lucas De Marchi wrote:
> > > With the introduction of Lunar Lake, igt won't be able to just
> > > re-use
> > > i915_pciids.h for both drivers anymore since the support for LNL
> > > is only
> > > coming with the xe module. Copy xe_pciids.h from the drm-xe-next
> > > branch
> > > (up to commit b9c9020fc816 ("drm/xe/pvc: Use fast copy engines as
> > > migrate engine on PVC") and start including it where needed. This
> > > brings
> > > all the LNL PCI IDs.
> > 
> > I couldn't find the commit hash b9c9020fc816. Is it lost due to
> > rebase?
> > In that case, does it make sense to use the commit hash or just
> > keep the
> > commit subject?
> 
> Rodrigo, did you tag the branch before force pushing? If so, can you
> push a tag like the archive/* ones? This would allow people to see
> the
> commit referenced, even if later we force push the branch.

ops, sorry for that. 
The tag is there now:

drm-xe-next-2023-08-29

btw, I noticed we are using 2 different styles, should we always add
'archive/' or should we drop that?

Is there any tag that is not an archive?

> 
> 
> Lucas De Marchi