From nobody Thu May 2 18:16:42 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Authentication-Results: mx.zohomail.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail(p=none dis=none) header.from=linux.intel.com ARC-Seal: i=1; a=rsa-sha256; t=1563261735; cv=none; d=zoho.com; s=zohoarc; b=VcqgNyvgSTotTOnVfGE3dX73WODGSNd3dccCd36fexol8UMImJ0zPJHx3qV5TemLmwnA2Y5Hqwn8+rhrlBN7q1ckWBoT9XPKO9jK4WDkNfdPBy9rSE1CeZWBxrB3EPajMtdr0/uPKFG22O/S89ObnWXPQFuB+NdmGDodn3Rf+nY= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zoho.com; s=zohoarc; t=1563261735; h=Cc:Date:From:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:Message-ID:Sender:Subject:To:ARC-Authentication-Results; bh=QPLBVOpPWOxggwlFA+FbCpa5ZoVYfe2S3Y/Q5eRNMnk=; b=I0coMKxrjrSpiJ8CdXPH/sYMGXe0hOcYQ5iVO2LOI9CJULjG84lZxJj2jjOhtqkWUOqjiP/FVsb9eegeVlpw7Tcp31ufu+SDGDSpjvNYEA3mweboBDj85nEOvRvq3w74v7krOAW0TuVqAcXSceGLXLpsv+seaPZ8yQJSIB+anIk= ARC-Authentication-Results: i=1; mx.zoho.com; spf=pass (zoho.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=fail header.from= (p=none dis=none) header.from= Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 15632617352691018.4425200985871; Tue, 16 Jul 2019 00:22:15 -0700 (PDT) Received: from localhost ([::1]:45844 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hnHmw-00049L-2Q for importer@patchew.org; Tue, 16 Jul 2019 03:22:14 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:44270) by lists.gnu.org with esmtp (Exim 4.86_2) (envelope-from ) id 1hnHml-0003l9-Nt for qemu-devel@nongnu.org; Tue, 16 Jul 2019 03:22:04 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hnHmk-0007zT-RP for qemu-devel@nongnu.org; Tue, 16 Jul 2019 03:22:03 -0400 Received: from mga12.intel.com ([192.55.52.136]:53917) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1hnHmi-0007ut-Sa for qemu-devel@nongnu.org; Tue, 16 Jul 2019 03:22:02 -0400 Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 Jul 2019 00:21:55 -0700 Received: from richard.sh.intel.com (HELO localhost) ([10.239.159.54]) by orsmga005.jf.intel.com with ESMTP; 16 Jul 2019 00:21:54 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.63,496,1557212400"; d="scan'208";a="342627492" From: Wei Yang To: qemu-devel@nongnu.org Date: Tue, 16 Jul 2019 15:21:27 +0800 Message-Id: <20190716072127.2000-1-richardw.yang@linux.intel.com> X-Mailer: git-send-email 2.17.1 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 192.55.52.136 Subject: [Qemu-devel] [PATCH] vl: make sure char-pty message displayed by moving setbuf to the beginning X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: pbonzini@redhat.com, armbru@redhat.com, Wei Yang Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: "Qemu-devel" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Recently we found a behavior change after commit 6ade45f2ac93611 ('char-pty: Print "char device redirected" message to stdout'). When we redirect output to a file, the message "char device redirected to PTY_NAME (label LABEL)" would not be seen at the beginning of the file. Instead, the message is displayed after QEMU quit. This will block test automation. The reason is this message is printed after we set line buffer mode. So move this to the beginning. Signed-off-by: Wei Yang --- vl.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/vl.c b/vl.c index 5089fce6c5..a5808f9a02 100644 --- a/vl.c +++ b/vl.c @@ -2890,6 +2890,8 @@ int main(int argc, char **argv, char **envp) char *dir, **dirs; BlockdevOptionsQueue bdo_queue =3D QSIMPLEQ_HEAD_INITIALIZER(bdo_queue= ); =20 + os_set_line_buffering(); + error_init(argv[0]); module_call_init(MODULE_INIT_TRACE); =20 @@ -4246,8 +4248,6 @@ int main(int argc, char **argv, char **envp) semihosting_arg_fallback(kernel_filename, kernel_cmdline); } =20 - os_set_line_buffering(); - /* spice needs the timers to be initialized by this point */ qemu_spice_init(); =20 --=20 2.17.1