Entity Manager error in tiogapass

Entity Manager error in tiogapass

[Jayashree D]

[-- Attachment #1: Type: text/plain, Size: 2784 bytes --]

Classification: Confidential
Hi Team,

In the latest build, I am facing the below issue in Entity Manager for tiogapass.
Already issue has been created in Entity Manager Repo - entity-manager terminated by 'std::filesystem::__cxx11::filesystem_error' * Issue #8 * openbmc/entity-manager (github.com)<https://github.com/openbmc/entity-manager/issues/8>

root@tiogapass:~# systemctl status xyz.openbmc_project.EntityManager.service -l
● xyz.openbmc_project.EntityManager.service - Entity Manager
     Loaded: loaded (]8;;file://tiogapass/lib/systemd/system/xyz.openbmc_project.EntityManager.service/lib/systemd/system/xyz.openbmc_project.EntityManager.service]8;;; enabled; vendor preset: enabled)
     Active: active (running) since Thu 1970-01-01 00:04:10 UTC; 25s ago
    Process: 851 ExecStartPre=/bin/mkdir -p /var/configuration (code=exited, status=0/SUCCESS)
    Process: 852 ExecStartPre=/bin/mkdir -p /tmp/overlays (code=exited, status=0/SUCCESS)
   Main PID: 853 (entity-manager)
     CGroup: /system.slice/xyz.openbmc_project.EntityManager.service
             └─853 /usr/bin/entity-manager

Jan 01 00:04:09 tiogapass systemd[1]: Starting Entity Manager...
Jan 01 00:04:10 tiogapass systemd[1]: Started Entity Manager.
Jan 01 00:04:27 tiogapass entity-manager[853]: Inventory Added
Jan 01 00:04:27 tiogapass entity-manager[853]: terminate called after throwing an instance of 'std::filesystem::__cxx11::filesystem_error'
Jan 01 00:04:27 tiogapass entity-manager[853]:   what():  filesystem error: recursive directory iterator cannot open directory: No such file or directory [/sys/bus/i2c/devices/i2c-16]

Regards,
Jayashree
::DISCLAIMER::
________________________________
The contents of this e-mail and any attachment(s) are confidential and intended for the named recipient(s) only. E-mail transmission is not guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or may contain viruses in transmission. The e mail and its contents (with or without referred errors) shall therefore not attach any liability on the originator or HCL or its affiliates. Views or opinions, if any, presented in this email are solely those of the author and may not necessarily reflect the views or opinions of HCL or its affiliates. Any form of reproduction, dissemination, copying, disclosure, modification, distribution and / or publication of this message without the prior written consent of authorized representative of HCL is strictly prohibited. If you have received this email in error please delete it and notify the sender immediately. Before opening any email and/or attachments, please check them for viruses and other defects.
________________________________

[-- Attachment #2: Type: text/html, Size: 5578 bytes --]

Re: Entity Manager error in tiogapass

[Johnathan Mantey]

[-- Attachment #1.1: Type: text/plain, Size: 3954 bytes --]

Jayashree,

On 4/26/21 9:00 AM, Jayashree D wrote:
>
> Classification: *Confidential*
>
> Hi Team,
>
> In the latest build, I am facing the below issue in Entity Manager for 
> tiogapass.
>
> Already issue has been created in Entity Manager Repo - entity-manager 
> terminated by 'std::filesystem::__cxx11::filesystem_error' · Issue #8 
> · openbmc/entity-manager (github.com) 
> <https://github.com/openbmc/entity-manager/issues/8>
>
> root@tiogapass:~# systemctl status 
> xyz.openbmc_project.EntityManager.service -l
>
> ● xyz.openbmc_project.EntityManager.service - Entity Manager
>
>      Loaded: loaded 
> (]8;;file://tiogapass/lib/systemd/system/xyz.openbmc_project.EntityManager.service/lib/systemd/system/xyz.openbmc_project.EntityManager.service]8;;; 
> enabled; vendor preset: enabled)
>
>      Active: active (running) since Thu 1970-01-01 00:04:10 UTC; 25s ago
>
>     Process: 851 ExecStartPre=/bin/mkdir -p /var/configuration 
> (code=exited, status=0/SUCCESS)
>
>     Process: 852 ExecStartPre=/bin/mkdir -p /tmp/overlays 
> (code=exited, status=0/SUCCESS)
>
>    Main PID: 853 (entity-manager)
>
>      CGroup: /system.slice/xyz.openbmc_project.EntityManager.service
>
>              └─853 /usr/bin/entity-manager
>
> Jan 01 00:04:09 tiogapass systemd[1]: Starting Entity Manager...
>
> Jan 01 00:04:10 tiogapass systemd[1]: Started Entity Manager.
>
> Jan 01 00:04:27 tiogapass entity-manager[853]: Inventory Added
>
> Jan 01 00:04:27 tiogapass entity-manager[853]: terminate called after 
> throwing an instance of 'std::filesystem::__cxx11::filesystem_error'
>
> Jan 01 00:04:27 tiogapass entity-manager[853]:   what():  filesystem 
> error: recursive directory iterator cannot open directory: No such 
> file or directory [/sys/bus/i2c/devices/i2c-16]
>

Check this portion of src/Overlay.cpp
static bool deviceIsCreated(const std::string& devicePath,
                             std::shared_ptr<uint64_t> bus,
                             std::shared_ptr<uint64_t> address,
                             const bool retrying)

I placed a comment describing the intent of the directory iterator. I 
have not seen the issue described on our systems, nor in QEMU (reported 
by another dev), so I'm not able to identify the issue.
>
> Regards,
>
> Jayashree
>
> ::DISCLAIMER::
> ------------------------------------------------------------------------
> The contents of this e-mail and any attachment(s) are confidential and 
> intended for the named recipient(s) only. E-mail transmission is not 
> guaranteed to be secure or error-free as information could be 
> intercepted, corrupted, lost, destroyed, arrive late or incomplete, or 
> may contain viruses in transmission. The e mail and its contents (with 
> or without referred errors) shall therefore not attach any liability 
> on the originator or HCL or its affiliates. Views or opinions, if any, 
> presented in this email are solely those of the author and may not 
> necessarily reflect the views or opinions of HCL or its affiliates. 
> Any form of reproduction, dissemination, copying, disclosure, 
> modification, distribution and / or publication of this message 
> without the prior written consent of authorized representative of HCL 
> is strictly prohibited. If you have received this email in error 
> please delete it and notify the sender immediately. Before opening any 
> email and/or attachments, please check them for viruses and other defects.
> ------------------------------------------------------------------------

-- 
Johnathan Mantey
Senior Software Engineer
*azad te**chnology partners*
Contributing to Technology Innovation since 1992
Phone: (503) 712-6764
Email: johnathanx.mantey@intel.com <mailto:johnathanx.mantey@intel.com>



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

Re: Entity Manager error in tiogapass

[Lei Yu]

On Tue, Apr 27, 2021 at 12:35 AM Johnathan Mantey
<johnathanx.mantey@intel.com> wrote:
>
> Jayashree,
>
> On 4/26/21 9:00 AM, Jayashree D wrote:
> >
> > Classification: *Confidential*
> >
> > Hi Team,
> >
> > In the latest build, I am facing the below issue in Entity Manager for
> > tiogapass.
> >
> > Already issue has been created in Entity Manager Repo - entity-manager
> > terminated by 'std::filesystem::__cxx11::filesystem_error' · Issue #8
> > · openbmc/entity-manager (github.com)
> > <https://github.com/openbmc/entity-manager/issues/8>
> >
> > root@tiogapass:~# systemctl status
> > xyz.openbmc_project.EntityManager.service -l
> >
> > ● xyz.openbmc_project.EntityManager.service - Entity Manager
> >
> >      Loaded: loaded
> > (]8;;file://tiogapass/lib/systemd/system/xyz.openbmc_project.EntityManager.service/lib/systemd/system/xyz.openbmc_project.EntityManager.service]8;;;
> > enabled; vendor preset: enabled)
> >
> >      Active: active (running) since Thu 1970-01-01 00:04:10 UTC; 25s ago
> >
> >     Process: 851 ExecStartPre=/bin/mkdir -p /var/configuration
> > (code=exited, status=0/SUCCESS)
> >
> >     Process: 852 ExecStartPre=/bin/mkdir -p /tmp/overlays
> > (code=exited, status=0/SUCCESS)
> >
> >    Main PID: 853 (entity-manager)
> >
> >      CGroup: /system.slice/xyz.openbmc_project.EntityManager.service
> >
> >              └─853 /usr/bin/entity-manager
> >
> > Jan 01 00:04:09 tiogapass systemd[1]: Starting Entity Manager...
> >
> > Jan 01 00:04:10 tiogapass systemd[1]: Started Entity Manager.
> >
> > Jan 01 00:04:27 tiogapass entity-manager[853]: Inventory Added
> >
> > Jan 01 00:04:27 tiogapass entity-manager[853]: terminate called after
> > throwing an instance of 'std::filesystem::__cxx11::filesystem_error'
> >
> > Jan 01 00:04:27 tiogapass entity-manager[853]:   what():  filesystem
> > error: recursive directory iterator cannot open directory: No such
> > file or directory [/sys/bus/i2c/devices/i2c-16]
> >
>
> Check this portion of src/Overlay.cpp
> static bool deviceIsCreated(const std::string& devicePath,
>                              std::shared_ptr<uint64_t> bus,
>                              std::shared_ptr<uint64_t> address,
>                              const bool retrying)
>
> I placed a comment describing the intent of the directory iterator. I
> have not seen the issue described on our systems, nor in QEMU (reported
> by another dev), so I'm not able to identify the issue.

The issue is reported at
https://github.com/openbmc/entity-manager/issues/8, and the commit
that introduces the issue is
https://github.com/openbmc/entity-manager/commit/9b86787adea3f8f29fac2acbb9fa0f48fbcf244a

Could you kindly investigate the exception and see how to make it not crash?

-- 
BRs,
Lei YU

Re: Entity Manager error in tiogapass

[Ed Tanous]

On Mon, Apr 26, 2021 at 7:23 PM Lei Yu <yulei.sh@bytedance.com> wrote:
>
> On Tue, Apr 27, 2021 at 12:35 AM Johnathan Mantey
> <johnathanx.mantey@intel.com> wrote:
> >
> > Jayashree,
> >
> > On 4/26/21 9:00 AM, Jayashree D wrote:
> > >
> > > Classification: *Confidential*
> > >
> > > Hi Team,
> > >
> > > In the latest build, I am facing the below issue in Entity Manager for
> > > tiogapass.
> > >
> > > Already issue has been created in Entity Manager Repo - entity-manager
> > > terminated by 'std::filesystem::__cxx11::filesystem_error' · Issue #8
> > > · openbmc/entity-manager (github.com)
> > > <https://github.com/openbmc/entity-manager/issues/8>
> > >
> > > root@tiogapass:~# systemctl status
> > > xyz.openbmc_project.EntityManager.service -l
> > >
> > > ● xyz.openbmc_project.EntityManager.service - Entity Manager
> > >
> > >      Loaded: loaded
> > > (]8;;file://tiogapass/lib/systemd/system/xyz.openbmc_project.EntityManager.service/lib/systemd/system/xyz.openbmc_project.EntityManager.service]8;;;
> > > enabled; vendor preset: enabled)
> > >
> > >      Active: active (running) since Thu 1970-01-01 00:04:10 UTC; 25s ago
> > >
> > >     Process: 851 ExecStartPre=/bin/mkdir -p /var/configuration
> > > (code=exited, status=0/SUCCESS)
> > >
> > >     Process: 852 ExecStartPre=/bin/mkdir -p /tmp/overlays
> > > (code=exited, status=0/SUCCESS)
> > >
> > >    Main PID: 853 (entity-manager)
> > >
> > >      CGroup: /system.slice/xyz.openbmc_project.EntityManager.service
> > >
> > >              └─853 /usr/bin/entity-manager
> > >
> > > Jan 01 00:04:09 tiogapass systemd[1]: Starting Entity Manager...
> > >
> > > Jan 01 00:04:10 tiogapass systemd[1]: Started Entity Manager.
> > >
> > > Jan 01 00:04:27 tiogapass entity-manager[853]: Inventory Added
> > >
> > > Jan 01 00:04:27 tiogapass entity-manager[853]: terminate called after
> > > throwing an instance of 'std::filesystem::__cxx11::filesystem_error'
> > >
> > > Jan 01 00:04:27 tiogapass entity-manager[853]:   what():  filesystem
> > > error: recursive directory iterator cannot open directory: No such
> > > file or directory [/sys/bus/i2c/devices/i2c-16]
> > >
> >
> > Check this portion of src/Overlay.cpp
> > static bool deviceIsCreated(const std::string& devicePath,
> >                              std::shared_ptr<uint64_t> bus,
> >                              std::shared_ptr<uint64_t> address,
> >                              const bool retrying)
> >
> > I placed a comment describing the intent of the directory iterator. I
> > have not seen the issue described on our systems, nor in QEMU (reported
> > by another dev), so I'm not able to identify the issue.
>
> The issue is reported at
> https://github.com/openbmc/entity-manager/issues/8, and the commit
> that introduces the issue is
> https://github.com/openbmc/entity-manager/commit/9b86787adea3f8f29fac2acbb9fa0f48fbcf244a
>
> Could you kindly investigate the exception and see how to make it not crash?

It sounds like Jonathan isn't able to reproduce.  Do you think you
could capture the full stack trace from the exception?  Do you have
anything odd on your systems that would cause the differing behavior?

>
> --
> BRs,
> Lei YU

Re: Entity Manager error in tiogapass

[Johnathan Mantey]

[-- Attachment #1.1: Type: text/plain, Size: 3887 bytes --]



On 4/27/21 9:54 AM, Ed Tanous wrote:
> On Mon, Apr 26, 2021 at 7:23 PM Lei Yu <yulei.sh@bytedance.com> wrote:
>> On Tue, Apr 27, 2021 at 12:35 AM Johnathan Mantey
>> <johnathanx.mantey@intel.com> wrote:
>>> Jayashree,
>>>
>>> On 4/26/21 9:00 AM, Jayashree D wrote:
>>>> Classification: *Confidential*
>>>>
>>>> Hi Team,
>>>>
>>>> In the latest build, I am facing the below issue in Entity Manager for
>>>> tiogapass.
>>>>
>>>> Already issue has been created in Entity Manager Repo - entity-manager
>>>> terminated by 'std::filesystem::__cxx11::filesystem_error' · Issue #8
>>>> · openbmc/entity-manager (github.com)
>>>> <https://github.com/openbmc/entity-manager/issues/8>
>>>>
>>>> root@tiogapass:~# systemctl status
>>>> xyz.openbmc_project.EntityManager.service -l
>>>>
>>>> ● xyz.openbmc_project.EntityManager.service - Entity Manager
>>>>
>>>>       Loaded: loaded
>>>> (]8;;file://tiogapass/lib/systemd/system/xyz.openbmc_project.EntityManager.service/lib/systemd/system/xyz.openbmc_project.EntityManager.service]8;;;
>>>> enabled; vendor preset: enabled)
>>>>
>>>>       Active: active (running) since Thu 1970-01-01 00:04:10 UTC; 25s ago
>>>>
>>>>      Process: 851 ExecStartPre=/bin/mkdir -p /var/configuration
>>>> (code=exited, status=0/SUCCESS)
>>>>
>>>>      Process: 852 ExecStartPre=/bin/mkdir -p /tmp/overlays
>>>> (code=exited, status=0/SUCCESS)
>>>>
>>>>     Main PID: 853 (entity-manager)
>>>>
>>>>       CGroup: /system.slice/xyz.openbmc_project.EntityManager.service
>>>>
>>>>               └─853 /usr/bin/entity-manager
>>>>
>>>> Jan 01 00:04:09 tiogapass systemd[1]: Starting Entity Manager...
>>>>
>>>> Jan 01 00:04:10 tiogapass systemd[1]: Started Entity Manager.
>>>>
>>>> Jan 01 00:04:27 tiogapass entity-manager[853]: Inventory Added
>>>>
>>>> Jan 01 00:04:27 tiogapass entity-manager[853]: terminate called after
>>>> throwing an instance of 'std::filesystem::__cxx11::filesystem_error'
>>>>
>>>> Jan 01 00:04:27 tiogapass entity-manager[853]:   what():  filesystem
>>>> error: recursive directory iterator cannot open directory: No such
>>>> file or directory [/sys/bus/i2c/devices/i2c-16]
>>>>
>>> Check this portion of src/Overlay.cpp
>>> static bool deviceIsCreated(const std::string& devicePath,
>>>                               std::shared_ptr<uint64_t> bus,
>>>                               std::shared_ptr<uint64_t> address,
>>>                               const bool retrying)
>>>
>>> I placed a comment describing the intent of the directory iterator. I
>>> have not seen the issue described on our systems, nor in QEMU (reported
>>> by another dev), so I'm not able to identify the issue.
>> The issue is reported at
>> https://github.com/openbmc/entity-manager/issues/8, and the commit
>> that introduces the issue is
>> https://github.com/openbmc/entity-manager/commit/9b86787adea3f8f29fac2acbb9fa0f48fbcf244a
>>
>> Could you kindly investigate the exception and see how to make it not crash?
> It sounds like Jonathan isn't able to reproduce.  Do you think you
> could capture the full stack trace from the exception?  Do you have
> anything odd on your systems that would cause the differing behavior?
>
>> --
>> BRs,
>> Lei YU

