diff options
author | rch <rch@chromium.org> | 2015-05-28 14:08:01 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-05-28 21:08:33 +0000 |
commit | e42933e7e947d45097fda7eb86605baa9c9a4602 (patch) | |
tree | 3798dece432557302d3de9eaa730176540a60169 | |
parent | efe2ac1d3e1650a1cec652566542ac492eafedb3 (diff) | |
download | chromium_src-e42933e7e947d45097fda7eb86605baa9c9a4602.zip chromium_src-e42933e7e947d45097fda7eb86605baa9c9a4602.tar.gz chromium_src-e42933e7e947d45097fda7eb86605baa9c9a4602.tar.bz2 |
Add a new histogram Net.QuicSession.PreferAesGcm to track if
the QUIC session prefers to use AES GCM because of hardware support.
Review URL: https://codereview.chromium.org/1152853003
Cr-Commit-Position: refs/heads/master@{#331853}
-rw-r--r-- | net/quic/quic_stream_factory.cc | 5 | ||||
-rw-r--r-- | tools/metrics/histograms/histograms.xml | 12 |
2 files changed, 16 insertions, 1 deletions
diff --git a/net/quic/quic_stream_factory.cc b/net/quic/quic_stream_factory.cc index b9a4123..dd979fd 100644 --- a/net/quic/quic_stream_factory.cc +++ b/net/quic/quic_stream_factory.cc @@ -596,7 +596,10 @@ QuicStreamFactory::QuicStreamFactory( new ChannelIDSourceChromium(channel_id_service)); } base::CPU cpu; - if (cpu.has_aesni() && cpu.has_avx()) + bool has_aes_hardware_support = cpu.has_aesni() && cpu.has_avx(); + UMA_HISTOGRAM_BOOLEAN("Net.QuicSession.PreferAesGcm", + has_aes_hardware_support); + if (has_aes_hardware_support) crypto_config_.PreferAesGcm(); if (!IsEcdsaSupported()) crypto_config_.DisableEcdsa(); diff --git a/tools/metrics/histograms/histograms.xml b/tools/metrics/histograms/histograms.xml index 3653cb4..5d192b9 100644 --- a/tools/metrics/histograms/histograms.xml +++ b/tools/metrics/histograms/histograms.xml @@ -21566,6 +21566,13 @@ Therefore, the affected-histogram name has to have at least one dot in it. </summary> </histogram> +<histogram name="Net.QuicSession.PreferAesGcm" enum="BooleanPreferred"> + <owner>rch@chromium.org</owner> + <summary> + True if the QUIC session prefers to use AES GCM because of hardware support. + </summary> +</histogram> + <histogram name="Net.QuicSession.PublicResetAddressMismatch" enum="QuicAddressMismatch"> <owner>wtc@chromium.org</owner> @@ -49007,6 +49014,11 @@ Therefore, the affected-histogram name has to have at least one dot in it. <int value="1" label="Populated"/> </enum> +<enum name="BooleanPreferred" type="int"> + <int value="0" label="Not Preferred"/> + <int value="1" label="Preferred"/> +</enum> + <enum name="BooleanPresent" type="int"> <int value="0" label="Not Present"/> <int value="1" label="Present"/> |