Re: [PATCH v4 1/3]mmc: set max_discard_sectors value for mmc queue

2011-02-14 Thread Adrian Hunter
On 12/02/11 08:22, ext Chuanxiao Dong wrote: max_discard_sectors value is UINT_MAX which means kernel block layer can pass down unlimited sectors to MMC driver to erase. But erasing so many sectors may delay some other important I/O requests. This is not preferred. So use 'pref_erase' to set a s

Re: [PATCH v4 1/3]mmc: set max_discard_sectors value for mmc queue

2011-02-14 Thread Arnd Bergmann
On Monday 14 February 2011, Dong, Chuanxiao wrote: > When I do trim with a 32GB eMMC card in my platform, sometimes I can get the > 10s > timeout errors but sometimes not. I am not much clear about the "discarding > partial > AU will take a lot longer". If this action is hide for driver, then I t

RE: [PATCH v4 1/3]mmc: set max_discard_sectors value for mmc queue

2011-02-13 Thread Dong, Chuanxiao
ject: Re: [PATCH v4 1/3]mmc: set max_discard_sectors value for mmc queue > > On Saturday 12 February 2011 11:42:51 Dong, Chuanxiao wrote: > > > From: Arnd Bergmann [mailto:a...@arndb.de] > > > On Saturday 12 February 2011 07:22:14 Chuanxiao Dong wrote: > > > >

Re: [PATCH v4 1/3]mmc: set max_discard_sectors value for mmc queue

2011-02-12 Thread Arnd Bergmann
On Saturday 12 February 2011 11:42:51 Dong, Chuanxiao wrote: > > From: Arnd Bergmann [mailto:a...@arndb.de] > > On Saturday 12 February 2011 07:22:14 Chuanxiao Dong wrote: > > > max_discard_sectors value is UINT_MAX which means kernel block layer can > > > pass > > > down unlimited sectors to MMC

RE: [PATCH v4 1/3]mmc: set max_discard_sectors value for mmc queue

2011-02-12 Thread Dong, Chuanxiao
ubject: Re: [PATCH v4 1/3]mmc: set max_discard_sectors value for mmc queue > > On Saturday 12 February 2011 07:22:14 Chuanxiao Dong wrote: > > max_discard_sectors value is UINT_MAX which means kernel block layer can > > pass > > down unlimited sectors to MMC driver t

Re: [PATCH v4 1/3]mmc: set max_discard_sectors value for mmc queue

2011-02-12 Thread Arnd Bergmann
On Saturday 12 February 2011 07:22:14 Chuanxiao Dong wrote: > max_discard_sectors value is UINT_MAX which means kernel block layer can pass > down unlimited sectors to MMC driver to erase. But erasing so many sectors may > delay some other important I/O requests. This is not preferred. > > So use