All of lore.kernel.org
 help / color / mirror / Atom feed
* smci: how do reboots work in scmi?
@ 2025-02-01 13:57 Dan Carpenter
  2025-02-02 22:12 ` Khaled Ali Ahmed
  0 siblings, 1 reply; 5+ messages in thread
From: Dan Carpenter @ 2025-02-01 13:57 UTC (permalink / raw)
  To: arm-scmi, Khaled Ali Ahmed, Girish Pathak

In scmi with pinctrl then we do a PINCTRL_REQUEST to claim the pins,
but then if we reboot without restarting the scmi server the the pins
are still claimed.  Is there some way to hook into VHOST_USER_NONE to
release all the pins?

regards,
dan carpenter

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: smci: how do reboots work in scmi?
  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
  0 siblings, 1 reply; 5+ messages in thread
From: Khaled Ali Ahmed @ 2025-02-02 22:12 UTC (permalink / raw)
  To: Dan Carpenter, arm-scmi@vger.kernel.org, Girish Pathak

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.

________________________________________
From: Dan Carpenter <dan.carpenter@linaro.org>
Sent: Saturday, February 1, 2025 1:57 PM
To: arm-scmi@vger.kernel.org <arm-scmi@vger.kernel.org>; Khaled Ali Ahmed <Khaled.AliAhmed@arm.com>; Girish Pathak <Girish.Pathak@arm.com>
Subject: smci: how do reboots work in scmi?

In scmi with pinctrl then we do a PINCTRL_REQUEST to claim the pins,
but then if we reboot without restarting the scmi server the the pins
are still claimed.  Is there some way to hook into VHOST_USER_NONE to
release all the pins?

regards,
dan carpenter
IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: smci: how do reboots work in scmi?
  2025-02-02 22:12 ` Khaled Ali Ahmed
@ 2025-02-03  6:23   ` Dan Carpenter
  2025-02-03  7:05     ` Vincent Guittot
  0 siblings, 1 reply; 5+ messages in thread
From: Dan Carpenter @ 2025-02-03  6:23 UTC (permalink / raw)
  To: Khaled Ali Ahmed; +Cc: arm-scmi@vger.kernel.org, Girish Pathak

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.

regards,
dan carpenter



^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: smci: how do reboots work in scmi?
  2025-02-03  6:23   ` Dan Carpenter
@ 2025-02-03  7:05     ` Vincent Guittot
  2025-02-03 12:52       ` Cristian Marussi
  0 siblings, 1 reply; 5+ messages in thread
From: Vincent Guittot @ 2025-02-03  7:05 UTC (permalink / raw)
  To: Dan Carpenter; +Cc: Khaled Ali Ahmed, arm-scmi@vger.kernel.org, Girish Pathak

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."

>
> regards,
> dan carpenter
>
>
>

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: smci: how do reboots work in scmi?
  2025-02-03  7:05     ` Vincent Guittot
@ 2025-02-03 12:52       ` Cristian Marussi
  0 siblings, 0 replies; 5+ messages in thread
From: Cristian Marussi @ 2025-02-03 12:52 UTC (permalink / raw)
  To: Vincent Guittot
  Cc: Dan Carpenter, Khaled Ali Ahmed, arm-scmi@vger.kernel.org,
	Girish Pathak, cristian.marussi

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

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-02-03 12:52 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
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 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.