Commit eea3678d authored by David Hildenbrand's avatar David Hildenbrand Committed by Christian Borntraeger

s390/mm: flush tlb of shadows in all situations

For now, the tlb of shadow gmap is only flushed when the parent is removed,
not when it is removed upfront. Therefore other shadow gmaps can reuse the
tables without the tlb getting flushed.

Fix this by simply flushing the tlb
1. Before the shadow tables are removed (analogouos to other unshadow functions)
2. When the gmap is freed and therefore the top level pages are freed.
Acked-by: default avatarMartin Schwidefsky <schwidefsky@de.ibm.com>
Signed-off-by: default avatarDavid Hildenbrand <dahi@linux.vnet.ibm.com>
Signed-off-by: default avatarChristian Borntraeger <borntraeger@de.ibm.com>
parent aa17aa57
...@@ -173,6 +173,9 @@ static void gmap_free(struct gmap *gmap) ...@@ -173,6 +173,9 @@ static void gmap_free(struct gmap *gmap)
{ {
struct page *page, *next; struct page *page, *next;
/* Flush tlb of all gmaps (if not already done for shadows) */
if (!(gmap_is_shadow(gmap) && gmap->removed))
gmap_flush_tlb(gmap);
/* Free all segment & region tables. */ /* Free all segment & region tables. */
list_for_each_entry_safe(page, next, &gmap->crst_list, lru) list_for_each_entry_safe(page, next, &gmap->crst_list, lru)
__free_pages(page, 2); __free_pages(page, 2);
...@@ -226,13 +229,10 @@ void gmap_remove(struct gmap *gmap) ...@@ -226,13 +229,10 @@ void gmap_remove(struct gmap *gmap)
{ {
struct gmap *sg, *next; struct gmap *sg, *next;
/* Flush tlb. */
gmap_flush_tlb(gmap);
/* Remove all shadow gmaps linked to this gmap */ /* Remove all shadow gmaps linked to this gmap */
if (!list_empty(&gmap->children)) { if (!list_empty(&gmap->children)) {
spin_lock(&gmap->shadow_lock); spin_lock(&gmap->shadow_lock);
list_for_each_entry_safe(sg, next, &gmap->children, list) { list_for_each_entry_safe(sg, next, &gmap->children, list) {
gmap_flush_tlb(sg);
list_del(&sg->list); list_del(&sg->list);
gmap_put(sg); gmap_put(sg);
} }
...@@ -1360,6 +1360,7 @@ static void gmap_unshadow(struct gmap *sg) ...@@ -1360,6 +1360,7 @@ static void gmap_unshadow(struct gmap *sg)
return; return;
sg->removed = 1; sg->removed = 1;
gmap_call_notifier(sg, 0, -1UL); gmap_call_notifier(sg, 0, -1UL);
gmap_flush_tlb(sg);
table = (unsigned long *)(sg->asce & _ASCE_ORIGIN); table = (unsigned long *)(sg->asce & _ASCE_ORIGIN);
switch (sg->asce & _ASCE_TYPE_MASK) { switch (sg->asce & _ASCE_TYPE_MASK) {
case _ASCE_TYPE_REGION1: case _ASCE_TYPE_REGION1:
......
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