Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fdborch] Fix for arm compilation #1592

Merged
merged 2 commits into from
Jan 12, 2021
Merged

[Fdborch] Fix for arm compilation #1592

merged 2 commits into from
Jan 12, 2021

Conversation

prsunny
Copy link
Collaborator

@prsunny prsunny commented Jan 11, 2021

What I did
Fix for arm compilation caused as part of #1275

Why I did it
Unblock PR build

How I verified it

Details if related

@prsunny prsunny requested a review from lguohan January 11, 2021 23:06
@prsunny prsunny changed the title Fix for arm compilation [Fdborch] Fix for arm compilation Jan 11, 2021
lguohan
lguohan previously approved these changes Jan 11, 2021
@prsunny
Copy link
Collaborator Author

prsunny commented Jan 11, 2021

@anish-n for viz

@lguohan
Copy link
Contributor

lguohan commented Jan 11, 2021

missing ?

In file included from fdborch.cpp:8:
fdborch.cpp: In member function 'bool FdbOrch::removeFdbEntry(const FdbEntry&, FdbOrigin)':
/usr/include/swss/logger.h:18:101: error: format '%lx' expects argument of type 'long unsigned int', but argument 6 has type 'sai_object_id_t' {aka 'long long unsigned int'} [-Werror=format=]
 #define SWSS_LOG_INFO(MSG, ...)        swss::Logger::getInstance().write(swss::Logger::SWSS_INFO,   ":- %s: " MSG, __FUNCTION__, ##__VA_ARGS__)
fdborch.cpp:1218:5: note: in expansion of macro 'SWSS_LOG_INFO'
     SWSS_LOG_INFO("Removed mac=%s bv_id=0x%lx port:%s",

@lguohan lguohan merged commit 17d0dae into sonic-net:master Jan 12, 2021
@prsunny prsunny deleted the fdborch branch January 12, 2021 02:31
DavidZagury pushed a commit to DavidZagury/sonic-swss that referenced this pull request Mar 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants