Hi Marcel,
On Wed, Feb 15, 2012 at 01:20:14PM +0100, Marcel Holtmann wrote:
> Hi Andrei,
>
> > Function state_to_string will be used in other files.
> >
> > Signed-off-by: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx>
> > ---
> > include/net/bluetooth/bluetooth.h | 27 +++++++++++++++++++++++++++
> > net/bluetooth/l2cap_core.c | 26 --------------------------
> > 2 files changed, 27 insertions(+), 26 deletions(-)
> >
> > diff --git a/include/net/bluetooth/bluetooth.h b/include/net/bluetooth/bluetooth.h
> > index 4a82ca0..0a47c20 100644
> > --- a/include/net/bluetooth/bluetooth.h
> > +++ b/include/net/bluetooth/bluetooth.h
> > @@ -129,6 +129,33 @@ enum {
> > BT_CLOSED
> > };
> >
> > +/* If unused will be removed by compiler */
> > +static inline char *state_to_string(int state)
> > +{
> > + switch (state) {
> > + case BT_CONNECTED:
> > + return "BT_CONNECTED";
> > + case BT_OPEN:
> > + return "BT_OPEN";
> > + case BT_BOUND:
> > + return "BT_BOUND";
> > + case BT_LISTEN:
> > + return "BT_LISTEN";
> > + case BT_CONNECT:
> > + return "BT_CONNECT";
> > + case BT_CONNECT2:
> > + return "BT_CONNECT2";
> > + case BT_CONFIG:
> > + return "BT_CONFIG";
> > + case BT_DISCONN:
> > + return "BT_DISCONN";
> > + case BT_CLOSED:
> > + return "BT_CLOSED";
> > + }
> > +
> > + return "invalid state";
> > +}
> > +
>
> so long term, we need to keep socket states inside l2cap_sock.c and
> L2CAP states inside l2cap_core.c. Since L2CAP core engine should use the
> states from the specification and not try to fit into socket states.
>
> I see a little bit of value here in having them in the debug log, but
> making them public like this seems a big overhead. Did you measure the
> size difference.
size show exactly the same size after applying this patch with my config.
(dynamic debug enabled)
size net/bluetooth/bluetooth.ko
text data bss dec hex filename
196404 12912 644 209960 33428 net/bluetooth/bluetooth.ko
> And most likely some state lookup table is better.
This can be done, BTW: unused functions will be removed by compiler, what
about lookup table? I assume the same.
Best regards
Andrei Emeltchenko
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
