fix doc links

frame/benchmarking/src/lib.rs

@@ -54,8 +54,6 @@ pub mod v1;
 pub use v1::*;
 
 /// Contains macros, structs, and traits associated with v2 of the pallet benchmarking syntax.
-/// This module contains macros, structs, and traits associated with v2 of the pallet
-/// benchmarking syntax.
 ///
 /// The [`v2::benchmarks`] and [`v2::instance_benchmarks`] macros can be used to designate a
 /// module as a benchmarking module that can contain benchmarks and benchmark tests. The
@@ -117,13 +115,14 @@ pub use v1::*;
 /// benchmarks using the `#[benchmark]` attribute, as shown in the example above.
 ///
 /// The `#[benchmark]` attribute expects a function definition with a blank return type and
-/// zero or more arguments whose names are valid [`BenchmarkParamater`]
-/// parameters, such as `x`, `y`, `a`, `b`, etc., and whose param types must implement
-/// [`ParamRange`]. At the moment the only valid type that implements
-/// [`ParamRange`] is [`Linear`].
+/// zero or more arguments whose names are valid
+/// [BenchmarkParameter](`crate::BenchmarkParameter`) parameters, such as `x`, `y`, `a`, `b`,
+/// etc., and whose param types must implement [ParamRange](`v2::ParamRange`). At the moment
+/// the only valid type that implements [ParamRange](`v2::ParamRange`) is
+/// [Linear](`v2::Linear`).
 ///
-/// The valid syntax for defining a [`Linear`] is `Linear<A, B>` where `A`, and `B` are
-/// valid integer literals (that fit in a `u32`), such that `B` >= `A`.
+/// The valid syntax for defining a [Linear](`v2::Linear`)is `Linear<A, B>` where `A`, and `B`
+/// are valid integer literals (that fit in a `u32`), such that `B` >= `A`.
 ///
 /// Note that the benchmark function definition does not actually expand as a function
 /// definition, but rather is used to automatically create a number of impls and structs
@@ -257,10 +256,10 @@ pub mod v2 {
 	///
 	/// See [`v2`] for more info.
 	pub trait ParamRange {
-		/// Represents the (inclusive) starting number of this [`ParamRange`].
+		/// Represents the (inclusive) starting number of this `ParamRange`.
 		fn start(&self) -> u32;
 
-		/// Represents the (inclusive) ending number of this [`ParamRange`]
+		/// Represents the (inclusive) ending number of this `ParamRange`.
 		fn end(&self) -> u32;
 	}
 

frame/benchmarking/src/v1.rs

@@ -15,7 +15,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-//! Macro for benchmarking a FRAME runtime.
+//! Macros for benchmarking a FRAME runtime.
 
 use super::*;
 

frame/examples/basic/src/benchmarking.rs

@@ -21,8 +21,11 @@
 #![cfg(feature = "runtime-benchmarks")]
 
 use crate::*;
-use frame_benchmarking::{impl_benchmark_test_suite, whitelisted_caller};
-use frame_support::benchmarking::{benchmarks, Linear};
+use frame_benchmarking::{
+	impl_benchmark_test_suite,
+	v2::{benchmarks, Linear},
+	whitelisted_caller,
+};
 use frame_system::RawOrigin;
 
 // To actually run this benchmark on pallet-example-basic, we need to put this pallet into the

frame/support/procedural/src/lib.rs

@@ -483,7 +483,7 @@ pub fn pallet(attr: TokenStream, item: TokenStream) -> TokenStream {
 /// An attribute macro that can be attached to a (non-empty) module declaration. Doing so will
 /// designate that module as a benchmarking module.
 ///
-/// See `frame_support::benchmarking` for more info.
+/// See `frame_benchmarking::v2` for more info.
 #[proc_macro_attribute]
 pub fn benchmarks(attr: TokenStream, tokens: TokenStream) -> TokenStream {
 	match benchmark::benchmarks(attr, tokens, false) {
@@ -495,7 +495,7 @@ pub fn benchmarks(attr: TokenStream, tokens: TokenStream) -> TokenStream {
 /// An attribute macro that can be attached to a (non-empty) module declaration. Doing so will
 /// designate that module as an instance benchmarking module.
 ///
-/// See `frame_support::benchmarking` for more info.
+/// See `frame_benchmarking::v2` for more info.
 #[proc_macro_attribute]
 pub fn instance_benchmarks(attr: TokenStream, tokens: TokenStream) -> TokenStream {
 	match benchmark::benchmarks(attr, tokens, true) {
@@ -508,7 +508,7 @@ pub fn instance_benchmarks(attr: TokenStream, tokens: TokenStream) -> TokenStrea
 /// attached to a function definition containing an `#[extrinsic_call]` or `#[block]`
 /// attribute.
 ///
-/// See `frame_support::benchmarking` for more info.
+/// See `frame_benchmarking::v2` for more info.
 #[proc_macro_attribute]
 pub fn benchmark(_attrs: TokenStream, _tokens: TokenStream) -> TokenStream {
 	quote!(compile_error!(
@@ -521,7 +521,7 @@ pub fn benchmark(_attrs: TokenStream, _tokens: TokenStream) -> TokenStream {
 /// used as a boundary designating where the benchmark setup code ends, and the benchmark
 /// verification code begins.
 ///
-/// See `frame_support::benchmarking` for more info.
+/// See `frame_benchmarking::v2` for more info.
 #[proc_macro_attribute]
 pub fn extrinsic_call(_attrs: TokenStream, _tokens: TokenStream) -> TokenStream {
 	quote!(compile_error!(
@@ -534,7 +534,7 @@ pub fn extrinsic_call(_attrs: TokenStream, _tokens: TokenStream) -> TokenStream
 /// enclosing benchmark function, This attribute is also used as a boundary designating where
 /// the benchmark setup code ends, and the benchmark verification code begins.
 ///
-/// See `frame_support::benchmarking` for more info.
+/// See `frame_benchmarking::v2` for more info.
 #[proc_macro_attribute]
 pub fn block(_attrs: TokenStream, _tokens: TokenStream) -> TokenStream {
 	quote!(compile_error!(