[PATCH v1 1/1] tools: env: use /run to store lockfile

Tom Rini trini at konsulko.com
Thu Feb 23 04:28:15 CET 2023


On Thu, Feb 23, 2023 at 01:06:20AM +0000, CHLI30 at nuvoton.com wrote:

> Hi Tom,
> 
> Is this commit valuable to merge into u-boot upstream?
> Thanks your reviewing, just ping this event friendly.

Yes, I'll be putting this in next soon, thanks.

> 
> Best regards,
> Tim
> 
> -----Original Message-----
> From: Tom Rini <trini at konsulko.com>
> Sent: Friday, February 10, 2023 11:30 PM
> To: Tim Lee <timlee660101 at gmail.com>
> Cc: sjg at chromium.org; CS20 CHLi30 <CHLI30 at nuvoton.com>; u-boot at lists.denx.de
> Subject: Re: [PATCH v1 1/1] tools: env: use /run to store lockfile
> 
> On Fri, Feb 10, 2023 at 05:05:39PM +0800, Tim Lee wrote:
> 
> > According this issue https://github.com/ppp-project/ppp/issues/339.
> > Eventually, the dt-utils changed lock directory to fix missing
> > /var/lock directory error then make dt-utils can run normally.
> >
> > We also have a similar issue with these two utilities fw_printenv and
> > fw_setenv will failed when the directory /var/lock is non-existent.
> >
> > We have a custom linux distribution built with yocto (OpenBMC) that
> > use systemd and it deprecated the /var/lock directory.
> > More discussion in systemd/systemd#15668.
> >
> > Thus, we sync with community's solution for uboot/tools/env utilities:
> > The current location /var/lock is considered legacy (at least by systemd).
> > Just use /run to store the lockfile and append the usual .lock suffix.
> >
> > Tested:
> > Verified /run/lock is now present and fw_printenv can work in OpenBMC.
> >
> > Signed-off-by: Tim Lee <timlee660101 at gmail.com>
> 
> Reviewed-by: Tom Rini <trini at konsulko.com>
> 
> --
> Tom
> ________________________________
> ________________________________
>  The privileged confidential information contained in this email is intended for use only by the addressees as indicated by the original sender of this email. If you are not the addressee indicated in this email or are not responsible for delivery of the email to such a person, please kindly reply to the sender indicating this fact and delete all copies of it from your computer and network server immediately. Your cooperation is highly appreciated. It is advised that any unauthorized use of confidential information of Nuvoton is strictly prohibited; and any information in this email irrelevant to the official business of Nuvoton shall be deemed as neither given nor endorsed by Nuvoton.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230222/9de13202/attachment.sig>


More information about the U-Boot mailing list