diff --git a/paddle/function/nnpack/NNPACKConvOp.cpp b/paddle/function/nnpack/NNPACKConvOp.cpp
index 5e4de55469d567ddea24d0dd9a660b627b3c9df5..d75fab040334f835a92c51babddc35a2b7ee0271 100644
--- a/paddle/function/nnpack/NNPACKConvOp.cpp
+++ b/paddle/function/nnpack/NNPACKConvOp.cpp
@@ -74,9 +74,9 @@ public:
 
   virtual void check(const BufferArgs& inputs,
                      const BufferArgs& outputs) override {
-    const TensorShape& output = inputs[0].shape();
+    const TensorShape& input = inputs[0].shape();
     const TensorShape& filter = inputs[1].shape();
-    const TensorShape& input = outputs[0].shape();
+    const TensorShape& output = outputs[0].shape();
     checkShape(input, filter, output);
   }
 
diff --git a/paddle/function/nnpack/NNPACKConvOpTest.cpp b/paddle/function/nnpack/NNPACKConvOpTest.cpp
index e7ce61cc6c425b7a7fdb98cefdfe66013f8ef379..48180112111c67f36ddd425008187201655089c9 100644
--- a/paddle/function/nnpack/NNPACKConvOpTest.cpp
+++ b/paddle/function/nnpack/NNPACKConvOpTest.cpp
@@ -60,12 +60,15 @@ public:
                             << " outputWidth=" << outputSize
                             << " stride=" << stride << " padding=" << padding;
 
+                  std::vector<size_t> paddings = {padding, padding};
+                  std::vector<size_t> strides = {stride, stride};
                   Compare2Function<DEVICE_TYPE_CPU, DEVICE_TYPE_CPU> test(
                       conv1,
                       conv2,
                       FuncConfig()
-                          .set("padding", padding)
-                          .set("stride", stride)
+                          .set("paddings", paddings)
+                          .set("strides", strides)
+                          .set("groups", (size_t)1)
                           .set("algo", algo));
 
                   TensorShape shape0{