It's generally better to group all of the module's static variables into a single struct, declare a single static pointer to it and then pvPortMalloc() it during init.  This way, when the module is optional and disabled, its static data consumes 0 bytes of RAM.

We haven't followed this in all of our modules unfortunately so not a blocker.  Preferably new modules would do this though since it helps us get more optional modules into a firmware load.