1. Making it a combobox is not for the purpose to simplify coding, it is for visual balance, for current kimchi UI style, you can imagine how a checkbox looks there.
2. Please clarify below

http://lists.ovirt.org/pipermail/kimchi-devel/2014-November/008660.html

---- For below, do you mean that I should always display SMT1, SMT2, SMT4 for the first 3, and for the last one, it will be SMT[threads_per_core]? I wonder why only 4 options.
	  Display a menu labled SMT, with up to four check-boxes.
          The check-boxes will be labeled SMT1, SMT2, SMT4, and SMT8.
          The max SMT value displayed will be dependant upon
          threads_per_core from above. The default checkbox should be the
          greatest SMT value (e.g. SMT8).

---- For below, seems like you mean the CPUs should be limited to a certain set of numbers, where to get this set of numbers.

	  Display the same box for CPUs as before. I had initially thought
          about having a drop-down for CPUs instead of the empty field,
          (with only legal vCPU amounts, like 1,2,4,8, etc.)
---- For below
curl -k -u user -X POST -H 'Content-Typ: application/json' -H 'Accept: application/json' \
 https://localhost:8001/templates -d'{"name": "test_topo", "cdrom": "/isos/DVD_name.iso", \
 "cpus":4, "cpu_info": {"topology": {"sockets": 1, "cores": 2, "threads":2}}}'

This is the content when create template, what content should be added for 'SMT' or 'Hyper-threading' when saving a template.


On 11/14/2014 2:21 AM, Crístian Viana wrote:
On 13-11-2014 08:58, Yu Xin Huo wrote:
x86


Power(for SMT, I think it is single selection)




Even though I know it might make the code a little harder, I think it would be better to display the HT field (on x86) as a checkbox element, where the user can enable and disable quickly. A combo box is not appropriate for on/off statuses. But I understand that it's easier because of how SMT works on POWER, and you'd need a combo box anyway.