This may be just wishful thinking....
Looking at the JSON input file for bus 16:
"Bus": 16,
             "Name": "MB_C2_AVA_RTEMP",

Would the recent sdbusplus encoding changes cause an issue?
The "_C2" is going to be translated into something that may be problematic.

Does applying 
https://gerrit.openbmc-project.xyz/c/openbmc/sdbusplus/+/42512 change 
anything?

-- 
Johnathan Mantey
Senior Software Engineer
*azad te**chnology partners*
Contributing to Technology Innovation since 1992
Phone: (503) 712-6764
Email: johnathanx.mantey@intel.com <mailto:johnathanx.mantey@intel.com>



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]

RE: Entity Manager error in tiogapass

[Jayashree D]

Classification: Public

Hi Johnathan,

In tiogapass json file, Bus 16 is present. But driver (i2c-16) is not created in my hardware. So the crash occurs in entity manager.
If this type of case occurs, can entity manager display this as error massage not as crash ?

Regards,
Jayashree.

-----Original Message-----
From: Johnathan Mantey <johnathanx.mantey@intel.com>
Sent: Tuesday, April 27, 2021 11:41 PM
To: Ed Tanous <edtanous@google.com>; Lei Yu <yulei.sh@bytedance.com>
Cc: openbmc@lists.ozlabs.org; Jayashree D <jayashree-d@hcl.com>
Subject: Re: Entity Manager error in tiogapass



On 4/27/21 9:54 AM, Ed Tanous wrote:
> On Mon, Apr 26, 2021 at 7:23 PM Lei Yu <yulei.sh@bytedance.com> wrote:
>> On Tue, Apr 27, 2021 at 12:35 AM Johnathan Mantey
>> <johnathanx.mantey@intel.com> wrote:
>>> Jayashree,
>>>
>>> On 4/26/21 9:00 AM, Jayashree D wrote:
>>>> Classification: *Confidential*
>>>>
>>>> Hi Team,
>>>>
>>>> In the latest build, I am facing the below issue in Entity Manager
>>>> for tiogapass.
>>>>
>>>> Already issue has been created in Entity Manager Repo -
>>>> entity-manager terminated by
>>>> 'std::filesystem::__cxx11::filesystem_error' · Issue #8 ·
>>>> openbmc/entity-manager (github.com)
>>>> <https://github.com/openbmc/entity-manager/issues/8>
>>>>
>>>> root@tiogapass:~# systemctl status
>>>> xyz.openbmc_project.EntityManager.service -l
>>>>
>>>> ● xyz.openbmc_project.EntityManager.service - Entity Manager
>>>>
>>>>       Loaded: loaded
>>>> (]8;;file://tiogapass/lib/systemd/system/xyz.openbmc_project.Entity
>>>> Manager.service/lib/systemd/system/xyz.openbmc_project.EntityManage
>>>> r.service]8;;;
>>>> enabled; vendor preset: enabled)
>>>>
>>>>       Active: active (running) since Thu 1970-01-01 00:04:10 UTC;
>>>> 25s ago
>>>>
>>>>      Process: 851 ExecStartPre=/bin/mkdir -p /var/configuration
>>>> (code=exited, status=0/SUCCESS)
>>>>
>>>>      Process: 852 ExecStartPre=/bin/mkdir -p /tmp/overlays
>>>> (code=exited, status=0/SUCCESS)
>>>>
>>>>     Main PID: 853 (entity-manager)
>>>>
>>>>       CGroup:
>>>> /system.slice/xyz.openbmc_project.EntityManager.service
>>>>
>>>>               └─853 /usr/bin/entity-manager
>>>>
>>>> Jan 01 00:04:09 tiogapass systemd[1]: Starting Entity Manager...
>>>>
>>>> Jan 01 00:04:10 tiogapass systemd[1]: Started Entity Manager.
>>>>
>>>> Jan 01 00:04:27 tiogapass entity-manager[853]: Inventory Added
>>>>
>>>> Jan 01 00:04:27 tiogapass entity-manager[853]: terminate called
>>>> after throwing an instance of 'std::filesystem::__cxx11::filesystem_error'
>>>>
>>>> Jan 01 00:04:27 tiogapass entity-manager[853]:   what():  filesystem
>>>> error: recursive directory iterator cannot open directory: No such
>>>> file or directory [/sys/bus/i2c/devices/i2c-16]
>>>>
>>> Check this portion of src/Overlay.cpp static bool
>>> deviceIsCreated(const std::string& devicePath,
>>>                               std::shared_ptr<uint64_t> bus,
>>>                               std::shared_ptr<uint64_t> address,
>>>                               const bool retrying)
>>>
>>> I placed a comment describing the intent of the directory iterator.
>>> I have not seen the issue described on our systems, nor in QEMU
>>> (reported by another dev), so I'm not able to identify the issue.
>> The issue is reported at
>> https://github.com/openbmc/entity-manager/issues/8, and the commit
>> that introduces the issue is
>> https://github.com/openbmc/entity-manager/commit/9b86787adea3f8f29fac
>> 2acbb9fa0f48fbcf244a
>>
>> Could you kindly investigate the exception and see how to make it not crash?
> It sounds like Jonathan isn't able to reproduce.  Do you think you
> could capture the full stack trace from the exception?  Do you have
> anything odd on your systems that would cause the differing behavior?
>
>> --
>> BRs,
>> Lei YU

This may be just wishful thinking....
Looking at the JSON input file for bus 16:
"Bus": 16,
             "Name": "MB_C2_AVA_RTEMP",

Would the recent sdbusplus encoding changes cause an issue?
The "_C2" is going to be translated into something that may be problematic.

Does applying
https://gerrit.openbmc-project.xyz/c/openbmc/sdbusplus/+/42512 change anything?

--
Johnathan Mantey
Senior Software Engineer
*azad te**chnology partners*
Contributing to Technology Innovation since 1992
Phone: (503) 712-6764
Email: johnathanx.mantey@intel.com <mailto:johnathanx.mantey@intel.com>


::DISCLAIMER::
________________________________
The contents of this e-mail and any attachment(s) are confidential and intended for the named recipient(s) only. E-mail transmission is not guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or may contain viruses in transmission. The e mail and its contents (with or without referred errors) shall therefore not attach any liability on the originator or HCL or its affiliates. Views or opinions, if any, presented in this email are solely those of the author and may not necessarily reflect the views or opinions of HCL or its affiliates. Any form of reproduction, dissemination, copying, disclosure, modification, distribution and / or publication of this message without the prior written consent of authorized representative of HCL is strictly prohibited. If you have received this email in error please delete it and notify the sender immediately. Before opening any email and/or attachments, please check them for viruses and other defects.
________________________________

