[Pdbg] [PATCH] main: Run as root

[Balbir Singh]

On Wed, Mar 28, 2018 at 4:31 PM, Alistair Popple <alistair at popple.id.au> wrote:
> On Wednesday, 28 March 2018 4:00:33 PM AEDT Amitay Isaacs wrote:
>> On Wed, 2018-03-28 at 15:19 +1100, Michael Neuling wrote:
>> > Thanks Amitay,
>> >
>> > There are some parts of pgdb that won't need root. Seems like we
>> > should add
>> > something like this when we actually try to open a device and get an
>> > EPERM.
>>
>> What parts are those?  From my (non-existent) understanding of pdbg,
>> all of it requires root privileges.
>
> The fake backend is the most obvious example. What we need here is some better
> error reporting when something that we expect to be there (or can't function
> without) isn't.
>
> What is happening is host_pib_probe() is failing but it isn't reported. This is
> because we don't currently have a way of figuring out if the failure is critical
> and should be reported as probing is allowed to fail (eg. because a piece of
> hardware like a core may not exist on this computer).
>
> Perhaps we should probably set/get the target status to figure out if a given
> target is critical or not to the current operation.
>

Doesn't this all depend on the backend? If the program has access to the backend
devices, everything works? If I used -b i2c and I had access to
/dev/i2c4 via a group,
I'd expect things to work

Balbir Singh.