RE: [GIT PULL 16/20] lightnvm: error handling when whole line is bad

2018-05-29 Thread Konopko, Igor J
> From: Javier Gonzalez [mailto:jav...@cnexlabs.com] > This case cannot occur on the only user of the function > (pblk_recov_l2p()). On the previous check (pblk_line_was_written()), we > verify the state of the line and the position of the first good chunk. In > the case of a bad line (less chunks

RE: [GIT PULL 16/20] lightnvm: error handling when whole line is bad

2018-05-29 Thread Konopko, Igor J
> From: Javier Gonzalez [mailto:jav...@cnexlabs.com] > This case cannot occur on the only user of the function > (pblk_recov_l2p()). On the previous check (pblk_line_was_written()), we > verify the state of the line and the position of the first good chunk. In > the case of a bad line (less chunks

Re: [GIT PULL 16/20] lightnvm: error handling when whole line is bad

2018-05-28 Thread Javier Gonzalez
> On 28 May 2018, at 10.58, Matias Bjørling wrote: > > From: Igor Konopko > > When all the blocks (chunks) in line are marked as bad (offline) > we shouldn't try to read smeta during init process. > > Currently we are trying to do so by passing -1

Re: [GIT PULL 16/20] lightnvm: error handling when whole line is bad

2018-05-28 Thread Javier Gonzalez
> On 28 May 2018, at 10.58, Matias Bjørling wrote: > > From: Igor Konopko > > When all the blocks (chunks) in line are marked as bad (offline) > we shouldn't try to read smeta during init process. > > Currently we are trying to do so by passing -1 as PPA address, > what causes multiple

[GIT PULL 16/20] lightnvm: error handling when whole line is bad

2018-05-28 Thread Matias Bjørling
From: Igor Konopko When all the blocks (chunks) in line are marked as bad (offline) we shouldn't try to read smeta during init process. Currently we are trying to do so by passing -1 as PPA address, what causes multiple warnings, that we issuing IOs to out-of-bound

[GIT PULL 16/20] lightnvm: error handling when whole line is bad

2018-05-28 Thread Matias Bjørling
From: Igor Konopko When all the blocks (chunks) in line are marked as bad (offline) we shouldn't try to read smeta during init process. Currently we are trying to do so by passing -1 as PPA address, what causes multiple warnings, that we issuing IOs to out-of-bound PPAs. Signed-off-by: Igor