> +/* pass flags explicitly */ which flags? > + TP_fast_assign( > + char *__name = "(none)"; > + > + __entry->wbc = (unsigned long)wbc & 0xffff; The wbc generally is allocated on stack, does it reall make sense to keep this? > + if (wbc->bdi) > + strncpy(__entry->name, dev_name(wbc->bdi->dev), 16); > + else > + strncpy(__entry->name, __name, 16); > + __entry->sb = !!wbc->sb; The bdi and sb fields are now gone from the writeback_control. But I think passing the bdi explicitly for the device name makes sense here. -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
