On 13/03/2021 12.22, menglong8.d...@gmail.com wrote:
From: Zhang Yunkai
'linux/ioctl.h' included in 'lightnvm.h' is duplicated.
It is also included in the 33th line.
Signed-off-by: Zhang Yunkai
---
include/uapi/linux/lightnvm.h | 1 -
1 file changed, 1 deletion(-)
diff --git
eturns NULL if
no vma is found, hence only the !vma check is needed in __do_munmap().
Signed-off-by: Gonzalo Matias Juarez Tello
---
mm/mmap.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/mm/mmap.c b/mm/mmap.c
index 3f287599a7a3..59ceaa474a00 100644
--- a/mm/mmap.c
Some lines in __do_munmap used the same logic as find_vma_intersection
(which is inlined) instead of directly using that function.
(Can't believe I made a typo in the first one, compiled this one,
sorry first patch kinda nervous for some reason)
Signed-off-by: Gonzalo Matias Juarez Tello
Some lines in __do_munmap used the same logic as find_vma_intersection
(which is inlined) instead of directly using that function.
Signed-off-by: Gonzalo Matias Juarez Tello
---
mm/mmap.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/mm/mmap.c b/mm/mmap.c
index
On 22/02/2021 07.06, Chaitanya Kulkarni wrote:
This fixs coccicheck warning:-
drivers/nvme//host/lightnvm.c:1243:60-61: WARNING opportunity for
kobj_to_dev()
Signed-off-by: Chaitanya Kulkarni
---
drivers/nvme/host/lightnvm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On 21/01/2021 20.49, Heiner Litz wrote:
there are a couple more, but again I would understand if those are
deemed not important enough to keep it.
device emulation of (non-ZNS) SSD block device
That'll soon be available. We will be open-sourcing a new device mapper
(dm-zap), which implements
ould still be maintained in the
github repository and available for researchers. Given the few changes
that have happened over the past year, it should be relatively easy to
rebase for each kernel release for quite a while.
Best, Matias
On Thu, Jan 21, 2021 at 5:57 AM Matias Bjørling wro
On 21/01/2021 13.47, Jens Axboe wrote:
On 1/21/21 12:22 AM, Pan Bian wrote:
The allocated page is not released if error occurs in
nvm_submit_io_sync_raw(). __free_page() is moved ealier to avoid
possible memory leak issue.
Applied, thanks.
General question for Matias - is lightnvm maintained
this queued up for 5.10? This is really helpful for e.g.
the zonefs test suite or xfstests when btrfs HMZONED support lands.
Thanks,
Johannes
Thanks, Niklas.
Reviewed-by: Matias Bjørling
On 18/08/2020 11.50, Javier Gonzalez wrote:
On 18.08.2020 09:12, Christoph Hellwig wrote:
On Tue, Aug 18, 2020 at 10:59:36AM +0530, Kanchan Joshi wrote:
If drive does not support zone-append natively, enable emulation using
regular write.
Make emulated zone-append cmd write-lock the zone,
> -Original Message-
> From: Javier González
> Sent: Tuesday, 7 July 2020 10.43
> To: Matias Bjorling
> Cc: Damien Le Moal ; ax...@kernel.dk;
> kbu...@kernel.org; h...@lst.de; s...@grimberg.me;
> martin.peter...@oracle.com; Niklas Cassel ; Hans
> Holmberg ; li
> -Original Message-
> From: Javier González
> Sent: Monday, 29 June 2020 21.39
> To: Damien Le Moal
> Cc: Matias Bjorling ; ax...@kernel.dk;
> kbu...@kernel.org; h...@lst.de; s...@grimberg.me;
> martin.peter...@oracle.com; Niklas Cassel ; Hans
> Holmberg ; li
> -Original Message-
> From: Bart Van Assche
> Sent: Monday, 29 June 2020 03.36
> To: Matias Bjorling ; ax...@kernel.dk;
> kbu...@kernel.org; h...@lst.de; s...@grimberg.me;
> martin.peter...@oracle.com; Damien Le Moal ;
> Niklas Cassel ; Hans Holmberg
>
> Cc:
> -Original Message-
> From: Niklas Cassel
> Sent: Monday, 29 June 2020 11.04
> To: Damien Le Moal
> Cc: Matias Bjorling ; ax...@kernel.dk;
> kbu...@kernel.org; h...@lst.de; s...@grimberg.me;
> martin.peter...@oracle.com; Hans Holmberg ;
> linux-s...@vger.
Descriptor Extension size from the
specific device.
Signed-off-by: Matias Bjørling
---
Documentation/block/queue-sysfs.rst | 6 ++
block/blk-sysfs.c | 15 ++-
drivers/nvme/host/zns.c | 1 +
drivers/scsi/sd_zbc.c | 1 +
include/linux
.
Best, Matias
Matias Bjørling (2):
block: add zone_desc_ext_bytes to sysfs
block: add BLKSETDESCZONE ioctl for Zoned Block Devices
Documentation/block/queue-sysfs.rst | 6 ++
block/blk-sysfs.c | 15 +++-
block/blk-zoned.c | 108
parameter.
Signed-off-by: Matias Bjørling
---
block/blk-zoned.c | 108 ++
block/ioctl.c | 2 +
drivers/nvme/host/core.c | 3 +
drivers/nvme/host/nvme.h | 9 +++
drivers/nvme/host/zns.c | 11
include/linux
On 19/06/2020 16.18, Jens Axboe wrote:
On 6/19/20 5:15 AM, Matias Bjørling wrote:
On 19/06/2020 11.41, javier.g...@samsung.com wrote:
Jens,
Would you have time to answer a question below in this thread?
On 18.06.2020 11:11, javier.g...@samsung.com wrote:
On 18.06.2020 08:47, Damien Le Moal
On 19/06/2020 17.20, Jens Axboe wrote:
On 6/19/20 9:14 AM, Matias Bjørling wrote:
On 19/06/2020 16.18, Jens Axboe wrote:
On 6/19/20 5:15 AM, Matias Bjørling wrote:
On 19/06/2020 11.41, javier.g...@samsung.com wrote:
Jens,
Would you have time to answer a question below in this thread
can allocate the completion entries in a 32MB memory range,
and then use base_address + index to get back to the completion data
structure specified in the sqe.
Best, Matias
On 18/06/2020 21.21, Kanchan Joshi wrote:
On Thu, Jun 18, 2020 at 10:04:32AM +0200, Matias Bjørling wrote:
On 17/06/2020 19.23, Kanchan Joshi wrote:
This patchset enables issuing zone-append using aio and io-uring
direct-io interface.
For aio, this introduces opcode IOCB_CMD_ZONE_APPEND
On 18/06/2020 10.39, Javier González wrote:
On 18.06.2020 10:32, Matias Bjørling wrote:
On 18/06/2020 10.27, Javier González wrote:
On 18.06.2020 10:04, Matias Bjørling wrote:
On 17/06/2020 19.23, Kanchan Joshi wrote:
This patchset enables issuing zone-append using aio and io-uring
direct-io
On 18/06/2020 10.27, Javier González wrote:
On 18.06.2020 10:04, Matias Bjørling wrote:
On 17/06/2020 19.23, Kanchan Joshi wrote:
This patchset enables issuing zone-append using aio and io-uring
direct-io interface.
For aio, this introduces opcode IOCB_CMD_ZONE_APPEND. Application
uses
On 17/06/2020 19.23, Kanchan Joshi wrote:
This patchset enables issuing zone-append using aio and io-uring direct-io
interface.
For aio, this introduces opcode IOCB_CMD_ZONE_APPEND. Application uses start LBA
of the zone to issue append. On completion 'res2' field is used to return
}
Thanks, Quishi.
Signed-off-by: Matias Bjørling
Jens, would you kindly pick up the patch?
Thank you, Matias
This patch adds support for MSG_PEEK. In such a case, packets are not
removed from the rx_queue and credit updates are not sent.
Signed-off-by: Matias Ezequiel Vara Larsen
Reviewed-by: Stefano Garzarella
Tested-by: Stefano Garzarella
---
net/vmw_vsock/virtio_transport_common.c | 55
This patch adds support for MSG_PEEK. In such a case, packets are not
removed from the rx_queue and credit updates are not sent.
Signed-off-by: Matias Ezequiel Vara Larsen
---
net/vmw_vsock/virtio_transport_common.c | 55 +++--
1 file changed, 52 insertions(+), 3
> > ...
> >
> > total += bytes;
> > off += bytes;
> > }
> > }
> >
> > What do you think?
> >
>
> Maybe, but I need to see a complete patch, evil is in the details :)
>
Thanks both for your comments, I will take them into account and submit
a second version.
Matias
This patch adds support for MSG_PEEK. In such a case, packets are not
removed from the rx_queue and credit updates are not sent.
Signed-off-by: Matias Ezequiel Vara Larsen
---
net/vmw_vsock/virtio_transport_common.c | 50 +++--
1 file changed, 47 insertions(+), 3
On Mon, Sep 23, 2019 at 09:58:30AM +0200, Stefano Garzarella wrote:
> Hi Matias,
> thanks for this patch!
>
> Since this patch only concerns virtio_transport,
> I'd use the 'vsock/virtio' prefix in the commit title:
> "vsock/virtio: add support for MSG_PEEK"
>
>
This patch adds support for MSG_PEEK. In such a case, packets are not
removed from the rx_queue and credit updates are not sent.
Signed-off-by: Matias Ezequiel Vara Larsen
---
net/vmw_vsock/virtio_transport_common.c | 59 +++--
1 file changed, 56 insertions(+), 3
On 6/25/19 2:36 PM, Damien Le Moal wrote:
On 2019/06/25 20:06, Matias Bjørling wrote:
On 6/22/19 3:02 AM, Damien Le Moal wrote:
On 2019/06/21 22:07, Matias Bjørling wrote:
From: Ajay Joshi
Implement REQ_OP_ZONE_OPEN, REQ_OP_ZONE_CLOSE and REQ_OP_ZONE_FINISH
support to allow explicit control
On 6/22/19 3:02 AM, Damien Le Moal wrote:
On 2019/06/21 22:07, Matias Bjørling wrote:
From: Ajay Joshi
Implement REQ_OP_ZONE_OPEN, REQ_OP_ZONE_CLOSE and REQ_OP_ZONE_FINISH
support to allow explicit control of zone states.
Signed-off-by: Ajay Joshi
Signed-off-by: Matias Bjørling
On 6/22/19 2:51 AM, Damien Le Moal wrote:
Matias,
Some comments inline below.
On 2019/06/21 22:07, Matias Bjørling wrote:
From: Ajay Joshi
Zoned block devices allows one to control zone transitions by using
explicit commands. The available transitions are:
* Open zone: Transition a zone
From: Ajay Joshi
Implement REQ_OP_ZONE_OPEN, REQ_OP_ZONE_CLOSE and REQ_OP_ZONE_FINISH
support to allow explicit control of zone states.
Signed-off-by: Ajay Joshi
---
drivers/md/dm-flakey.c| 7 +++
drivers/md/dm-linear.c| 2 +-
drivers/md/dm.c | 5 +++--
From: Heiner Litz
bio_add_pc_page() may merge pages when a bio is padded due to a flush.
Fix iteration over the bio to free the correct pages in case of a merge.
Signed-off-by: Heiner Litz
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-core.c | 16
- with default op value it is approx.
110MB of DRAM less for every 1TB of media.
Signed-off-by: Igor Konopko
Reviewed-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-core.c | 8
drivers/lightnvm/pblk-init.c | 7 +++
drivers
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-gc.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/drivers/lightnvm/pblk-gc.c b/drivers/lightnvm/pblk-gc.c
index 901e49951ab5..65692e6d76e6 100644
--- a/drivers/lightnvm/pblk-gc.c
+++ b/drivers/lightnvm/pblk-gc.c
easily get rid of this lock for a time when we are
removing nvm targets.
Signed-off-by: Igor Konopko
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/core.c | 34 --
1 file changed, 16 insertions(+), 18 deletions(-)
diff --git
On 3/18/19 2:32 PM, Marcin Dziegielewski wrote:
On 3/14/19 2:56 PM, Matias Bjørling wrote:
On 3/14/19 6:41 AM, Marcin Dziegielewski wrote:
Open channel devices are not able to handle traditional
IO requests addressed by LBA, so following current
approach to exposing special nvme devices
be used as a traditional zoned block
device, and use the existing infrastructure. Which is really neat.
It is not upstream, since it depends on some features that we introduce
with zoned namespaces, but in general, tools can read/write from a block
device as any other, just honoring the special write rules that are for
OCSSD/zoned block devices.
-Matias
translates into a stall when
an I/O of the largest possible size is submitted.
Fixes: f9f9d1ae2c66 ("lightnvm: pblk: prevent stall due to wb threshold")
Signed-off-by: Javier González
---
Matias: Can you apply this as a fix to 5.1. This is a case I missed when fixing
the wb thresh
From: Hans Holmberg
As chunk metadata is allocated using vmalloc, we need to free it
using vfree.
Fixes: 090ee26fd512 ("lightnvm: use internal allocation for chunk log page")
Signed-off-by: Hans Holmberg
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-core.c | 2 +-
1 fi
: 766c8ceb16fc ("lightnvm: pblk: guarantee that backpointer is respected
on writer stall")
Signed-off-by: Javier González
Reviewed-by: Hans Holmberg
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-rb.c | 25 +++--
drivers/lightnvm/pblk-rl.c | 5 ++--
)/arch/$(SRCARCH)/include as a header
search path, but we should not rely on it.
Signed-off-by: Masahiro Yamada
Reviewed-by: Hans Holmberg
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-trace.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/lightnvm/pblk
.
This will render us a warning during recovery if something unexpected
has happened to the chunk write pointers (i.e. powerloss, a spurious
chunk reset, ..).
Reported-by: Zhoujie Wu
Tested-by: Zhoujie Wu
Reviewed-by: Javier González
Signed-off-by: Hans Holmberg
Signed-off-by: Matias Bjørling
in the
L2P. Lines with a counter of greater than zero will not be selected
for GC.
Signed-off-by: Heiner Litz
Reviewed-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-core.c | 1 +
drivers/lightnvm/pblk-gc.c| 22
From: Andy Shevchenko
There are new types and helpers that are supposed to be used in new code.
As a preparation to get rid of legacy types and API functions do
the conversion here.
Signed-off-by: Andy Shevchenko
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers
.
Change types of lba_list_mem and lba_list_media members to be u64.
Signed-off-by: Andy Shevchenko
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/lightnvm/pblk.h b/drivers
Hi Jens,
Would you please pick up the following patches for 5.1?
It is a bunch of misc patches this time. A couple of fixes and cleanups.
Andy Shevchenko (2):
lightnvm: Use u64 instead of __le64 for CPU visible side
lightnvm: pblk: Switch to use new generic UUID API
Hans Holmberg (3):
.
Signed-off-by: Hans Holmberg
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-init.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/lightnvm/pblk-init.c b/drivers/lightnvm/pblk-init.c
index f9a3e47b6a93..eb0135c77805 100644
--- a/drivers/lightnvm/pblk-init.c
+++ b/drivers
sed by Dan and
the " Zoned Block Devices" from Matias and Damien.
Cheers
Stephen
[1]
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/net/ethernet/netronome/nfp/bpf/offload.c?h=v5.0-rc5
If we're going down that road, we can also look at the bl
On 2/1/19 3:38 AM, Heiner Litz wrote:
This patch fixes a race condition where a write is mapped to the last
sectors of a line. The write is synced to the device but the L2P is not
updated yet. When the line is garbage collected before the L2P update is
performed, the sectors are ignored by the
On 2/5/19 7:50 AM, Javier González wrote:
In order to respect mw_cuinits, pblk's write buffer maintains a
backpointer to protect data not yet persisted; when writing to the write
buffer, this backpointer defines a threshold that pblk's rate-limiter
enforces.
On small PU configurations, the
On 1/30/19 11:26 AM, Javier González wrote:
In order to respect mw_cuinits, pblk's write buffer maintains a
backpointer to protect data not yet persisted; when writing to the write
buffer, this backpointer defines a threshold that pblk's rate-limiter
enforces.
On small PU configurations, the
On 1/30/19 9:18 AM, h...@owltronix.com wrote:
From: Hans Holmberg
pblk stripes writes of minimal write size across all non-offline chunks
in a line, which means that the maximum write pointer delta should not
exceed the minimal write size.
Extend the line write pointer balance check to cover
On 1/25/19 11:09 AM, Javier González wrote:
In order to respect mw_cuinits, pblk's write buffer maintains a
backpointer to protect data not yet persisted; when writing to the write
buffer, this backpointer defines a threshold that pblk's rate-limiter
enforces.
On small PU configurations, the
On 1/25/19 9:01 AM, Hans Holmberg wrote:
On Fri, Jan 25, 2019 at 8:35 AM Masahiro Yamada
wrote:
As the comment block in include/trace/define_trace.h says,
TRACE_INCLUDE_PATH should be a relative path to the define_trace.h
../../drivers/lightnvm is the correct relative path.
On 1/24/19 3:31 PM, Andy Shevchenko wrote:
There are new types and helpers that are supposed to be used in new code.
As a preparation to get rid of legacy types and API functions do
the conversion here.
Signed-off-by: Andy Shevchenko
---
v2:
- convert instance_uuid to guid_t and get rid of
On 1/22/19 11:15 AM, h...@owltronix.com wrote:
From: Hans Holmberg
pblk_line_meta_free might sleep (it can end up calling vfree, depending
on how we allocate lba lists), and this can lead to a BUG()
if we wake up on a different cpu and release the lock.
As there is no point of grabbing the
On 1/22/19 11:17 AM, h...@owltronix.com wrote:
From: Hans Holmberg
As chunk metadata is allocated using vmalloc, we need to free it
using vfree.
Fixes: 090ee26fd512 ("lightnvm: use internal allocation for chunk log page")
Signed-off-by: Hans Holmberg
---
drivers/lightnvm/pblk-core.c | 2 +-
On 12/22/18 8:39 AM, Gustavo A. R. Silva wrote:
Remove one of the calls to function bio_put(), so *bio* is only
freed once.
Notice that bio is being dereferenced in bio_put(), hence leading to
a use-after-free bug once *bio* has already been freed.
Addresses-Coverity-ID: 1475952 ("Use after
From: Hans Holmberg
The chunk metadata is allocated with vmalloc, so we need to use
vfree to free it.
Fixes: 090ee26fd512 ("lightnvm: use internal allocation for chunk log page")
Signed-off-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
driver
("lightnvm: move bad block and chunk state logic to
core")
Fixes: a294c199455187d1 ("lightnvm: implement get log report chunk helpers")
Signed-off-by: Geert Uytterhoeven
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/core.c | 2 +-
drivers/nvme/host/lightnvm.c | 3 ++-
From: Hans Holmberg
The check for chunk closes suffers from an off-by-one issue, leading
to chunk close events not being traced.
Fixes: 4c44abf43d00 ("lightnvm: pblk: add trace events for chunk states")
Signed-off-by: Hans Holmberg
Signed-off-by: Matias Bjørling
---
drivers/lig
From: Hans Holmberg
ADDR_POOL_SIZE is not used anymore, so remove the macro.
Signed-off-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-init.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/lightnvm/pblk-init.c b/drivers
-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-write.c | 17 +++--
1 file changed, 15 insertions(+), 2 deletions(-)
diff --git a/drivers/lightnvm/pblk-write.c b/drivers/lightnvm/pblk-write.c
index 3ddd16f47106..750f04b8a227 100644
--- a/drivers/lightnvm
From: Hans Holmberg
If mapping fails (i.e. when running out of lines), handle the error
and stop writing.
Signed-off-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-map.c | 47 +--
drivers/lightnvm
with the standard
(11%) over provisioning.
Signed-off-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-init.c | 40
drivers/lightnvm/pblk-rl.c | 5 ++---
drivers/lightnvm/pblk.h | 12 ++-
3
From: Zhoujie Wu
The smeta area l2p mapping is empty, and actually the
recovery procedure only need to restore data sector's l2p
mapping. So ignore the smeta oob scan.
Signed-off-by: Zhoujie Wu
Reviewed-by: Javier González
Reviewed-by: Hans Holmberg
Signed-off-by: Matias Bjørling
From: Hua Su
Protect the list_add on the pblk_line_init_bb() error
path in case this code is used for some other purpose
in the future.
Signed-off-by: Hua Su
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-core.c | 13 ++---
1 file changed, 10
From: Hua Su
Signed-off-by: Hua Su
Updated description.
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-rb.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/lightnvm/pblk-rb.c b/drivers/lightnvm/pblk-rb.c
index b1f4b51783f4..9f7fa0fe9c77 100644
From: Hans Holmberg
Remove the call to pblk_line_replace_data as it returns
directly because we have not set l_mg->data_next yet.
Signed-off-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-recovery.c | 1 -
1 file changed, 1 delet
-by: Matias Bjørling
---
drivers/lightnvm/core.c | 12 +---
drivers/nvme/host/core.c | 18 +-
drivers/nvme/host/lightnvm.c | 18 ++
drivers/nvme/host/nvme.h | 2 --
4 files changed, 20 insertions(+), 30 deletions(-)
diff --git a/drivers/lightnvm
. This is also
more explicit about what actually happens on cache creation.
In the process, do a small refactoring to use kref helpers.
Fixes: 1864de94ec9d6 "lightnvm: pblk: stop recreating global caches"
Signed-off-by: Javier González
Reviewed-by: Hans Holmberg
Signed-off-by: Matia
is supported.
Reviewed-by: Javier González
Signed-off-by: Igor Konopko
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-core.c | 5 ++--
drivers/lightnvm/pblk-init.c | 6
drivers/lightnvm/pblk-map.c | 20 -
drivers/lightnvm/pblk-read.c | 48
and write path.
- Fix from Hua Su to fix a race condition in the pblk recovery code.
- Scan optimization added to pblk recovery from Zhoujie.
- Small geometry cleanup from me.
Thank you,
Matias
Geert Uytterhoeven (1):
lightnvm: Fix uninitialized return value in nvm_get_chunk_meta()
Hans Holmberg
and enables drives
without per IO metadata to recover open lines.
After this patch, drives with OOB size <16B will use packed metadata
and metadata size larger than16B will continue to use the device per
IO metadata.
Reviewed-by: Javier González
Signed-off-by: Igor Konopko
Signed-off-by: Matias Bjørl
in the
mapping table (as the lba is marked as invalid).
Signed-off-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-write.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/lightnvm/pblk-write.c b/drivers
,
such as 128 bytes, the static size does not cover this use-case.
This patch adds support for I/O metadata above 56 bytes by changing DMA
pool size based on device meta size and allows pblk to use OOB metadata
>=16B.
Reviewed-by: Javier González
Signed-off-by: Igor Konopko
Signed-off-by: Mat
From: Javier González
pblk's recovery path is single threaded and therefore a number of
assumptions regarding concurrency can be made. To avoid confusion, make
this explicit with a couple of comments in the code.
Signed-off-by: Javier González
Signed-off-by: Matias Bjørling
---
drivers
-by: Matias Bjørling
---
drivers/lightnvm/pblk-read.c | 20 +---
drivers/lightnvm/pblk.h | 2 ++
2 files changed, 7 insertions(+), 15 deletions(-)
diff --git a/drivers/lightnvm/pblk-read.c b/drivers/lightnvm/pblk-read.c
index 9fba614adeeb..19917d3c19b3 100644
--- a/drivers/lightnvm
done for pblk_get_sec_meta()
since we also cannot point to the same memory address in meta buffer
when we are using it for pblk recovery process
Reported-by: Hans Holmberg
Tested-by: Hans Holmberg
Signed-off-by: Igor Konopko
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk.h | 7
IO operation.
This patch prevents pblk to be instantiated in case device only
supports interleaved metadata.
Reviewed-by: Javier González
Signed-off-by: Igor Konopko
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/pblk-init.c | 6 ++
drivers/nvme/host/lightnvm.c | 1 +
include/linux
On 11/03/2018 07:37 PM, Matias Bjørling wrote:
The numa_slit variable used by node_distance is available to a
module as long as it is linked at compile-time. However, it is
not available to loadable modules. Leading to errors such as:
ERROR: "numa_slit" [drivers/nvme/host/nv
On 11/03/2018 07:37 PM, Matias Bjørling wrote:
The numa_slit variable used by node_distance is available to a
module as long as it is linked at compile-time. However, it is
not available to loadable modules. Leading to errors such as:
ERROR: "numa_slit" [drivers/nvme/host/nv
On 11/06/2018 02:33 PM, Hans Holmberg wrote:
From: Hans Holmberg
This series is a slew of bugfixes and cleanups for PBLK, mostly
fixing issues found during corner-case testing in QEMU.
Changes since v1:
Messed up from:, now the patches apply with the correct author
Pardon the
On 11/06/2018 02:33 PM, Hans Holmberg wrote:
From: Hans Holmberg
This series is a slew of bugfixes and cleanups for PBLK, mostly
fixing issues found during corner-case testing in QEMU.
Changes since v1:
Messed up from:, now the patches apply with the correct author
Pardon the
ty into account when selecting a
path"
Fixes: 73569e11032f "lightnvm: remove dependencies on BLK_DEV_NVME and PCI"
Signed-off-by: Matias Bjøring
---
arch/ia64/include/asm/numa.h | 4 +++-
arch/ia64/kernel/acpi.c | 6 +++---
arch/ia64/mm/numa.c | 6 ++
3 files changed,
ty into account when selecting a
path"
Fixes: 73569e11032f "lightnvm: remove dependencies on BLK_DEV_NVME and PCI"
Signed-off-by: Matias Bjøring
---
arch/ia64/include/asm/numa.h | 4 +++-
arch/ia64/kernel/acpi.c | 6 +++---
arch/ia64/mm/numa.c | 6 ++
3 files changed,
On 09/11/2018 01:35 PM, Javier González wrote:
On the OCSSD 2.0 spec, the device populates the metadata pointer (if
provided) when a chunk is reset. Implement this path in pblk and use it
for sanity chunk checks.
For 1.2, reset the write pointer and the state on core so that the erase
path is
On 09/11/2018 01:35 PM, Javier González wrote:
On the OCSSD 2.0 spec, the device populates the metadata pointer (if
provided) when a chunk is reset. Implement this path in pblk and use it
for sanity chunk checks.
For 1.2, reset the write pointer and the state on core so that the erase
path is
On 09/04/2018 12:38 PM, Hans Holmberg wrote:
From: Hans Holmberg
On 1.2-devices, the mapping-out of remaning sectors in the
failed-write's block can result in an infinite loop,
stalling the write pipeline, fix this.
Fixes: 6a3abf5beef6 ("lightnvm: pblk: rework write error recovery path")
On 09/04/2018 12:38 PM, Hans Holmberg wrote:
From: Hans Holmberg
On 1.2-devices, the mapping-out of remaning sectors in the
failed-write's block can result in an infinite loop,
stalling the write pipeline, fix this.
Fixes: 6a3abf5beef6 ("lightnvm: pblk: rework write error recovery path")
r, and let the rest be
dependent on the command opcode, which is trivial to detect and
set.
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/core.c | 20
drivers/lightnvm/pblk-core.c | 13 -
drivers/lightnvm/pblk-read.c | 8 +---
drivers/lig
r, and let the rest be
dependent on the command opcode, which is trivial to detect and
set.
Signed-off-by: Matias Bjørling
---
drivers/lightnvm/core.c | 20
drivers/lightnvm/pblk-core.c | 13 -
drivers/lightnvm/pblk-read.c | 8 +---
drivers/lig
From: Matias Bjørling
Adds support for exposing a null_blk device through the zone device
interface.
The interface is managed with the parameters zoned and zone_size.
If zoned is set, the null_blk instance registers as a zoned block
device. The zone_size parameter defines how big each zone
From: Matias Bjørling
Adds support for exposing a null_blk device through the zone device
interface.
The interface is managed with the parameters zoned and zone_size.
If zoned is set, the null_blk instance registers as a zoned block
device. The zone_size parameter defines how big each zone
From: Hans Holmberg
Smeta write errors were previously ignored. Skip these
lines instead and throw them back on the free
list, so the chunks will go through a reset cycle
before we attempt to use the line again.
Signed-off-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias
From: Hans Holmberg
Smeta write errors were previously ignored. Skip these
lines instead and throw them back on the free
list, so the chunks will go through a reset cycle
before we attempt to use the line again.
Signed-off-by: Hans Holmberg
Reviewed-by: Javier González
Signed-off-by: Matias
1 - 100 of 2084 matches
Mail list logo