lundman commented on this pull request.
> + pa.pa_zhp = zhp;
+ pa.pa_fd = outfd;
+ pa.pa_parsable = flags->parsable;
+ pa.pa_progress = flags->progress;
+ pa.pa_siginfo = flags->siginfo;
+
+ if (pa.pa_siginfo) {
+ /* Make sure this thread ignores signal, or ioctl
aborts */
+ sigset_t sigset;
+ (void) sigemptyset(&sigset);
+ (void) sigaddset(&sigset, SIGINFO);
+ (void) pthread_sigmask(SIG_BLOCK, &sigset, NULL);
+ }
+
+ error = pthread_create(&tid, NULL,
+ send_progress_thread, &pa);
Yes, it does create a thread that does nothing. It was simply a style decision
to avoid having
```
if (progress || siginfo) phtread_create(...)
```
and same again for `phread_join()` at the end. But really I have no strong
opinion either way.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openzfs/openzfs/pull/638#discussion_r218269956
------------------------------------------
openzfs: openzfs-developer
Permalink:
https://openzfs.topicbox.com/groups/developer/T000e5d283b7db7e5-M09247026cf57fb863e98e574
Delivery options: https://openzfs.topicbox.com/groups/developer/subscription
