Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 2 Jul 2017 04:09:05 +0000 (UTC)
From:      Marcelo Araujo <araujo@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org
Subject:   svn commit: r320566 - stable/11/sys/geom/part
Message-ID:  <201707020409.v62495Pm087522@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: araujo
Date: Sun Jul  2 04:09:05 2017
New Revision: 320566
URL: https://svnweb.freebsd.org/changeset/base/320566

Log:
  MFC r320390:
  
  With r318394 seems it breaks gpart(8) in some embedded systems such like PCEngines,
  RPI1-B, Alix and APU2 boards as well as NanoBSD with the following message:
  
  vnode_pager_generic_getpages_done: I/O read error 5
  
  Seems the breakage was because it was missed to include acr in glabel update.
  
  Reported by:	Peter Blok <pblok@bsd4all.org>,
  		madpilot, imp and trasz.
  Reviewed by:	trasz
  Tested by:	Peter Blok and madpilot.
  Sponsored by:	iXsystems, Inc.
  Differential Revision:	https://reviews.freebsd.org/D11365

Modified:
  stable/11/sys/geom/part/g_part.c
Directory Properties:
  stable/11/   (props changed)

Modified: stable/11/sys/geom/part/g_part.c
==============================================================================
--- stable/11/sys/geom/part/g_part.c	Sun Jul  2 03:50:22 2017	(r320565)
+++ stable/11/sys/geom/part/g_part.c	Sun Jul  2 04:09:05 2017	(r320566)
@@ -886,7 +886,8 @@ g_part_ctl_commit(struct gctl_req *req, struct g_part_
 		if (!entry->gpe_deleted) {
 			/* Notify consumers that provider might be changed. */
 			if (entry->gpe_modified && (
-			    entry->gpe_pp->acw + entry->gpe_pp->ace) == 0)
+			    entry->gpe_pp->acw + entry->gpe_pp->ace +
+			    entry->gpe_pp->acr) == 0)
 				g_media_changed(entry->gpe_pp, M_NOWAIT);
 			entry->gpe_created = 0;
 			entry->gpe_modified = 0;



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201707020409.v62495Pm087522>