Re: [PATCH] dbus: export roam time, roam complete, and session length to dbus

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, 2018-05-04 at 17:08 +0000, Matthew Wang wrote:
> No reason, should I resend the patch?

Yeah, if you could.

Dan

> On Fri, May 4, 2018 at 7:43 AM Dan Williams <dcbw@xxxxxxxxxx> wrote:
> 
> > On Thu, 2018-05-03 at 16:32 -0700, Matthew Wang wrote:
> > > Added new Interface properties "RoamTime", "RoamComplete", and
> > > "SessionLength".
> > > 
> > > "RoamTime" carries the roam time of the most recent roam in
> > > milliseconds.
> > > "RoamComplete" carries True or False corresponding to the success
> > > status of the
> > > most recent roam.
> > > "SessionLength" carries the number of milliseconds corresponding
> > > to
> > > how long
> > > the connection to the last AP was before a roam or disconnect
> > > happened.
> > > 
> > > These properties will all be logged in UMA to compare FT vs non-
> > > FT
> > > network
> > > behaviors.
> > > 
> > > Signed-off-by: Matthew Wang <matthewmwang@xxxxxxxxxxxx>
> > > ---
> > >  doc/dbus.doxygen                        | 15 ++++++
> > >  wpa_supplicant/dbus/dbus_new.c          | 27 ++++++++++
> > >  wpa_supplicant/dbus/dbus_new.h          |  3 ++
> > >  wpa_supplicant/dbus/dbus_new_handlers.c | 65
> > > +++++++++++++++++++++++++
> > >  wpa_supplicant/dbus/dbus_new_handlers.h |  3 ++
> > >  wpa_supplicant/notify.c                 | 27 ++++++++++
> > >  wpa_supplicant/notify.h                 |  3 ++
> > >  wpa_supplicant/wpa_supplicant.c         | 34 +++++++++++++
> > >  wpa_supplicant/wpa_supplicant_i.h       |  4 ++
> > >  9 files changed, 181 insertions(+)
> > > 
> > > diff --git a/doc/dbus.doxygen b/doc/dbus.doxygen
> > > index 2ca40ae9b..d3003b31d 100644
> > > --- a/doc/dbus.doxygen
> > > +++ b/doc/dbus.doxygen
> > > @@ -769,6 +769,21 @@ fi.w1.wpa_supplicant1.CreateInterface.
> > >       <p>The most recent IEEE 802.11 status code for association
> > > rejection.</p>
> > >        </li>
> > > 
> > > +      <li>
> > > +     <h3>RoamTime - i - (read)</h3>
> > > +     <p>The most recent roam time in milliseconds.</p>
> > > +      </li>
> > 
> > Any reason to have RoamTime and SessionLength be int, rather than
> > uint?
> >  Are they ever going to be less than zero?
> > 
> > Dan
> > 
> > > +      <li>
> > > +     <h3>RoamComplete - b - (read)</h3>
> > > +     <p>The most recent roam success or failure.</p>
> > > +      </li>
> > > +
> > > +      <li>
> > > +     <h3>SessionLength - i - (read)</h3>
> > > +     <p>The most recent BSS session length in milliseconds.</p>
> > > +      </li>
> > > +
> > >        <li>
> > >       <h3>EapolVersion - s - (read/write)</h3>
> > >       <p>IEEE 802.1X/EAPOL version number</p>
> > > diff --git a/wpa_supplicant/dbus/dbus_new.c
> > > b/wpa_supplicant/dbus/dbus_new.c
> > > index e0f16bbda..168e9baf6 100644
> > > --- a/wpa_supplicant/dbus/dbus_new.c
> > > +++ b/wpa_supplicant/dbus/dbus_new.c
> > > @@ -2162,6 +2162,15 @@ void wpas_dbus_signal_prop_changed(struct
> > > wpa_supplicant *wpa_s,
> > >               prop = "AssocStatusCode";
> > >               flush = TRUE;
> > >               break;
> > > +     case WPAS_DBUS_PROP_ROAM_TIME:
> > > +             prop = "RoamTime";
> > > +             break;
> > > +     case WPAS_DBUS_PROP_ROAM_COMPLETE:
> > > +             prop = "RoamComplete";
> > > +             break;
> > > +     case WPAS_DBUS_PROP_SESSION_LENGTH:
> > > +             prop = "SessionLength";
> > > +             break;
> > >       default:
> > >               wpa_printf(MSG_ERROR, "dbus: %s: Unknown Property
> > > value %d",
> > >                          __func__, property);
> > > @@ -2731,6 +2740,24 @@ static const struct wpa_dbus_property_desc
> > > wpas_dbus_bss_properties[] = {
> > >         NULL,
> > >         NULL
> > >       },
> > > +     {
> > > +       "RoamTime", WPAS_DBUS_NEW_IFACE_INTERFACE, "i",
> > > +       wpas_dbus_getter_roam_time,
> > > +       NULL,
> > > +       NULL
> > > +     },
> > > +     {
> > > +       "RoamComplete", WPAS_DBUS_NEW_IFACE_INTERFACE, "b",
> > > +       wpas_dbus_getter_roam_complete,
> > > +       NULL,
> > > +       NULL
> > > +     },
> > > +     {
> > > +       "SessionLength", WPAS_DBUS_NEW_IFACE_INTERFACE, "i",
> > > +       wpas_dbus_getter_session_length,
> > > +       NULL,
> > > +       NULL
> > > +     },
> > >       { NULL, NULL, NULL, NULL, NULL, NULL }
> > >  };
> > > 
> > > diff --git a/wpa_supplicant/dbus/dbus_new.h
> > > b/wpa_supplicant/dbus/dbus_new.h
> > > index e68acb7a1..08f4858b1 100644
> > > --- a/wpa_supplicant/dbus/dbus_new.h
> > > +++ b/wpa_supplicant/dbus/dbus_new.h
> > > @@ -31,6 +31,9 @@ enum wpas_dbus_prop {
> > >       WPAS_DBUS_PROP_BSSS,
> > >       WPAS_DBUS_PROP_DISCONNECT_REASON,
> > >       WPAS_DBUS_PROP_ASSOC_STATUS_CODE,
> > > +     WPAS_DBUS_PROP_ROAM_TIME,
> > > +     WPAS_DBUS_PROP_ROAM_COMPLETE,
> > > +     WPAS_DBUS_PROP_SESSION_LENGTH,
> > >  };
> > > 
> > >  enum wpas_dbus_bss_prop {
> > > diff --git a/wpa_supplicant/dbus/dbus_new_handlers.c
> > > b/wpa_supplicant/dbus/dbus_new_handlers.c
> > > index a3c98fadd..4e3412cdc 100644
> > > --- a/wpa_supplicant/dbus/dbus_new_handlers.c
> > > +++ b/wpa_supplicant/dbus/dbus_new_handlers.c
> > > @@ -3158,6 +3158,71 @@ dbus_bool_t
> > > wpas_dbus_getter_assoc_status_code(
> > >  }
> > > 
> > > 
> > > +/**
> > > + * wpas_dbus_getter_roam_time - Get most recent roam time
> > > + * @iter: Pointer to incoming dbus message iter
> > > + * @error: Location to store error on failure
> > > + * @user_data: Function specific data
> > > + * Returns: TRUE on success, FALSE on failure
> > > + *
> > > + * Getter for "RoamTime" property.
> > > + */
> > > +dbus_bool_t wpas_dbus_getter_roam_time(
> > > +     const struct wpa_dbus_property_desc *property_desc,
> > > +     DBusMessageIter *iter, DBusError *error, void *user_data)
> > > +{
> > > +     struct wpa_supplicant *wpa_s = user_data;
> > > +     dbus_int32_t roam_time = wpa_s->roam_time.sec * 1000 +
> > > +                              wpa_s->roam_time.usec / 1000;
> > > +
> > > +     return wpas_dbus_simple_property_getter(iter,
> > > DBUS_TYPE_INT32,
> > > +                                             &roam_time, error);
> > > +}
> > > +
> > > +/**
> > > + * wpas_dbus_getter_roam_complete - Get most recent roam success
> > > or
> > > failure
> > > + * @iter: Pointer to incoming dbus message iter
> > > + * @error: Location to store error on failure
> > > + * @user_data: Function specific data
> > > + * Returns: TRUE on success, FALSE on failure
> > > + *
> > > + * Getter for "RoamComplete" property.
> > > + */
> > > +dbus_bool_t wpas_dbus_getter_roam_complete(
> > > +     const struct wpa_dbus_property_desc *property_desc,
> > > +     DBusMessageIter *iter, DBusError *error, void *user_data)
> > > +{
> > > +     struct wpa_supplicant *wpa_s = user_data;
> > > +     dbus_bool_t roam_complete = os_reltime_initialized(&wpa_s-
> > > > roam_time) ?
> > > 
> > > +                                 TRUE : FALSE;
> > > +
> > > +     return wpas_dbus_simple_property_getter(iter,
> > > DBUS_TYPE_BOOLEAN,
> > > +                                             &roam_complete,
> > > error);
> > > +}
> > > +
> > > +
> > > +/**
> > > + * wpas_dbus_getter_session_length - Get most recent BSS session
> > > length
> > > + * @iter: Pointer to incoming dbus message iter
> > > + * @error: Location to store error on failure
> > > + * @user_data: Function specific data
> > > + * Returns: TRUE on success, FALSE on failure
> > > + *
> > > + * Getter for "SessionLength" property.
> > > + */
> > > +dbus_bool_t wpas_dbus_getter_session_length(
> > > +     const struct wpa_dbus_property_desc *property_desc,
> > > +     DBusMessageIter *iter, DBusError *error, void *user_data)
> > > +{
> > > +     struct wpa_supplicant *wpa_s = user_data;
> > > +     dbus_int32_t session_length = wpa_s->session_length.sec *
> > > 1000 +
> > > +                                   wpa_s->session_length.usec /
> > > 1000;
> > > +
> > > +     return wpas_dbus_simple_property_getter(iter,
> > > DBUS_TYPE_INT32,
> > > +                                             &session_length,
> > > error);
> > > +}
> > > +
> > > +
> > >  /**
> > >   * wpas_dbus_getter_bss_expire_age - Get BSS entry expiration
> > > age
> > >   * @iter: Pointer to incoming dbus message iter
> > > diff --git a/wpa_supplicant/dbus/dbus_new_handlers.h
> > > b/wpa_supplicant/dbus/dbus_new_handlers.h
> > > index 26652ad3d..7a03b7f9f 100644
> > > --- a/wpa_supplicant/dbus/dbus_new_handlers.h
> > > +++ b/wpa_supplicant/dbus/dbus_new_handlers.h
> > > @@ -148,6 +148,9 @@
> > > DECLARE_ACCESSOR(wpas_dbus_setter_fast_reauth);
> > >  DECLARE_ACCESSOR(wpas_dbus_getter_disconnect_reason);
> > >  DECLARE_ACCESSOR(wpas_dbus_getter_disassociate_reason);
> > >  DECLARE_ACCESSOR(wpas_dbus_getter_assoc_status_code);
> > > +DECLARE_ACCESSOR(wpas_dbus_getter_roam_time);
> > > +DECLARE_ACCESSOR(wpas_dbus_getter_roam_complete);
> > > +DECLARE_ACCESSOR(wpas_dbus_getter_session_length);
> > >  DECLARE_ACCESSOR(wpas_dbus_getter_bss_expire_age);
> > >  DECLARE_ACCESSOR(wpas_dbus_setter_bss_expire_age);
> > >  DECLARE_ACCESSOR(wpas_dbus_getter_bss_expire_count);
> > > diff --git a/wpa_supplicant/notify.c b/wpa_supplicant/notify.c
> > > index 83df04f39..a2b04716d 100644
> > > --- a/wpa_supplicant/notify.c
> > > +++ b/wpa_supplicant/notify.c
> > > @@ -149,6 +149,33 @@ void wpas_notify_assoc_status_code(struct
> > > wpa_supplicant *wpa_s)
> > >  }
> > > 
> > > 
> > > +void wpas_notify_roam_time(struct wpa_supplicant *wpa_s)
> > > +{
> > > +     if (wpa_s->p2p_mgmt)
> > > +             return;
> > > +
> > > +     wpas_dbus_signal_prop_changed(wpa_s,
> > > WPAS_DBUS_PROP_ROAM_TIME);
> > > +}
> > > +
> > > +
> > > +void wpas_notify_roam_complete(struct wpa_supplicant *wpa_s)
> > > +{
> > > +     if (wpa_s->p2p_mgmt)
> > > +             return;
> > > +
> > > +     wpas_dbus_signal_prop_changed(wpa_s,
> > > WPAS_DBUS_PROP_ROAM_COMPLETE);
> > > +}
> > > +
> > > +
> > > +void wpas_notify_session_length(struct wpa_supplicant *wpa_s)
> > > +{
> > > +     if (wpa_s->p2p_mgmt)
> > > +             return;
> > > +
> > > +     wpas_dbus_signal_prop_changed(wpa_s,
> > > WPAS_DBUS_PROP_SESSION_LENGTH);
> > > +}
> > > +
> > > +
> > >  void wpas_notify_network_changed(struct wpa_supplicant *wpa_s)
> > >  {
> > >       if (wpa_s->p2p_mgmt)
> > > diff --git a/wpa_supplicant/notify.h b/wpa_supplicant/notify.h
> > > index 3ca933c76..bde20e83d 100644
> > > --- a/wpa_supplicant/notify.h
> > > +++ b/wpa_supplicant/notify.h
> > > @@ -24,6 +24,9 @@ void wpas_notify_state_changed(struct
> > > wpa_supplicant *wpa_s,
> > >                              enum wpa_states old_state);
> > >  void wpas_notify_disconnect_reason(struct wpa_supplicant
> > > *wpa_s);
> > >  void wpas_notify_assoc_status_code(struct wpa_supplicant
> > > *wpa_s);
> > > +void wpas_notify_roam_time(struct wpa_supplicant *wpa_s);
> > > +void wpas_notify_roam_complete(struct wpa_supplicant *wpa_s);
> > > +void wpas_notify_session_length(struct wpa_supplicant *wpa_s);
> > >  void wpas_notify_network_changed(struct wpa_supplicant *wpa_s);
> > >  void wpas_notify_ap_scan_changed(struct wpa_supplicant *wpa_s);
> > >  void wpas_notify_bssid_changed(struct wpa_supplicant *wpa_s);
> > > diff --git a/wpa_supplicant/wpa_supplicant.c
> > > b/wpa_supplicant/wpa_supplicant.c
> > > index dcd787bec..df27c55ad 100644
> > > --- a/wpa_supplicant/wpa_supplicant.c
> > > +++ b/wpa_supplicant/wpa_supplicant.c
> > > @@ -839,6 +839,40 @@ void wpa_supplicant_set_state(struct
> > > wpa_supplicant *wpa_s,
> > >               wpa_supplicant_state_txt(wpa_s->wpa_state),
> > >               wpa_supplicant_state_txt(state));
> > > 
> > > +     if (state == WPA_AUTHENTICATING && wpa_s->wpa_state ==
> > > WPA_COMPLETED) {
> > > +             if (wpa_s->sme.prev_bssid_set &&
> > > +                 os_memcmp(wpa_s->sme.prev_bssid, wpa_s-
> > > > pending_bssid, ETH_ALEN) != 0) {
> > > 
> > > +                     os_get_reltime(&wpa_s->roam_start);
> > > +             }
> > > +     } else if (state == WPA_COMPLETED &&
> > > +                os_reltime_initialized(&wpa_s->roam_start)) {
> > > +             os_reltime_age(&wpa_s->roam_start, &wpa_s-
> > > > roam_time);
> > > 
> > > +             wpa_s->roam_start.sec = 0;
> > > +             wpa_s->roam_start.usec = 0;
> > > +             wpas_notify_auth_changed(wpa_s);
> > > +             wpas_notify_roam_time(wpa_s);
> > > +             wpas_notify_roam_complete(wpa_s);
> > > +     } else if (state == WPA_DISCONNECTED &&
> > > +                os_reltime_initialized(&wpa_s->roam_start)) {
> > > +             wpa_s->roam_start.sec = 0;
> > > +             wpa_s->roam_start.usec = 0;
> > > +             wpa_s->roam_time.sec = 0;
> > > +             wpa_s->roam_time.usec = 0;
> > > +             wpas_notify_roam_complete(wpa_s);
> > > +     }
> > > +
> > > +     if (state == WPA_COMPLETED && wpa_s->wpa_state !=
> > > WPA_COMPLETED) {
> > > +             wpas_notify_auth_changed(wpa_s);
> > > +             os_get_reltime(&wpa_s->session_start);
> > > +     } else if (state != WPA_COMPLETED &&
> > > +                wpa_s->wpa_state == WPA_COMPLETED &&
> > > +                os_reltime_initialized(&wpa_s->session_start)) {
> > > +             os_reltime_age(&wpa_s->session_start, &wpa_s-
> > > > session_length);
> > > 
> > > +             wpa_s->session_start.sec = 0;
> > > +             wpa_s->session_start.usec = 0;
> > > +             wpas_notify_session_length(wpa_s);
> > > +     }
> > > +
> > >       if (state == WPA_INTERFACE_DISABLED) {
> > >               /* Assure normal scan when interface is restored */
> > >               wpa_s->normal_scans = 0;
> > > diff --git a/wpa_supplicant/wpa_supplicant_i.h
> > > b/wpa_supplicant/wpa_supplicant_i.h
> > > index 4d18177fb..a4d56b810 100644
> > > --- a/wpa_supplicant/wpa_supplicant_i.h
> > > +++ b/wpa_supplicant/wpa_supplicant_i.h
> > > @@ -492,6 +492,10 @@ struct wpa_supplicant {
> > >       struct wpa_supplicant *next;
> > >       struct l2_packet_data *l2;
> > >       struct l2_packet_data *l2_br;
> > > +     struct os_reltime roam_start;
> > > +     struct os_reltime roam_time;
> > > +     struct os_reltime session_start;
> > > +     struct os_reltime session_length;
> > >       unsigned char own_addr[ETH_ALEN];
> > >       unsigned char perm_addr[ETH_ALEN];
> > >       char ifname[100];

_______________________________________________
Hostap mailing list
Hostap@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/hostap



[Index of Archives]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux