[devel] [PATCH 1/1] mds: improve thread safety in mdstest - part 2 [#2746]

- Remove thread-local declaration of svc_to_mds_info --- src/mds/apitest/mdstipc_api.c | 7 ++- src/mds/apitest/mdstipc_conf.c | 43 +- 2 files changed, 27 insertions(+), 23 deletions(-) diff --git a/src/mds/apitest/mdstipc_api.c b/src/mds/apitest/mds

Re: [devel] [PATCH 1/1] mds: improve thread safety in mdstest - part 2 [#2746]

Hi Hans, In version 2 of the patch, I have removed the thread-local declaration of svc_to_mds_info as your suggestion and use it as local variable for each MDS service request to avoid Data race issues. Please help review it again. Thank you. -- Best regards, Hoa Le On 03/27/2018 01:41 PM,

[devel] [PATCH 0/1] Review Request for mds: improve thread safety in mdstest - part 2 V2 [#2746]

Summary: mds: improve thread safety in mdstest - part 2 V2 [#2746] Review request for Ticket(s): 2746 Peer Reviewer(s): Hans, Anders Pull request to: Hans, Anders Affected branch(es): develop, release Development branch: ticket-2746 Base revision: 9c846d28a5dac616b2619d1fe274105d463d0d20 Personal r

Re: [devel] [PATCH 1/1] mds: improve thread safety in mdstest - part 2 [#2746]

ack, review only. /Thanks HansN On 03/27/2018 10:34 AM, Hoa Le wrote: - Remove thread-local declaration of svc_to_mds_info --- src/mds/apitest/mdstipc_api.c | 7 ++- src/mds/apitest/mdstipc_conf.c | 43 +- 2 files changed, 27 insertions(+), 23 d

[devel] [PATCH 1/1] dtm: Document update for transportd.conf and osaflog options [#2820]

modified: 00-README.conf modified: src/dtm/README --- 00-README.conf | 15 +++ src/dtm/README | 23 +++ 2 files changed, 38 insertions(+) diff --git a/00-README.conf b/00-README.conf index 94cb3d3..5f3927b 100644 --- a/00-README.conf +++ b/00-RE

[devel] [PATCH 0/1] Review Request for dtm: Documentation update for transportd.conf and osaflog options [#2820]

Summary: dtm: Document update for transportd.conf and osaflog options [#2820] Review request for Ticket(s): 2820 Peer Reviewer(s): anders.wid...@ericsson.com Pull request to: *** LIST THE PERSON WITH PUSH ACCESS HERE *** Affected branch(es): develop Development branch: ticket-2820 Base revision: 2e

[devel] [PATCH 1/1] dtm: Use base::StrToUint64 to parse command-line option arguments [#2815]

Use the function base::StrToUint64 to pass command-line option arguments. This gives us error checking, and support for the k, M and G suffixes. --- src/dtm/tools/osaflog.cc| 44 + src/dtm/transport/log_server.cc | 44 +--

[devel] [PATCH 0/1] Review Request for dtm: Use base::StrToUint64 to parse command-line option arguments [#2815]

Summary: dtm: Use base::StrToUint64 to parse command-line option arguments [#2815] Review request for Ticket(s): 2815 Peer Reviewer(s): Ravi Pull request to: Affected branch(es): develop Development branch: ticket-2815 Base revision: 2e1b1228d0bb8f7aa66773f11ff915d5a39b0748 Personal repository: g