#undef TRACE_SYSTEM #define TRACE_SYSTEM hda #if !defined(__HDAC_TRACE_H) || defined(TRACE_HEADER_MULTI_READ) #define __HDAC_TRACE_H #include #include #include #ifndef HDAC_MSG_MAX #define HDAC_MSG_MAX 500 #endif struct hdac_bus; struct hdac_codec; TRACE_EVENT(hda_send_cmd, TP_PROTO(struct hdac_bus *bus, unsigned int cmd), TP_ARGS(bus, cmd), TP_STRUCT__entry(__dynamic_array(char, msg, HDAC_MSG_MAX)), TP_fast_assign( snprintf(__get_str(msg), HDAC_MSG_MAX, "[%s:%d] val=0x%08x", dev_name((bus)->dev), (cmd) >> 28, cmd); ), TP_printk("%s", __get_str(msg)) ); TRACE_EVENT(hda_get_response, TP_PROTO(struct hdac_bus *bus, unsigned int addr, unsigned int res), TP_ARGS(bus, addr, res), TP_STRUCT__entry(__dynamic_array(char, msg, HDAC_MSG_MAX)), TP_fast_assign( snprintf(__get_str(msg), HDAC_MSG_MAX, "[%s:%d] val=0x%08x", dev_name((bus)->dev), addr, res); ), TP_printk("%s", __get_str(msg)) ); TRACE_EVENT(hda_unsol_event, TP_PROTO(struct hdac_bus *bus, u32 res, u32 res_ex), TP_ARGS(bus, res, res_ex), TP_STRUCT__entry(__dynamic_array(char, msg, HDAC_MSG_MAX)), TP_fast_assign( snprintf(__get_str(msg), HDAC_MSG_MAX, "[%s:%d] res=0x%08x, res_ex=0x%08x", dev_name((bus)->dev), res_ex & 0x0f, res, res_ex); ), TP_printk("%s", __get_str(msg)) ); DECLARE_EVENT_CLASS(hdac_stream, TP_PROTO(struct hdac_bus *bus, struct hdac_stream *azx_dev), TP_ARGS(bus, azx_dev), TP_STRUCT__entry( __field(unsigned char, stream_tag) ), TP_fast_assign( __entry->stream_tag = (azx_dev)->stream_tag; ), TP_printk("stream_tag: %d", __entry->stream_tag) ); DEFINE_EVENT(hdac_stream, snd_hdac_stream_start, TP_PROTO(struct hdac_bus *bus, struct hdac_stream *azx_dev), TP_ARGS(bus, azx_dev) ); DEFINE_EVENT(hdac_stream, snd_hdac_stream_stop, TP_PROTO(struct hdac_bus *bus, struct hdac_stream *azx_dev), TP_ARGS(bus, azx_dev) ); #endif /* __HDAC_TRACE_H */ /* This part must be outside protection */ #undef TRACE_INCLUDE_PATH #define TRACE_INCLUDE_PATH . #undef TRACE_INCLUDE_FILE #define TRACE_INCLUDE_FILE trace #include a9c00980552fe508c'>commitdiff
diff options
context:
space:
mode:
authorDexuan Cui <decui@microsoft.com>2017-01-28 11:46:02 -0700
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2017-01-31 10:59:48 +0100
commit433e19cf33d34bb6751c874a9c00980552fe508c (patch)
treece6547ef2987fbb289fa28f03536328a42781651 /sound/pci/mixart/mixart.c
parent191e885a2e130e639bb0c8ee350d7047294f2ce6 (diff)
Drivers: hv: vmbus: finally fix hv_need_to_signal_on_read()
Commit a389fcfd2cb5 ("Drivers: hv: vmbus: Fix signaling logic in hv_need_to_signal_on_read()") added the proper mb(), but removed the test "prev_write_sz < pending_sz" when making the signal decision. As a result, the guest can signal the host unnecessarily, and then the host can throttle the guest because the host thinks the guest is buggy or malicious; finally the user running stress test can perceive intermittent freeze of the guest. This patch brings back the test, and properly handles the in-place consumption APIs used by NetVSC (see get_next_pkt_raw(), put_pkt_raw() and commit_rd_index()). Fixes: a389fcfd2cb5 ("Drivers: hv: vmbus: Fix signaling logic in hv_need_to_signal_on_read()") Signed-off-by: Dexuan Cui <decui@microsoft.com> Reported-by: Rolf Neugebauer <rolf.neugebauer@docker.com> Tested-by: Rolf Neugebauer <rolf.neugebauer@docker.com> Cc: "K. Y. Srinivasan" <kys@microsoft.com> Cc: Haiyang Zhang <haiyangz@microsoft.com> Cc: Stephen Hemminger <sthemmin@microsoft.com> Cc: <stable@vger.kernel.org> Signed-off-by: K. Y. Srinivasan <kys@microsoft.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'sound/pci/mixart/mixart.c')