Re: [PATCH 1/1] drm/i915: Export LMEM max memory bandwidth via sysfs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Thanks Ashutosh for the clarification.

BR
Himal Ghimiray

> -----Original Message-----
> From: Dixit, Ashutosh <ashutosh.dixit@xxxxxxxxx>
> Sent: 23 November 2022 11:29
> To: Ghimiray, Himal Prasad <himal.prasad.ghimiray@xxxxxxxxx>
> Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx>; intel-
> gfx@xxxxxxxxxxxxxxxxxxxxx
> Subject: Re:  [PATCH 1/1] drm/i915: Export LMEM max memory
> bandwidth via sysfs
> 
> On Tue, 22 Nov 2022 21:10:01 -0800, Ghimiray, Himal Prasad wrote:
> >
> > > -----Original Message-----
> > > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx>
> > > Sent: 21 November 2022 17:17
> > > To: Ghimiray, Himal Prasad <himal.prasad.ghimiray@xxxxxxxxx>; intel-
> > > gfx@xxxxxxxxxxxxxxxxxxxxx
> > > Subject: Re:  [PATCH 1/1] drm/i915: Export LMEM max
> > > memory bandwidth via sysfs
> > >
> > >
> > > On 21/11/2022 10:01, Himal Prasad Ghimiray wrote:
> > > > Export lmem maximum memory bandwidth to the userspace via sysfs
> > > >
> > > > (v2)
> > > > Add TODO comment to have client parts specific condition
> > > > (Anshuman) Remove prelim prefix from the sysfs node name (Aravind)
> > >
> > > Link to userspace consumer?
> > [Ghimiray, Himal Prasad]
> >
> > Does above comment means stating name of sysfs node in commit
> message ?
> 
> No. It means that there has to be a "real" open source userspace program
> (like Mesa, Level-0/OneApi or another accepted program or UMD) which is
> actually reading the sysfs file exposed. Without such a consumer, the sysfs
> addition will not be accepted upstream.
> 
> E.g. see cover letter here:
> 
> https://patchwork.freedesktop.org/series/106460/
> 
> "An approved Level-0/oneAPI UMD pull request which consumes the
> exposed defaults can be seen here:
> 	https://github.com/intel/compute-runtime/pull/552
> "
> 
> Thanks.
> --
> Ashutosh




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux