Skip to content

cgroup2: Don't retry on EINTR - #262

Merged
AkihiroSuda merged 1 commit into
containerd:mainfrom
dcantah:cg2-noretry-eintr
Dec 31, 2022
Merged

AkihiroSuda merged 1 commit into
containerd:mainfrom
dcantah:cg2-noretry-eintr

Conversation

@dcantah

@dcantah dcantah commented Dec 31, 2022 •

Copy link
Copy Markdown
Member

Follow-up from a similar change for cgroup1 package here. Since Go 1.15 the stdlib automatically retries IO on EINTR, and since this package uses things from go 1.16, all consumers of this pkg should be on > 1.15 making any manual retries on EINTR unnecessary.

Follow-up from a similar change for cgroup1 package here. Since Go 1.15
the stdlib automatically retries IO on EINTR, and since this package
uses things from go 1.16, all consumers of this pkg should be on > 1.15
making any manual retries on EINTR unnecessary.

Signed-off-by: Danny Canter <danny@dcantah.dev>
Comment thread cgroup2/manager.go
Comment on lines +757 to +758
//
// /sys/fs/cgroup/my.slice/my-group.slice/my-group-112233.slice

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gofmt having fun I assume

@AkihiroSuda
AkihiroSuda merged commit e91481e into containerd:main Dec 31, 2022
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