All of lore.kernel.org
 help / color / mirror / Atom feed
From: Cristian Marussi <cristian.marussi@arm.com>
To: Vincent Guittot <vincent.guittot@linaro.org>
Cc: Dan Carpenter <dan.carpenter@linaro.org>,
	Khaled Ali Ahmed <Khaled.AliAhmed@arm.com>,
	"arm-scmi@vger.kernel.org" <arm-scmi@vger.kernel.org>,
	Girish Pathak <Girish.Pathak@arm.com>,
	cristian.marussi@arm.com
Subject: Re: smci: how do reboots work in scmi?
Date: Mon, 3 Feb 2025 12:52:10 +0000	[thread overview]
Message-ID: <Z6C7-vLTOa4k7vMf@pluto> (raw)
In-Reply-To: <CAKfTPtAJe5E2RdLvhEzE0vcKhy_0uQ_8x4STfcT_teLam6PuBw@mail.gmail.com>

On Mon, Feb 03, 2025 at 08:05:30AM +0100, Vincent Guittot wrote:
> On Mon, 3 Feb 2025 at 07:25, Dan Carpenter <dan.carpenter@linaro.org> wrote:
> >
> > On Sun, Feb 02, 2025 at 10:12:17PM +0000, Khaled Ali Ahmed wrote:
> > > Hello Dan,
> > > the following message still has not been implemented yet:
> > > - PINCTRL_REQUEST.
> > > - PINCTRL_RELEASE.
> > > - PINCTRL_SET_PERMISSIONS.
> > > - NEGOTIATE_PROTOCOL_VERSION.
> > >
> > > thank you so much for your attention and participation.
> > >
> >
> > Ah right, I was thinking of PINCTRL_SETTINGS_CONFIGURE.
> >
> > When the system reboots, it would be nice to have a hook in SCP to just
> > reset everything.
> 
> Maybe BASE_RESET_AGENT_CONFIGURATION :
> " This command is used to reset platform resource settings that were
> previously configured by an agent.
> Platform resource settings refer to power domain, performance domain,
> clock, sensors and other
> settings associated with a device that the agent has access to. This
> command can also be used to reset
> agent-specific permission configurations to access devices and protocols."

You mean Linux OSPM Agent invoking this commands for itself upon reboot ?
(and the platform acting accordignly..)

First I think we should clarify here what we mean with "system reboot".

- a single Agent rebooting by its own decision
    BASE_RESET_AGENT_CONFIGURATION(self) could be an option

- some Trusted agent causing other agents to shutdown/reboot
    in this case maybe there could be some interactions via SystemPower
    protocol, with the server asking upfront for a graceful shutdown
    which would trigger similarly a BASE_RESET_AGENT_CONFIGURATION(self).
    This would give the server a notion of an "agent resetting" which is
    something that currently is not present in SCMI because the server
    cannot know really if an agent is alive (and I am NOT saying that
    this would be a bad thing.. :P)

    ...BUT, on the other side, if a not-so-graceful shutdown is required
    the victim-agent wont have a chance for a clean reboot/shutdown so
    somehow the TrustedAgent could have to issue a BASE_RESET_AGENT_CONFIGURATION
    on behalf of that agent, OR maybe the server could just cleanup the
    config on its own for the disappearing agent...

- the whole system at the platform level, including the server itself:
  does it matter in this case to reset the agent configs ?
  ...IOW is there a scenario in which an SCMI server could reset, needing
  all its agents' states to be reset too, WHILE the platform that is running
  on somehow survives and retains some state ?

I think we never discussed these scenarios, especially in a virtualized
environement...

Thanks,
Cristian

      reply	other threads:[~2025-02-03 12:52 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-02-01 13:57 smci: how do reboots work in scmi? Dan Carpenter
2025-02-02 22:12 ` Khaled Ali Ahmed
2025-02-03  6:23   ` Dan Carpenter
2025-02-03  7:05     ` Vincent Guittot
2025-02-03 12:52       ` Cristian Marussi [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=Z6C7-vLTOa4k7vMf@pluto \
    --to=cristian.marussi@arm.com \
    --cc=Girish.Pathak@arm.com \
    --cc=Khaled.AliAhmed@arm.com \
    --cc=arm-scmi@vger.kernel.org \
    --cc=dan.carpenter@linaro.org \
    --cc=vincent.guittot@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.