[QUESTION] inconsistent use of smp_mb()

From: Zilin Guan
Date: Wed Dec 04 2024 - 02:03:57 EST


Hello,

I have a question regarding the use of smp_rmb() to enforce
memory ordering in two related functions.

In the function netfs_unbuffered_write_iter_locked() from the file
fs/netfs/direct_write.c, smp_rmb() is explicitly used after the
wait_on_bit() call to ensure that the error and transferred fields are
read in the correct order following the NETFS_RREQ_IN_PROGRESS flag:

105 wait_on_bit(&wreq->flags, NETFS_RREQ_IN_PROGRESS,
106 TASK_UNINTERRUPTIBLE);
107 smp_rmb(); /* Read error/transferred after RIP flag */
108 ret = wreq->error;
109 if (ret == 0) {
110 ret = wreq->transferred;
111 iocb->ki_pos += ret;
112 }

However, in the function netfs_end_writethrough() from the file
fs/netfs/write_issue.c, there is no such use of smp_rmb() after
the corresponding wait_on_bit() call, despite accessing the same filed
of wreq->error and relying on the same NETFS_RREQ_IN_PROGRESS flag:

681 wait_on_bit(&wreq->flags, NETFS_RREQ_IN_PROGRESS,
TASK_UNINTERRUPTIBLE);
682 ret = wreq->error;

My question is why does the first function require a CPU memory barrier
smp_rmb() to enforce ordering, whereas the second function does not?

Thank you for your time and assistance!

Best Regards,
Zilin Guan