connman.lists.linux.dev archive mirror
 help / color / mirror / Atom feed
* [PATCH] service: Report errors to user in native mode
@ 2021-10-01 12:20 VAUTRIN Emmanuel (Canal Plus Prestataire)
  2021-10-04  6:34 ` Daniel Wagner
  0 siblings, 1 reply; 6+ messages in thread
From: VAUTRIN Emmanuel (Canal Plus Prestataire) @ 2021-10-01 12:20 UTC (permalink / raw)
  To: connman

When the native mode is activated, the user still needs to be informed
of errors in failure state, via the agent.
---
 src/service.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/service.c b/src/service.c
index 8097f5373250..70ff26e6a4a1 100644
--- a/src/service.c
+++ b/src/service.c
@@ -6139,7 +6139,8 @@ static int service_indicate_state(struct connman_service *service)
 		break;
 
 	case CONNMAN_SERVICE_STATE_FAILURE:
-		if (service->connect_reason == CONNMAN_SERVICE_CONNECT_REASON_USER) {
+		if (service->connect_reason == CONNMAN_SERVICE_CONNECT_REASON_USER ||
+		service->connect_reason == CONNMAN_SERVICE_CONNECT_REASON_NATIVE) {
 			connman_agent_report_error(service, service->path,
 						error2string(service->error),
 						report_error_cb,
-- 
2.25.1


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

* Re: [PATCH] service: Report errors to user in native mode
  2021-10-01 12:20 [PATCH] service: Report errors to user in native mode VAUTRIN Emmanuel (Canal Plus Prestataire)
@ 2021-10-04  6:34 ` Daniel Wagner
  2021-10-04  8:08   ` VAUTRIN Emmanuel (Canal Plus Prestataire)
  0 siblings, 1 reply; 6+ messages in thread
From: Daniel Wagner @ 2021-10-04  6:34 UTC (permalink / raw)
  To: connman, VAUTRIN Emmanuel (Canal Plus Prestataire); +Cc: Daniel Wagner

On Fri, 1 Oct 2021 12:20:27 +0000, VAUTRIN Emmanuel (Canal Plus Prestataire) wrote:
> When the native mode is activated, the user still needs to be informed
> of errors in failure state, via the agent.
> 

Applied, thanks!

[1/1] service: Report errors to user in native mode
      commit: b9a0a039ccc5697d413f61c9afe7c8c2b6dbd607

Best regards,
-- 
Daniel Wagner <wagi@monom.org>

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

* RE: [PATCH] service: Report errors to user in native mode
  2021-10-04  6:34 ` Daniel Wagner
@ 2021-10-04  8:08   ` VAUTRIN Emmanuel (Canal Plus Prestataire)
  2021-10-04  8:41     ` Daniel Wagner
  0 siblings, 1 reply; 6+ messages in thread
From: VAUTRIN Emmanuel (Canal Plus Prestataire) @ 2021-10-04  8:08 UTC (permalink / raw)
  To: Daniel Wagner, connman

> Applied, thanks!
Great news for this patch and the others! Thank you Daniel!

Emmanuel

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

* Re: [PATCH] service: Report errors to user in native mode
  2021-10-04  8:08   ` VAUTRIN Emmanuel (Canal Plus Prestataire)
@ 2021-10-04  8:41     ` Daniel Wagner
  2021-10-04  9:47       ` VAUTRIN Emmanuel (Canal Plus Prestataire)
  0 siblings, 1 reply; 6+ messages in thread
From: Daniel Wagner @ 2021-10-04  8:41 UTC (permalink / raw)
  To: VAUTRIN Emmanuel (Canal Plus Prestataire), connman

On 04.10.21 10:08, VAUTRIN Emmanuel (Canal Plus Prestataire) wrote:
>> Applied, thanks!
> Great news for this patch and the others! Thank you Daniel!

I was blocked doing the stable-rt update, hence I had no cycles left on
ConnMan. Now since the stable-rt release is out the door I try to get
the clang fixes sorted out as next.

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

* RE: [PATCH] service: Report errors to user in native mode
  2021-10-04  8:41     ` Daniel Wagner
@ 2021-10-04  9:47       ` VAUTRIN Emmanuel (Canal Plus Prestataire)
  2021-10-06 12:41         ` Daniel Wagner
  0 siblings, 1 reply; 6+ messages in thread
From: VAUTRIN Emmanuel (Canal Plus Prestataire) @ 2021-10-04  9:47 UTC (permalink / raw)
  To: Daniel Wagner, connman

> I was blocked doing the stable-rt update, hence I had no cycles left on
> ConnMan. Now since the stable-rt release is out the door I try to get
> the clang fixes sorted out as next.
Allright, good that you have been able to integrate the 4 patches.
There is no hurry for hidden networks, which is a bit tricky.
I have tried to add the minimal code to support it.
There are probably a lot of codes from wifi.c that still need to be adapted
in iwd.c.

Emmanuel

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

* Re: [PATCH] service: Report errors to user in native mode
  2021-10-04  9:47       ` VAUTRIN Emmanuel (Canal Plus Prestataire)
@ 2021-10-06 12:41         ` Daniel Wagner
  0 siblings, 0 replies; 6+ messages in thread
From: Daniel Wagner @ 2021-10-06 12:41 UTC (permalink / raw)
  To: VAUTRIN Emmanuel (Canal Plus Prestataire); +Cc: connman

On Mon, Oct 04, 2021 at 09:47:47AM +0000, VAUTRIN Emmanuel (Canal Plus Prestataire) wrote:
> There are probably a lot of codes from wifi.c that still need to be adapted
> in iwd.c.

I really hope this is not the case.

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

end of thread, other threads:[~2021-10-06 12:41 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-01 12:20 [PATCH] service: Report errors to user in native mode VAUTRIN Emmanuel (Canal Plus Prestataire)
2021-10-04  6:34 ` Daniel Wagner
2021-10-04  8:08   ` VAUTRIN Emmanuel (Canal Plus Prestataire)
2021-10-04  8:41     ` Daniel Wagner
2021-10-04  9:47       ` VAUTRIN Emmanuel (Canal Plus Prestataire)
2021-10-06 12:41         ` Daniel Wagner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).