On Tue, 19 Sep 2017 13:41:35 -0700 Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > Call it blk_trace mutex and move it right next to the blk_trace > structure: > > ifdef CONFIG_BLK_DEV_IO_TRACE > struct blk_trace *blk_trace; > struct mutex blk_trace_mutex; > #endif > > which makes it completely obvious to any read what you are protecting > with it. As a 1000ft away bystander, this appears to be the most logical solution. -- Steve