Thanks for the review Eelco, Em seg., 5 de fev. de 2024 às 04:32, Eelco Chaudron <[email protected]> escreveu:
> > > On 5 Feb 2024, at 2:24, Roberto Bartzen Acosta via dev wrote: > > > Updating the reference documentation with the inclusion of possible > building > > problems with libjemalloc and solution suggestions. > > > > Reported-at: > https://bugs.launchpad.net/ubuntu/+source/openvswitch/+bug/2015748 > > Signed-off-by: Roberto Bartzen Acosta <[email protected]> > > --- > > Documentation/intro/install/general.rst | 16 ++++++++++++++++ > > 1 file changed, 16 insertions(+) > > > > diff --git a/Documentation/intro/install/general.rst > b/Documentation/intro/install/general.rst > > index ab6209482..eeb55075c 100644 > > --- a/Documentation/intro/install/general.rst > > +++ b/Documentation/intro/install/general.rst > > @@ -308,6 +308,22 @@ you wish to link with jemalloc add it to LIBS:: > > > > $ ./configure LIBS=-ljemalloc > > > > +.. note:: > > + Linking Open vSwitch with the jemalloc shared library may not work as > > + expected in certain operating system development environments. You can > > + override the automatic compiler decision to avoid possible linker > issues by > > + passing ``-fno-lto`` or ``-fno-builtin`` flag since the jemalloc > override > > + standard built-in memory allocation functions such as malloc, calloc, > etc. > > + Both options can solve possible jemalloc linker issues with pros and > cons for > > + each case, feel free to choose the path that appears best to you. > Disabling > > + LTO flag example:: > > + > > + $ ./configure LIBS=-ljemalloc CFLAGS=-fno-lto > > + > > + Disabling built-in flag example:: > > + > > + ./configure LIBS=-ljemalloc CFLAGS=-fno-builtin > > Should we also include the default CFLAGS? As this option overrides all > CFLAGS, so now we also remove the default “-g -O2” options. > Sure, I took a look at the configuration examples and others also pass the "-g -O2" in CFLAGS as your suggestion. So, I will change this and send a new patch version. Thanks, Roberto > > > + > > Example usage:: > > $ # Clone OVS repo > > $cd /home/foo/ovs > > -- > > 2.25.1 > > > > > > -- > > > > > > > > > > _'Esta mensagem é direcionada apenas para os endereços constantes no > > cabeçalho inicial. Se você não está listado nos endereços constantes no > > cabeçalho, pedimos-lhe que desconsidere completamente o conteúdo dessa > > mensagem e cuja cópia, encaminhamento e/ou execução das ações citadas > estão > > imediatamente anuladas e proibidas'._ > > > > > > * **'Apesar do Magazine Luiza tomar > > todas as precauções razoáveis para assegurar que nenhum vírus esteja > > presente nesse e-mail, a empresa não poderá aceitar a responsabilidade > por > > quaisquer perdas ou danos causados por esse e-mail ou por seus anexos'.* > > > > > > > > _______________________________________________ > > dev mailing list > > [email protected] > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > -- _‘Esta mensagem é direcionada apenas para os endereços constantes no cabeçalho inicial. Se você não está listado nos endereços constantes no cabeçalho, pedimos-lhe que desconsidere completamente o conteúdo dessa mensagem e cuja cópia, encaminhamento e/ou execução das ações citadas estão imediatamente anuladas e proibidas’._ * **‘Apesar do Magazine Luiza tomar todas as precauções razoáveis para assegurar que nenhum vírus esteja presente nesse e-mail, a empresa não poderá aceitar a responsabilidade por quaisquer perdas ou danos causados por esse e-mail ou por seus anexos’.* _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
