Thread: C->C++ conversion

How to convert the code below to use std::vector and std::copy. Is it even possible? I'm just having some fun and trying to convert a piece of my code to c++ style.

Code:

// Just an example, missing some stuff.
std::string vendor; // already in c++
unsigned int registers[4];

          asm volatile
          (
            "pushq %%rbx\n\t"
            "cpuid\n\t"
            "movq %%rax, (%1)\n\t"
            "movq %%rbx, 4(%1)\n\t"
            "movq %%rdx, 8(%1)\n\t"
            "movq %%rcx, 12(%1)\n\t"
            "popq %%rbx\n\t"
           :
           : "a"(cpuidOperation), "S"(registers)
          );

memcpy((void *)vendor.c_str(),&registers[1],3*sizeof(unsigned int));

This is what I have done so far, but I'm a bit stuck in std::copy.

Code:

std::string vendor;
std::vector<unsigned int> registers_cpuinfo;
registers_cpuinfo.reserve(4);

          asm volatile
          (
            "pushq %%rbx\n\t"
            "cpuid\n\t"
            "movq %%rax, (%1)\n\t"
            "movq %%rbx, 4(%1)\n\t"
            "movq %%rdx, 8(%1)\n\t"
            "movq %%rcx, 12(%1)\n\t"
            "popq %%rbx\n\t"
           :
           : "a"(cpuidOperation), "S"(&registers_cpuinfo)
          );

 std::copy(registers_cpuinfo.begin(), registers_cpuinfo.begin()
                 + 3*sizeof(unsigned int),vendor.begin()); ??

Thanks for any hints.

Here's an example on how you could do it with an uint vector.
It's in intel assembler syntax though. I have no idea how to convert it to AT&T, but hopefully you'll get the idea.

Code:

const std::string GetVendorName_uint()
{
	std::vector<unsigned int> data(5, 0); // 1 extra element to null-terminate the string
	unsigned int* p = &data[0];

	__asm
	{
		pushad
		mov eax, 0
		cpuid
		mov esi, p
		mov [esi], eax
		mov [esi+4], ebx
		mov [esi+8], edx
		mov [esi+12], ecx
		popad
	}

	char* str = reinterpret_cast<char*>(&data[1]); // skip eax
	return std::string(str);
}

In this case it works albeit its not correct.

No. It doesn't.

It may appear to work under these circumstances for your environment. I doubt even that much.

This does not goes well on what I want to do.

The first code block attempts to do part of what I suggested.

You can't do what you seem to want to do with the tools you are trying to do it with.

It looks to me as if you are trying to get the vendor ID string into a `std::string' variable in a very round about or illegal fashion. (This can be done, for x86, far simpler.) Is this what you are trying to do? If not, maybe you should describe in words what you are trying to do.

the gcc inline assembly code?

That's GCC/GAS inline assembler? Wow. There is a couple of features there I haven't seen before.

Soma

Here's an example on how you could do it with an uint vector.

That should work, but I still wonder why you would want to pay for the "ACDD Dance" of `std::vector<???>' when you wind up by punning the data at the relevant address anyway?!

Also, you don't need to explicitly store the value of `EAX' after `CPUID' returns if you are only going to throw it away.

Soma

Originally Posted by phantomotap

That should work, but I still wonder why you would want to pay for the "ACDD Dance" of `std::vector<???>' when you wind up by punning the data at the relevant address anyway?!

No idea really, I'm just answering the question

Also, you don't need to explicitly store the value of `EAX' after `CPUID' returns if you are only going to throw it away.

Yes I know. But I included it just to show how to make a string from a vector where it's included.
I noticed the "cpuidOperation" variable in the OPs post so I'm assuming he wants to keep eax for the other cpuid calls besides vendor name.

Originally Posted by phantomotap

No. It doesn't.

It may appear to work under these circumstances for your environment. I doubt even that much.

It works, at least in my conditions. May fail in other ones, I don't disagree there.

You can't do what you seem to want to do with the tools you are trying to do it with.

It looks to me as if you are trying to get the vendor ID string into a `std::string' variable in a very round about or illegal fashion. (This can be done, for x86, far simpler.) Is this what you are trying to do? If not, maybe you should describe in words what you are trying to do.

In this specific case, I'm just getting the vendor name, but the assembly code is supposed to be reused, to get other features of the CPU. Feel free to explain how I can do this in other ways.

That's GCC/GAS inline assembler? Wow. There is a couple of features there I haven't seen before.

Yep

Also, you don't need to explicitly store the value of `EAX' after `CPUID' returns if you are only going to throw it away.
Yes I know. But I included it just to show how to make a string from a vector where it's included.
I noticed the "cpuidOperation" variable in the OPs post so I'm assuming he wants to keep eax for the other cpuid calls besides vendor name.

Exactly!

That should work, but I still wonder why you would want to pay for the "ACDD Dance" of `std::vector<???>' when you wind up by punning the data at the relevant address anyway?!

What is punning? English is not my native language.