Re: [PATCH v2 1/2] drm: Add GPU reset sysfs event

From: Simon Ser <contact@emersion.fr>
To: "Christian König" <ckoenig.leichtzumerken@gmail.com>
Cc: Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com>,
	Shashank Sharma <shashank.sharma@amd.com>,
	Amaranath Somalapuram <amaranath.somalapuram@amd.com>,
	amd-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org,
	Alexandar Deucher <alexander.deucher@amd.com>,
	Shashank Sharma <contactshashanksharma@gmail.com>,
	Christian Koenig <christian.koenig@amd.com>
Subject: Re: [PATCH v2 1/2] drm: Add GPU reset sysfs event
Date: Wed, 09 Mar 2022 10:28:27 +0000	[thread overview]
Message-ID: <iZTBpyxPhm4Q7lobMcS6nj3qX3Tt2qh8bfiZWEek-lgn9S-PQcBHKN3uUEcgVwSOeVocniOKYIRNI1kvWlfMB3XZ32D4CQIX9Ob8T04SdLw=@emersion.fr> (raw)
In-Reply-To: <1e2ad919-e336-cf8f-c675-0ddd62796a0f@gmail.com>

On Wednesday, March 9th, 2022 at 11:24, Christian König <ckoenig.leichtzumerken@gmail.com> wrote:

> Am 09.03.22 um 11:10 schrieb Simon Ser:
> > On Wednesday, March 9th, 2022 at 10:56, Pierre-Eric Pelloux-Prayer <pierre-eric.pelloux-prayer@amd.com> wrote:
> >
> >> Would it be possible to include the app parameters as well?
> > Can all processes read sysfs events?
>
> No, but application parameters are usually not secret.

It's a bad practice, yes. But people still do it.

> > There might be security implications here. The app parameters might
> > contain sensitive information, like passwords or tokens.
>
> It's a well known security vulnerably to provide password etc.. as
> application parameter since everybody can read them through procfs.

I was thinking mostly about Flatpak apps here. Flatpak apps are running
in a process namespace so they can't query the CLI parameters of PIDs
outside of the namespace. But they might still receive sysfs uevents.