Re: Entity Manager error in tiogapass

[Johnathan Mantey]

[-- Attachment #1.1: Type: text/plain, Size: 6913 bytes --]

On 4/28/21 1:35 AM, Jayashree D wrote:
> Classification: Public
>
> Hi Johnathan,
>
> In tiogapass json file, Bus 16 is present. But driver (i2c-16) is not created in my hardware. So the crash occurs in entity manager.
> If this type of case occurs, can entity manager display this as error massage not as crash ?
It's still unclear to me what the failure mode is based on the input 
that has been provided.

The directory_iterator only iterates over directories/files that are 
present. If i2c-16 isn't there, there won't be a path iterator value 
assigned, and the code should continue looking for files that do exist.
There is also a secondary test to make sure the item in the path 
iterator is a directory, and if it is not, the directory is ignored via 
a "continue" statement.

The code prior to this commit used directory_iterator in a loop instead 
of recursive_directory_iterator. The latter was suggested by Ed Tanous 
to simplify the logic/indent level. I agreed the solution looked cleaner 
and implemented it. Perhaps this introduced an unexpected artifact that 
did not exist in the preceding implementation of the directory recursion.

> Regards,
> Jayashree.
>
> -----Original Message-----
> From: Johnathan Mantey <johnathanx.mantey@intel.com>
> Sent: Tuesday, April 27, 2021 11:41 PM
> To: Ed Tanous <edtanous@google.com>; Lei Yu <yulei.sh@bytedance.com>
> Cc: openbmc@lists.ozlabs.org; Jayashree D <jayashree-d@hcl.com>
> Subject: Re: Entity Manager error in tiogapass
>
>
>
> On 4/27/21 9:54 AM, Ed Tanous wrote:
>> On Mon, Apr 26, 2021 at 7:23 PM Lei Yu <yulei.sh@bytedance.com> wrote:
>>> On Tue, Apr 27, 2021 at 12:35 AM Johnathan Mantey
>>> <johnathanx.mantey@intel.com> wrote:
>>>> Jayashree,
>>>>
>>>> On 4/26/21 9:00 AM, Jayashree D wrote:
>>>>> Classification: *Confidential*
>>>>>
>>>>> Hi Team,
>>>>>
>>>>> In the latest build, I am facing the below issue in Entity Manager
>>>>> for tiogapass.
>>>>>
>>>>> Already issue has been created in Entity Manager Repo -
>>>>> entity-manager terminated by
>>>>> 'std::filesystem::__cxx11::filesystem_error' · Issue #8 ·
>>>>> openbmc/entity-manager (github.com)
>>>>> <https://github.com/openbmc/entity-manager/issues/8>
>>>>>
>>>>> root@tiogapass:~# systemctl status
>>>>> xyz.openbmc_project.EntityManager.service -l
>>>>>
>>>>> ● xyz.openbmc_project.EntityManager.service - Entity Manager
>>>>>
>>>>>        Loaded: loaded
>>>>> (]8;;file://tiogapass/lib/systemd/system/xyz.openbmc_project.Entity
>>>>> Manager.service/lib/systemd/system/xyz.openbmc_project.EntityManage
>>>>> r.service]8;;;
>>>>> enabled; vendor preset: enabled)
>>>>>
>>>>>        Active: active (running) since Thu 1970-01-01 00:04:10 UTC;
>>>>> 25s ago
>>>>>
>>>>>       Process: 851 ExecStartPre=/bin/mkdir -p /var/configuration
>>>>> (code=exited, status=0/SUCCESS)
>>>>>
>>>>>       Process: 852 ExecStartPre=/bin/mkdir -p /tmp/overlays
>>>>> (code=exited, status=0/SUCCESS)
>>>>>
>>>>>      Main PID: 853 (entity-manager)
>>>>>
>>>>>        CGroup:
>>>>> /system.slice/xyz.openbmc_project.EntityManager.service
>>>>>
>>>>>                └─853 /usr/bin/entity-manager
>>>>>
>>>>> Jan 01 00:04:09 tiogapass systemd[1]: Starting Entity Manager...
>>>>>
>>>>> Jan 01 00:04:10 tiogapass systemd[1]: Started Entity Manager.
>>>>>
>>>>> Jan 01 00:04:27 tiogapass entity-manager[853]: Inventory Added
>>>>>
>>>>> Jan 01 00:04:27 tiogapass entity-manager[853]: terminate called
>>>>> after throwing an instance of 'std::filesystem::__cxx11::filesystem_error'
>>>>>
>>>>> Jan 01 00:04:27 tiogapass entity-manager[853]:   what():  filesystem
>>>>> error: recursive directory iterator cannot open directory: No such
>>>>> file or directory [/sys/bus/i2c/devices/i2c-16]
>>>>>
>>>> Check this portion of src/Overlay.cpp static bool
>>>> deviceIsCreated(const std::string& devicePath,
>>>>                                std::shared_ptr<uint64_t> bus,
>>>>                                std::shared_ptr<uint64_t> address,
>>>>                                const bool retrying)
>>>>
>>>> I placed a comment describing the intent of the directory iterator.
>>>> I have not seen the issue described on our systems, nor in QEMU
>>>> (reported by another dev), so I'm not able to identify the issue.
>>> The issue is reported at
>>> https://github.com/openbmc/entity-manager/issues/8, and the commit
>>> that introduces the issue is
>>> https://github.com/openbmc/entity-manager/commit/9b86787adea3f8f29fac
>>> 2acbb9fa0f48fbcf244a
>>>
>>> Could you kindly investigate the exception and see how to make it not crash?
>> It sounds like Jonathan isn't able to reproduce.  Do you think you
>> could capture the full stack trace from the exception?  Do you have
>> anything odd on your systems that would cause the differing behavior?
>>
>>> --
>>> BRs,
>>> Lei YU
> This may be just wishful thinking....
> Looking at the JSON input file for bus 16:
> "Bus": 16,
>               "Name": "MB_C2_AVA_RTEMP",
>
> Would the recent sdbusplus encoding changes cause an issue?
> The "_C2" is going to be translated into something that may be problematic.
>
> Does applying
> https://gerrit.openbmc-project.xyz/c/openbmc/sdbusplus/+/42512 change anything?
>
> --
> Johnathan Mantey
> Senior Software Engineer
> *azad te**chnology partners*
> Contributing to Technology Innovation since 1992
> Phone: (503) 712-6764
> Email: johnathanx.mantey@intel.com <mailto:johnathanx.mantey@intel.com>
>
>
> ::DISCLAIMER::
> ________________________________
> The contents of this e-mail and any attachment(s) are confidential and intended for the named recipient(s) only. E-mail transmission is not guaranteed to be secure or error-free as information could be intercepted, corrupted, lost, destroyed, arrive late or incomplete, or may contain viruses in transmission. The e mail and its contents (with or without referred errors) shall therefore not attach any liability on the originator or HCL or its affiliates. Views or opinions, if any, presented in this email are solely those of the author and may not necessarily reflect the views or opinions of HCL or its affiliates. Any form of reproduction, dissemination, copying, disclosure, modification, distribution and / or publication of this message without the prior written consent of authorized representative of HCL is strictly prohibited. If you have received this email in error please delete it and notify the sender immediately. Before opening any email and/or attachments, please check them for viruses and other defects.
> ________________________________

-- 
Johnathan Mantey
Senior Software Engineer
*azad te**chnology partners*
Contributing to Technology Innovation since 1992
Phone: (503) 712-6764
Email: johnathanx.mantey@intel.com <mailto:johnathanx.mantey@intel.com>



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]