From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36250) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YiMlB-0005OZ-AY for qemu-devel@nongnu.org; Wed, 15 Apr 2015 08:49:42 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YiMl8-0008Ug-0R for qemu-devel@nongnu.org; Wed, 15 Apr 2015 08:49:41 -0400 Received: from mailapp01.imgtec.com ([195.59.15.196]:50899) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YiMl7-0008U8-Oe for qemu-devel@nongnu.org; Wed, 15 Apr 2015 08:49:37 -0400 Message-ID: <552E5E59.90108@imgtec.com> Date: Wed, 15 Apr 2015 13:49:29 +0100 From: Leon Alrae MIME-Version: 1.0 References: <1427894283-31953-1-git-send-email-leon.alrae@imgtec.com> <9DE872C8-4AE3-48FC-B4A1-61ADE4DB7F32@livius.net> <551CF623.7040506@imgtec.com> <551D086D.9010604@imgtec.com> <551D1BC2.10408@imgtec.com> <1C30C43D-CD89-4EBF-991A-9CC53019ED7B@livius.net> <6D39441BF12EF246A7ABCE6654B023532101BA54@LEMAIL01.le.imgtec.org> <1741D364-48FC-4682-B660-82A5B48F8DE4@livius.net> <55255556.8020504@imgtec.com> <2A76D025-D3C9-4023-B48B-F6870998C99A@livius.net> <552E514E.1030204@imgtec.com> <428A086F-92EB-4A25-9A96-849317AD4E02@livius.net> In-Reply-To: <428A086F-92EB-4A25-9A96-849317AD4E02@livius.net> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC PATCH] vl.c: add -semihosting-config "arg" sub-argument List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Liviu Ionescu Cc: "peter.maydell@linaro.org" , "christopher.covington@linaro.org" , Matthew Fortune , "qemu-devel@nongnu.org" On 15/04/2015 13:06, Liviu Ionescu wrote: > >> On 15 Apr 2015, at 14:53, Leon Alrae wrote: >> >> On 15/04/2015 10:09, Liviu Ionescu wrote: >>> Peter/Leon, >>> >>> I'm planning for my future releases, and, to avoid incompatible versions, I would like to help with the semihosting configuration issue, to speed up things. is there any chance to close this issue in the near future? >> >> I'm hoping so :) I'm holding up UHI patches respin until we resolve this. > > how do you suggest we should proceed? If there aren't any objections for the proposed solution, then let's just cook up the patch. It would be nice to get ACK/NACK from Peter as initially it was suggested to avoid adding top level arguments. But I think there were reasonable enough arguments for having top level --semihosting-cmdline placed at the end of command line. > >>> we currently have two solutions: >>> >>> - multiple: --semihosting-config arg="..." >>> - a single option, placed at the end, and followed by any number of arguments: --semihosting-cmdline $@ \n >>> >>> both can generate the array of args required by UHI and both can pass args with spaces. >>> >>> the second one is more scripting friendly, and is somehow in the spirit of other unix programs that need to pass such arguments. >> >> I don't have a strong preference here, but I think I would be leaning >> towards the second one as it just looks better. On the other hand it >> reserves the last position in the QEMU command line, which I'm not sure >> if can become a problem at some point? > > do you have anything specific in mind? I don't know qemu very well, but apart from setting semihosting and kernel args, I doubt there are other use cases to require variable number of arguments. No, I don't. Leon