[Pdbg] [PATCH v2 13/15] main: Convert getcfam/putcfam to use path based targeting

Amitay Isaacs amitay at ozlabs.org
Wed Nov 14 16:42:59 AEDT 2018


On Tue, 2018-11-13 at 16:01 +1100, Alistair Popple wrote:
> Hi Amitay,
> 
> >  test_result 0 <<EOF
> > -p0:0xc09 = HEX8
> > +/kernelfsi at 0: 0xc09 = HEX8
> 
> We may have discussed this sorry if I'm changing my mind but can we
> keep the 
> output the same for the time being? ie. Something like ("p%d", 
> pdbg_target_index(fsi))?

Well the question is what targets we want for getcfam/putcfam?

As per your thinking, if we want pib target to be selected for
getcfam/putcfam commands, then the the existing output makes sense.
In that case, the implementation should find the fsi target for
selected pib target for actual operations.

Does that work?

> 
> >  EOF
> > 
> >  do_skip
> > -test_run $PDBG -p0 getcfam 0xc09
> 
> This test should still pass though? Do you think it would it be
> better to just 
> add another test or are you do think the other selection tests cover
> this case 
> adequately?
> 
> - Alistair
> 
> > +test_run $PDBG -P fsi getcfam 0xc09
> > 
> >  test_result 0 <<EOF
> >  p0:0xf000f = HEX16

Amitay.
-- 

Art is man's attempt to improve on nature.



More information about the Pdbg mailing list