[PATCH] drm/xe/hwmon: Remove unwanted write permission for currN_label
Rodrigo Vivi
rodrigo.vivi at intel.com
Mon Apr 22 19:28:47 UTC 2024
On Fri, Apr 19, 2024 at 07:55:11PM +0530, Riana Tauro wrote:
>
>
> On 4/19/2024 6:29 PM, Karthik Poosa wrote:
> > Change umode of currN_label from 0644 to 0444 as write permission
> > not needed for label.
> >
> > Signed-off-by: Karthik Poosa <karthik.poosa at intel.com>
> Looks good to me
> Reviewed-by: Riana Tauro <riana.tauro at intel.com>
There's something strange with the CI runs with this patch.
I even triggered a retest, but got exactly same strange failure
on both runs.
Maybe CI was down or bad during these tests and it needs a third rerun?!
> > ---
> > drivers/gpu/drm/xe/xe_hwmon.c | 9 +++++++--
> > 1 file changed, 7 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/xe/xe_hwmon.c b/drivers/gpu/drm/xe/xe_hwmon.c
> > index 453e601ddd5e..dca275117232 100644
> > --- a/drivers/gpu/drm/xe/xe_hwmon.c
> > +++ b/drivers/gpu/drm/xe/xe_hwmon.c
> > @@ -550,12 +550,17 @@ xe_hwmon_curr_is_visible(const struct xe_hwmon *hwmon, u32 attr, int channel)
> > {
> > u32 uval;
> > + /* hwmon sysfs attribute of current available only for package */
> > + if (channel != CHANNEL_PKG)
> > + return 0;
> > +
> > switch (attr) {
> > case hwmon_curr_crit:
> > - case hwmon_curr_label:
> > - if (channel == CHANNEL_PKG)
> > return (xe_hwmon_pcode_read_i1(hwmon->gt, &uval) ||
> > (uval & POWER_SETUP_I1_WATTS)) ? 0 : 0644;
> > + case hwmon_curr_label:
> > + return (xe_hwmon_pcode_read_i1(hwmon->gt, &uval) ||
> > + (uval & POWER_SETUP_I1_WATTS)) ? 0 : 0444;
> > break;
> > default:
> > return 0;
More information about the Intel-xe
mailing list