From owner-freebsd-current@FreeBSD.ORG Sat Mar 21 01:00:45 2015 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id 45F8A330; Sat, 21 Mar 2015 01:00:45 +0000 (UTC) Received: from mail-wg0-x22d.google.com (mail-wg0-x22d.google.com [IPv6:2a00:1450:400c:c00::22d]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (Client CN "smtp.gmail.com", Issuer "Google Internet Authority G2" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id DCE428D4; Sat, 21 Mar 2015 01:00:44 +0000 (UTC) Received: by wgbcc7 with SMTP id cc7so102164420wgb.0; Fri, 20 Mar 2015 18:00:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=9sdzZF1vZ6uUxanKu0S+SEXCg9NnyeFjltdUIFVWQEw=; b=os2omYscDFBS1MNe4xCo+k1EMUjksHTjvAootZkm9mSdXlLUniMVFoMXcYIJL7Zd+1 yuvsx3tknRSt7K6MVcqsmltYbtVQlwayRXWI8NLBkIwv/1jb+6z7fJY2ewpHnifg2f+m WU5KiLgkdGb5W0+MSwA+KBNYEs6qswtiyiITUCA8F0l4BzuVvyjeOolVCXh48qJfCCcw KxoxEdCZh1PE95I2rdmX5rx6jZNwEMe8qii7t5CbD0OBT2JUs42ZKD55ipqS8/LYSACK LVW377Lx97dnL1rdsXwOYIRMigTnd/tZEkGmktHvzI0zq96+j+5rehw6LUgM46Z2Be4i xTiw== X-Received: by 10.180.74.135 with SMTP id t7mr686145wiv.72.1426899643337; Fri, 20 Mar 2015 18:00:43 -0700 (PDT) Received: from localhost.localdomain (ip-89-176-114-84.net.upcbroadband.cz. [89.176.114.84]) by mx.google.com with ESMTPSA id q6sm303207wix.3.2015.03.20.18.00.41 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 20 Mar 2015 18:00:42 -0700 (PDT) From: Mateusz Guzik To: Konstantin Belousov Subject: [PATCH 0/3] fix up some recent proc fallout and more Date: Sat, 21 Mar 2015 02:00:37 +0100 Message-Id: <1426899640-6599-1-git-send-email-mjguzik@gmail.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <20150320122125.GP2379@kib.kiev.ua> References: <20150320122125.GP2379@kib.kiev.ua> Cc: freebsd-current@freebsd.org, jenkins-admin@freebsd.org, Mateusz Guzik X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 21 Mar 2015 01:00:45 -0000 From: Mateusz Guzik Patches in this series fix a bug introduced in r280130 and deal with additional issues. I'm not happy with how sutff is being done at the moment. In particular we zero out various fields on process exit, which puts it into a state state which is not guaranteed when we deal when it is first allocated. But I guess we can leave it as it is for now. I don't believe we have a good solution at the moment which would poison memory returned by uma_zalloc, that would definitely make the issue apparent right of the bat. Adding such a feature sould be reasonably simple but fallout may take some time to deal with, I'll make a follow up thread. Mateusz Guzik (3): fork: assign refed credentials earlier cred: add proc_set_cred_init helper proc: use MTX_NEW flag in proc_init sys/kern/init_main.c | 2 +- sys/kern/kern_fork.c | 15 +++++++-------- sys/kern/kern_proc.c | 11 +++++------ sys/kern/kern_prot.c | 16 ++++++++++++++-- sys/sys/ucred.h | 1 + 5 files changed, 28 insertions(+), 17 deletions(-) -- 2.3.2