[CRIU] Re: [PATCH] crtools: cleanup fdset initalization

[Cyrill Gorcunov]

On Wed, Feb 08, 2012 at 07:17:20PM +0400, Pavel Emelyanov wrote:
> 
> > Давай тогда поменяем на cr_get_dump_fdset() и cr_get_show_fdset().
> > Нормально?
> > Или даже что-нить типа cr_get_fdset_for_dump() и cr_get_fdset_for_show().
> 
> cr_fdset_open(what_for)

Почему нельзя просто сделать cr_fdset_try_open и для всего, что не получилось
открыть, просто оставить fd = -1, не вываливаясь с ошибкой?

Более того, cr_fdset_open может юзать cr_fdset_try_open, просто проверяя
по маске, что файл не удалось открыть и тогда уже вываливаться с ошибкой.

Не?

Типа

cr_fdset_try_open
	...
		ret = open(path, O_RDWR | O_CREAT | O_EXCL, CR_FD_PERM);
		if (ret < 0)
			fdset->fds[i] = -errno;

а на

cr_fdset_open

	if ((use_mask & CR_FD_DESC_USE(i)))
		if (fdset->fds[i] < 0)
			pr_err("shit %s\n", strerrno(fdset->fds[i]))

	Cyrill