On Sun, Oct 29, 2023 at 10:09:42AM +0800, Edward Adam Davis wrote:
> @@ -585,7 +596,6 @@ ssize_t ptp_read(struct posix_clock_context *pccontext, uint rdflags,
> free_event:
> kfree(event);
> exit:
> - if (result < 0)
> - ptp_release(pccontext);
> + mutex_unlock(&ptp->tsevq_mux);
> return result;
> }
This is the only hunk that makes sense. Keep this, but remove the
rest, just like in your previous patches.
Thanks,
Richard