Commit 253d6a28 authored by Andy Shevchenko's avatar Andy Shevchenko Committed by Chris Ball

mmc: mmc_test: minor clean up, use t = &test.area everywhere

Signed-off-by: default avatarAndy Shevchenko <ext-andriy.shevchenko@nokia.com>
Signed-off-by: default avatarChris Ball <cjb@laptop.org>
parent aea253ec
...@@ -1386,8 +1386,9 @@ static int mmc_test_area_io(struct mmc_test_card *test, unsigned long sz, ...@@ -1386,8 +1386,9 @@ static int mmc_test_area_io(struct mmc_test_card *test, unsigned long sz,
*/ */
static int mmc_test_area_fill(struct mmc_test_card *test) static int mmc_test_area_fill(struct mmc_test_card *test)
{ {
return mmc_test_area_io(test, test->area.max_tfr, test->area.dev_addr, struct mmc_test_area *t = &test->area;
1, 0, 0);
return mmc_test_area_io(test, t->max_tfr, t->dev_addr, 1, 0, 0);
} }
/* /*
...@@ -1400,7 +1401,7 @@ static int mmc_test_area_erase(struct mmc_test_card *test) ...@@ -1400,7 +1401,7 @@ static int mmc_test_area_erase(struct mmc_test_card *test)
if (!mmc_can_erase(test->card)) if (!mmc_can_erase(test->card))
return 0; return 0;
return mmc_erase(test->card, t->dev_addr, test->area.max_sz >> 9, return mmc_erase(test->card, t->dev_addr, t->max_sz >> 9,
MMC_ERASE_ARG); MMC_ERASE_ARG);
} }
...@@ -1527,8 +1528,10 @@ static int mmc_test_area_prepare_fill(struct mmc_test_card *test) ...@@ -1527,8 +1528,10 @@ static int mmc_test_area_prepare_fill(struct mmc_test_card *test)
static int mmc_test_best_performance(struct mmc_test_card *test, int write, static int mmc_test_best_performance(struct mmc_test_card *test, int write,
int max_scatter) int max_scatter)
{ {
return mmc_test_area_io(test, test->area.max_tfr, test->area.dev_addr, struct mmc_test_area *t = &test->area;
write, max_scatter, 1);
return mmc_test_area_io(test, t->max_tfr, t->dev_addr, write,
max_scatter, 1);
} }
/* /*
...@@ -1568,18 +1571,19 @@ static int mmc_test_best_write_perf_max_scatter(struct mmc_test_card *test) ...@@ -1568,18 +1571,19 @@ static int mmc_test_best_write_perf_max_scatter(struct mmc_test_card *test)
*/ */
static int mmc_test_profile_read_perf(struct mmc_test_card *test) static int mmc_test_profile_read_perf(struct mmc_test_card *test)
{ {
struct mmc_test_area *t = &test->area;
unsigned long sz; unsigned long sz;
unsigned int dev_addr; unsigned int dev_addr;
int ret; int ret;
for (sz = 512; sz < test->area.max_tfr; sz <<= 1) { for (sz = 512; sz < t->max_tfr; sz <<= 1) {
dev_addr = test->area.dev_addr + (sz >> 9); dev_addr = t->dev_addr + (sz >> 9);
ret = mmc_test_area_io(test, sz, dev_addr, 0, 0, 1); ret = mmc_test_area_io(test, sz, dev_addr, 0, 0, 1);
if (ret) if (ret)
return ret; return ret;
} }
sz = test->area.max_tfr; sz = t->max_tfr;
dev_addr = test->area.dev_addr; dev_addr = t->dev_addr;
return mmc_test_area_io(test, sz, dev_addr, 0, 0, 1); return mmc_test_area_io(test, sz, dev_addr, 0, 0, 1);
} }
...@@ -1588,6 +1592,7 @@ static int mmc_test_profile_read_perf(struct mmc_test_card *test) ...@@ -1588,6 +1592,7 @@ static int mmc_test_profile_read_perf(struct mmc_test_card *test)
*/ */
static int mmc_test_profile_write_perf(struct mmc_test_card *test) static int mmc_test_profile_write_perf(struct mmc_test_card *test)
{ {
struct mmc_test_area *t = &test->area;
unsigned long sz; unsigned long sz;
unsigned int dev_addr; unsigned int dev_addr;
int ret; int ret;
...@@ -1595,8 +1600,8 @@ static int mmc_test_profile_write_perf(struct mmc_test_card *test) ...@@ -1595,8 +1600,8 @@ static int mmc_test_profile_write_perf(struct mmc_test_card *test)
ret = mmc_test_area_erase(test); ret = mmc_test_area_erase(test);
if (ret) if (ret)
return ret; return ret;
for (sz = 512; sz < test->area.max_tfr; sz <<= 1) { for (sz = 512; sz < t->max_tfr; sz <<= 1) {
dev_addr = test->area.dev_addr + (sz >> 9); dev_addr = t->dev_addr + (sz >> 9);
ret = mmc_test_area_io(test, sz, dev_addr, 1, 0, 1); ret = mmc_test_area_io(test, sz, dev_addr, 1, 0, 1);
if (ret) if (ret)
return ret; return ret;
...@@ -1604,8 +1609,8 @@ static int mmc_test_profile_write_perf(struct mmc_test_card *test) ...@@ -1604,8 +1609,8 @@ static int mmc_test_profile_write_perf(struct mmc_test_card *test)
ret = mmc_test_area_erase(test); ret = mmc_test_area_erase(test);
if (ret) if (ret)
return ret; return ret;
sz = test->area.max_tfr; sz = t->max_tfr;
dev_addr = test->area.dev_addr; dev_addr = t->dev_addr;
return mmc_test_area_io(test, sz, dev_addr, 1, 0, 1); return mmc_test_area_io(test, sz, dev_addr, 1, 0, 1);
} }
...@@ -1614,6 +1619,7 @@ static int mmc_test_profile_write_perf(struct mmc_test_card *test) ...@@ -1614,6 +1619,7 @@ static int mmc_test_profile_write_perf(struct mmc_test_card *test)
*/ */
static int mmc_test_profile_trim_perf(struct mmc_test_card *test) static int mmc_test_profile_trim_perf(struct mmc_test_card *test)
{ {
struct mmc_test_area *t = &test->area;
unsigned long sz; unsigned long sz;
unsigned int dev_addr; unsigned int dev_addr;
struct timespec ts1, ts2; struct timespec ts1, ts2;
...@@ -1625,8 +1631,8 @@ static int mmc_test_profile_trim_perf(struct mmc_test_card *test) ...@@ -1625,8 +1631,8 @@ static int mmc_test_profile_trim_perf(struct mmc_test_card *test)
if (!mmc_can_erase(test->card)) if (!mmc_can_erase(test->card))
return RESULT_UNSUP_HOST; return RESULT_UNSUP_HOST;
for (sz = 512; sz < test->area.max_sz; sz <<= 1) { for (sz = 512; sz < t->max_sz; sz <<= 1) {
dev_addr = test->area.dev_addr + (sz >> 9); dev_addr = t->dev_addr + (sz >> 9);
getnstimeofday(&ts1); getnstimeofday(&ts1);
ret = mmc_erase(test->card, dev_addr, sz >> 9, MMC_TRIM_ARG); ret = mmc_erase(test->card, dev_addr, sz >> 9, MMC_TRIM_ARG);
if (ret) if (ret)
...@@ -1634,7 +1640,7 @@ static int mmc_test_profile_trim_perf(struct mmc_test_card *test) ...@@ -1634,7 +1640,7 @@ static int mmc_test_profile_trim_perf(struct mmc_test_card *test)
getnstimeofday(&ts2); getnstimeofday(&ts2);
mmc_test_print_rate(test, sz, &ts1, &ts2); mmc_test_print_rate(test, sz, &ts1, &ts2);
} }
dev_addr = test->area.dev_addr; dev_addr = t->dev_addr;
getnstimeofday(&ts1); getnstimeofday(&ts1);
ret = mmc_erase(test->card, dev_addr, sz >> 9, MMC_TRIM_ARG); ret = mmc_erase(test->card, dev_addr, sz >> 9, MMC_TRIM_ARG);
if (ret) if (ret)
...@@ -1646,12 +1652,13 @@ static int mmc_test_profile_trim_perf(struct mmc_test_card *test) ...@@ -1646,12 +1652,13 @@ static int mmc_test_profile_trim_perf(struct mmc_test_card *test)
static int mmc_test_seq_read_perf(struct mmc_test_card *test, unsigned long sz) static int mmc_test_seq_read_perf(struct mmc_test_card *test, unsigned long sz)
{ {
struct mmc_test_area *t = &test->area;
unsigned int dev_addr, i, cnt; unsigned int dev_addr, i, cnt;
struct timespec ts1, ts2; struct timespec ts1, ts2;
int ret; int ret;
cnt = test->area.max_sz / sz; cnt = t->max_sz / sz;
dev_addr = test->area.dev_addr; dev_addr = t->dev_addr;
getnstimeofday(&ts1); getnstimeofday(&ts1);
for (i = 0; i < cnt; i++) { for (i = 0; i < cnt; i++) {
ret = mmc_test_area_io(test, sz, dev_addr, 0, 0, 0); ret = mmc_test_area_io(test, sz, dev_addr, 0, 0, 0);
...@@ -1669,20 +1676,22 @@ static int mmc_test_seq_read_perf(struct mmc_test_card *test, unsigned long sz) ...@@ -1669,20 +1676,22 @@ static int mmc_test_seq_read_perf(struct mmc_test_card *test, unsigned long sz)
*/ */
static int mmc_test_profile_seq_read_perf(struct mmc_test_card *test) static int mmc_test_profile_seq_read_perf(struct mmc_test_card *test)
{ {
struct mmc_test_area *t = &test->area;
unsigned long sz; unsigned long sz;
int ret; int ret;
for (sz = 512; sz < test->area.max_tfr; sz <<= 1) { for (sz = 512; sz < t->max_tfr; sz <<= 1) {
ret = mmc_test_seq_read_perf(test, sz); ret = mmc_test_seq_read_perf(test, sz);
if (ret) if (ret)
return ret; return ret;
} }
sz = test->area.max_tfr; sz = t->max_tfr;
return mmc_test_seq_read_perf(test, sz); return mmc_test_seq_read_perf(test, sz);
} }
static int mmc_test_seq_write_perf(struct mmc_test_card *test, unsigned long sz) static int mmc_test_seq_write_perf(struct mmc_test_card *test, unsigned long sz)
{ {
struct mmc_test_area *t = &test->area;
unsigned int dev_addr, i, cnt; unsigned int dev_addr, i, cnt;
struct timespec ts1, ts2; struct timespec ts1, ts2;
int ret; int ret;
...@@ -1690,8 +1699,8 @@ static int mmc_test_seq_write_perf(struct mmc_test_card *test, unsigned long sz) ...@@ -1690,8 +1699,8 @@ static int mmc_test_seq_write_perf(struct mmc_test_card *test, unsigned long sz)
ret = mmc_test_area_erase(test); ret = mmc_test_area_erase(test);
if (ret) if (ret)
return ret; return ret;
cnt = test->area.max_sz / sz; cnt = t->max_sz / sz;
dev_addr = test->area.dev_addr; dev_addr = t->dev_addr;
getnstimeofday(&ts1); getnstimeofday(&ts1);
for (i = 0; i < cnt; i++) { for (i = 0; i < cnt; i++) {
ret = mmc_test_area_io(test, sz, dev_addr, 1, 0, 0); ret = mmc_test_area_io(test, sz, dev_addr, 1, 0, 0);
...@@ -1709,15 +1718,16 @@ static int mmc_test_seq_write_perf(struct mmc_test_card *test, unsigned long sz) ...@@ -1709,15 +1718,16 @@ static int mmc_test_seq_write_perf(struct mmc_test_card *test, unsigned long sz)
*/ */
static int mmc_test_profile_seq_write_perf(struct mmc_test_card *test) static int mmc_test_profile_seq_write_perf(struct mmc_test_card *test)
{ {
struct mmc_test_area *t = &test->area;
unsigned long sz; unsigned long sz;
int ret; int ret;
for (sz = 512; sz < test->area.max_tfr; sz <<= 1) { for (sz = 512; sz < t->max_tfr; sz <<= 1) {
ret = mmc_test_seq_write_perf(test, sz); ret = mmc_test_seq_write_perf(test, sz);
if (ret) if (ret)
return ret; return ret;
} }
sz = test->area.max_tfr; sz = t->max_tfr;
return mmc_test_seq_write_perf(test, sz); return mmc_test_seq_write_perf(test, sz);
} }
...@@ -1726,6 +1736,7 @@ static int mmc_test_profile_seq_write_perf(struct mmc_test_card *test) ...@@ -1726,6 +1736,7 @@ static int mmc_test_profile_seq_write_perf(struct mmc_test_card *test)
*/ */
static int mmc_test_profile_seq_trim_perf(struct mmc_test_card *test) static int mmc_test_profile_seq_trim_perf(struct mmc_test_card *test)
{ {
struct mmc_test_area *t = &test->area;
unsigned long sz; unsigned long sz;
unsigned int dev_addr, i, cnt; unsigned int dev_addr, i, cnt;
struct timespec ts1, ts2; struct timespec ts1, ts2;
...@@ -1737,15 +1748,15 @@ static int mmc_test_profile_seq_trim_perf(struct mmc_test_card *test) ...@@ -1737,15 +1748,15 @@ static int mmc_test_profile_seq_trim_perf(struct mmc_test_card *test)
if (!mmc_can_erase(test->card)) if (!mmc_can_erase(test->card))
return RESULT_UNSUP_HOST; return RESULT_UNSUP_HOST;
for (sz = 512; sz <= test->area.max_sz; sz <<= 1) { for (sz = 512; sz <= t->max_sz; sz <<= 1) {
ret = mmc_test_area_erase(test); ret = mmc_test_area_erase(test);
if (ret) if (ret)
return ret; return ret;
ret = mmc_test_area_fill(test); ret = mmc_test_area_fill(test);
if (ret) if (ret)
return ret; return ret;
cnt = test->area.max_sz / sz; cnt = t->max_sz / sz;
dev_addr = test->area.dev_addr; dev_addr = t->dev_addr;
getnstimeofday(&ts1); getnstimeofday(&ts1);
for (i = 0; i < cnt; i++) { for (i = 0; i < cnt; i++) {
ret = mmc_erase(test->card, dev_addr, sz >> 9, ret = mmc_erase(test->card, dev_addr, sz >> 9,
...@@ -1808,11 +1819,12 @@ static int mmc_test_rnd_perf(struct mmc_test_card *test, int write, int print, ...@@ -1808,11 +1819,12 @@ static int mmc_test_rnd_perf(struct mmc_test_card *test, int write, int print,
static int mmc_test_random_perf(struct mmc_test_card *test, int write) static int mmc_test_random_perf(struct mmc_test_card *test, int write)
{ {
struct mmc_test_area *t = &test->area;
unsigned int next; unsigned int next;
unsigned long sz; unsigned long sz;
int ret; int ret;
for (sz = 512; sz < test->area.max_tfr; sz <<= 1) { for (sz = 512; sz < t->max_tfr; sz <<= 1) {
/* /*
* When writing, try to get more consistent results by running * When writing, try to get more consistent results by running
* the test twice with exactly the same I/O but outputting the * the test twice with exactly the same I/O but outputting the
...@@ -1829,7 +1841,7 @@ static int mmc_test_random_perf(struct mmc_test_card *test, int write) ...@@ -1829,7 +1841,7 @@ static int mmc_test_random_perf(struct mmc_test_card *test, int write)
if (ret) if (ret)
return ret; return ret;
} }
sz = test->area.max_tfr; sz = t->max_tfr;
if (write) { if (write) {
next = rnd_next; next = rnd_next;
ret = mmc_test_rnd_perf(test, write, 0, sz); ret = mmc_test_rnd_perf(test, write, 0, sz);
...@@ -1859,17 +1871,18 @@ static int mmc_test_random_write_perf(struct mmc_test_card *test) ...@@ -1859,17 +1871,18 @@ static int mmc_test_random_write_perf(struct mmc_test_card *test)
static int mmc_test_seq_perf(struct mmc_test_card *test, int write, static int mmc_test_seq_perf(struct mmc_test_card *test, int write,
unsigned int tot_sz, int max_scatter) unsigned int tot_sz, int max_scatter)
{ {
struct mmc_test_area *t = &test->area;
unsigned int dev_addr, i, cnt, sz, ssz; unsigned int dev_addr, i, cnt, sz, ssz;
struct timespec ts1, ts2; struct timespec ts1, ts2;
int ret; int ret;
sz = test->area.max_tfr; sz = t->max_tfr;
/* /*
* In the case of a maximally scattered transfer, the maximum transfer * In the case of a maximally scattered transfer, the maximum transfer
* size is further limited by using PAGE_SIZE segments. * size is further limited by using PAGE_SIZE segments.
*/ */
if (max_scatter) { if (max_scatter) {
struct mmc_test_area *t = &test->area;
unsigned long max_tfr; unsigned long max_tfr;
if (t->max_seg_sz >= PAGE_SIZE) if (t->max_seg_sz >= PAGE_SIZE)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment